该程序中的分段错误是什么

Question

 #include<stdio.h>
 #include <stdlib.h>
 int main()
 {
         int n,small=0,large=0,s,l,temp;
         printf("this should work");
         scanf("%d",&n);
          //   printf("%d",n);//
         int a[n];
        for(int i=0;i<n;i++)
             {
              scanf("%d",&a[i]);
              }
             /*    for(int i=0;i<n;i++)
                          printf("%d",a[i]);*/
        small=a[0];
        large=a[n-1];
        for(int i=0;i<n;i++)
        {
               if(a[i]<small && i!=0)
               {  
                       small=a[i];
                       s=i;
                }
                if(a[i]>large && i!=n-1)
                { 
                        large=a[i];
                        l=i;
                 }
        }
        temp=a[s];
        a[s]=a[l];
        a[l]=a[s];
        for(int i=0;i<n;i++)
              printf("%d ",a[i]);
         return 0;
  }

这是一个交换数组中最大和最小数字并打印新数组的简单程序。 当我试图运行这个程序时，我遇到了分段错误。 通常，当我们尝试访问超出范围的内存位置时会发生分段错误。 所以我添加了 printf 语句来找出错误在哪里。 但是没有执行任何打印语句。这里有什么错误？

Answer 1

一个问题是你实际上并没有将s 和l 设置为任何东西，除非你找到一个小于/大于当前元素的元素。

这意味着（例如），如果第一个元素是最小的，s 将被设置为任意值，并且尝试使用它来索引数组可能会出现问题。

要解决这个问题，在哪里设置small 和large，您还应该设置：

s = 0;
l = n - 1;

另外，你的交换码是错误的，应该是：

temp = a[s];
a[s] = a[l];
a[l] = temp;

Answer 2

您应该将s 和l 初始化为某个值，因为当if 条件不起作用时，它们的值将保持未初始化的垃圾值。因此，a[l] 或 a[s] 将不起作用，因为这些索引是未定义的值。这就是为什么您会得到segmentation fault 的原因，因为您正在访问数组的未定义区域。

因此，请使用 s=0,l=0 等数组范围内的随机值来初始化变量，或者您可以添加一些标志来检查条件是否有效。

if (l != 0 && s != 0) {
    temp=a[s];
    a[s]=a[l];
    a[l]=a[s];
}

另外，我认为您在最后一行 a[l]=temp 而不是 a[l]=a[s] 中交换了值。

ideone link

Answer 3

你不能声明一个基于动态大小的数组，除非编译器支持它，即使那样它通常也是不可移植的。

int a[n]

你实际上需要使用 malloc 或 calloc。

int *a;

a = (int *)malloc(n * sizeof(int));或 a = (int *)calloc(n, sizeof(int));