スイッチ内からループを抜け出す方法は？

次のようなコードを書いています。

while(true) {
    switch(msg->state) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
    case DONE:
        break; // **HERE, I want to break out of the loop itself**
    }
}

それを行う直接的な方法はありますか？

フラグを使用して、スイッチの直後に条件付きブレークを置くことでループから抜けることができることは知っています。C ++にこのための構造が既にあるかどうかを知りたいだけです。

前提

次のコードは、言語や目的の機能に関係なく、不適切な形式と見なされます。

while( true ) {
}

サポートする引数

次のwhile( true )理由により、ループは不適切な形式です。

• whileループの暗黙のコントラクトを解除します。
  • whileループ宣言では、唯一の終了条件を明示的に記述する必要があります。
• それは永遠にループすることを意味します。
  • ループ内のコードは、終了句を理解するために読み取る必要があります。
  • 永久に繰り返されるループは、ユーザーがプログラム内からプログラムを終了できないようにします。
• 非効率的です。
  • 「true」のチェックを含む、複数のループ終了条件があります。
• バグが発生しやすい。
  • 各反復で常に実行されるコードを配置する場所を簡単に決定できません。
• 不必要に複雑なコードにつながります。
• 自動ソースコード分析。
  • バグの発見、プログラムの複雑さの分析、セキュリティチェック、またはコードを実行せずに他のソースコードの動作を自動的に導出するには、初期破壊条件を指定することで、アルゴリズムが有用な不変条件を特定できるようになり、自動ソースコード分析の指標が向上します。
• 無限ループ。
  • 誰while(true)もが無限ではないforループを常に使用する場合、ループに実際に終了条件がないときに簡潔に通信する能力を失います。（おそらく、これはすでに起こっているので、要点は議論の余地があります。）

「移動」の代わり

次のコードはより良い形式です：

while( isValidState() ) {
  execute();
}

bool isValidState() {
  return msg->state != DONE;
}

メリット

フラグなし。いいえgoto。例外なし。簡単に変更できます。読みやすい。簡単に修正できます。さらに、コード：

1. ループ自体からループのワークロードの知識を分離します。
2. コードを保守している人が機能を簡単に拡張できるようにします。
3. 複数の終了条件を1つの場所に割り当てることができます。
4. 実行するコードから終了句を分離します。
5. 原子力発電所にとってより安全です。;-)

2番目のポイントは重要です。コードがどのように機能するかを知らずに、メインループに他のスレッド（またはプロセス）に一定のCPU時間を与えるように依頼された場合、2つの解決策が思い浮かびます。

オプション1

一時停止を簡単に挿入します。

while( isValidState() ) {
  execute();
  sleep();
}

オプション＃2

上書き実行：

void execute() {
  super->execute();
  sleep();
}

このコードは、が埋め込まれたループよりも単純です（したがって、読みやすくなっています）switch。このisValidStateメソッドは、ループを継続するかどうかを決定するだけです。メソッドの主力は、メソッドに抽象化する必要がexecuteあります。これにより、サブクラスがデフォルトの動作をオーバーライドできるようになります（組み込みのswitchおよびを使用する難しいタスクgoto）。

Pythonの例

StackOverflowに投稿された次の回答（Pythonの質問に対する）と比較してください。

1. 永久にループします。
2. 選択肢を入力するようユーザーに依頼します。
3. ユーザーの入力が「再開」の場合は、ループを永久に継続します。
4. それ以外の場合は、ループを永久に停止します。
5. 終わり。

while True: 
    choice = raw_input('What do you want? ')

    if choice == 'restart':
        continue
    else:
        break

print 'Break!' 

対：

1. ユーザーの選択を初期化します。
2. ユーザーの選択が「再起動」という単語である間ループします。
3. 選択肢を入力するようユーザーに依頼します。
4. 終わり。

choice = 'restart';

while choice == 'restart': 
    choice = raw_input('What do you want? ')

print 'Break!'

ここではwhile True、誤解を招く複雑なコードになります。

使用できますgoto。

while ( ... ) {
   switch( ... ) {
     case ...:
         goto exit_loop;

   }
}
exit_loop: ;

別の解決策はcontinue、と組み合わせてキーワードを使用することですbreak。つまり、

for (;;) {
    switch(msg->state) {
    case MSGTYPE
        // code
        continue; // continue with loop
    case DONE:
        break;
    }
    break;
}

continueステートメントを使用して、ループを続行する各ケースラベルを終了し、ステートメントを使用して、ループをbreak終了する必要があるケースラベルを終了します。

もちろん、このソリューションは、switchステートメントの後に実行する追加のコードがない場合にのみ機能します。

これを行うためのきちんとした方法は、これを関数に入れることです：

int yourfunc() {

    while(true) {

        switch(msg->state) {
        case MSGTYPE: // ... 
            break;
        // ... more stuff ...
        case DONE:
            return; 
        }

    }
}

オプション（ただし、「悪い習慣」）：すでに示唆したように、gotoを使用するか、スイッチ内で例外をスローできます。

