我是否正确使用数组？ 如果是这样，为什么我会收到核心转储错误？

Question

我不明白为什么会收到此错误。 我是数组的新手，所以我可以理解

tand that I might have made a mistake somewhere.

make -s
 ./main
Enter number of students: 1

For student 1,
Enter Surname: test
Enter Mark for MOD100: 1
Enter Mark for MOD101: 2
Enter Mark for MOD102: 3
Enter Mark for MOD103: 4
Enter Mark for MOD104: 5
Enter Mark for MOD105: 6
Student Name    MOD100  MOD101  MOD102  MOD103MOD104  MOD105  Overall
signal: segmentation fault (core dumped)


据我所知，我的代码是正确的，第二部分是一个 WIP。

这是我的代码：

`

#include <stdio.h>

int main() {
  int numofstudents = 0;
  int i;
  int j;
  int sum;

  char names[numofstudents][1];
  int modules[numofstudents][6];
  
  printf("Enter number of students: ");
  scanf("%i", &numofstudents);


  for (i = 0; i < numofstudents; ++i) 
  
  {
    
    printf("\nFor student %d,\n", i + 1);
    printf("Enter Surname: ");
    scanf("%s", &names[i]);
    printf("Enter Mark for MOD100: ");
    scanf("%i", &modules[1][i]);
    printf("Enter Mark for MOD101: ");
    scanf("%i", &modules[2][i]);
    printf("Enter Mark for MOD102: ");
    scanf("%i", &modules[3][i]);
    printf("Enter Mark for MOD103: ");
    scanf("%i", &modules[4][i]);
    printf("Enter Mark for MOD104: ");
    scanf("%i", &modules[5][i]);
    printf("Enter Mark for MOD105: ");
    scanf("%i", &modules[6][i]);
  }

  
  printf("Student\t");
  printf("Name\t");
  printf("MOD100\t");
  printf("MOD101\t");
  printf("MOD102\t");
  printf("MOD103\t");
  printf("MOD104\t");
  printf("MOD105\t");
  printf("Overall\n");
  
  for (i = 0; i < numofstudents; ++i)
  {

    printf("%4d\t", i + 1);
    printf("%4s\t", names[i][1]);
    printf("%4i\%\t", modules[1][i]);
    printf("%4i\%\t", modules[2][i]);
    printf("%4i\%\t", modules[3][i]);
    printf("%4i\%\t", modules[4][i]);
    printf("%4i\%\t", modules[5][i]);
    printf("%4i\%\t", modules[6][i]);

  }




  return 0;
}

`

我尝试以不同的方式输入接收到的值，但每次都会出现此错误。

Answer 1

您的数组没有任何元素。 当您尝试访问此类数组时，您总是会在边界之外访问它。 这是一个未定义的行为。

扫描用户输入后，您需要定义数组。

int main() {
  int numofstudents = 0;
  int i;
  int j;
  int sum;

  
  printf("Enter number of students: ");
  if(scanf("%i", &numofstudents) != 1) {/* error handling*/}

  char names[numofstudents][1];
  int modules[numofstudents][6];

您还需要对数组进行索引，即： scanf("%i", &modules[i][1]);

另一个问题是索引是从零开始的，而不是从一个开始，你需要从scanf("%i", &modules[i][0]);开始。 并完成scanf("%i", &modules[i][5]);

Answer 2

0_________ 是正确的，您的索引是错误的。 我只是想指出您正在创建一个可变长度数组 (VLA)。 int numofstudents = something; 其次是char names[numofstudents][1]; 正在创建一个 VLA，因为numofstudents的值在编译时是未知的。 为避免这种情况，只需制作一个常量大小的数组：

#define STUDENTS_MAX 100

int numofstudents = 0;
char names[STUDENTS_MAX][1];

scanf("%i", &numofstudents);

// Now, don't allow the user to enter anything over STUDENTS_MAX
if (numofstudents > STUDENTS_MAX) {
    numofstudents = STUDENTS_MAX;
}

这也将解决您写越界的问题。

为避免意外创建 VLA，您可以在 gcc 或 clang 中使用-Werror=vla作为构建选项。 不确定 MSVC 是否有这样的选项。