0

我从另一个职位搬到这里。但是这次我能够获得某种类型的输出。但我似乎无法遍历我的节点并让它们单独打印出来,就像它在输出中的外观一样。这是我到目前为止所拥有的,也是程序应该输出的屏幕截图。

在此处输入图像描述

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;
}
4

3 回答 3

1
out << p -> next;

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

于 2013-02-14T04:44:06.917 回答
1

operator<<将不会打印任何内容,因为LList::front从未分配给。它始终为空。

于 2013-02-14T04:46:04.353 回答
1

您的推送算法毫无意义。要将某些内容推到列表的后面,您只想head在列表为空时进行修改,但您有:

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

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

结局大概应该是:

if (_head == 0)
    _head = p;

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

于 2013-02-14T04:50:10.800 回答