c# - switchステートメントを含むwhileループから抜け出します

Question

switchステートメントを含むループから抜け出す方法を理解するのに問題があります。ブレークは、ループではなく、スイッチからブレークアウトします。

これにはおそらくもっとエレガントな解決策があります。trueで始まり、falseに設定され、ループを終了するフラグを実装しました。より良い解決策を提供できますか？

背景：このコードは、バーコードワークフローシステムで使用されます。バーコードスキャナーが組み込まれたPocketPCがあります。このコードは、これらの機能の1つで使用されます。ルーチン全体でさまざまなデータの入力をユーザーに求めます。このピースを使用すると、PocketPC端末にその情報を表示するいくつかのインベントリレコード（ページングされた結果）をスクロールでき、完了の場合は「D」、終了するには「Q」を入力できます。

改善が必要な現在のC＃の例を次に示します。

do
{
    switch (MLTWatcherTCPIP.Get().ToUpper())
    {
        case "": //scroll/display next inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "P": //scroll/display previous inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "D": //DONE (exit out of this Do Loop)
            // break; // this breaks out of the switch, not the loop
            // return; // this exists entire method; not what I'm after
            keepOnLooping = false;
            break;
        case "Q": //QUIT (exit out to main menu)
            return;
        default:
            break;
    }
} while (keepOnLooping);

これは、VB.NETでこれを行うコードの例です。

Do
    Select Case MLTWatcherTCPIP.Get().ToUpper
        Case "" ''#scroll/display next inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown()
        Case "P" ''#scroll/display previous inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextUp()
        Case "D" ''#DONE (exit out of this Do Loop)
            Exit Do
        Case "Q" ''#QUIT (exit out to main menu)
            Return
    End Select
Loop

ありがとう、

score 53 · Accepted Answer

私はそれを避けようとしますが、あなたは使うことができます...

後藤

ただし、熊手を持った怒っている暴徒は、そうすることを選択した場合、職業上の危険になります。

score 50 · Accepted Answer

このフォームは、これまでになく少し読みやすくなっています。

bool done = false;
while (!done) 
{ 
    switch (MLTWatcherTCPIP.Get().ToUpper()) 
    { 
        case "": //scroll/display next inventory location 
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown(); 
            break; 
        case "P": //scroll/display previous inventory location 
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown(); 
            break; 
        case "D": //DONE (exit out of this Do Loop) 
            done = true;
            break; 
        case "Q": //QUIT (exit out to main menu) 
            return; 
        default: 
            break; 
    } 
}

score 31 · Accepted Answer

ここでの1つのオプションは、このループをメソッド（ "extract method"）にリファクタリングし、を使用することreturnです。

score 12 · Accepted Answer

私が知っている他の唯一の方法は恐ろしい後藤です。 MSDNもこれを言っています。

ただし、この場合に使用する理由はわかりません。実装した方法は正常に機能し、gotoよりも保守しやすくなっています。私はあなたが持っているものを保持します。

score 10 · Accepted Answer

マルチレベルのブレークにはgotoステートメントを使用する必要があります。これは、C＃で唯一の「クリーンな」方法のようです。フラグを使用することも有用ですが、ループを実行するための他の苦境がある場合は、追加のコードが必要です。

http://msdn.microsoft.com/en-us/library/aa664756(VS.71).aspx

他のいくつかの非C言語には、実行することによってマルチレベルのブレークがあることに注意するのは興味深いかもしれません（ただし、Javaは、続行を装ったgotoを使用するので、同じように役に立ちません。。：P）break levels;

score 8 · Accepted Answer

ループを継続するためにブール値を返すメソッドにスイッチをラップしてみませんか？コードを読みやすくするという副次的な利点があります。結局、gotoステートメントは必要ないと誰かが論文を書いたのには理由があります;）

do
{
    bool keepOnLooping = TryToKeepLooping();
} while (keepOnLooping);

private bool TryToKeepLooping()
{
    switch (MLTWatcherTCPIP.Get().ToUpper())
    {
        case "": //scroll/display next inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "P": //scroll/display previous inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "D": //DONE (exit out of this Do Loop)
            // break; // this breaks out of the switch, not the loop
            // return; // this exists entire method; not what I'm after
            return false;
        case "Q": //QUIT (exit out to main menu)
            return true;
        default:
            break;
    }

    return true;
}

