为什么这段代码没有输出预期的输出？

Question

这对于查找错误来说是一个很好的问题。 不？至少对初学者来说还可以。

#define SIZE 4
int main(void){
  int chars_read = 1;
  char buffer[SIZE + 1] = {0};  
  setvbuf(stdin, (char *)NULL, _IOFBF, sizeof(buffer)-1);  
  while(chars_read){
    chars_read = fread(buffer, sizeof('1'), SIZE, stdin);
    printf("%d, %s\n", chars_read, buffer);
  }
  return 0;
}

使用上面的代码，我正在尝试使用重定向./a.out < data 从文件中读取。输入文件内容：

1line
2line
3line
4line

但我没有得到预期的输出，而是混入了一些图形字符。 怎么了？

---

提示：（由 Alok 提供）

• sizeof('1') == sizeof(int)
• sizeof("1") == sizeof(char)*2

所以，用 1 代替 :-)

查看this post 使用 fread 的缓冲 IO 示例。

Answer 1

'1' 的类型在 C 中是 int，而不是 char，因此您正在读取每个 fread 中的 SIZE*sizeof(int) 字节。如果 sizeof(int) 大于 1（在大多数现代计算机上是），那么您正在读取 buffer 的存储空间。这是 C 和 C++ 不同的地方之一：在 C 中，字符文字的类型为 int，在 C++ 中，它们的类型为 char。

所以，您需要chars_read = fread(buffer, 1, SIZE, stdin);，因为sizeof(char) 根据定义为1。

事实上，我会把你的循环写成：

while ((chars_read = fread(buffer, 1, sizeof buffer - 1)) > 0) {
    buffer[chars_read] = 0; /* In case chars_read != sizeof buffer - 1.
                               You may want to do other things in this case,
                               such as check for errors using ferror. */
    printf("%d, %s\n", chars_read, buffer);
}

回答你的另一个问题，'\0' 是int 0，所以{'\0'} 和{0} 是等价的。

对于setvbuf，我的文档说：

size 参数可以设为零，以像往常一样获得延迟的最佳大小缓冲区分配。

你为什么评论\\而不是//或/* */？ :-)

编辑：根据您对问题的编辑，sizeof("1") 是错误的，sizeof(char) 是正确的。

sizeof("1") 是 2，因为 "1" 是一个包含两个元素的 char 数组：'1' 和 0。

Answer 2

这是一种使用重定向逐字节读取文件行的​​方法 ./a.out

至少产生预期的输出... :-)

/*

Why does this code not output the expected output ?,
http://stackoverflow.com/questions/2378264/why-does-this-code-not-output-the-expected-output

compile with:
gcc -Wall -O3 fread-test.c

create data:
echo $'1line\n2line\n3line\n4line' > data

./a.out < data

*/

#include <stdio.h>

#define SIZE 5

int main(void) 
{

   int i=0, countNL=0;
   char singlechar = 0;
   char linebuf[SIZE + 1] = {0};
   setvbuf(stdin, (char *)NULL, _IOFBF, sizeof(linebuf)-1);  

   while(fread(&singlechar, 1, 1, stdin))     // fread stdin byte-by-byte
   {
      if ( (singlechar == '\n') )
      {
         countNL++;
         linebuf[i] = '\0';
         printf("%d:  %s\n", countNL, linebuf);
         i = 0;
      } else {
         linebuf[i] = singlechar; 
         i++;
      }
   }

   if ( i > 0 )    // if the last line was not terminated by '\n' ...
   {
      countNL++;
      linebuf[i] = '\0';
      printf("%d:  %s\n", countNL, linebuf);
   }

 return 0;

}

Answer 3

字符缓冲区[SIZE + 1] = {0};

这不是你所期望的，它使缓冲区指向程序常量数据段中的一个字节区域。即这将损坏 SIZE 字节数，并可能导致内存保护错误。始终使用 strcpy() 或等效方法初始化 C 字符串。