入力された行をコマンドの引数だけ繰り返すプログラムについて

Question

コード
#include <stdio.h>
#include <stdlib.h>

int main(int argc, char* argv[]){
    char buf[80];
    int i, n;

    if(argc != 2);
    n = atoi(argv[1]); 
    
    while (!feof(stdin)){
        gets(buf);
        for (i = 0; i<n; i++){
            puts(buf);
        }
    }
    return 0;
}

$seq 1 5 | ./a.out 2
1
1
2
2
3
3
4
4
5
5
5
5
出力が上記のようになるのですが、なぜ5が4つ出力されるのでしょうか？
教えていただけると幸いです。

Answer 1

feof()は終端指示子(End-of-file indicator)をチェックする関数であって、ファイルが終端に達したかどうかをチェックする関数ではありません。
終端指示子は、gets()などの関数の実行結果によって設定されます。
つまり質問のコードでは、gets()の読み込みが失敗してからでないと、feof()の結果はTrueになりません。
ループの判定をwhile(gets(buf) != NULL) {とすれば意図したとおりになります。

Answer 2

入力の最後、5に到達後にEOFが来ますが、その時のgets()呼出が失敗します。
しかしbufには前の入力結果5が残っており、これを処理しているので、最後だけ２回処理されています。
gets(buf); -> if( !gets(buf)) break;と変更すると意図どおりになります。

Comments

[mo_chi_o]

ありがとうございます。理解できました。