Question

程序：

#include<stdio.h>
#include<string.h>

char *f_gets(char *s, int n, FILE *iop)
{
    int c=0;
    char *cs;
    cs = s;

    while (--n > 0 && (c = getc(iop)) != EOF)
    {
        if ((*cs++ = c) == '\n')
            break;
    }
    *cs = '\0';
    return (c == EOF && cs == s) ? NULL : s;
}


main(int argc, char *argv[])
{
    FILE *fp1,*fp2;
    char s2[100],s1[100];
    if (argc <= 2 )
        printf("2 argument needed \n"); 
    else
        if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)
            printf("cat: can't open The file\n");
        else
        {
            while(1)
            {
                f_gets(s1,100,fp1); // 1st iteration
                f_gets(s2,100,fp2); // 2nd iteration
                if(!strcmp(s1,s2)) 
                    printf("%s %s",s1,s2);
            }
            fclose(fp1);
            fclose(fp2);
        }
}

输出：

$ ./a.out a b
Segmentation fault (core dumped)
$

在上面的程序中，当我们第二次调用f_gets时会发生段错误。即使我两次检查程序很难找到问题。有没有人解释为什么它会产生问题。

Answer 1

您拨打电话时未打开第二个档案。

问题是你从短路路径呼叫fopen：

if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)

由于代码中存在错误，当fp1打开正常时，fp2将始终保持关闭状态。这是因为(fp1=fopen(argv[1],"r"))== NULL将评估为0，并确保永远不会调用(fp2=fopen(argv[2],"r"))==NULL。

您可以将&&替换为||来解决此问题，但更好的方法是一次打开一个文件。

Answer 2

if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)

表示，＆＃34;如果程序未能同时打开fp1和fp2＆＃34;。您可以使用||来解决此问题。但最好不要编写长而复杂的表达式，因为它们只会增加写错误的机会。

相反，这样做：

fp1=fopen(argv[1],"r");
if(fp1 == NULL)
{
  // error handling
}

fp2=fopen(argv[2],"r");
if(fp2 == NULL)
{
  fclose(fp1);
  // error handling
}

作为奖励：仅仅因为我们将程序重新编写成可读的东西，我们现在也发现了第二个错误。如果程序打开fp1但无法打开fp2，它在中止之前从未关闭fp1。

getc函数产生段错误

2 个答案: