链接列表中的运算符重载

时间:2013-02-14 04:33:53

标签: c++ linked-list

我从其他帖子搬到这里。但这次我能够获得某种类型的输出。但我似乎无法遍历我的节点并让它们单独打印出来,看它在输出上的外观。这是我到目前为止所拥有的内容以及该程序应该输出的截图。

enter image description here

LLIST.H

#ifndef LList_h
#define LList_h

#include <iostream>
#include "node.h"

class LList
{
    public:
        LList(void);            //constructor
        LList(const LList &);   //copy constructor
        ~LList();               //destructor
        LList *next;            //points to next node
        void push_back(const string &str);
        void push_front(const string &str);
        friend ostream& operator<<(ostream& out, const LList& llist);
        LList &operator=(const LList &l);       

    private:
        Node *_head;
        Node *_tail;
        LList *front;       //points to front of the list
};

inline LList::LList(void)
{
    cerr << "head = tail = 0 at 0024f8d0\n";

    _head = 0;
    _tail = 0;
    front = 0;
}

inline void LList::push_back(const string &str)
{
    Node *p = new Node(str);
    if (_tail == 0)
    {
        _head = _tail = p;
    }
    else
    {
        _tail ->next(p);
        _tail = p;
    }

    if (_head == 0)
    {
        _head = _tail = p;
    }
    else
    {
        _head ->next(p);
        _head = p;
    }
}

inline void LList::push_front(const string &str)
{
    Node *p = new Node(str);
    if (_tail == 0)
    {
        _head = _tail = p;
    }
    else
    {
        _tail ->next(p);
        _tail = p;
    }

    if (_head == 0)
    {
        _head = _tail = p;
    }
    else
    {
        _head ->next(p);
        _head = p;
    }
}

LList & LList::operator=(const LList &l)
{
    _head = l._head;
    _tail = l._tail;
    front = l.front;
    return *this;
}

inline LList::~LList()
{
}


#endif

maind.cpp

#include "LList.h"
#include <iostream>

using namespace std;

ostream& operator<<(ostream& out, const LList& llist);

int main( )
{
    LList a;

    a.push_back(  "30" );
    a.push_front( "20" );
    a.push_back(  "40" );
    a.push_front( "10" );
    a.push_back(  "50" );

    cout << "list a:\n" << a << '\n';
    return 0;
}

ostream &operator <<( ostream &out, const LList & llist )
{
    for( LList *p = llist.front; p != 0; p = p -> next )
        out << p -> next;

    return out;
}

3 个答案:

答案 0 :(得分:1)

out << p -> next;

此行将跳过您的第一个元素,并在您的最后一个元素上导致未定义的行为(可能是段错误)。这应该是out<<p

答案 1 :(得分:1)

您的operator<<将不会打印任何内容,因为永远不会分配LList::front。它总是空的。

答案 2 :(得分:1)

你的推送算法毫无意义。要在列表后面推送一些内容,如果列表为空,您只需要修改head,但您有:

if (_head == 0)
{
    _head = _tail = p;
}
else
{
    _head ->next(p);
    _head = p;
}

如果列表中已有条目,为什么要将_head设置为p?你的代码有许多类似的错误 - 逻辑不正确。

结局应该只是:

if (_head == 0)
    _head = p;

如果头部已有节点,则在后面添加一个条目根本不会影响头部。