C++ - 链表栈运算符重载函数

Question

我目前正在研究实现链表的堆栈。在重载“=”运算符时遇到问题。我不知道该怎么做。如果有人能指出一个好的方向，那就太棒了。

//operator overload
template <class S>
const Stack<S>::operator=( const Stack& s )
{

    if (s.isEmpty())
        theFront = theTop = 0
    else
    {
        NodePointer temp = q->theFront;

        while(temp != 0)
        {
            push(temp->data);
            temp = temp->next;
        }
    }

    return *this;
}

我也收到此错误： 堆栈，std::allocator > >::Node::Node(std::basic_string, std::allocator >)' 引用自 C:\USERS\JOHNNY\DESKTOP\STACK\INFIX_TO_RPN.OBJ

我的操作符重载函数可以解决这个问题吗？

Answer 1

您需要先清空当前堆栈，然后再向其推送数据。您应该添加一个 removeAll 函数，并在分配的顶部调用它（在检查自分配之后，这也是一个好主意）。否则，它看起来是正确的。所以，最终的结果是：

//operator overload 
template <class S> 
const Stack<S>::operator=( const Stack& s ) 
{ 
    // Check for self assignment
    if (&s==this)
        return *this;

    // Clear the current stack
    removeAll();

    // Copy all data from stack s
    if (!s.isEmpty())
    { 
        NodePointer temp = q->theFront; 

        while(temp != 0) 
        { 
            push(temp->data); 
            temp = temp->next; 
        } 
    } 

    return *this; 
} 

这里是一个示例 removeAll 函数：

template <class S> 
void Stack<S>::removeAll()    
{ 
    while (s.theFront)
    {
        NodePointer p = s.theFront;

        s.theFront = s.theFront->next;
        delete p;
    }

    s.theTop = s.theFront;
}

Answer 2

不要为您的班级手动实现复制赋值运算符，而是使用the copy-and-swap idiom。

一旦你为你的类实现了swap() 函数（我在上面链接的那篇文章很好地描述了如何做到这一点），operator= 重载就变得简短而简单：

Stack& operator=(Stack rhs)
{
    swap(rhs);
    return *this;
}