Использует «goto» в этой ситуации?

Question

Ниже псевдокод:

myGoto: 
try 
{ 
    // do some db updating 
    myDB.doOptimisticConcurrency(); 

} catch (MyConcExeption ex) { 

    if (tried < fiveTimes) { 
     myDB.Refresh(); 
     tried++; 
     goto myGoto; 
    } 

} 

У меня есть несколько примерочных уловы блоков в одном методе, и я не хочу, чтобы reinvoke моего метода с самого начала для каждого брошенного исключения. Использует ли в этой ситуации goto?

Answer 1

Вы можете изменить его:

while (tried < fiveTimes) 
try 
{ 
    // do some db updating 
    myDB.doOptimisticConcurrency(); 
    break; 
} 
catch (MyConcExeption ex) 
{ 
    tried++; 
    myDB.Refresh(); 
} 

Answer 2

Использование goto почти никогда не является приемлемым, оно приводит к spaghetti code и делает ваш код менее читаемым.

В вашем случае простой цикл сделает ваш код более читаемым.

Answer 3

Вы можете просто использовать цикл.

Answer 4

Вы не должны бросать исключение, если ситуация не, ну, исключительный. Почему бы вам не добавить аргумент повтора к этому методу и исключить исключение изнутри только в случае превышения количества повторов?

РЕДАКТИРОВАТЬ: Как и другие, предложенная петля также является лучшей альтернативой. Тем не менее, это выглядит как ваш метод, а не метод, завернутый в какую-либо библиотеку, которую вы не можете изменить. Если я прав, я все равно поеду с параметром повтора и исключим только исключение, если все попытки не пройдут. Если вы ожидаете, что этот метод будет иногда терпеть неудачу с первой попытки, это не должно быть исключением.

Answer 5

Я бы не использовал «goto» - но вы можете написать небольшой вспомогательный метод. Например:

public static void TryMultiple<E>(Action action, int times) where E : Exception 
{ 
    E lastException = null; 
    for (int i = 0; i < times; i++) 
    { 
     try 
     { 
      action(); 
      return; // Yay, success! 
     } 
     catch (E exception) 
     { 
      // Possibly log? 
      lastException = exception; 
     } 
    } 
    throw new RetryFailedException("Retry failed " + times + " times", 
            lastException); 
} 

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

TryMultiple<MyConcurrencyException>(myDB.doOptimisticConcurrency, 5); 

Answer 6

Это намного лучше:

private void MyMethod() 
{ 
    MyMethod(5); 
}  

private void MyMethod(int triesLeft) 
{ 
    if(triesLeft == 0) 
     return; // or throw 

    try 
    { 
     // do some db updating 
     myDB.doOptimisticConcurrency(); 
    }  
    catch (MyConcExeption ex) 
    { 
     myDB.Refresh(); 
     MyMethod(triesLeft - 1); 
    } 
}