strcmpを使ってファイル操作

Question

前提・実現したいこと

C言語でテキストファイルを読み込み別のファイルに書き込むプログラムを作りたいのですが

発生している問題・エラーメッセージ

strcmpを使って、"Y"が打ち込まれたときだけファイルを読み込む処理を実装したいのですが、"Y"と打ち込んでもexit(1)が反応してしまいます。

kpra.exe

読み込むファイル名を指定してください。：test.txt
test.txtの内容をrev2.txtに書き込みます。よろしいですか？ (Y/N)y

ここでプログラム文が終了しました。

該当のソースコード

C言語

ソースコード
include <stdio.h>
include <stdlib.h>
include <string.h>

int main() {
//ファイル構造体へのポインタを宣言
FILE *fr, *fw;
char str[256], filename[256], *ans;

//読み込むファイルを開く
printf("読み込むファイル名を指定してください。：");
scanf("%s", filename);

fr = fopen(filename,"r");

//ファイルが開けなかったとき
if(fr==NULL) {
    //失敗を表示して終了
    printf("ファイルを開けませんでした\n");
    return -1;
}

//読み込ませるファイルを開く
fw = fopen("rev2.txt","a");

//ファイルが開けなかったとき
if(fw==NULL){
    //失敗を表示して終了
    printf("ファイルを開けませんでした\n");
    return -1;
}
printf("%sの内容をrev2.txtに書き込みます。よろしいですか？ (Y/N)", filename);
scanf("%s", ans);
if(strcmp(ans,"Y")) {
  printf("続行します。\n");
} else {
  printf("中断します。\n");
  exit(1);
}

//ファイルへの書き込み
while(fgets(str, 256, fr)){
    printf("%s", str);
    fputs(str, fw);
}
printf("書き込み完了しました。\n");
fclose(fw);
fclose(fr);
return 0;

}

Answer 1

私自身がポインタの使い方を間違っていて、ポインタの配列を確保しておりませんでした。
解答してくださったみなさまご迷惑おかけしました。

Answer 2

改行とか制御コードとか見えない文字の可能性もあるので、自分ならstrncmpを使います。

Answer 3

strcmp(str1,str2)は、str1とstr2が同じ文字列なら0を返します。

"Y"が打ち込まれたとき、ansは"Y"となり、strcmp(ans,"Y")は strcmp("Y","Y")を演算することになりますから、返り値は0です。

そして、C言語では、0でFalseを、0以外の整数でTrueを表しますから、if(strcmp(ans,"Y")) は分岐条件がFalseとなってelse節の以下のコードが実行されているのです。
"printf("中断します。\n");
exit(1);"

＝＝
「strcmp(ans,"Y")」の部分に否定の演算子"!"を付け加えて「!strcmp(ans,"Y")」とすれば、Ruizinさんが希望するように動作するはずです。

Comments

[Ruizin]

動かしてみましたが
>kpra.exe
読み込むファイル名を指定してください。：test.txt
test.txtの内容をrev2.txtに書き込みます。よろしいですか？ (Y/N)Y

となりました。

Answer 4

strcmpの戻り値は、第一引数と第二引数が一致すれば0です。
つまり
if(0){
になるのでelseの方に行ってしまいます。
素直に、
if(strcmp(ans,"Y") == 0) {
と書くのがよろしいかと思います。

Comments

[Ruizin]

ソースコードを変えてみましたが
>kpra.exe
読み込むファイル名を指定してください。：test.txt
test.txtの内容をrev2.txtに書き込みます。よろしいですか？ (Y/N)Y

となり、変化がありませんでした。

[ttyp03]

改行コードも含んでるのかも？
scanfのあと、printf("[%s]\n",ans);などとして、何が格納されているか確認してみてください。
[Y]と出るなら改行コードを含んでいないのでうまく動くはず。
[Y
]となるなら、改行コードを含んでいるので、対策が必要です。