どこかおかしいのでしょうか？

質問

大学の授業で以下のような問題を出されました。

　■I have a ball.
　　と入力したら
　　1 : 1 I
　　2 : 4 have
　　3 : 1 a
　　4 : 5 ball.
　　と出力されるようなプログラムを作りなさい。

自分なりに以下のようなプログラムを書きました。

#include <stdio.h>

#define MAX 1000

struct A
{
int vol;
char x[MAX];
};

main()
{

int p;
struct A a[100];
void readcalcprint(struct A []);

for(p=0; p<100; p++)
a[p].vol = 0;

readcalcprint(a);

}

void readcalcprint(struct A a[100])
{
int p, q;
for(p=0; p<100; p++)
{
q=0;
while((a[p].x[q]=getchar()) != '\0')
q++;
a[p].vol = q;
if(a[p].vol==0) break;
}

for(p=0; a[p].vol>0; p++)
printf("%d : %d %s\n", (p+1), a[p].vol, a[p].x);
}

コンパイルは出来るのですが、実行すると入力文の最後にスペースを何回入力しても、プロセスが終わりません。どこがおかしいのでしょうか。ずっと考えていて分からなかったので、書き込ませてもらいました。

回答よろしくお願いします。

回答No.3

ko_kinta · ベストアンサー

入力された文字を以下の3種類に区別しなければいけません。
(1) 文字列の終了を表す改行('\n')
(2) 文字列内の語の区切りを表す空白(' '==0x20)
(3) カウント対象の文字

このコードでは(1)と(2)の区別がありません。
また、ループ終了の判定に'\0'を使っていますが、これはコード上の文字列終端記号であり、標準入力は出来ないので、whileは無限ループに陥ってます。

それとmainで変数の初期化を中途半端に行っています。memset()を使って一気にやっつけた方がいいでしょう。
memset(a, 0, sizeof(a));

readcalcprint()の修正としては、こんなところでしょうか。
1. forのループ条件は、語数が100以下であること、および、'\n'が入力されていないこととします。
2. whileのループは語の長さがMAX-1以下であることを条件とすること。
3. getchar()した文字は一度char型変数に格納し、(3)の文字である場合のみa[p].x[q]に格納すること。
4. 入力文字が(1)だった場合はwhileループを終了すること。
5. 入力文字が(2)で語の長さが1以上であればwhileループを終了すること。
6. 入力文字が(2)で語の長さが0であれば入力文字を無視してwhile先頭に戻ること。(文字列区切りに空白が2個以上連続して入力された場合の対応処理)

お礼

丁寧な回答ありがとうございます。

たった今、ようやく自分のプログラムを修正することで正しく動作するものが書けました。

本当に細かいところから懇切丁寧に説明していただきありがとうございます。

実は（24時過ぎたんで）今日プログラミングの試験があり、この単語帳のところでちょうどつまずいていたところだったんです。

単位とれるように頑張ります！

回答No.2

funaho

ぱっと見たところ、
>while((a[p].x[q]=getchar()) != '\0')
getchar()で入力をしていますが、スペースが入力されたにpの値が増える処理が無いと今の出力部で正しく出力されません。

また、
>for(p=0; p<100; p++)
>{
>q=0;
>while((a[p].x[q]=getchar()) != '\0')
>q++;
>a[p].vol = q;
>if(a[p].vol==0) break;
>}

無限ループになっています。

私が、似たようなプログラムを書いたので参考にしてみてください。
#include <stdio.h>
#include <string.h>
#include <ctype.h>

void main()
{
char str[51],str2[20][51],*pst[20];
int i,j,l,n;

l=0;
n=0;

printf("文->");
gets(str);

for(i=0;str[i]!='\0';i++){
if(isgraph(str[i])){
str2[l][n]=str[i];
n++;
}else{
str2[l][n]='\0';
pst[l]=str2[l];
n=0;
l++;
}
}
if(isgraph(str[i-1])){
str2[l][n]='\0';
pst[l]=str2[l];
l++;
}

for(i=0;i<l;i++){
printf("%d : %d %s\n", i+1, strlen(pst[i]), pst[i]);
}
}

お礼

回答ありがとうございます。

わざわざプログラムを書いて頂きとても役に立ちました。

プログラム参考にさせてもらい、自分なりにアレンジもできました。ありがとうございます。

回答No.1

hello_world

問題に対するgetchar()の使い方が間違っています。
getchar()の仕様をよく読んでください。
問題の解き方もちょっと変わっていて、恐らく「I have a ball.」の'.'の直後は改行なんですよね？その改行判断はいらないと思われているのでしょうか？(ちょっとだけ先述のヒント)

お礼

回答ありがとうございます。

getchar()について復習します。