C言語の勉強のため問題を解いていますがエラーの理由がわからない

Question

###問題
太郎が花子と一緒にトランプ遊びをしようとしたところ、52枚あるはずのカードが n 枚のカードしか手元にありません。これらの n 枚のカードを入力として、足りないカードを出力するプログラムを作成して下さい。

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

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

最初の行に太郎が持っているカードの枚数 n (n ≤ 52)が与えられます。

続いて n 組のカードがそれぞれ１行に与えられます。各組は１つの空白で区切られた文字と整数です。文字はカードの絵柄を表し、スペードが'S'、ハートが'H'、クラブが'C'、ダイヤが'D'で表されています。整数はそのカードのランク(1 〜 13)を表しています。

URL:
https://onlinejudge.u-aizu.ac.jp/courses/lesson/2/ITP1/6/ITP1_6_B

#include <stdio.h>
#include <string.h>

int main (void) {
int number, i, j, n;
int cards[4][14];
char picture;

for (j=0;j<=3;j++) { 
for (i=0;i<=13;i++) {
if (j==0 && i==0) {
cards[j][i]='S';
} else if (j==1 && i==0) {
cards[j][i]='H';
} else if (j==2 && i==0) {
cards[j][i]='C';
} else if (j==3 && i==0) {
cards[j][i]='D';
}
cards[j][i]=i;
}
}

scanf ("%d",&n);

for (i=1;i<=n;i++) {
scanf ("%c %d",&picture,&number);
if (picture=='S') {
picture=0;
} else if (picture=='H') {
picture=1;
} else if (picture=='C') {
picture=2;
} else if (picture=='D') {
picture=3;
}
cards[picture][number]=0;
}

for (j=0;j<=3;j++) { 
for (i=0;i<=13;i++) {
if (cards[j][i]!=0) {
if (j==0) {
printf ("S %d",i);
} else if (j==1) {
printf ("H %d",i);
} else if (j==2) {
printf ("C %d",i);
} else if (j==3) {
printf ("D %d",i);
}
}

}
}

return 0;
}

###自分のコードの意味
まずcards[][]でSの１〜１３、Hの１〜１３、Cの１〜１３、Dの１〜１３の表のようなものを作ります。
次にscanfしたもののところに0を代入します。
そして0が代入されていないところを見つけてそこを出力します。

###エラー内容
zsh: bus error
と言うエラーが
S 12
などのように入力しているときに出ます

Answer 1

おそらくですが、scanfの%cが、前の入力の末尾に残った改行文字を取り込んでしまっている状況ではないでしょうか？改行文字の文字コードは10なので、cards[picture]のようにすれば配列の領域を突破してしまいます。

scanfを直接操るのは煩雑なので、「fgetsで1行取得」→「sscanfでパターン認識」という2段階に分けたほうがいいでしょう。

Comments

[yuruyuru]

maisumakunさん
ありがとうございます。
無事正常に動きました！

Answer 2

- scanf ("%c %d",&picture,&number);
+ scanf (" %c%d",&picture,&number);

%c の前にスペースを置くだけで解決するでしょう。

質問とは無関係ですが、
最初の for の2重ループで cards[j][i] に 'S' や 'H' などを
代入した後で i を上書きしているので、'S' などは入りません。