EOFがうまくいきません

Question

標準入力から入力した文字とファイルから読み込んだ文字を比較してCtrl＋Dを入力したらループを抜け出せるものを作っていたのですが、このコードだと抜け出せるものの実行結果が次のようになってしまいます。自分はこれでいけると思っていたので何を直せばいいのかわかりません。
実行結果
Input a word: zoo
The words zoo is found.
Input a word: daf
The words daf is not found.
Input a word: The words daf is not found.
Input a word: The words daf is not found.
Input a word: The words daf is not found.
Input a word: The words daf is not found.
Input a word: The words daf is not found.
・
・
・
・
このまま最後の文がかなり長く続いて元のコマンド入力待ちの状態にもどります。なぜ最後の行が何十行も続くのでしょうか、fgetsでNを100に定義しているのと関係ありそうな気はしているのですが何を直せばいいのかわかりません。

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define N 100

int main(){
    int i;
    char input[N],buf[N],*in,*bu,c;
    FILE *fp;

//printf("%s\n",input);
    if((fp=fopen("words","r"))==NULL){//読み込んだ値をfpに代入
        exit(1);
    }
//文字列を一つずつコピーしていき、比較する
i=0;
    while(1){
        fgets(buf,N,fp); 
        strtok(buf,"\n");
        if(buf[i]==EOF）break;
        bu=buf;
        //printf("%s\n",bu);

        printf("Input a word: ");
        fgets(input,N,stdin);
        strtok(input,"\n");
        in=input;

     if(strcmp(in,bu)==0){
     printf("The words %s is found.\n",input);
     }
     else{
      printf("The words %s is not found.\n",input);
     }
     i++;
    }
    fclose(fp);
    return 0;
}

Answer 1

まず、 katoyさんの書かれたように、 fgets()関数の使い方確認をされる事を勧めます。

strtok(buf,"\n");

こちらについても目的が分かりませんが、行の分割でしょうか? もし、そうならば、fgets()が行単位で読み込む事が理解されていないと思います。

if(buf[i]==EOF）break;

こちらは、ファイルの最後まで読み込んだ事の確認と思われますが、読み込んだ文字列中に EOF(-1)が来る事はあり得ません。(バイナリだったら、、はありますが)
fgets()の読込みが失敗した場合、buffの値は保証されませんが、大体は、そのままの事が多いと思われます。 ---> 最後の行が連続して出力される理由。

EOF検出には、feof関数もあり、そちらでの確認もありますが、この関数は、EOF状態で無いと、検出しないという問題があり、ちょっと使いづらい。 fgets()の後で、feof() して fgets()がEOFで無い事の確認には使えるでしょうか。

Comments

[Teemro_431265]

strtok(buf,"\n");を入れた理由としてはfgetsを使った際%sの後になぜか改行が入ってしますので調べて見たらこれが出て来たので使っています。これを入れると改行がなくなるようです。

[pepperleaf]

あ、行末の改行コードでしたか。ちょっと見落としていました。

Answer 2

まずは、fgets を使って、ファイルを読み込み EOF を認識して、ファイルを閉じるコードをかいてください。
それができてから、 文字列を一つずつコピーしていき比較する 機能を追加するとよいです。

参考情報

• fgetsで単純にファイルを読み取る

https://mementoo.info/archives/788

• char * fgets( char *row , int len , FILE *fp )

http://hitorilife.com/fgets.php

...
■戻り値：
ファイルポインタfpから１行、またはlenバイト読み込んでその先頭アドレスを返す。
全行を読み込んだ場合、又はエラー時にNULL(\0)を返す。
...

追記：
変更例を示します。
(動作は質問文コードとは一致していませんが、ファイルからの読み込み、キーボードからの入力、改行コードの削除は動作しています)

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

#define N (100)

int main() {
  FILE *fp;
  char line[N], input[N];
  char *p;

  if ((fp =fopen("words.txt", "r")) == NULL) {
    printf("can not open words.txt\n");
    exit(1);
  }

  // 文字列を比較する
  while (1) {
    char* s = fgets(line, N - 1, fp);
    if (s == NULL) {
      break;
    }

    p = strchr(line, '\n');
    if(p != NULL) {
      *p = '\0';
    }
    printf("[%s]\n", line);

    printf("Input a words: ");
    fgets(input, N - 1, stdin);
    p = strchr(input, '\n');
    if(p != NULL) {
      *p = '\0';
    }
    // printf("[%s]\n", input);

    if (strcmp(line, input) == 0) {
      printf("'%s' == '%s'\n", line, input);
    } else {
      printf("'%s' != '%s'\n", line, input);
    }
  }
  fclose(fp);
  return 0;
}

Comments

[Teemro_431265]

while(fgets(buf, N, fp) != NULL)
このような感じに書くことはわかったんですけど、これだとstrtok(buf,"\n");が適用（？）されなくなってしまいprintf("%s",input);の後に謎の改行が入ってしまいます。

[katoy]

改行コードを削除する方法例を追記しました。

[Teemro_431265]

なるほど、ありがとうございます。

[Teemro_431265]

できないことはないのですがCntr+Dを押すと残りの単語も全て比較するものになってしまいました、文字列に入っている文字列の初期化みたいなのってできないですよね？

[katoy]

fgets(input, N - 1, stdin);
を
s = fgets(input, N - 1, stdin);
if (s == NULL) {
break;
}
に変更すれば Ctrl-D を入力すると、プログラム終了するようにできます。違う動作にしたければ、break の部分を書き換える必要があります。

[Teemro_431265]

その方法って文字列でも使うことができるんですか？

訂正
ポインタにすることで終了することができました。ありがとうございます。