1

我已经写了这段代码

#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 之前打印出任何消息，这让我得出结论，事情的命令行部分发生了一些事情/打开的文件。

4

2 回答 2

1

一些观察，

您应该阅读关于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;
}
于 2013-10-19T01:38:21.893 回答
0

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

假设您的应用程序被称为“foo”：

> gdb foo
# run

当它崩溃时：

# bt
于 2013-10-18T23:44:03.157 回答