队列 C++ 的运算符重载

Question

我试图使用重载运算符方法将一个队列的条目复制到另一个队列中，但我的函数出错了。我不知道如何以其他方式访问队列“原始”的值，而不是以下方式：

struct Node
{
   int item;
   Node* next;
};

class Queue
{
public:
    // Extra code here
    void operator = (const Queue &original);
protected:
    Node *front, *end;
};

void Queue::operator=(const Queue &original)
{
    //THIS IS WHERE IM GOING WRONG
    while(original.front->next != NULL) {
        front->item = original.front->item;
        front->next = new Node;
        front = front->next;
        original.front = original.front->next;
    }
}

Answer 1

你有一个功能复制构造函数吗？如果是这样，我会根据您的复制构造函数来实现您的赋值运算符，如下所示：

#include <algorithm>  // <utility> for C++11

void Queue::operator=(const Queue &other)
{
    // Assumes your only field is the "front" pointer.

    Queue tmp(other);   // May throw.
    std::swap(front, tmp.front);  // Will not throw.
}

这个想法是你在一个临时对象中执行任何可能引发异常的操作（比如你对operator new()的调用），该对象将清理资源，然后通过交换内容来“提交”你的更改一个非抛出操作，这样即使tmp 的构造过程中抛出异常，你的Queue 的状态也是正常的。指针分配保证不会抛出，这就是为什么在这种情况下对std::swap() 的调用不会抛出。离开赋值运算符tmp 的范围后，析构函数应该清理旧的链接列表，因为它的front 已与旧的front 交换。

有关此“copy-to-temporary-and-swap”习语的详细信息，以及它与强大的异常安全保证的关系，请参阅GotW #59。

Answer 2

void Queue::operator=(const Queue &original)
{
    Node* tmp = original.front;
    //THIS IS WHERE IM GOING WRONG
    while(tmp->next != NULL) {
        front->item = tmp->item;
        front->next = new Node;
        front = front->next;
        tmp = tmp->next;
    }
}