C语言fgets读取文件时i++计数失效但printf输出正常问题

• 核心原因是栈缓冲区溢出，fgets的使用不符合API要求导致相邻变量i的内存被意外篡改，具体逻辑如下：
  1. fgets是字符串读取函数，调用时指定的第二个参数是缓冲区总长度，它会自动在读取到的内容末尾追加一个\0作为字符串结束符。你传入的缓冲区是单个char变量a的地址，指定长度为2，意味着每次读取成功后，除了把1个有效字符写入a的内存，还会向&a + 1的地址写入1字节的\0。
  2. 局部变量a和i在栈内存中是相邻分配的，在常见的小端序架构下，&a + 1刚好对应i变量的最低有效字节。每次fgets执行成功都会把这个字节清零：每次循环中i++将i加1后，下一次调用fgets就会把i的最低位刷成0，相当于i每次都被重置回0。
  3. 最后一次fgets读取到文件末尾返回NULL，不会执行写\0的操作，所以最后一次循环执行的i++结果1被保留，最终输出的i值就是1。
  4. printf("%c", a)输出正常是因为a本身的内存写入是正确的，溢出修改的是i的内存，所以打印内容符合预期。

如果要按字符读取文件，不要用fgets，改用fgetc即可，修改后代码如下：

int getLevelWidth(FILE *level){
  int a; // fgetc返回int类型，需要存储EOF标记所以不能用char
  int i = 0;
  while((a = fgetc(level)) != EOF) {
    printf("%c",a);
    i++;
  }
  printf("%i", i);
  return 0;
}

如果一定要用fgets，需要给它分配足够的缓冲区，缓冲区长度至少要比指定的读取长度大1，避免溢出。

内容的提问来源于stack exchange，提问作者Marius-Daniel Petrut