Разорвать вложенные циклы

goto!

Я не понимаю этого упорного мема, в котором говорится, что goto считается «вредным». При правильном использовании он очень мощный, и это именно тот случай.

• рефакторинг, поэтому вам не нужно выходить из вложенных циклов таким образом.
  Использование return часто возможно, если поместить циклы в отдельную функцию.
• используйте goto.
• Использовать флаг (некрасиво)

Используйте флаг, чтобы сигнализировать о завершении:

for(int y = 0; y < 100; y+=10)
{
     bool flag = false;
     for(int x = 0; x < 100; x += 10)
     {
        if (x == 56)
        {
           flag = true;
           break;
        }
     }

     if(flag) break;
}

Некоторые люди пристрелили бы меня за предложение использовать оператор goto, но выход из нескольких циклов — это одно из мест, где он может быть очень полезным (и эффективным):

/*Main loop*/
for(int y = 0; y < 100; y+=10)
{
    /*Sub loop*/
    for (int x = 0; x < 100; x += 10)
    {
        if(x == 56) 
        { 
            goto MainLoopDone;
        }
    }
}

MainLoopDone:
// carry on here

часто лучше поместить это в отдельную функцию, а затем выполнить «возврат»

void loop_de_loop()
{
  for(int y = 0; y < 100; y+=10)
  {
      /*Sub loop*/
      for (int x = 0; x < 100; x += 10)
      {
          if(x == 56) 
          { 
              return;
          }
      }
  }
}

Я не знаю, есть ли способ выйти из вложенных циклов в C#, но позвольте мне предложить обходной путь.

Вы можете бросить основной цикл в функцию и вернуться из этой функции. Вы можете return false; указать преждевременный разрыв и return true; указать, что цикл прошел полностью, если это имеет значение.

Флаги, как предложено в комментариях, вероятно, лучший метод:

boolean someFlag = true;

for(int y = 0; i < 100 && someFlag; y += 10) {
  for(int x = 0; x < 100 && somFlag; x += 10) {
    if(x == 56)
      someFlag = false;
  }
}

Нет хорошего универсального ответа. «Правильный путь» зависит от реальной проблемы. Лучший способ — поместить внешний цикл в функцию, а затем использовать return;, чтобы выйти из него. Это может быть x=100; y=100;. Это может быть done=true;. Черт возьми, это может быть даже goto (шучу).

Не рекомендуется, но вы можете использовать goto. См. это.

public class GotoTest1
{
    static void Main()
    {
        int x = 200, y = 4;
        int count = 0;
        string[,] array = new string[x, y];

        // Initialize the array:
        for (int i = 0; i < x; i++)

            for (int j = 0; j < y; j++)
                array[i, j] = (++count).ToString();

        // Read input:
        Console.Write("Enter the number to search for: ");

        // Input a string:
        string myNumber = Console.ReadLine();

        // Search:
        for (int i = 0; i < x; i++)
        {
            for (int j = 0; j < y; j++)
            {
                if (array[i, j].Equals(myNumber))
                {
                    goto Found;
                }
            }
        }

        Console.WriteLine("The number {0} was not found.", myNumber);
        goto Finish;

    Found:
        Console.WriteLine("The number {0} is found.", myNumber);

    Finish:
        Console.WriteLine("End of search.");


        // Keep the console open in debug mode.
        Console.WriteLine("Press any key to exit.");
        Console.ReadKey();
    }
}
/*
Sample Input: 44

Sample Output
Enter the number to search for: 44
The number 44 is found.
End of search.
*/

Не рекомендуется, потому что это затрудняет понимание потока. Другой вариант, конечно, - установить какой-то флаг во внутреннем цикле и проверить его во внешнем цикле, я не принял во внимание это, поскольку это очевидно, и предполагаю, что вы все равно это знаете.. :)

Поскольку, как вы упомянули, в команде break нет меток, вы можете сделать что-то вроде этого:

/*Main loop*/
bool fFound;
for(int y = 0; y < 100 && !fFound; y+=10)
{
    /*Sub loop*/
    for (int x = 0; x < 100; x += 10)
    {
        if(x == 56) 
        { 
            //Break the main loop 
            fFound = true;
            break; //Break inner loop
        }
    }
}

Как уже говорили другие, «правильный» ответ зависит от проблемы, которую вы решаете. Если вы можете, лучше разбить его на более мелкие части. Что-то по этой модели:

object MainLoop ()
{
    object result = null;
    for(int y = 0; y < 100; y+=10)
    {
        result = SubLoop(y);
        if (result != null)
        {
            break;
        }
    }
    return result;
}

object SubLoop (int y)
{
    object result = null;
    for (int x = 0; x < 100; x += 10)
    {
        if(x == 56)
        {
            result = objectInstance;
            break;
        }
    }
    return result;
}

На мой взгляд, в разной степени некрасиво иметь несколько операторов return из одной функции, использовать дополнительные флаги или (вздрагивать) использовать операторы goto. Но иногда один из них необходим.

Редактировать: Это демонстрирует использование этого метода для возврата полезного объекта какого-либо типа, вы должны иметь "Customer" или "IDataAccess" или "bool" или что-то отличное от "object" в качестве возвращаемых типов, если используя это по-настоящему.

Сначала я использовал LINQ для сбора интересных объектов, а затем выполнял операции с результатами запроса LINQ. Тем самым удаляя вложенные циклы и заменяя их одним циклом.