番号、氏名を入力しテキストファイルに出力するプログラム

Question

c言語において番号、氏名を入力しテキストファイルに出力するプログラムをつくっているのですが、番号だけファイルに出力され、氏名が出力されません。どうしたらよいのでしょうか。

#include<stdio.h>
#include<string.h>
int main(){
  int a,b;
  char e[256];
  char x[1],y[1];
FILE *fp;
 printf("Touroku\n");
  printf("shinki=1,tsuika=2:");
  scanf("%d",&a);
 while(1){
  printf("1 bango:");
  scanf("%d",&b);
  printf("2 name:");
  scanf("%s",e);

  printf("OK?(y/n):");
  scanf("%s",x);
  printf("Continue?(y/n):");
  scanf("%s",y);
 if(!strcmp(y, "n")) {
    break;
  }
  
}
 fp = fopen("gakusei.txt","a");
 if(fp==NULL){
   printf("file open failed.\n");
   return -1;
 }
 fprintf(fp,"%d %s",b,e);
 fprintf(fp,"\n");
 fclose(fp);
 return 0;
}

実行例はこのようにしたいです。
<実行例>
Touroku
shinki = 1, tsuika = 2 : 1
1 bango : 123456
2 name : mamoru
OK? (y/n) : y
Continue? (y/n) : y
1 bango : 654321
2 name : motoki
OK? (y/n) : y
Continue? (y/n) : n
<テキストファイル>
123456 mamoru
654321 motoki

Comments

[y_waiwai]

実際の出力結果を提示しましょう

[ksyiwk]

1 bango : 123456
2 name : mamoruと入力したら
123456しか出力しません。

Answer 1

c の文字列は最後に '\0' が付きます.
y/n の入力では2バイト以上の領域が必要ですので, char x[2],y[2]; としてみては如何でしょうか.

なお, このコードではこれ以外でも問題があるので, これを直してもご希望の結果にはならないと思います.

Comments

[ksyiwk]

char x[2],y[2];にしたところ文字は出力されましたが、実行例どうりにしたら2つ目の654321 motokiしかファイルに書かれていません。1つ目は上書きされたと思うのですがどうしたらよいのでしょうか

[jimbe]

お考えの通り, 入力のループの繰り返しで同じ変数を使用していますので, 上書きされています.
入力のループが終わってからファイルへの書き出しを行っているのが問題と思いますので, 一回の入力毎に("OK? (y/n)" で "y" と入力されたら)ファイルに書き出すようにしては如何でしょうか.

[ksyiwk]

OK? (y/n)の後
if(!strcmp(x,"y")){
fprintf(fp,"%d %s\n",b,e);
}
としたら出来ました。ありがとうございました。

Answer 2

まず、

char x[1],y[1];

これはまずいです
文字列というのは後端に必ず'\0'が入るので、たとえ１文字だけの入力だとしてもここで領域外のアクセスになります。

また、１文字だけを入力してくれることを期待するのは無茶だと思いますがどうでしょうか。

＃まあ、２５５文字だけを入力してくれるのもどーすんだという話に

Answer 3

eじゃなくて&eですよね？

問題部分

scanf("%s",e);

他の人も言ってる問題点は割愛します・・・。

Comments

[LouiS0616]

配列を実引数として取り渡す際には先頭要素のポインタに読み替えられるので、その部分は問題無いです。
むしろ&eを使ってしまうと型の不整合で警告を吐きます。

[LineOfLightning]

おお、そうでした。
失礼しました。