试图模拟grep命令

时间:2013-05-31 16:44:06

标签: c linux

编译代码时出现分段错误。

如果有人可以帮助我,我的代码会很高兴我不会感到高兴。

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

int main(int argc,char *argv[])
{
FILE *fp;
char fline[100];
char *newline;
int i,count=0,occ=0;

fp=fopen(argv[1],"r");

while(fgets(fline,100,fp)!=NULL)
{
count++;
    if(newline=strchr(fline,'\n'))
        *newline='\0';
    if(strstr(fline,argv[2])!=NULL)
    {
        printf("%s %d %s",argv[1],count,fline);
    occ++;  
    }


}

printf("\n Occurence= %d",occ);

return 1;
}

1 个答案:

答案 0 :(得分:5)

请参阅man openman fopen

FILE *fp;
...
fp=open(argv[1],"r");

open返回一个整数,而不是文件指针。只需将该行更改为

即可
fp=fopen(argv[1],"r");

注意:对于那些想知道这是什么的人来说,OP在问题中的代码中编辑了这个错误

这导致我们(其他一些小问题也得到解决 - 见评论):

+ EDIT:指向应该进行错误检查的地方:

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

int main(int argc, char *argv[]) {
    FILE *fp;
    char fline[100];
    char *newline;
    int i, count = 0, occ = 0;

    // for starters, ensure that enough arguments were passed:
    if (argc < 3) {
      printf("Not enough command line parameters given!\n");
      return 3;
    } 

    fp = fopen(argv[1], "r");
    // fopen will return if something goes wrong.  In that case errno will
    // contain the error code describing the problem (could be used with
    // strerror to produce a user friendly error message
    if (fp == NULL) {
      printf("File could not be opened, found or whatever, errno is %d\n",errno);
      return 3;
    } 


    while (fgets(fline, 100, fp) != NULL) {
        count++;
        if (newline = strchr(fline, '\n'))
            *newline = '\0';
        if (strstr(fline, argv[2]) != NULL) {
            // you probably want each found line on a separate line,
            // so I added \n
            printf("%s %d %s\n", argv[1], count, fline);
            occ++;
        }
    }
    // it's good practice to end your last print in \n
    // that way at least your command prompt stars in the left column
    printf("\n Occurence= %d", occ);

    return 1;
}

ps:所以错误发生在运行时期间,而不是在编译时期间 - 这种区别非常重要,因为找不到编译器故障并解决库使用错误需要相当不同的技术...