L'utilisation de "goto" est-elle acceptable dans cette situation?

Question

Voici pseudocode:

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

} catch (MyConcExeption ex) { 

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

} 

J'ai plusieurs blocs try-catch dans une méthode, et je ne veux pas en relançant ma méthode depuis le début de toute exception lancée. L'utilisation de goto est-elle acceptable dans cette situation?

Answer 1

Vous pouvez le changer:

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

Answer 2

L'utilisation de goto n'est presque jamais acceptable, elle conduit à spaghetti code et rend votre code moins lisible.

Dans votre cas, une simple boucle rendrait votre code plus lisible.

Answer 3

Vous pouvez simplement utiliser une boucle.

Answer 4

Vous ne devriez pas être lancer une exception si la situation est, bien, exceptionnel. Pourquoi n'ajoutez-vous pas un nouvel argument à cette méthode et ne lancez l'exception en interne que si le nombre de tentatives est dépassé?

EDIT: Comme d'autres l'ont suggéré, une boucle est également une meilleure alternative. Cependant, cela ressemble à votre méthode, pas une méthode enveloppée dans une bibliothèque que vous ne pouvez pas modifier. Si je suis correct, je continuerais avec le paramètre retry et ne lancerais une exception que si toutes les tentatives échouaient. Si vous pensez que cette méthode échouera parfois lors de la première tentative, cela ne devrait pas être une exception.

Answer 5

Je n'utiliserais pas "goto" - mais vous pourriez écrire une petite méthode d'aide. Par exemple:

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

Ceci est juste un croquis de la solution - vous voudrez l'adapter en conséquence. Quoi qu'il en soit, cela vous permet essentiellement d'effectuer des tentatives dans le cadre d'une exception semi-acceptée de manière réutilisable. Vous utiliserez probablement une expression lambda pour exprimer l'action, ou parfois juste un groupe de méthode pour un seul appel de méthode:

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

Answer 6

Ceci est beaucoup mieux:

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