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 fault

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

如果我修改

printf（“％s％s％s”，argv [1]，argv [2]，argv [3]）;

到

printf（“％s％s％s \ n”，argv [1]，argv [2]，argv [3]）;

输出是'hello'，后跟文件的三个名称，然后是段错误。

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

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

Answer 1

一些观察，

您应该阅读fflush(stdout)，因为它可以帮助您完成调试语句，

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

你分配给char指针数组list[20]和story[100]，但你有循环（由臭名昭着的＆＃39; i＆＃39;索引），这可能很容易走出尽头列表或故事。

您尝试打开文件名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;
}

Answer 2

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

让我们说你的app被称为“foo”：

> gdb foo
# run

当它崩溃时：

# bt

stdout缓冲问题

2 个答案: