特定の文字が入力されるまで・・・

プログラムで、
A～Cの記号が選ばれるまで、
（つまり他の文字が入力されたらもう一度入力しなすようにしたい）
入力コマンドを繰り返す制御文を作りたいのですが
自分で作ってもうまくいきません。

char c;
while((c=getchar())!='A'||'B'||'C'){
}
以下はA,B,Cのいずれかが入力されたら実行される制御文が続く


しかし、実際はA～Cを入力しても、
次の制御文に移りません。
どこがおかしいでしょうか？
また、正しく動かすにはどうすればよいでしょうか？

No.4 ベストアンサー 回答者： crew21 回答日時： 2007/12/14 10:00

試してないけどこのコードは無限ループでしょうね。

人間の考え方と、コンピュータの考え方の違いがまさに現れてるコードに見える。

このコードは簡単に言うと、while(判定文１or判定文２or判定文３)てことです（それはわかりますか？）。

で、判定文１は今仮に置いとくとしても、判定文２と判定文３は絶対にTrueになってしまうんですよ。

それはわかりますか？
判定文というのは、要は０ならFalse、０以外ならTrueですから、判定文２に'B'という０以外の値（'B'は0x42）を書いた時点で必ずTrueになってしまいます。

だから、たとえ判定文１がFalseであっても判定文２は絶対Trueになるから、判定文３は評価されることすらなく、while()の中全体の評価は必ずTrueとなってしまい、無限ループになります（次の制御文に行きません）。

この回答へのお礼

ご回答ありがとうございます。
while((c=getchar())!='A'||'B'||'C')
ここが変だと気づきました。
へぼミスです。
親切な回答ありがとうございました。

お礼日時：2007/12/18 12:22

No.5 回答者： aris-wiz 回答日時： 2007/12/14 11:37

無限ループになる理由はNo4さんが回答しているので、

蛇足ですが、一度は必ず入力するならば、
int c;
do{
　c = getchar();
}while( c!='A' && c!='B' && c!='C' );
でいいかと。

親切なのはNo1さんのが親切かな。

ちなみに細かいですが、getcharは入力ストリームから文字を、
unsigned char型の値として読み取り、int型に変換して返してきます。

この回答へのお礼

ご回答ありがとうございます。
間違えていた箇所がわかりました。
ありがとうございました。

お礼日時：2007/12/18 12:10

No.3 回答者： koko_u_ 回答日時： 2007/12/14 03:44

対称性が崩れてるからヤダ。

この回答へのお礼

意味がわかないのですが

お礼日時：2007/12/18 12:22

No.2 回答者： yaemon_2006 回答日時： 2007/12/14 03:37

　ちょっとすっきり。

while ( (c = getchar()) != 'A' && c != 'B' && c != 'C' ) {
　 fprintf(stderr, "re-enter a character\n");
}

この回答へのお礼

ご回答ありがとうございます。
そうですね＾＾

お礼日時：2007/12/18 12:23

No.1 回答者： koko_u_ 回答日時： 2007/12/14 02:46

>どこがおかしいでしょうか？

明らかに

(c=getchar())!='A'||'B'||'C'

ですね。

char c = getchar();
while ( c != 'A' && c != 'B' && c != 'C' ) {
　 fprintf(stderr, "re-enter a character\n");
　 c = getchar();
}

という風か？

この回答へのお礼

ご回答ありがとうございます。
間違っていた箇所がわかりました。

お礼日時：2007/12/18 12:23