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

Question

我在这里遇到了一个奇怪的错误：882 Segmentation fault ./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

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

您的第一步应该是在 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;。

Answer 2

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

Answer 3

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

1. 在你的条目中类

   operator T* () {return &stud;}
   

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

1. 在你的llist 班级中

   if(temp==temp->towardsback){
           delete this;
   

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

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

Answer 4

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