C＃のループは、私が書いてきた

Question

番目のステートメントに移動する前に最初の文を3回ループこれを行う-ながら、C＃でコンソールアプリでループ：

do 
{ 
    ThisHelp.ShowMenu(); 
    userChoice = (char) Console.Read(); 
    ThisHelp.Helpon(userChoice); 
}while(ThisHelp.IsValid(userChoice) == false); 

書き込み+ WriteLineメソッドのちょうど束を持っているThisHelp.ShowMenu()方法メソッドには、ユーザーに1から8までの数値を入力するよう要求されます。 メソッドThisHelp.Helpon()には、渡されたユーザー入力に応じて異なるテキストが表示されるスイッチがあります。 だけユーザ入力が1から8

の数がここで問題だされていることを確認ThisHelp.IsValid方法：ユーザが番号を書き込み、プレスが入力すると、ループの最初の文は、第二の文を実行する前に3回実行少なくとも一度は。なぜこうなった？

編集：

public bool IsValid(char ch) 
{ 
    if (ch < '1' | ch > 8 & ch != 'q') return false; 
    else return true; 
} 

Answer 1

問題は、Console.Read()がのすべての文字を返します。ユーザーが入力します。あなたはむしろ使用すること

• 8
• \r（キャリッジリターン）と
• \n（改行）

：彼は8とに入るのであれば、その後プレスは、次の3つの文字を取得をENTERこれは：

char userChoice = Console.ReadLine().Trim()[0]; 

しかし、空の入力にはエラー処理を追加してください。

Answer 2

Console.Read()一般のみ戻り、ユーザがリターンを押したとき - れるもがされるキャリッジ・リターンおよび改行を指す：要求としてここThisHelp.IsValidメソッドのコードは、ありますその後のConsole.Read()の呼び出しで返されました。各反復でuserChoiceの値をログに記録する（またはデバッガで調べる）ことにより、検証できます。

Console.ReadLine()を呼び出すだけで、行全体を消費し、その値を整数に変換する方が簡単かもしれません。

Answer 3

ループの2番目の2行を1つにまとめることができます。このような何か試してみてください：

ThisHelp.Helpon(Char.Parse(Console.Readline()));