Simplyfing Code mit vielen ifs

Question

ich eine Methode, die auf dem ENUM basiert und am Anfang klar zu sein, haben wir diese Situation:

public void MyMetohd(Somestatus status) 
{ 
if(status == Somestatus.Enum1) 
{ 
DoA(); 
DoB(); 
DoC(); 
DoD(); 
DoE(); 
} 
if(status == Somestatus.Enum2) 
{ 
DoA(); 
DoB(); 
DoC(); 
DoD(); 
} 

if(status == Somestatus.Enum3) 
{ 
DoA(); 
DoB(); 
DoC(); 
} 

if(status == Somestatus.Enum4) 
{ 
DoA(); 
DoB(); 
} 

if(status == Somestatus.Enum5) 
{ 
DoA(); 
} 
} 

Wie würden Sie diese Art von Code zu optimieren (es isn‘ t meins)?

Answer 1

Überprüfen Sie, ob Sie Ihren Code nicht umstellen können, um eine Strategy Pattern zu verwenden. (Überprüfen Sie auch this). Sie können es mit einer Fabrikmethode oder Fabrikklasse kombinieren.

Answer 2

Auf den ersten Blick ein switch Aussage scheint der beste Ansatz zu sein, aber Sie haben eine Menge wiederholt Code sogar in das bekam:

switch (status) 
{ 
    case Somestatus.Enum1: 
     DoA(); 
     DoB(); 
     DoC(); 
     DoD(); 
     DoE(); 
     break; 
    case Somestatus.Enum2: 
     DoA(); 
     DoB(); 
     DoC(); 
     DoD(); 
     break; 
    ... 
} 

Während dies besser ist es immer noch nicht ideal ist, wie Sie haben Die Anrufe an DoA usw. wiederholt.

Answer 3

Wenn Sie an der gleichen Bedingung und anderen Wert dafür arbeiten, zu dieser Zeit besser verwenden Sie switch..case ... Verwenden Sie If..else, wenn Sie mehrere Bedingungen testen möchten.

Answer 4

könnte man auch „invertieren“ die Logik, wenn das es einfacher macht (abhängig von der Anzahl der Aufzählungen gegen die Anzahl der verschiedenen Aktionen zur Deckung):

if(status == Somestatus.Enum1 || status == Somestatus.Enum2) 
DoA(); 

if(status == Somestatus.Enum1 || status == Somestatus.Enum4) 
DoB(); 

... 

Answer 5

Sie einen Wörterbuch Schlüssel auf die verwenden können, Enum-Wert und mit einer Liste von Action oder Action<T> ausgeführt werden.

Dictionary<int,IList<Action>> actionsPerEnumValue; 

Füllen Sie dieses Wörterbuch mit den Enum-Werten und den Aktionen für jedes.

In Ihrer Funktion erhalten Sie die Liste der Funktionen pro Wert und rufen Sie jede Aktion auf.

foreach(var act in actionsPerEnumValue[status]) 
{ 
    act(); 
} 

Siehe this SO für ein Beispiel beantworten.

Answer 6

klingt wie Sie State Pattern

Answer 7

verwenden, sollten Sie einen Vergleich verwenden können, wenn Sie die Werte der einzelnen Mitglieder des enum eingestellt.

Dann verwenden Sie einfach Vergleich, um Ihren Code zu tun. Weil Sie immer DoA() machen, fangen Sie damit an.

if(status <= Somestatus.Enum5) 
    DoA(); 

if(status <= Somestatus.Enum4) 
    DoB(); 

if(status <= Somestatus.Enum4) 
    DoC(); 
... 

Fahren Sie so weiter. Auf diese Weise werden alle Ihre Funktionen aufgerufen, wenn der Wert Enum1 ist.

Answer 8

EDITED:

// ANOTHER WAY 
public void MyMetohd(Somestatus status) 
{ 
    switch(status) 
    { 
     case Somestatus.Enum1: 
      do_("ABCDE"); 
      break; 
     case Somestatus.Enum2: 
      do_("ABCD"); 
      // and so on... 
     } 
} 

public static void do_(string s) 
{ 
    foreach(char ch in s) 
    { 
     switch(ch) 
     { 
      case 'A': 
       doA(); 
       break; 
      case 'B': 
       doB(); 
       break; 
      case 'C': 
       doC(); 
       break; 
      case 'D': 
       doD(); 
       break; 
      case 'E': 
       doE(); 
       break    
     } 
    } 
} 

Answer 9

von optimize werde ich nehme an, Sie bedeuten "Dryer machen".

Du wird eine Balance zwischen Code hat zu schlagen, die leicht zu lesen ist (was, was Sie haben, ist, wenn auch etwas wiederholend) und Code, die so wenig wie möglich

einfach wiederholt eingeben das macht mich sich schmutzig anfühlen, aber wenn Sie wollen, ist TROCKEN und weniger LOC, ich denke, es würde tun, was Sie wollen.

switch (status) 
      { 
       case Somestatus.Enum1: 
        DoE(); 
        goto SomeStatus.Enum2; 
       case Somestatus.Enum2: 
        DoD(); 
        goto SomeStatus.Enum3; 
       case Somestatus.Enum3: 
        DoC(); 
        goto SomeStatus.Enum4; 
       case Somestatus.Enum4: 
        DoB(); 
        goto SomeStatus.Enum5; 
       case Somestatus.Enum5: 
        DoA(); 
        break; 
       default: 
        throw new InvalidArgumentException("Unknown Status"); 
      } 

Answer 10

Verwenden Sie ein Muster, um dies zu lösen, wird die beste Lösung sein, wie die anderen vorgeschlagen haben. Ich wollte noch eine andere Lösung anbieten.

Ich empfehle sehr, dies jedoch nicht zu tun.

public void MyMetohd(Somestatus status) 
    DoA(); 
    if (status != SomeStatus.Enum5) { 
     DoB(); 
     if (status != SomeStatus.Enum4) { 
      DoC(); 
      if (status != SomeStatus.Enum3) { 
       DoD(); 
       if (status != SomeStatus.Enum2) { 
        DoE(); 
       } 
      } 
     } 
    } 
} 

Answer 11

Obwohl ich mich für die Lösung von Brook selbst entscheiden würde, möchte ich auf eine andere elegante und kurze Lösung hinweisen.

public void MyMethod(Somestatus status) 
{ 
    foreach (Action toDo in new Action[] { DoA, DoB, DoC, DoD, DoE }.Take(5 - (int)status)) 
     toDo(); 
} 

Dies setzt jedoch voraus Somestatus ist wie folgt definiert:

enum Somestatus 
{ 
    Enum1, 
    Enum2, 
    Enum3, 
    Enum4, 
    Enum5 
} 

Ich mag diese Lösung als akademische und da ist sehr kurz, aber es ist sicherlich nicht groß Lesbarkeit oder Wartbarkeit haben.