这个分段错误的根源是什么？

Question

我这里有一个奇怪的错误： 882 Segmentation failure ./a.out

代码段：

int end=array.Length, loop=0;
cout<<end<<" about to print";
for(;loop<end;loop++){
    cout<<"\nMr. loop says: ";
    array.get(loop).print(fout);
}

我的程序的输出：

enpty initializer called for llist
enpty initializer called for entry
adding
adding
999deleting999
done deleting
123deleting123
done deleting
333deleting333
done deleting
printing
2 about to print
./g+: line 7:   882 Segmentation fault      ./a.out

和重要的输出：

printing
2 about to print
./g+: line 7:   882 Segmentation fault      ./a.out

意味着错误是这一行：

for(;loop<end;loop++){

这已经证明良好的价值观，并且在语法上是正确的（是的，我知道它的风格很糟糕）。

有想法吗？我的大学里似乎没有人能够帮助我解决这个问题。

以下是文件：

• core.cpp
• entry.h
• in.data
• llist.h
• mmm.h
• student.h

Answer 1

这很可能是因为你管理列表的方式。您正在添加/删除动态分配的对象，并通过引用传递它们。这是一个坏主意。

Answer 2

好的，我粗略地浏览了一下您的代码，发现了几个问题。

1. 在您的条目中<>类

   operator T* () {return &stud;}
   

stud 已经是一个 T*。您现在返回该指针的地址，而不是指针本身。

1. 在你的llist类中

   if(temp==temp->向后){
           删除这个；
   

如果此条件为真，则实际上会删除您的列表类。哎哟？

就我个人而言，我认为你需要重新开始你的列表实现:)

Answer 3

我主要使用 C#，但数组在 C++ 中的索引是否为 0？因此，您需要将 end 设置为 array.Length-1。至少在 C# 中，数组中的第一个元素是 array[0]。

Answer 4

除非您正在执行低级内存操作（并且您没有在发布的程序中执行此操作），否则 分段错误是内存损坏的标志，即您搞乱了一些编码。请注意，内存损坏通常发生在错误触发之前。在极端情况下，初始内存损坏和实际分段错误可能相隔数小时和模块。

第一步应该是在 valgrind 或 gdb 中运行程序以找出分段错误的详细信息。另外，始终至少使用 gcc -Wall 进行编译并记下每个警告 - 除非您正在修改编译器，否则每个警告的更改都表明存在错误。

就您而言，错误几乎肯定是在 llist 的实现中。该实现存在许多问题：

• 实现不属于 .h 文件，它们应该位于 .cpp 文件中。
• entry.towardsback 和 entry.towardsfront 通常称为 prev(ious) 和 next - 这些名称更短，更容易区分。
• 根据链表的一般命名约定，hold 可能应该称为 head。
• 长度为 0 的列表不应包含任何条目，即使用 hold == NULL 进行初始化。
• 从第 61 行开始（在 add 中）：

hold->towardsback = node;
hold->towardsback->towardshead = node; // node->towardshead = node;

第二行几乎肯定是错误的。您可能想先配置node，然后设置hold->towardsback = node;。