链表,C ++分段错误

时间:2012-10-02 08:47:43

标签: c++ list pointers linked-list

我一直无法完成作业,因为我似乎无法确定此分段错误的来源。

我正在尝试将节点添加到文件中的链接列表中。我已经运行了多次测试并将问题缩小了很多,但是,我不知道究竟是什么造成了问题,因此当我尝试更改其他细节时,我会产生新的问题。

这是我的第二道菜,所以,希望我的代码不是那么糟糕,以至于它无法帮助。 这是add方法:

    bool OrderedList::add (CustomerNode* newEntry)
{
if (newEntry != 0)
{
    CustomerNode * current;
    CustomerNode * previous = NULL;
    if(!head)
        head = newEntry;
    current = head;
  // initialize "current" & "previous" pointers for list traversal
   while(current && *newEntry < *current) // location not yet found (use short-circuit evaluation)
   {
    // move on to next location to check
    previous = current;
    current = current->getNext();
   }

  // insert node at found location (2 cases: at head or not at head)
  //if previous did not acquire a value, then the newEntry was
  //superior to the first in the list. 
  if(previous = NULL)
    head = newEntry;
  else
  {
    previous->setNext(newEntry); //Previous now needs to point to the newEntry
    newEntry->setNext(current); //and the newEntry points to the value stored in current.
  }
}
    return newEntry != 0;  // success or failure
    }

好的,有一个重载的运算符&lt;包含在程序中,外部测试并不表示操作员有问题,但我也会将其包含在内:

    bool CustomerNode::operator< (const CustomerNode& op2) const
    {
       bool result = true;
       //Variable to carry & return result
       //Initialize to true, and then:
       if (strcmp(op2.lastName, lastName))
        result = false;

        return result;
       }

这是来自gdb的回溯:

    #0  0x00401647 in CustomerNode::setNext(CustomerNode*) ()
    #1  0x00401860 in OrderedList::add(CustomerNode*) ()
    #2  0x004012b9 in _fu3___ZSt4cout ()
    #3  0x61007535 in _cygwin_exit_return () from /usr/bin/cygwin1.dll
    #4  0x00000001 in ?? ()
    #5  0x800280e8 in ?? ()
    #6  0x00000000 in ?? ()

这是尝试纠正不同段错误的大量工作的结果,而这一点更令人惊讶。我不知道我的setNext方法是如何导致问题的,这里是:

void CustomerNode::setNext (CustomerNode* newNext)
{
    //set next to newNext being passed
    next = newNext;
    return;
}

在此先感谢,如果有必要确定此问题,我将很乐意发布更多代码。

3 个答案:

答案 0 :(得分:4)

这是

if(previous = NULL)

而不是

if(previous == NULL)

这会将previous设置为NULL,然后进入else分支:

previous->setNext(newEntry); //Previous now needs to point to the newEntry
newEntry->setNext(current);

导致未定义的行为。

答案 1 :(得分:1)

if(previous = NULL)

似乎有点可疑,因为它总是评估为false

您可以通过两种主要方式避免此类错误:

  • const慷慨,几乎到处都可以撒上

  • 与值比较时,请将该值放在左侧。

,例如,写

if( NULL = previous )

并获得编译错误,而不是崩溃或不正确的结果。

就我个人而言,我没有做左边的价值,因为我从来没有遇到过这个问题。我怀疑部分是因为我对const非常慷慨。但作为初学者,我认为这是一个好主意。

答案 2 :(得分:0)

您可以发布所有代码,但我能看到的第一个明显问题是:

if(previous = NULL)

当你的意思是==时,使用C / C ++ / Java是一个非常非常常见的错误。