在 C 中以某种格式从文件中读取会产生段错误

Question

目前，我正在尝试从代表数独谜题解决方案的文件中读取数字。该文件应该被格式化为 9 个数字，中间没有空格，后跟一个换行符，在第 80 个数字之后结束。

在阅读时，我正在使用 fscanf，我可以让它与一个有效的谜题一起工作，但不能与一个不是有效格式的谜题一起工作；那些会产生分段错误。我试图检查数组的长度，但是由于在我声明 s 时它设置为 10，所以它不会捕获太短或太长的字符串。我会以错误的方式解决这个问题吗？我将在下面发布代码。

FILE* puzzlefile;
char s[10];
int i=0, j=0, skip;
int form;

//opens the file so we can access the puzzle
puzzlefile = fopen(argv[1], "r");

for (i=0; i<9; i++){
    form = fscanf(puzzlefile, "%s\n",s);
    for (j=0; j<10; j++){
        if (form == 0){
            printf("Invalid format on input. Bad newline.\n");
            return 1;
        }
        if(j<9){
            if(s[j] == '1' || s[j] == '2' || s[j] == '3' || s[j] == '4' || s[j] == '5' || s[j] == '6' || s[j] == '7' || s[j] == '8' || s[j] == '9'){
                puzzle[i][j] = (int)(s[j]-'0');
            }
            else{
                printf("Invalid format on input.\n");
                return 1;
            }
        }
    }
}

我是不是走错了路？如果是这样，我只是在滥用 fscanf() 吗？或者我应该使用其他类型的输入函数？我正在测试的文件包括一些遵循格式但使用字母而不是数字的文件，一个在第一行带有额外数字的拼图，一个每 5 个数字有换行符的拼图，以及一个全是数字而没有换行符的拼图。所有这些都会产生段错误。

Answer 1

1. NULL不检查puzzlefile，要确保文件已经打开，万一不是fopen()返回NULL，所以检查@987654325的返回值@ 是一件非常重要的事情。

2. 您似乎没有检查argv[1] != NULL，如果您忘记将参数传递给程序会发生这种情况，后果将是程序出现故障，您将不知道原因。

3. s的大小太小，容易溢出，让fscanf()知道目标字符串还有多少空间，像这样

   form = fscanf(puzzlefile, "%9s\n", s);
   /*                          ^ 9 characters + '\0' = 10 */
   

4. 为了防止读取超出字符串的末尾，请执行此操作

   for (i = 0 ; s[i] != '\0' ; ++i)
   

   而不是

   for (i=0; i<10; i++)
   

   和内循环一样。

5. 如果你这样做会更好

   if ((s[j] > '0') && (s[j] < 1 + '9'))
   

   而不是

   if(s[j] == '1' || s[j] == '2' || s[j] == '3' || s[j] == '4' || s[j] == '5' || s[j] == '6' || s[j] == '7' || s[j] == '8' || s[j] == '9')
   

---

你没有发布puzzle的声明，如果你在它结束后写，也可能是问题，你应该非常小心c中的绑定检查，因为写超过缓冲区的末尾，不会有一个明确的行为，在某些情况下，当你测试它时不会发生任何不好的事情，当然，这个 bug 会一直静静地呆在那里，直到它爆炸。