テキストファイルの単語数

Question

テキストファイルを読み込み、行数と単語数をカウントし、テキストファイルで出力するプログラムなのですが、何故か単語数が0と出力されてしまいます。
どこがおかしいのか教えてください。

#include <stdio.h>

int main(void)
{
    FILE *fp;
    char str[2048];
    int f;
    int w=0,l=0;
    
    fp = fopen("list.txt", "r");
   
    if(fp == NULL)
    {
        printf("Cannot open file\n");
        return 1;
    }

    while(fgets(str, 2048, fp) != NULL)
    {
        l++;
    };
    
    while((f = fgetc(fp)) != EOF)
    {
        if(f == ' ' || f == '\t' || f == '\n' || f == ',' || f == '.')
        {
            w++;
        }
    };

    fclose(fp);

    fp = fopen("out.txt", "w");

    fprintf(fp, "Number of lines = %d\n", l);
    fprintf(fp, "Number of words = %d\n", w);

    fclose(fp);

    return 0;
}

Comments

[1T2R3M4]

debugとかしないのですか。

[kaguya85]

お恥ずかしながらdebugの仕方がわからないんです。

[dodox86]

> debugの仕方がわからないんです。

Visual Studio(20xxやCode)、EclipseなどのIDE、gdbなどを使ってください。もしそれが無くても各変数の動き（変化の仕方）をprintfなどで確認することでもできます。それが基本です。

[dodox86]

「printfなどで確認することが基本」は言い過ぎだったかもしれませんが、自分が書いたプログラムが想定通りに各ステップで動作しているか、動作していなければどこでおかしくなっているか初心者なりにでも工夫して確認し、場所を特定することは大事なことです。

Answer 1

while(fgets(str, 2048, fp) != NULL)

{
    l++;
};

ここでファイルの終了までファイルを読んでいるので、

while((f = fgetc(fp)) != EOF)

このループ条件が成立することはありませんね

Comments

[kaguya85]

なるほど、そういう事なんですね。
ということは、fgetcの方は消してfgetsの読み込みで単語数も数えるようにすれば良いのですか？

[y_waiwai]

そうやってもいいですし、ファイルポインタを先頭に戻してfgetcを実行してもいいです。

まあ、どっちかいうと、fgetsで全てを済ませたほうがいいですね

[kaguya85]

fgetsで
if(f == ' ' || f == '\t' || f == '\n' || f == ',' || f == '.')
{
w++;
}
の部分は利用できないですよね…？

[y_waiwai]

strに１行分の文字列が読み込まれてるので、
str[なんたら]で、なんたら番目の文字が読めます
まあしかし、単純にスペース文字をカウントするという方法では単語数を取るのは無理かと思います

＃スペースが複数連続してる場合とか、行末の改行なんかもカウントしてしまう

[kaguya85]

すみません、fgetsでの単語数の数え方が全く分からないので、差し支えなければ指標など示して貰えませんか。