c言語を用いた文書検索に関する質問です。

質問

いつもお世話になっています。
c言語にあまり詳しくないので、どなたか詳しい方お願いします。

今回、ディレクトリの複数文書内で特定の単語（今回はクエリと呼びます）を含む文書を探し出すして、テキストファイル名を表示する処理を行うプログラムを作成していました。

以下に添付したソースは、コンパイル自体は通るのですがコマンドラインを
./ex12 "green" corpus/*.txt
と入力するとsegmentation faultとなってしまいます。
（ex12は実行ファイル名、greenに特に意味はありません。別の単語でも同じ結果が出てしまいます。）

複数ファイルではなく１つの文書内での検索は問題なかったのですが、何が原因なのかがよく分かりません。
１つの文書内検索の際は、メインループの最初のfor文を除いての処理となります。

printfを用いていろいろ試した結果、ファイル自体はきちんと読み込めているようなのでfor文を入れたのが原因ではないようなのですが・・・。

ちょっと原因が分からなくて詰まってしまったので、どなたか詳しいかたよろしくお願いします。

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

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

FILE* fp;
char buffer[2048];
char kueri[256];
char* s;
char* delimiter = " .,"; //単語の区切れの定義
int i;
int flag = 0;

/*コマンドラインが正しく入力されているかの確認*/
if(argc < 3){
printf("error1\n");
exit(1);
}

/*入力されたクエリをkueriに格納する*/
strcpy(kueri,argv[1]);

/*英小文字を英大文字に変換する処理を\0まで繰り返す*/
s = kueri;
while(*s != '\0'){
*s = toupper(*s);
s++;
}

/*メインループ*/
for(i = 2; i < argc; i++){

/*ファイルを開く*/
if((fp = fopen(argv[i], "r")) == NULL){
printf("error2");
exit(1);
}

while(fgets(buffer,sizeof(buffer),fp) != NULL){

/*最後に\0を格納する*/
buffer[strlen(buffer)-1] = '\0';

/*英小文字を英大文字に変換する処理を\0まで繰り返す*/
s = buffer;
while(*s != '\0'){
*s = toupper(*s);
s++;
}

/*strtokを用いて単語毎に区切っていく（１単語目）*/
s = strtok(buffer,delimiter);

if(strcmp(s,kueri) == 0){
printf("入力されたクエリを文書内に発見しました。\n");
printf("file name: %s\n",argv[i]);
flag = 1;
}else{

/*strtokを用いて単語毎に区切っていく（2単語目以降）*/
while((s = strtok(NULL, delimiter)) != NULL){

if(strcmp(s,kueri) == 0){
printf("入力されたクエリを文書内に発見しました。\n");
printf("file name: %s\n",argv[i]);
flag = 1;
}
}
}
}

fclose(fp);

}

if(flag == 0){
printf("クエリを文書内に発見できませんでした。\n");
}

exit(0);
}

回答No.1

tyatsumi · ベストアンサー

入力ファイル内に空行があると、
次の部分でSegmentation faultになるようです。
----
/*strtokを用いて単語毎に区切っていく（１単語目）*/
s = strtok(buffer,delimiter);

if(strcmp(s,kueri) == 0){
----
bufferが空行だとstrtokが返すsがNULLになるようなんですが、
そのせいでstrcmpがSegfaultになる、ということのようです。
strtokの2回目以降の呼び出しのように、sのNULLチェックを加えれがよいと思います。

ちなみに解析にはgdbというデバッガを使いました。
Segmentation faultが何処で起きたかを調べるにはgdbが便利です。
もしgccを使っているようなら、gdbの使いかたも調べてみてください。

お礼

ありがとうございます！
無事作成することができました。

gccを使っていたにで、gdbについても調べてみます。

本当にありがとうございました。