score 7 · Accepted Answer

外側のループから簡単に抜け出すことはできませんが、できcontinueます。

ロジックを逆にすると、これが得られます。breakswitchステートメントの直後にループを終了する必要があることに注意してください。

私の意見では、これはあまり読みやすいコードではなく、フラグが最適だと思います。

   do
         {
            switch (Console.ReadKey().KeyChar.ToString())
            {
                case "U":
                    Console.WriteLine("Scrolling up");
                    continue;

                case "J":
                    Console.WriteLine("Scrolling down");
                    continue;

                case "D": //DONE (exit out of this Do Loop)
                    break;

                case "Q": //QUIT (exit out to main menu)
                    return;

                default:
                    Console.WriteLine("Continuing");
                    continue;
            }

            break;

        } while (true);

        Console.WriteLine("Exited");

score 4 · Accepted Answer

フラグはこれを行うための標準的な方法です。私が知っている他の唯一の方法は、を使用することgotoです。

score 2 · Accepted Answer

switchステートメントをステートメントに置き換えることができますif/else。不要gotoで、breakステートメントはループを終了します。

do
{
  String c = MLTWatcherTCPIP.Get().ToUpper();

  if (c = "")
    MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
  else if (c = "P")
    MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextUp();
  else if (c = "D")
     break;
  else if (c = "Q")
    return;
  else
  {
    // Handle bad input here.
  }
} while (keepLooping)

score 1 · Accepted Answer

whileIMO、これはループから抜け出すための完璧な方法のようです。それは副作用なしであなたが期待することをします。私はすることを考えることができました

if(!keepOnLooping)
  break;

しかし、それは実行に関しては実際には何の違いもありません。

score 1 · Accepted Answer

それを関数にラップし、returnステートメントを使用して終了します。どのようにそのことについて？

score 1 · Accepted Answer

次のように記述します。

case "Exit/Break" :
                  //Task to do
                    if(true)
                      break;

この休憩は、どのような場合にも関連付けられません。whileループに属します。

score 0 · Accepted Answer

switchステートメントをfor/foreachループに変更できます。条件が満たされたら、「keepOnLooping」をfalseに設定し、breakを使用してループから抜け出します。残りはそれ自身の世話をする必要があります。

score 0 · Accepted Answer

もう1つの（それほど優れていない）代替手段は、case「ループから抜け出す」必要がある場所をifすぐに処理して、switchブロックから移動することです。スイッチケースが非常に長い場合、それほどエレガントではありません。

do
{
    var expression = MLTWatcherTCPIP.Get().ToUpper();
    if (expression = "D") //DONE (exit out of this Do Loop)
    {   
        statement;
        break;
    }

    switch (expression)
    {
        case "": //scroll/display next inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "P": //scroll/display previous inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "Q": //QUIT (exit out to main menu)
            return;
        default:
            break;
    }
} while (true); //or whatever your condition is

また、ループから抜け出すだけで、式自体の計算は簡単である（変数の読み取りなど）ことを考慮して、caseそれ自体をループの条件の一部にすることもできます。while

do
{
    switch (expression)
    {
        case "": //scroll/display next inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "P": //scroll/display previous inventory location
            MLTWatcherTCPIP.TerminalPrompt.ScrollBodyTextDown();
            break;
        case "Q": //QUIT (exit out to main menu)
            return;
        default:
            break;
    }
} while (condition && expression != "D");

また、すべてを新しいメソッド（これに対する最も洗練されたソリューション）にリファクタリングすることが何らかの理由で受け入れられない場合は、匿名のデリゲートを使用して既存のメソッド内で同じことを行うこともできます。

score -2 · Accepted Answer

うまくいくかもしれないし、うまくいかないかもしれないが、ラムダは楽しみのためだけにそれを試してみませんか

while(  (expr) => (){
switch(expr){
case 1: dosomething; return true; 
case 2 : something;return true;
case exitloop:return false;}
});

c# - switchステートメントを含むwhileループから抜け出します

15 に答える 15

後藤

Related

Reference