sscanf が１つしか読み取ってくれない

Question

c言語の sscanf について質問させてください。
マックのterminalです。
int main(){
char linebuf[256];
int parcel[256][2];
int i=0;

while(fgets(linebuf, 256, stdin)!=NULL){
printf("linebuf[0]=%c \n", linebuf[0]);
printf("linebuf[1]=%c \n", linebuf[1]);
printf("linebuf[2]=%c \n", linebuf[2]);
printf("linebuf[3]=%c \n", linebuf[3]);
printf("linebuf[4]=%c \n", linebuf[4]);

 sscanf(linebuf, "%d, %d", &parcel[i][0], &parcel[i][1]);
 printf("parcel[%d][0]=%d ", i, parcel[i][0]);
 printf("parcel[%d][1]=%d \n", i, parcel[i][1]);
 ++i;      

}
return 0;
}

これを./a.out で実行し、標準入力で 12 25 を入れると、
./a.out
12 25
linebuf[0]=1
linebuf[1]=2
linebuf[2]=
linebuf[3]=2
linebuf[4]=5
parcel[0][0]=12 parcel[0][1]=0
となって、linebuf にはちゃんと入ってるのですが、
２５が入ってて欲しい parcel[0][1]
には何も入らず。
ネットで色々調べましたがうまく行かず、こちらで似た質問があって、それでもうまく行かなかったので、質問させていただきました。よろしくお願いします！

Comments

[jimbe]

ご質問として解決されましたら、ベストアンサーを選んで解決済みとしてください。

[yonitugu]

すいません。初めてなもので、あまりよくわかっておりませんで。
いただいたご回答、どれも素晴らしく、そのままにしておいたのですが、それだと解決済みにならないのですね。了解しました。ご教示ありがとうございます。

Answer 1

sscanfで"%d, %d"としてカンマの入力を期待しているけど実際の入力にカンマが無いからでは。

Comments

[yonitugu]

ありがとうございます。よく理解できました。

[yonitugu]

Sscanfの側でカンマを入れたら、入力でも入れないといけなくて、カンマを入れないなら入れないで、入力の方で入れない、というわけですね。理解が深まりました！感謝です。

Answer 2

ネットで色々調べました

「頑張ったんだもん」の主張のためではないのですから、調べて得た情報が何で、そこから解決に至らないまでもどういう考えを経たかを書かないと、「色々調べた」こと自体に意味がなくなってしまいます。(宿題ができなかった言い訳なら「頑張った」事実は重要かもしれませんが、問題の解決には全く寄与しません)

さて。
(scanf系の話はときどき書くんだけど、どっかいっちゃうんだよなぁ...)
scanf(以下sscanfを含めscanfで代表とします)は、初心者向けみたいな扱いになっていますが、「定番」から一歩踏み出すと、ちゃんとした知識がないと結構痛い目を見るかもしれません。
scanfは、「書式指定文字列にマッチするデータ(文字列)を取り込んで、そこから指定されたデータを取り出す」機能を持っています。

"%d"は、「読み込み位置から、0個以上の空白文字(スペース、改行、タブ文字等)を読み飛ばした後に整数として解釈できる数字の並びがあればデータとして取り出す(解釈できない文字が出てくるまで取り込む)」という指示です。

では、"%d, %d"とは?
「0個以上の空白文字を読み飛ばした後に整数として解釈できる数字の並びがあって(ここまで先の%d)、','があり、0個以上の空白を読み飛ばし(これは' 'の働き)、0個以上の空白文字を読み飛ばした後に整数として解釈できる数字の並びがある(後の%d)」ものにマッチします。
例を挙げるなら、
"123,456"、" 123, 456"、"123, [改行]456"等はOK。しかし、"123 456"はマッチしません。最初の%dまでは一致するのでデータを取りますが、そこまで。ついでに言えば"123 , 456"も駄目ですね。
入力が書式指定文字にマッチしない場合は、scanfはデータの取り込みを諦めて先に進みます。"123,456"みたいなデータ形式に対してのみマッチするのに、そういうデータを与えなければデータの取得は失敗します。

そうでない入力を与えたいのなら、それに合った(一致する)書式指定をしてやらなければいけません。
"123 456"を受け取りたいのなら、「0個以上の空白文字を読み飛ばした後に整数として解釈できる数字の並びがあって(ここまで先の%d)、0個以上の空白文字を読み飛ばした後に整数として解釈できる数字の並びがある(ここまで後の%d)」つまり"%d%d"。あるいは、「見た目」のままに間に空白を挟んで"%d %d"でも特に問題はないです。書式指定中の空白は、「0個以上の空白を読み飛ばす」ので、「(%dのあと)、0個以上の空白を読み飛ばして(' ')、0個以上の空白を読みどばした後の整数を取得(後の%d)」としても書式が一致/データが取得できることになります。

---

初心者的ミスのあるあるの一例にscanf("%d\n",&val);というのがあります。これがどういう入力をうけつけることになるか、考えてわかるようならscanfに引っかかることはもうないんじゃないかしら。

Comments

[yonitugu]

ありがとうございます。じっくり読ませていただきます。

Answer 3

カンマ(,)があるから,その後は読み込めずに不定になります。

sscanf(linebuf, "%d, %d", &parcel[i][0], &parcel[i][1]);
↓
sscanf(linebuf, "%d %d", &parcel[i][0], &parcel[i][1]);

~/Project/c >./a.out
12 25
linebuf[0]=1
linebuf[1]=2
linebuf[2]=
linebuf[3]=2
linebuf[4]=5
parcel[0][0]=12 parcel[0][1]=25

Comments

[cateye]

~/Project/c >./a.out
12 25
linebuf[0]=1
linebuf[1]=2
linebuf[2]=
linebuf[3]=2
linebuf[4]=5
error
~/Project/c >echo $?
1
~/Project/c >cat tera.c
#include <stdio.h>

int main() {
char linebuf[256];
int parcel[256][2];
int i = 0;

while (fgets(linebuf, 256, stdin) != NULL) {
printf("linebuf[0]=%c \n", linebuf[0]);
printf("linebuf[1]=%c \n", linebuf[1]);
printf("linebuf[2]=%c \n", linebuf[2]);
printf("linebuf[3]=%c \n", linebuf[3]);
printf("linebuf[4]=%c \n", linebuf[4]);

if(sscanf(linebuf, "%d, %d", &parcel[i][0], &parcel[i][1]) !=2){
puts("error");
return 1;
}
printf("parcel[%d][0]=%d ", i, parcel[i][0]);
printf("parcel[%d][1]=%d \n", i, parcel[i][1]);
++i;
}
return 0;
}

[yonitugu]

ありがとうございます。カンマをとれば通ること、理解できました。

[yonitugu]

カンマを取ったらうまくいきました。ありがとうございます！

[cateye]

scanf()系のライブラリ(sscanf,fscanfなど )は、戻り値は必ずチェックしましょうd(^^ ・・・（上記の!=2など）

[yonitugu]

scanf系の戻り値、重要とのこと、了解です。頭に入れておきます！