Рекомендации по перекрестному соединению

Question

У меня очень большой цикл, который объединяет 1000 строк. Я выхожу из цикла, если найдено магическое значение 1. Если магическое значение 1 не найдено, но найдено магическое значение 2, цикл должен перейти к началу. Прямо сейчас я использую переключатель, некоторые ifs и goto. Я читал, что goto - не лучший способ. Есть ли лучший способ сделать эту работу?

Answer 1

Чтобы выйти из цикла, вы можете использовать оператор break, чтобы перейти к следующей записи, вы можете использовать оператор continue.

for(int i = 0; i < 1000; i++) 
{ 
    if(magicValue1) 
     break; 
    if(magicValue2) 
     continue; 
} 

Я НЕ попустительство использование GOTO ЗАЯВЛЕНИЕ Я просто указываю на возможную случай использования

Вы можете использовать Гото заявление прыжок, чтобы начать/выход петли, однако я бы держитесь подальше от этой опции, если вы не используете вложенный цикл. Я думаю, что инструкция goto по-прежнему использует свои возможности для оптимизации, выходя из чистого ect .. но в целом лучше всего использовать ее довольно экономно.

for(int i = 0; i < 100; i++) 
{ 
    start: 

    for(int i = 0; i < 10; i++) 
    { 
    if(magicValue1) 
     goto end; 
    if(magicValue2) 
     goto start; 
    } 
} 
end : 

Answer 2

Как об этом:

for(int i = 0; i < 1000; i++) { 
    if(values[i] == MAGIC_VALUE_1) { 
     break; 
    } else if(values[i] == MAGIC_VALUE_2) { 
     i = 0; 
    } 
} 

Если на «перейти к началу» вы имеете в виду «пропустить эту запись и обработать следующий,» заменить i = 0 с continue.

Answer 3

while варьирование не break:

bool continue = true; int i = 0; 
while (i < 1000 && continue){ 
    if(values[i] == MAGIC_VALUE_1) { 
     continue=false; 
    } else if(values[i] == MAGIC_VALUE_2) { 
     i = 0; 
    } 
    i++; 
} 

Answer 4

Я принимаю # 2 случай означает, что вы хотите, чтобы не выполнять (т.е. пропустить) тело цикла в случае # 2, и не то, что вы хотите чтобы сбросить цикл до 0. (См. комментарии к коду, если у меня это есть.)

Это предложение может быть спорным из-за менее условного состояния в цикле for, можно сказать, документируя масштаб, но если это вас не беспокоит, то сжатый способ написать то, что я думаю, вы хотите:

 for (int i= 0; i<values.Length && values[i]!= MAGIC_1; i++) 
     { 
      if (values[i] == MAGIC_2) 
      { 
       // Don't do the loop body for this case but continue on looping 
       continue; 
       // If you want to reset the loop to zero instead of skip the 2 case, 
       // comment-out the continue; and un-comment the line below: 
       // i=0; 
      } 
      // Do long loop body here 
     } 

Answer 5

Я не могу комментировать еще (1 пункт респ прочь)

, но не это было бы лучше:

for (int i = 0; i < 1000; i++) 
{ 
    if (magicValue1) 
    { 
     break; 
    } 
    else if (magicValue2) 
    { 
     dosomething(); 
     i=0; 
    } 
} 

, и я не уверен, что к Что подразумевается под «рестарта Поиск".

Answer 6

Обратите внимание, что если вы установите счетчик на 0, если MagicValue равно 2, и ваш код никогда не изменяет значения, вы, вероятно, будете в бесконечном цикле.

Answer 7

Более сложный может быть:

Определим 2 методы расширения.

public static class Extensions 
{ 
    public static bool isMagic_1(this int i) 
    { 
     return i == 1; 
    } 

    public static bool isMagic_2(this int i) 
    { 
     return i == 2; 
    } 
} 

Теперь вы можете сделать это:

for(int i = 0; i < 1000; i++) 
    { 
    if(i.isMagic_1()) 
     break; 
    if(i.isMagic_2()) 
     continue; 
    } 

надеюсь, что это помогает!