标准输出缓冲问题

Question

这段代码我已经写好了

#include<stdio.h>
#include<assert.h>
#include<stdlib.h>

char *list[20],*story[100];

FILE*listfile;
FILE*infile;
FILE*outfile;

int check(char*string)
{
  int i=0;
  while(list[i]!=NULL)
    {
      if(strcmp(string,list[i])==0){return 1;};
      i++;
    };
  return 0;
};

void print_d(int d){ printf(" debug %d ",d);};

int main(int argc,char**argv){

  assert(argc==4);
  printf("hello \n");
  //assigning the file pointers in their respective modes
  printf("%s %s %s ",argv[1],argv[2],argv[3]);
  listfile=fopen(argv[1],"r");
  print_d(12);
  infile=fopen(argv[2],"r");
  outfile=fopen(argv[3],"w");
  print_d(0);
  int i=0; /* the infamous 'i' */

  while(1)
    {

      if(feof(listfile)!=0)
        { break;};
      list[i]=malloc(sizeof(char [15]));
      fscanf(listfile,"%s[^\n]",list[i]);
      i++;


    }
  i=0;
  print_d(1);
  while(1)
    {

      if(feof(infile)!=0)
        { break;};
      story[i]=malloc(sizeof(char [25]));
      fscanf(infile,"%s",story[i]);
      i++;

    }
  fclose(infile);
  fclose(listfile);
  i=0;

  print_d(2);
  while(1)
    {

      if(check(story[i])==1)
        { fprintf(outfile,"%s","censored");}
      else
        {
          fprintf(outfile,"%s",story[i]);
        };
    };
  print_d(3);
  fclose(outfile);
  i=0;
  while(list[i]!=NULL)
    { free(list[i]);};
  return 0;


}

以下问题随之而来

[1] 输出是一个 hello 后跟一个 seg 错误

这就是事情变得有趣的地方

如果我修改

printf("%s %s %s ",argv[1],argv[2],argv[3]);

到

printf("%s %s %s\n ",argv[1],argv[2],argv[3]);

输出是一个“你好”，后跟三个文件名，然后是一个段错误。

在用户 danfuzz 指出我应该将 print_d 调试更改为打印到 stderr 之后（我这样做了）..调试打印现在工作正常。所以我想一个更好的问题是为什么会首先发生这种情况和步骤防止此类事情发生？

对于经验丰富的程序员来说，这似乎是一个微不足道的问题，但请注意，早期版本（上面代码中的那个）未能在 seg fault 之前打印出任何消息，这使我得出结论，在命令行部分发生了一些事情东西/打开文件。

Answer 1

无需在论坛中猜测或询问，只需运行调试器即可。如果在 linux 上工作，gdb 会直接带您到分段错误的位置。

假设您的应用名为“foo”：

> gdb foo
# run

当它崩溃时：

# bt

Answer 2

一些观察，

您应该阅读有关 fflush(stdout) 的信息，因为它将帮助您进行调试语句，

void print_d(int d)
{
    printf(" debug %d ",d); fflush(stdout);
};

您分配给 char 指针数组 list[20] 和 story[100]，但您有循环（由臭名昭著的“i”索引），很容易离开列表或故事的末尾。

您尝试打开文件名 argv[2] 和 argv[3] 的文件，其中一个为读取，另一个为写入，将这些行更改为以下内容，

printf("%s %s %s ",argv[1],argv[2],argv[3]); fflush(stdout);

if( !(listfile=fopen(argv[1],"r")) )
{
    printf("cannot open %s\n",argv[1]); fflush(stdout);
    return(1);
}
print_d(1);
if( !(infile=fopen(argv[2],"r")) )
{
    printf("cannot open %s\n",argv[2]); fflush(stdout);
    return(2);
}
print_d(2);
if( !(outfh=fopen(argv[3],"w+")) ) //notice the "w+" to create missing file
{
    printf("cannot open %s\n",argv[3]); fflush(stdout);
    return(3);
}
print_d(3);

现在文件可以正常打开，因此将调试 print_d 参数更改为递增的数字顺序，这样您就可以按顺序发现哪个，并且由于您使用的是计数器，所以 for(;;) 循环有效，

int check(char*string)
{
    int i;
    for(i=0; list[i]!=NULL; i++)
    {
        if(strcmp(string,list[i])==0){return 1;};
    };
    return 0;
};

更改循环以成功读取两个文件，

for(i=0; i<20; ++i)
{
    if(feof(listfh)!=0) { break; };
    list[i]=malloc(sizeof(char [15]));
    fscanf(listfh,"%s[^\n]",list[i]);
}
fclose(listfh);
debug(4);

还有，

for(i=0; i<20; ++i)
{
    if(feof(infh)!=0) { break; };
    story[i]=malloc(sizeof(char [25]));
    fscanf(infh,"%s",story[i]);
}
fclose(infh);
debug(5);

现在对循环进行简单更改以扫描故事，检查审查（eek！），因此我们避免比较和打印空指针（您遇到的另一个问题），

for(i=0; i<100 && (story[i]); ++i)
{
    if(check(story[i])==1)
    {
        fprintf(outfh,"%s","censored"); fflush(outfh);
    }
    else
    {
        fprintf(outfh,"%s",story[i]); fflush(outfh);
    };
};

但是，请注意，您真的不需要将故事读入数组，您可以一次读取一行，然后打印结合这两个循环，您可以扫描任意大的文件，而无需分配大量空间，

for(i=0; 1; ++i)
{
    if(feof(infh)!=0) { break; };
    story[0]=malloc(sizeof(char [25]));
    fscanf(infh,"%s",story[0]);
    if(check(story[0])==1)
    {
        fprintf(outfh,"%s","censored"); fflush(outfh);
    }
    else
    {
        fprintf(outfh,"%s",story[0]); fflush(outfh);
    };
}
fclose(infh);
fclose(outfh);

您还需要确保只释放您分配的行，

for(i=0; list[i] && list<20; i++)
{
    free(list[i]);
}

这应该可以解决您的问题。

---

添加一个usage()函数，

void usage(char*progname)
{
    printf("need 3 files\n");
    printf("%s <restricted> <story> <censorted>\n",progname);
}

然后调用它，

if( argc < 4 )
{
    usage(argv[0]);
    return 0;
}