コンパイルするとエラーが発生する。

./〇<〇.txtでテキストファイル〇.datを読み込んでテキストファイル内の数字、空白、改行、その他の文字の数を表示するプログラムを書きたいのですが、コンパイルすると以下のエラーメッセージが表示されてしまいます。
コードの何処がおかしいのか教えて頂けないでしょうか。

7.c: In function ‘main’:
7.c:8:16: error: too many arguments to function ‘getchar’
8 | while((f = getchar(fp)) != EOF){
| ^~~~~~~
In file included from 7.c:1:
/usr/include/stdio.h:492:12: note: declared here
492 | extern int getchar (void);
| ^~~~~~~
7.c:9:26: error: expected expression before ‘>’ token
9 | if(f>='0' && '9'=>f) d++;
| ^

c
1
2#include <stdio.h>
3
4int main(void){
5    int ch,f;
6    int d=0,o=0,nl=0,s=0;
7    FILE *fp;
8
9    while((f = getchar(fp)) != EOF){
10        if(f>='0' && '9'=>f) d++;
11        if(f==' ') s++;
12        if(f=='\n') nl++;   
13        if(f!=' ' && f!='\n') o++;
14    }
15
16    printf("%3d  digit\n%3d  spaces\n%3d  new lines\n%3d  others\n",d,s,nl,o);
17    return 0;
18}
19

行動規範の内容に同意します

回答4件

エラーメッセージはとりあえず、読んでみると原因が書かれてます。
英語なんて読めなくても、単語だけでも調べればだいたいわかります。

7.c:8:16: error: too many arguments to function ‘getchar’

8 | while((f = getchar(fp)) != EOF){

too many(多すぎる) arguments(引数) to function(関数) ‘getchar’
↓
「getchar関数の引数が多い」というエラーなので、getchar関数の仕様を確認してみます。
すると、

getchar

int getchar(void);
標準入力 (standard input) から 1 文字単位で文字を読み取ります．

と書かれているので、引数は必要ありません。そして、説明に書かれているとおり、
stdioからの入力を受け取るのがgetcharです。

ファイル情報をfpの中に入れたくてFILE *fpとしましたが、なにか不味かったでしょうか…？

getcharでは、それはできません。

/usr/include/stdio.h:492:12: note: declared here

492 | extern int getchar (void);
| ^~~

あ。エラーメッセージの続きにも、getcharの定義はこれだぞ。って親切に書いてありました。

7.c:9:26: error: expected expression before ‘>’ token

9 | if(f>='0' && '9'=>f) d++;

expected(予期する) expression(式) before(前に) ‘>’ token(単語)
↓
>の前に式を予期しました。
↓
>の前に式が必要です。

何かというと、このぶぶん。'9'=>f コンパイラは頭からコードを読んでいって、要素ごとに分解します。
この内容であれば、こうです。
'9'(リテラル) =(代入演算子) >(比較演算子) f(変数)
以上の4っつに分解したうえで、
代入演算子=の定義は、『式 = 式』という形になっているので、
=の次には、式が来ることを、予期した(expected)のですが、
なぜかその次に書かれているのは、>という演算子です。
これでは、演算ができませんので、

(=の次には)「式を予期したのに、その前に>が来ちゃったよ。ナニコレ？」

というエラーになっています。

原因はというと、=>という演算子はC言語にはありません。
その直前で書いているように、>=と書きます。
それであれば、コンパイラはきちんと、

'9'(リテラル) >=(比較演算子) f(変数)

と解釈できます。

投稿2020/08/30 10:24

amiya

総合スコア1218

torimoti

2020/08/30 11:27

とてもわかりやすい解説ありがとうございます。大変助かりました。ご指摘の点をしっかりと意識してエラーメッセージもある程度読めるようになりたいと思います。

行動規範の内容に同意します

とりあえず一点
×　'9'=>f
○　'9'>=f

あと、このままだとotherで
数字が再カウントされますね。

投稿2020/08/30 09:58

編集2020/08/30 10:19

hana_yama_san

総合スコア923

torimoti

2020/08/30 10:06

確かに仰る通り数字を再カウントしてしまっておりました。適切な助言ありがとうございます。

行動規範の内容に同意します

コンパイラさんは、とにかく文法解釈の余地があるならとにかくソースを読み進めます。で、どうしようもなくなった段階で初めて悲鳴(エラー)をあげます。

つまり、ソースコードに何かの間違いがあった時に、「その結果として」どうにもならなくなったところがエラーメッセージの場所で、つまり「間違い」そのものはエラーメッセージより少し(あるいはだいぶ)前にある、というのは普通にあることです。

今回であれば、ソースを読み進めていって、8行目先頭の'while'に出会ったところで、文法的には‘=’, ‘,’, ‘;’, ‘asm’ or ‘attribute’ がなければおかしいのに'while'が出てきちゃった...と文句を言っているわけで。つまり、whileが間違っているわけではなく、File *fpからwhileにつながったところに問題がありそう、ということになります。
そういう目で見ると、File *fpの後に;がないんだ...というのに気がつけるんじゃないかと思います。

それと、時々「合っている/間違えていないはずなのにエラーが出る」と主張する人がいます。コンパイラのバグだとでも言いたいのでしょうが(質問者さんのことではありませんが)。世界中の人たちが散々使い倒しているコンパイラに、初心者がホイホイと見つけられるバグが残っている可能性はとても低いです(え? internal error? いや、それは...)。エラーが出るとか期待した動きをしない以上は自分がどこか間違えているはず、と思って間違いを探さないと、目の前にある間違いも見えなくなってしまいますのでご注意ください。

投稿2020/08/30 07:53

thkana

総合スコア7703

thkana

2020/08/30 07:57

書いてるうちに話はどんどん進んじゃった。もう少し自分でも悩もうよ... エラーが出るたびに全部ここで聞く気?

torimoti

2020/08/30 08:18

度々すみません。このプログラムですが、微力ながら2~3日程かけ、調べながら作ったものになります。また、回答頂いてる合間には実際に調べています。ただ、理解しようとは思っているのですが、此方で聞いた方が求めている回答を得られるため甘えてしまった節はあります。お目汚しになってしまった事に関しては本当に申し訳ないです。

行動規範の内容に同意します

ベストアンサー