Я нахожу эту форму немного более читабельной:

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; 
    } 
}

Я бы постарался избежать этого, но вы могли бы использовать...

перейти к

Однако разъяренные толпы с вилами становятся профессиональной опасностью, если вы решите это сделать.

Один из вариантов здесь — преобразовать этот цикл в метод («метод извлечения») и использовать return.

Единственный другой способ, о котором я знаю, это ужасный goto. MSDN также говорит об этом.

Однако я не вижу причин, по которым вы могли бы использовать его в этом случае. То, как вы реализовали, отлично работает и более удобно в обслуживании, чем goto. Я бы сохранил то, что у тебя есть.

Вы должны использовать оператор goto для многоуровневых разрывов. Кажется, это единственный «чистый» способ в С#. Использование флага также полезно, но требует дополнительного кода, если цикл имеет другие трудности для выполнения.

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

Может быть интересно отметить, что некоторые другие не-c языки имеют многоуровневые разрывы, выполняя break levels; (однако Java так же бесполезна, поскольку использует goto, замаскированный под continue.. :P)

Почему бы не обернуть переключатель в метод, который возвращает логическое значение, чтобы продолжать цикл? Это имело бы побочное преимущество, сделав код более читаемым. Есть причина, по которой кто-то написал статью, в которой говорится, что нам не нужны инструкции 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;
}

Вы не можете легко вырваться из внешнего цикла, но вы можете его continue.

Если вы измените свою логику, то вы получите это. Обратите внимание, что сразу после оператора switch есть break для выхода из цикла.

На мой взгляд, это не очень читаемый код, и я думаю, что флаг все же лучше.

   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");

Флаг — стандартный способ сделать это. Единственный другой способ, который я знаю, это использовать goto.

Вы можете заменить оператор 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)

Оберните его в функцию и используйте оператор return для выхода. Как насчет этого?

ИМО, это кажется прекрасным способом выйти из цикла while. Он делает то, что вы ожидаете, без побочных эффектов. я мог подумать о том, чтобы сделать

if(!keepOnLooping)
  break;

Но на самом деле это не отличается с точки зрения исполнения.

Напишите что-то вроде:

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

Этот перерыв не будет связан ни с одним делом. Он будет принадлежать циклу while.

Вы можете изменить оператор switch на цикл for/foreach. Как только условие выполнено, установите «keepOnLooping» в false, а затем используйте break, чтобы выйти из цикла. Остальное должно позаботиться о себе.

Другая (не очень хорошая) альтернатива состоит в том, чтобы однозначно обрабатывать 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");

Кроме того, если рефакторинг всего этого в новый метод (что является наиболее элегантным решением) по какой-то причине неприемлем, вы также можете положиться на анонимного делегата, чтобы сделать то же самое внутри существующего метода.

Может сработать, а может и не сработать, но лямда, почему бы не попробовать просто ради удовольствия

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