AIZU ONLINE JUDGE Lesson-ITP1 ITP1_6_Bの問題 構造体を用いて

Question

前提・実現したいこと

AIZU ONLINE JUDGE Lesson-ITP1 ITP1_6_Bの問題を構造体を用いたコードで解決したいです。
｛太郎が花子と一緒にトランプ遊びをしようとしたところ、52枚あるはずのカードが n 枚のカードしか手元にありません。これらの n 枚のカードを入力として、足りないカードを出力するプログラムを作成して下さい。

太郎が最初に持っていたトランプはジョーカーを除く52枚のカードです。

52枚のカードは、スペード、ハート、クラブ、ダイヤの４つの絵柄に分かれており、各絵柄には13のランクがあります。｝

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

エラーメッセージはないんですが

scanf("%d",&n);
  Card card[n];
  for(i=0;i<n;i++){
    scanf("%c %d",&card[i].name,&card[i].num);
  }
```の部分のコードがうまく機能しておらずn個のカード読み込むことができません。どこがおかしいか解説していただけると嬉しいです。
### 該当のソースコード

```c
#include <stdio.h>

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

void cardcheck(Card *sample,Card *card,int n){
  int i,j;
  int judge;
  for(i=0;i<52;i++){
    judge=0;
    for(j=0;j<n;j++){
      if((sample[i].name==card[j].name)&&(sample[i].num==card[j].num)){
        judge=1;
        break;
      }
    }
    if(judge==0){
      printf("%c %d\n",sample[i].name,sample[i].num);
    }
  }
}
int main(void){
  int n;
  int i;
  Card sample[52]={
    {'S',1},
    {'S',2},
    {'S',3},
    {'S',4},
    {'S',5},
    {'S',6},
    {'S',7},
    {'S',8},
    {'S',9},
    {'S',10},
    {'S',11},
    {'S',12},
    {'S',13},
    {'H',1},
    {'H',2},
    {'H',3},
    {'H',4},
    {'H',5},
    {'H',6},
    {'H',7},
    {'H',8},
    {'H',9},
    {'H',10},
    {'H',11},
    {'H',12},
    {'H',13},
    {'C',1},
    {'C',2},
    {'C',3},
    {'C',4},
    {'C',5},
    {'C',6},
    {'C',7},
    {'C',8},
    {'C',9},
    {'C',10},
    {'C',11},
    {'C',12},
    {'C',13},
    {'D',1},
    {'D',2},
    {'D',3},
    {'D',4},
    {'D',5},
    {'D',6},
    {'D',7},
    {'D',8},
    {'D',9},
    {'D',10},
    {'D',11},
    {'D',12},
    {'D',13},
  };
  scanf("%d",&n);
  Card card[n];
  for(i=0;i<n;i++){
    scanf("%c %d",&card[i].name,&card[i].num);
  }
  cardcheck(sample,card,n);
  return 0;
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

> ～の部分のコードがうまく機能しておらずn個のカード読み込むことができません

どのような入力でどのようになるはずが, どのように機能せず出来ないのでしょうか.

[kelt22]

例えばn=30といれても30までいかずn=20で読み込みが終わって下の関数が実行してしまう感じです。

[jimbe]

http://freeh.minim.ne.jp/minimini/develop/cscanf.html

[kelt22]

scanf("%*c%c")を使うことで一応解決することができました。ありがとうございます。

Answer 1

構造体を使う必要は無いのでは.

#include <stdio.h>
#include <string.h>
int main() {
  int i,n,r;
  char c,cards[52]={0};
  char *suits="SHCD",*p;
  scanf("%d",&n);
  for(i=0;i<n;i++){
    scanf(" %c %d",&c,&r);
    p=strchr(suits,c);
    if(p==NULL||r<1||r>13) return 1; //エラー
    cards[(p-suits)*13+r-1]=1;
  }
  for(i=0;i<52;i++){
    if(cards[i]==0) printf("%c %d\n",suits[i/13],i%13+1);
  }
  return 0;
}

Comments

[kelt22]

構造体を使わないやり方は思いつかなかったのでありがとうございます。

Answer 2

scanf(" %c%d",&card[i].name,&card[i].num);
にすればいいのでは？

Comments

[PingHermit]

あ、解説がいるのか。
%c は、空白文字を含めて1文字読み込もうとします。
前に入力した数値の後の改行を読み込んでしまうと思うので、
空白文字を読み飛ばすように、%c の前に ' ' や、'\n'、'\t' など
空白文字を読み飛ばす文字を入れると、改行を読み飛ばしてくれます。
%d は数値以外は受け付けませんが、空白文字は勝手に読み飛ばしてくれる仕様なので、
%d の前に空白文字は必要ありません。

scanf の所しか見ていません。

[PingHermit]

よく、改行を読み飛ばそうと、
"%d\n" とする人がいますが、
数値入力の後の空白文字を読み飛ばそうとします。
だから、空白文字以外の文字が来るか、eof になるまで、
scanf を抜け出せなくなります。
最後に付けるのだけはやめましょう。

[kelt22]

すいません。やってみたのですが、前のコードと同じような感じで入力がうまくいきませんでした。

[PingHermit]

じゃ、入力ミスしているかですかね。
scanf の返却値で、チェックしましょう。
当然、成功しているなら、２が返ってくるはずですから。

まあ、デバッグ時は、printf とかでどういう状態になっているかなど
出力してチェックしておかないと、
何で動作がおかしくなっているか、わからないでしょうから。

scanf の入力後に、回数：scanfの返却値、入力値を出力するくらいはしましょう。