1

在我的代码中有 operator+ 重载。在这个范围内，我定义了 object ans，我想构建并返回它，但似乎析构函数ans在我返回它之前就已经解构了，所以这个方法返回了一些未定义的值。

我不明白我错在哪里？是析构函数，构建函数，还是在我的 operator+ 重载中？

这是我的代码：

class vector1{
    int * arr;
int size;
public:
//constructors
vector1(){}
vector1(int n){
    size=n;
    arr= new int[size];
}
//functions
int get_size(){return size;}

void init(){  //initialize all array cells to 0
    for(int i=0;i<size;i++)
        arr[i]=0;
}
int get_value_in_index(int index){
    return arr[index];
}
void set_value_in_index(int i, int num){
    arr[i]=num;
}
int & operator[](int i){
    int default_val=0;
    if (i<0 || i>size){
        cout<<"index not valid"<<endl;
        return default_val;
    }
    return arr[i];
}
vector1 operator+(vector1 & ob);

//destructor
~vector1(){
    delete [] arr;
}
};

vector1 vector1:: operator+(vector1 & ob){
vector1 ans(size);
if (ob.get_size()!=size){  //if the arrays are not the same size return array of     '0', this array size
    cout<<"vectors not the same length. can't sum them"<<endl;
    //test
    //exit(1);
    ans.init();
}
else{
    for (int i=0;i<size;i++)
        ans.set_value_in_index(i,arr[i]+ob.get_value_in_index(i));
}
return ans;
}

感谢您的时间和帮助。

4

3 回答 3

5

您的 operator+ 返回一个新vector1类的副本。
但是原始的（在函数开头声明的那个）在块的末尾（右括号}之前）被销毁。

然后析构函数删除内部数组arr。
所以复制的vector1对象指向一个已删除的数组。

您应该创建一个复制构造函数，该构造函数也将复制内部数组。

vector1(const vector1& _other ){
 //copy .arr from the other one to this one
}
于 2013-05-19T12:58:17.617 回答
1

当您从operator+浅拷贝返回时，会留下两个指向相同数据的ans实例。vector1当第一个实例超出范围时，它会删除两者都指向的数据。要更正此问题，您需要添加一个复制构造函数，该构造函数对vector1

vector1(const vector1& other)
{
    // make a deep copy from "other.
}
于 2013-05-19T12:58:39.823 回答
0

您没有定义复制构造函数，因此编译器创建了默认构造函数。

您operator+返回向量的浅表副本。当ans操作员结束时超出范围时，它会随身携带数据。

请阅读更多关于什么是三法则？

于 2013-05-19T13:12:33.357 回答