Question

我正在测试一个简单的运算符重载代码，当测试它时，这段代码只是在“ nd.print（）”处崩溃（将内核转储）。有什么建议吗？

崩溃发生在Ubuntu 16.04 64位上。当我尝试使用某些在线shell环境（例如https://www.onlinegdb.com/online_c++_compiler）时，看起来还可以。

#include <iostream>
using namespace std;

class Node
{
    int d;
    public:
    Node (int dd = 0):d(dd){}
    Node &operator=(Node &nd){ d = nd.d; }
    void print(){ cout<<d<<endl; }
};

int main()
{
    Node nd1(1), nd2(2);
    Node nd;
    nd = nd2 = nd1;
    nd.print();    //*******Crash here
    return 0;
}

我希望它只是打印一个值而不会崩溃。

Answer 1

operator=方法需要返回分配的变量。照原样，它没有返回任何内容（尽管签名表明您会-可能会有编译器警告），所以nd = ...位正在分配一个未定义的值。然后，您尝试对该未定义的值调用print方法。

在这种情况下，您要返回分配的值，即*this：

Node &operator=(Node &nd)
{
    d = nd.d;
    return *this;
}

用户定义的简单类，带有操作符=重载代码崩溃

1 个答案: