C言語の構造体に対する標準入力

Question

c言語の標準入力と構造体に関する質問です。
例えばトランプのカードなどを入力する際に入力が"S3"や"D10"などで与えられるとして、これを構造体Cardの配列に保存していきたいです。
入力が一つの時は下記のようなコードでうまくいっています。

#include "stdio.h"
#include "stdlib.h"

typedef struct Card Card;
struct Card{char mark;int num;};

int main(){
    Card x[100];
#入力は例えば"S3"など
    scanf("%c%d",&x[0].mark,&x[0].num);
    printf("%c %d",x[0].mark,x[0].num);
    return 0;
}

これをn枚入力に拡張したコードを実行すると思ったように動作してくれません。
(入力は初めカード数nを入力してその後n枚分H3 K6...などを入力しています。
ex:
5
H4 C9 S4 D2 C3
)

#include "stdio.h"
#include "stdlib.h"
typedef struct Card Card;
struct Card{char mark;int num;};

int main(){
    int n,i;
    Card x[100];

    scanf("%d",&n);#カード数n
    for(i=0;i<=n;i++){
        scanf("%c%d",&x[i].mark,&x[i].num);
    }
    for(i=0;i<=n;i++){
        printf("%c %d",x[i].mark,x[i].num);
    }

    return 0;
}

コードのミスや改善法などを教えていただければ幸いです。よろしくお願いします。

Answer 1

scanf の "%d" や "%s" などは先行する空白(スペース、タブ、改行など)を読み飛ばして
くれるので想定通りの入力ができますが、"%c" だけは空白の読み飛ばし機能がないので、
想定通りでない入力になることがよくあります。
" %c" のように直前にスペースを指定して 0個以上の連続する空白を読み飛ばしてから
1文字読み込むようにしてください。
scanf では "\n" は " " と全く同じ機能であり、通常は使用しません。
はまる例としても次のものがあります。

    printf("a: "); scanf("%d\n", &a);
    printf("b: "); scanf("%d\n", &b);

キーボードから入力する場合、プロンプトの a: が出て、123 Enter と入力しても次の
プロンプト b: は出てきません。"\n" が空白でない文字が出てくるまで待ってしまうからです。

Comments

[Miso___]

補足まで頂いてありがとうございます、大変勉強になりました。
お陰様で解決することができました！ありがとうございました！

Answer 2

以下の修正で期待通りにはなりました。

scanf("%d",&n);
↓
scanf("%d\n",&n);

Comments

[Miso___]

お陰様で解決することができました、ありがとうございます！

[thkana]

そのパターンは今回「だけ」はいいけど、

#include <stdio.h>
int main(void) {
int a,b;
printf("Enter..\n");
scanf("%d\n", &a);
scanf("%d\n", &b);
printf("%d %d\n", a, b);
return 0;
}
なんてやってハマっているのをしばしば見かけるから止めたほうがいいと思う。

[thkana]

って、kazuma-sさんが書いてたか。

[ttyp03]

thkanaさん
この手の処理なんて毎度決まったフォーマットがあるわけじゃないので、別に今回「だけ」でもいいと思いますけどね。
まあ何がダメでどうすればいいかを試す能力は鍛える必要はありますが。
もっとも他の回答にあるようにscanfははまるだけなのでgetsを使ってもらいたいですが。

[thkana]

なぜscanfが嫌らしいかと言うと、本当は知ってなきゃいけないことをすっ飛ばして初心者に「お約束」だけで処理させようとするからで、そうするとデータとパターンがアンマッチしてハマるんじゃないですか。だったら、ハマりやすいパターンをわざわざ教えて「何がダメでどうすればいいかを試す能力は鍛え」てやるのは、ちょっとイジワルですよね。

で、そこでgetsお薦めですか? 標準から外されちゃいましたけど。

[ttyp03]

随分食い下がりますねｗ
日常的にscanfを使いたい、scanfマスターになりたい、という人ならキッチリお勉強すればよいでしょうけど、この手の学校の課題みたいなやつはscanfを勉強しているわけじゃないですよね？
そこに時間をかける必要はない、と個人的には思います。
仕事で使うということも私の場合は一切ありませんし、全然重要視していないのですよ。
thkanaさんがscanf大好きでお勧めしたい人なら意見が対立するのは仕方ないので、あとは質問者さんがどう受け止めるかだけじゃないですかね？
getsが標準から外されたかどうかはしりませんが、getsでもfgetsでもgets_sでもまあなんでもいいですよ。「gets系」と書けばよかったですかね。

Answer 3

そもそも文字列の入力はscanfでやってはいけません。
fgetsで１行取得してそこから分解させて、代入していきましょう

Comments

[Miso___]

fgetsをあまり使ったことがなかったため勉強になります、ありがとうございました！