私の知る限り、C ++には「ダブルブレーク」や同様の構成要素はありません。最も近いものはgoto-ですが、その名前には悪い意味が含まれていますが、ある理由で言語に存在しています-慎重にそして控えめに使用されている限り、それは実行可能なオプションです。

スイッチを次のような別の関数に入れることができます。

bool myswitchfunction()
{
    switch(msg->state) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
    case DONE:
        return false; // **HERE, I want to break out of the loop itself**
    }
    return true;
}

while(myswitchfunction())
    ;

gotoが気に入らない場合でも、例外を使用してループを終了しないでください。次のサンプルは、それがいかに醜いかを示しています。

try {
  while ( ... ) {
    switch( ... ) {
      case ...:
        throw 777; // I'm afraid of goto
     }
  }
}
catch ( int )
{
}

この回答のgotoように使用します。この場合、コードは他のどのオプションよりも明確になります。この質問がお役に立てば幸いです。goto

しかしgoto、ここでは文字列のため、使用することが唯一のオプションだと思いますwhile(true)。ループのリファクタリングを検討する必要があります。次の解決策を考えます：

bool end_loop = false;
while ( !end_loop ) {
    switch( msg->state ) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
    case DONE:
        end_loop = true; break;
    }
}

または次のように：

while ( msg->state != DONE ) {
    switch( msg->state ) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
}

この場合、ループから抜け出すためのC ++構造はありません。

フラグを使用してループを中断するか、（必要に応じて）コードを関数に抽出してを使用しますreturn。

gotoを使用することもできますが、ループを停止するフラグを設定することをお勧めします。次に、スイッチから抜け出します。

whileループの条件を修正して、問題を解消してみませんか？

while(msg->state != DONE)
{
    switch(msg->state) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
    case DONE:
        // We can't get here, but for completeness we list it.
        break; // **HERE, I want to break out of the loop itself**
    }
}

いいえ、キーワード "break"がスイッチブロックを終了するためにすでに予約されている場合、C ++にはこのための構成はありません。あるいは、終了フラグを指定したdo..while（）で十分です。

do { 
    switch(option){
        case 1: ..; break;
        ...
        case n: .. ;break;
        default: flag = false; break;
    }
} while(flag);

おもう;

while(msg->state != mExit) 
{
    switch(msg->state) 
    {
      case MSGTYPE: // ...
         break;
      case DONE:
      //  .. 
      //  ..
      msg->state =mExit;
      break;
    }
}
if (msg->state ==mExit)
     msg->state =DONE;

それを行う最も簡単な方法は、SWITCHを実行する前に単純なIFを置くことです。そのIFは、ループを終了するための条件をテストします..........可能な限り単純です

breakC ++ のキーワードは、最もネストされた囲み反復またはswitchステートメントのみを終了します。したがって、ステートメントwhile (true)内でループを直接抜けることはできませんswitch。ただし、次のコードを使用することもできます。これは、この種の問題の優れたパターンだと思います。

for (; msg->state != DONE; msg = next_message()) {
    switch (msg->state) {
    case MSGTYPE:
        //...
        break;

    //...
    }
}

msg->state等しいときに何かを行う必要がある場合DONE（クリーンアップルーチンを実行するなど）、forループの直後にそのコードを配置します。つまり、現在以下の場合：

while (true) {
    switch (msg->state) {
    case MSGTYPE:
        //... 
        break;

    //...

    case DONE:
        do_cleanup();
        break;
    }

    if (msg->state == DONE)
        break;

    msg = next_message();
}

次に代わりに使用します：

for (; msg->state != DONE; msg = next_message()) {
    switch (msg->state) {
    case MSGTYPE:
        //...
        break;

    //...
    }
}

assert(msg->state == DONE);
do_cleanup();

説明の深さを考えると、これがいかに単純であるかに驚かされます。

bool imLoopin = true;

while(imLoopin) {

    switch(msg->state) {

        case MSGTYPE: // ... 
            break;

        // ... more stuff ...

        case DONE:
            imLoopin = false;
            break;

    }

}

笑！！本当に！それだけで十分です！1つの追加変数！

while(MyCondition) {
switch(msg->state) {
case MSGTYPE: // ... 
    break;
// ... more stuff ...
case DONE:
   MyCondition=false; // just add this code and you will be out of loop.
    break; // **HERE, you want to break out of the loop itself**
}
}

同じ問題が発生し、フラグを使用して解決しました。

bool flag = false;
while(true) {
    switch(msg->state) {
    case MSGTYPE: // ... 
        break;
    // ... more stuff ...
    case DONE:
        flag = true; // **HERE, I want to break out of the loop itself**
    }
    if(flag) break;
}

C ++構文をよく覚えている場合はbreak、と同じように、ステートメントにラベルを追加できますgoto。だからあなたが望むものは簡単に書かれるでしょう：

while(true) {
    switch(msg->state) {
    case MSGTYPE: // ...
        break;
    // ... more stuff ...
    case DONE:
        break outofloop; // **HERE, I want to break out of the loop itself**
    }
}

outofloop:
// rest of your code here

  while(true)
  {
    switch(x)
    {
     case 1:
     {
      break;
     }
    break;
   case 2:
    //some code here
   break;
  default:
  //some code here
  }
}