Unterbrechung lang Methode Muster läuft

Question

ich im Moment etwas langweilige Muster bin mit Fehlermeldung für den Benutzer zu erzeugen, eine lange Operation ausgeführt wird:

string _problem; 

void SomeLongRunningMethod() 
{ 
    try 
    { 
     _problem = "Method1 had problem"; 
     Method1(); 
     _problem = "Unexpected error during doing something in Method2"; 
     if(Method2()) 
     { 
      _problem = "Method3 fails"; 
      Method3(); 
     } 
     _problem = "Not possible to obtain data"; 
     var somedata = Method4(); 
    } 
    catch(Exception) 
    { 
     MessageBox.Show("Problem with some long running method: " + _problem); 
    } 
} 

Beide Methoden werfen kann, und ich will den Benutzer sagen, an dem Scheitern Schritt tritt ein. Dies geschieht durch Setzen von _problemvor mit einem von ihnen.

In einigen Fällen kann ich verschiedene Exception Arten zu fangen, aber das funktioniert nicht immer, z. sowohl Method1 als auch Method2 können throw InvalidOperationException().

Dieser wiederholte Code sieht wie ein Muster aus. Obwohl ich es nicht erkennen kann. Irgendwelche Ideen? Wie verbessert man die Lesbarkeit?

Answer 1

Sie when im catch verwenden könnte zwischen denselben Ausnahmetypen zu differenzieren und welche Methode zu überprüfen, diese Ausnahme ausgelöst hat:

void SomeLongRunningMethod() 
{ 
    try 
    { 
     Method1(); 
     if (Method2()) 
     { 
      Method3(); 
     } 
     var somedata = Method4(); 
    } 
    catch (InvalidOperationException invEx) when (invEx.TargetSite?.Name == nameof(Method1)) 
    { 
     // ... 
    } 
    catch (InvalidOperationException invEx) when (invEx.TargetSite?.Name == nameof(Method2)) 
    { 
     // ... 
    } 
    catch (Exception ex) 
    { 
     // ... 
    } 
} 

Answer 2

Sie können die Methode, die die Ausnahme verursacht hat, mit error.TargetSite abrufen. Das einzige, was müssen Sie ist Linie Ihren Fang ändern: catch (Exception error)

Answer 3

Ich würde Machen Sie eine Reihenfolge der Dinge, die Sie tun und durchlaufen möchten:

var methodList = new[]{ 
    new{action = (Action)Method1, identifier = "Method1"}, 
    new{action = (Action)Method2, identifier = "Method2"}, 
    new{action = (Action)Method3, identifier = "Method3"}, 
}; 
string problem = null; 
foreach(var info in methodList) 
{ 
    try 
    { 
     info.action(); 
    } 
    catch(InvalidOperationException) 
    { 
     problem = string.Format("{0} failed", info.identifier); 
     break; 
    } 
} 
if(problem != null) 
{ 
    //notify 
}