主页 / 问题 / 1437344
Accepted
Stanley
Asked:2022-09-08 16:12:09 +0800 CST2022-09-08 16:12:09 +0800 CST

“飞机上的折线”类

  • 772

问题的条件和陈述：平面上的折线由一系列坐标对给出，这些坐标确定了该折线的点在坐标平面上的位置。对于两条折线，求和 - 一条折线，依次由第一条折线和第二条折线的点组成。在几何上，这可以通过将第一条折线的最后一点和第二条折线的第一点用线段连接起来来表示。编写类“平面上的折线”和算子“折线之和”；

作为代码，我执行了以下操作：输入 2 个任意长度的数组，然后通过重载“+”运算符将它们组合起来。但是，出现错误“在 tcache 2 中检测到双重释放”。

#include <iostream>
using namespace std;
int i=0;
class Array
{
private:
    int size;
   unsigned int* data;
public:
// конструктор
    Array(){}
    Array(int asize) {
        size=asize;
        data=(unsigned int*)new int[size];
        if(data) cout<<"Massiv inicializir\n";
        else cout<<"ne hvatka pamiati\n";
    }
~Array(){
    delete[] data;
    cout<<"Object uni4tojen\n";
}
        //Далее методы
Array(const Array &ob)   //Определение конструктора //копирования
{                           
 size=ob.size;
 data=(unsigned int*)new int [size];
 for(int i=0;i<ob.size;i++)
  data[i]=ob.data[i];
}
void print(){
    for(i=0;i<size;i++){
    cout<<*(data+i)<<" ";
}
cout<<endl;
}

void scan(){
cout<<"Vvedite massiv iz:"<<size<<"simvolov \n";
for(int i=0;i<size;i++){
cin>>*(data+i);
}
}
Array operator+(const Array& ob){
    this->size=size;
    Array tmp(size+ob.size);
    for(i=0;i<size;i++) 
    {
        tmp.data[i]=*(data+i);//this->data[i]=data[i];
    }
    for(i=size;i<size+ob.size;i++) tmp.data[i]=ob.data[i-size];
 return tmp;
}
};
int main(){
    Array first(3);
    Array second(2);
    first.scan();
    second.scan();
    Array third(5);
    third=first+second;
    third.print();
 return 0;
}

请解释为什么会发生这个错误（很可能是由于分配了额外的内存）以及如何修复它。谢谢你。

对了，我之前问了一个关于Lomanaya的问题，条件一样，但是代码不适合，所以我决定单独问这个问题，但是用我需要的代码。

c++

1 个回答

  1. Best Answer

    您没有赋值运算符，但赋值本身是：

    third=first+second;

    因此，它是由编译器生成的，同时执行一个简单的逐个成员赋值（即只分配指针，而不是复制内存的内容），结果导致内存泄漏（对象的内存third没有被删除） +导致两次释放相同的内存（一个由求和创建的临时对象，一个对象third）。

    记住三大规则 :) - 如果您已经在使用内存 - 需要析构函数、复制构造函数和赋值运算符...

    我推荐通过复制进行赋值的习惯用法：编写一个函数swap来交换两个对象的成员，然后创建一个临时对象，与之交换——你就完成了。在你的版本中是这样的：

    void swap(Array& a) {
    std::swap(size,a.size);
    std::swap(data,a.data);
}

Array& operator=(const Array& a)
{
    if (this != &a) { // Необязательно!!
        Array tmp(a);
        swap(tmp);
    }
    return *this;
}
    • 1

相关问题