1

我整天都在摸索运算符重载的新手，我的 operator+ 函数可以正确计算数据，但是我需要将 temp 类的数据传递给 operator= 以将其分配给我的类的单独实例并且返回 temp 不会工作（我想数据在退出时被破坏？）整个想法来自 main x = y + z 被调用来添加来自 y 和 z 的两个向量的数据并将其分配给 x 并且我得到 y + 的计算z 正确，但是将它传递给 x 我撞到了砖墙有什么问题？或者有人知道吗？这是我班上的代码

VecXd& operator=(const VecXd& rhs)
{
    VecXd<V> temp;
    temp.dimension = rhs.dimension;
    cout << "operator= check dimension is..." << temp.dimension << endl;

    for(int i = 0; i < rhs.dimension; i++)  //Passing data to x?
    {
        cout << "test" << endl;
        temp.vecArr[i] = rhs.vecArr[i];
        cout << temp.vecArr[i] << " our new value" << endl;
    }
}

friend VecXd& operator+(VecXd& lhs, VecXd& rhs){

    VecXd<V> temp;
    cout << lhs.dimension << "-lhs d-" << rhs.dimension << "-rhs d-" << endl; //works
    if(lhs.dimension == rhs.dimension) //dimension level check
    {
        temp.vecArr = new V[lhs.dimension];
        for(int i = 0; i < lhs.dimension; i++)
        {
            temp.vecArr[i] = lhs.vecArr[i] + rhs.vecArr[i];
            cout << temp.vecArr[i] << " our new value" << endl;
        }
        //return *temp.vecArr;
        return temp; //***? how to pass data?
    }
    else{
        cout << "Dimensions do not match!!! Error!" << endl;
    }
}

任何想法？不要太苛刻......哈哈...... :l

4

2 回答 2

1

您的赋值运算符是虚假的：赋值应该执行的操作是使指向this的对象与右侧的对象相同。您的赋值运算符创建一个临时对象并设置该对象。但是，退出赋值运算符时它会消失。此外，您声明您的赋值运算符返回一个引用，该引用通常返回一个引用，*this但没有返回语句。

除非我有很好的理由采取不同的做法，否则我会根据复制构造函数、析构函数和swap()函数来实现赋值运算符：

VecXd& VecXd::operator=(VecXd rhs) 
{
    this->swap(rhs);
    return *this;
}

调用复制构造函数来创建函数的参数，很好地复制对象。然后新创建的副本与被分配的对象的内容交换，并在返回*this原始内容后由创建的临时对象的析构函数释放rhs。所需要的只是一个相对的功能swap()：

void VecXd::swap(VecXd& other)
{
     std::swap(this->dimension, other.dimension);
     std::swap(this->vecArr, other.vecArr); 
}

当然，该swap()函数假定两个成员可以交换。由于我还没有看到您的声明，VecXd因此我无法判断这是否可行，但通常应该可行。

我还没有真正看过你的加法运算符。

于 2013-09-18T23:56:16.880 回答
0

这个Caltech C++ 运算符重载页面应该有你需要的一切，并且对所涉及的逻辑有一个相对清晰的解释。绝不要求您遵守其中列出的规则和准则，但它们在一般情况下运作良好。

在算术运算符的特定情况下，操作通常根据 += 运算符定义，如 chris 对您的原始帖子的评论中所述。这意味着您通常传入左右参数，复制 lhs，然后使用 lhs_copy += rhs; 找到结果。考虑到这一点，更容易避免代码重复；您只会在 += 中创建一次加法逻辑，然后在 + 中调用它。

您返回的结果不能是引用 - 在函数中创建的任何内容都不会保留，但不应更改 lhs 或 rhs 参数。相反，您只需返回作为实例递增的重复 lhs - 在您的情况下，函数签名是：

const VecXd operator+(const VecXd& lhs, const VecXd& rhs).

您的输入可以而且应该是 const，因为这两个项目都不会在函数中更改，并且 Caltech 页面建议返回类型是 const 实例以禁止某些奇怪的操作，这在许多情况下也是很好的做法。

于 2013-09-18T23:47:23.510 回答