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

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 ./a.out 从文件中读取内容。数据。输入文件的内容：

1line
2line
3line
4line

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

---

提示：（Courtesy Alok）

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

所以，使用 1相反:-)

看看这个post 使用 fread 的缓冲 IO 示例。

Answer 1

'1' 的类型在 C 中是 int，而不是 char，因此您正在阅读 SIZE*sizeof(int)fread 中的 code> 字节。如果 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'}< /code> 和 {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 字符串。