用单链表重载 + 运算符

Question

对于我的家庭作业，我必须创建一个 int 链接列表。我重载了复制构造函数和赋值运算符，但似乎使 + 运算符重载。我有一个已定义的析构函数来清除列表。

List List::operator+(const List &add)
{
     List result;
     result += *this;
     result += add;
     return result;
}

+= 正在工作。另外，当我执行以下操作时： 列表列表 3 = 列表 1 + 列表 2； 有用。似乎析构函数在它返回之前就被调用了，所以如果我这样做的话，我不会得到 List3 的任何东西

List list3;    
list3 = list1 + list2;

这里是复制构造函数、赋值重载和+=重载

List& List::operator=(const List &assign)
{
    Node *traverse = assign.head;
    int x;
    int *passX = &x;
    while (traverse != nullptr)
    {
        x = traverse->getItem();
        this->Insert(passX);
        traverse = traverse->getNext();
    }
    return *this;
}

List342& List342::operator+=(const List342 &add)
{
    Node *traverse = add.head;
    int x;
    int *passX = &x;
    while (traverse != nullptr)
    {
        x = traverse->getItem();
        this->Insert(passX);
        traverse = traverse->getNext();
    }
    return *this;
}

List342::List342(const List342 &copy)
{
    *this = copy;
}

struct Node
    {
        int item;
        Node *next = nullptr;
        int getItem() const;
        Node* getNext() const;
        void setItem(const int &val);
        void setNext(Node* nodePtr);
    };
    Node *head;
    int itemCount;

最后一部分是节点的结构体，以及这个类的任何对象都会有的两个变量。

谢谢

Answer 1

所以我设法解决了这个问题

List& List::operator+(const List &add)
{
    List *result = new List;
    *result = *this;
    *result += add;
     return result;
}

我的析构函数在返回之前清除了列表...在堆中分配空间会阻止析构函数这样做。

感谢大家的帮助！

Answer 2

很好地将 operator+ 实现为 operator+= 的函数。但是，在重载运算符 + 时，您应该同时声明 LHS 和 RHS const。

const List List::operator+(const List &add) const
{
     List result;
     result += *this;
     result += add;
     return result;
}

第一个const表示函数返回一个const List，所以不能：

List A, B, C;
A + B = C;

第二个表示函数没有修改它的成员变量，所以你可以相信A+B不会修改A。

假设默认构造函数是一个空列表并且您的类不使用指针，请尝试正确声明 const。 const 变量的作用域与非 const 不同。如果您的类使用指针，请确保您实际上是在复制相关内存，而不仅仅是内存地址。