为什么此指针/整数比较会生成分段错误

Why does this pointer/integer comparison generate a segmentation fault?

本文关键字：分段 错误 比较 指针 整数 为什么      更新时间：2023-10-16

我正在开发一个程序，该程序会生成分段错误，我无法理解原因。如果我从"最低"answers"最大"变量中删除指针声明，并将它们用作无指针整数，则程序运行良好。

然而，当我尝试使用指针时，问题就出现了，并且我收到了分段错误。我意识到这可能是一个非常简单且解决得很好的问题，但我试图通过查看其他类似的问题来理解代码。我还没有找到解决我问题的办法。我也不明白出了什么问题。

这是产生问题的代码（下面是完整来源的链接）：

     cout << "This is the array containing the random numbers:n";
 for(int *i=numbers; i != numbers + arrLength; i++) {
     if((*i % 200) == 0 && *i > 200) {
         cin.get();
         cout << endl;
     }
     else
         cout << *i << ' ';
     // Get statistics
     // In the continuation of getting, lowest, largest then adding to sum.
     // THIS PART IS MAKING SEGMENTATION FAULT.
     if(*i < *lowest)
         lowest =  i;
     if(*i > *largest)
         largest = i;
     sum += *i;
 }

i变量指向用户输入后声明的旧引用：

cout << "You entered: " << arrLength << "nn";
 int *numbers = new int[arrLength];
 // Fill the array with random numbers
 srand(time(NULL));
 int x;
 int range = 5001;
 for(int index=0; index<arrLength; index++){
     *(numbers + index) = rand() % range;
     x = rand() % 2;
     if(x > 0) {
         *(numbers + index) = *(numbers + index) * -1;
     }
 }

请解释为什么我的程序不起作用，以及我做错了什么。正如我之前所说的，除了之外，其他一切都有效

if(*i < *lowest)
         lowest =  i;
     if(*i > *largest)
         largest = i;

完整来源：http://pastie.org/2105963

提前感谢！

int *largest = 0, *lowest = 0, sum = 0;

这是你的问题。您永远不会为它们分配内存或分配有效地址。当你取消引用一个NULL指针时，比如这里的

if(*i < *lowest) //lowest is NULL

您得到了Undefined Behavior，其中包括分段错误、鼻恶魔和其他任何

它不起作用，因为它不比较指针。您正在比较存储在某些内存地址上的值，这些地址很可能不属于您的程序，因此您会被SIGSEGV击落。

您无法访问不属于程序的内存。

在取消引用lowest和largest之前，它们是否已初始化？如果最低地址未指向有效地址，则使用*lowest将导致seg故障。

您可以尝试在开始时的某个位置设置lowest = largest = numbers;。也许这会对你有所帮助。

您正在初始化最低的&最大值指针指向0，然后稍后取消对它们的引用，而不将它们指向有效值。

int *largest = 0, *lowest = 0, sum = 0;

您的编程使用了无效内存（最小和最大指针）。但是

此代码中不需要指针

事实上，这个程序有一种不需要的复杂性。既然可以使用索引，为什么要使用指针？

您可能会发现您的分配/初始化错误。

 for(int index=0; index<arrLength; index++){
     numbers[index] = rand() % range;
     x = rand() % 2;
     if(x > 0) {
         numbers[index] = numbers[index] * -1;
     }
 }

您只需要一个指针即可创建数组。

int *numbers = new int[arrayLength];

最后你的程序会更简单：

int lowest =numbers[0], largest = lowest;
 for(int i=0; i < arrLength; i++) {
     int ii = numbers[i ];                      )
     if((ii % 200) == 0 && ii > 200) {
         cin.get();
         cout << endl;
     }
     else
         cout << ii << ' ';
     if(ii < lowest)
         lowest =  ii;
     if(ii > largest)
         largest = ii;
     sum += ii;
 }

您正在将最低值和最大值初始化为0，并且解引用通过空指针访问内存，因此它失败了。试试这个：

if (!lowest || *i < *lowest)
    lowest =  i;
if (!largest || *i > *largest)
    largest = i;

或：

int *numbers = new int[arrLength];
lowest = largers = numbers; // initialise to something non-null
相关文章：