C言語で特定の文字が何文字あるのかカウントしたい。

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
C言語について質問です。
キーボードから入力したファイル名のファイルに含まれている小文字のaの数を求めたいのですが上手く機能しません。カウントができていないようで、何度やっても0になってしまいます。　どなたかわからないでしょうか?

作ったソースは以下の通りです。
include <stdio.h>

int main(void)

{

int a_count=0;
char fname[100];
int i;
FILE*fp;

printf("ファイル名:");
scanf("%s",fname);

fp=fopen(fname,"r");
if(fp==NULL){
printf("指定したファイルは開けません.");
return (1);
}

for(i=0; fname[i]!='\0'; i++){
if(fname[i] == 'a'){a_count++;}
break;
}

fclose(fp);

printf("aは%d文字\n",a_count);

return 0;

}

Comments

[1T2R3M4]

何度やっても0になるってことはコンパイルできたってこと？
信じられない。

ところで、こちらはお知合いですか。
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q10235956255

[kanato_gungun]

知らない人です。

Answer 1

for(i=0; fname[i]!='\0'; i++){
if(fname[i] == 'a'){a_count++;}
break;
}

いろいろな間違いはともかく、質問に対する答えとしてはここでaを見つけてカウントした後にfor文をbreakで抜けているからです。

Answer 2

カウントが出来ない以前に、変数textが宣言されていないので、コンパイルエラーです。

あと、変数countが初期化されていません。

Answer 3

• fgetsで読んだ内容を格納する配列textが宣言されていない。コンパイルできないよね。
• textの添字として使うiは、内側のwhileの直前で0を代入する必要がある。
• 内側のwhileの条件を、ヌル文字と等しくないにしておかないと、長い行を分割して読むときに困る。
• countの初期値を設定していない。
• fopenに失敗したときでもfgetsが実行されるため、実行時エラーを引き起こすだろう。

修正後の問題点

• fgetsしておらず、ファイル名の文字列のaを数えている。
• forの中でbreakしているため、実質的に文字列の先頭の文字しか調べていない。