プログラムが動きません。どうすれば動くのでしょうか？

質問

このプログラム（test.cとする）の内容を標準入力から読み込み、copied.cという名前のファイルにwモードで書き込むプログラム。（標準入力からの読み込みとファイルへの書き込みでは、１文字単位での入出力）なのですが、エラーが出て動きません。動かすにはどうすればよいでしょうか？
＜ソース＞
#include<stdio.h>

int main(void)
{
char *filename = "copied.c";
FILE *wfp;
char ch;

wfp = fopen("filename", "w");
if( wfp == NULL ) {
printf("ファイルが見つかりません: %s\n", filename);
return 1;
}

while( (ch = getchar()) ) != EOF ) {
fputc(ch, wfp);
}

fclose( wfp );
return 0;
}
＜エラー＞
% gcc -o test test.c
test.c: 関数 `main' 内:
test.c:15: error: 文法エラー before '!=' token
test.c: トップレベル:
test.c:19: 警告: 関数宣言中に（型の無い）仮引数名があります
test.c:19: 警告: data definition has no type or storage class
test.c:20: error: 文法エラー before "return"
%
です。よろしくお願いします。

回答No.1

titokani · ベストアンサー

> while( (ch = getchar()) ) != EOF ) {
閉じカッコが１個多いですよ。

お礼

気付きませんでした。ありがとうございました。

回答No.6

Interest

コンパイラが吐き出すエラーを省略せずそのままコピー＆ペーストして質問したのは、非常に上手な質問の仕方ですね。

なにせ、コンパイラが
test.c:15: error: 文法エラー before '!=' token
と言っているのですから、これが
「test.cの１５行目に文法エラーがあるよ。 != の前になんか忘れてねぇ？」
と言っているのだと気づけば、これからはコンパイラが吐き出すメッセージを読むだけでどこが悪いのか気づくようになるでしょう。

回答No.5

Tacosan

「int の方がいい」ではなく「int でないとダメ」ですね＞#4.

回答No.4

redfox63

getcharの戻り値を受けるのは int方のほうがいいと思いますよ
char型ですと EOFとの比較が意図しない結果を生むことがありますので

お礼

わかりました。変えておきます。

回答No.3

jjk65536

エラーではないですが、間違いを見つけたので
指摘させてください。

wfp = fopen("filename", "w");

ここはダブルクォートをはずして

wfp = fopen(filename, "w");

としないと、「filename」というファイルを開いて
しまいますよ？

お礼

そうなんですか。回答ありがとうございました。

回答No.2

redfox63

> while( (ch = getchar()) ) != EOF ) {
閉じカッコ『)』の数があっていないように思います

ココの閉じカッコの数が違うためにココ以降で文法エラーが発生しているのではないかと思います

お礼

わかりました。ありがとうございました。