我一直无法完成作业,因为我似乎无法确定此分段错误的来源。
我正在尝试将节点添加到文件中的链接列表中。我已经运行了多次测试并将问题缩小了很多,但是,我不知道究竟是什么造成了问题,因此当我尝试更改其他细节时,我会产生新的问题。
这是我的第二道菜,所以,希望我的代码不是那么糟糕,以至于它无法帮助。 这是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;
}
在此先感谢,如果有必要确定此问题,我将很乐意发布更多代码。
答案 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是一个非常非常常见的错误。