C言語で1文字を受け入れるメニュー

Question

1文字を受け入れるCプログラムで簡単なメニューを作成したいと思います。メニューは次のようになります：

1. [S]プログラムが印刷されるか
2. [E] XIT

ユーザーが '1'、 'S' または 'S' を入力した場合 " Hello "を入力して再度入力を促します ユーザーが '2'、Eまたは 'E'を入力すると、プログラムは終了します。 それ以外の場合は、「無効な入力」が表示され、再度プロンプトが表示されます。

私はプログラムを作成することができますが、ユーザーが12,13,14,15,16などと入力すると、1から始まり、Helloと他のオプションも同じように表示されます。

私のコードは次のとおりです。

#include <stdio.h> 

void clearBuffer(); 


int main() { 

    int i = 0; 
    char selection; 

    do 
    { 
     printf("\t1. [S]how\n"); 
     printf("\t2. [E]xit\n"); 
     printf("Enter your selection from the number or character noted above: "); 
     scanf("%s", &selection); 
     clearBuffer(); 

     if (selection == '1' || selection == 's' || selection == 'S') 
      printf("Hello"); 
     else if (selection == '2' || selection == 'E' || selection == 'x') 
      i = 0; 

} while(i != 0); 



} 

void clearBuffer() 
{ 

    while(getchar() != '\n'); 

} 

Answer 1

あなたはscanfので返される文字列の長さをチェックし、1つの文字よりも長いエントリ拒否し、標準Cライブラリの一部である、strlenを使用することができます。

を

また、getchar（）を使用してユーザーからの文字を1つだけ受け入れることができます。つまり、Enterキーを押す必要はありません。

Answer 2

あなただけの1文字を受信しようとしている場合）（GETCHARのためのscanf（）関数を置き換えることを検討機能：

printf("Enter your selection from the number or character noted above: "); 
selection = getchar(); 

Answer 3

は、すでに述べたように、あなたが唯一の文字をしたい場合、あなたはgetchar()を使用する必要があります。何らかの理由でscanf()を使用したい場合は、ではなく、"%c"という正しい形式が使用されます。

1文字を探している場合は、ifブロックがちょっと「忙しい」（読んだり、ぎこちない）ように見えることをお勧めします...スイッチは、より洗練された、よりエレガントな方法です（IMHO ）。

/* something like this ... */ 
switch (selection) { 
    case '1': 
    case 's': 
    case 'S': 
    printf ("Hello\n"); 
    break; 

    case '2': 
    case 'e': 
    case 'E': 
    i = 0; 
    break; 
} 

ものの他のカップル...あなたは（'s'と'S'は同じことを行います、つまり）文字の場合は読まれて気にしないならば、あなたがあなたの前に大文字にselectionを変換することができますif-ブロックまたはswitch-ブロックtoupper()を使用してください。また、これはスタイルの提案に過ぎません。終了フラグにiを使用しないでください。一般的なプラクティスは、カウンターやインデックスにiとjのようなものを使用することです。quit_nowまたはuser_doneのようなものを使用すると、変数の意味をより正確に伝えることができます。