Cette logique if-else-else peut-elle être réécrite en quelque chose de plus propre?

Question

Pourrais-je écrire la logique suivante d'une manière plus simple et plus facile à lire? Le ci-dessous est ce que je dois, mais il est très salissant:

if (IsChanged == true) 
{ 
    return; 
} 

else if (Status == "" && IsChanged == false) // Executed when the close (x) button is pressed, as the Status string is not yet set to a real value... 
{ 
    CancelClose(); 
} 

else if (IsChanged == false && Status == "saving") // saving logic falls to here... 
{ 
    // IsChanged = false; 
} 

Merci

Answer 1

C'est un peu plus propre:

if (IsChanged) 
{ 

} 
else if (Status == "saving") 
{ 

} 
else if (Status == "") 
{ 

} 
else 
{ 

} 

Je vous recommande d'utiliser un enum pour représenter l'état. Cela permettra à votre code d'être fortement typé.

public enum Status 
{ 
    Closing, 
    Saving, 
    Changed, 
} 

Ensuite, vous pouvez utiliser une instruction switch agréable de décider des mesures à prendre.

switch (_status) 
{ 
    case Status.Saving: 
     break; 
    case Status.Closing: 
     break; 
    case Status.Changed: 
     break; 
    default: 
     break; 
} 

Answer 2

if (isChanged) return; 

switch (Status) { 
    case "": 
     CancelClose(); 
     break; 
    case "saving": 
     // IsChanged = false; 
     break; 
} 

Ceci est à peu près aussi propre qu'il obtient. Notez que parce que vous renvoyez si isChanged est vrai, vous pouvez toujours supposer que isChanged est false.

Answer 3

oui:

if (IsChanged) return; 
    if (String.IsNullOrEmpty(Status)) CancelClose(); 

Answer 4

if(IsChanged) 
    return; 

if(Status == "saving") 
{ 
    // save  
} 
else if(string.IsNullOrEmpty(Status)) 
{ 
    CancelClose();  
} 

Answer 5

Il peut être simplifed pour

if (IsChanged) 
    { 
     return; 
    } 

    else if (Status == "") // Executed when the close (x) button is pressed, as the Status string is not yet set to a real value... 
    { 
     CancelClose(); 
    } 

    else if (Status == "saving") // saving logic falls to here... 
    { 
     //  IsChanged = false; 
    } 

Vous n'avez pas besoin == vrai dans le premier chèque, car il est déjà vrai ou faux. vous n'avez pas besoin de vérifier false dans les autres choix car si ce n'est pas vrai, il doit être faux.

Answer 6

Puisque vous revenez si IsChanged == true, vous n'en avez pas besoin dans les autres ifs.

if (IsChanged == true) 
     return; 

    switch (Status) 
    { 
     case "": 
     CancelClose(); 
     break; 
     case "saving": 
     break; 
    } 

Answer 7

if (IsChanged) return; 

if (Status == "saving") 
{ 
    //IsChanged = false; 
} 
else if (Status = "") 
{ 
    CancelClose(); 
} 

il faut éviter de commencer vos noms de variables avec majuscules.

Answer 8

• couper le premier sinon si juste. Si IsChanged est vrai, le "else" ne sera jamais atteint.
• supprime le IsChanged == false de vos autres ifs car ils sont toujours vrais.
• Pensez à enums au lieu de chaînes pour votre statut.

Je recommande:

if (IsChanged) 
{ 
     return; 
} 

if (CurrentStatus == Status.None) 
{ 
    CancelClose(); 
    return; 
} 

if (CurrentStatus == Status.Saving) 
{ 
    //  IsChanged = false; 
} 

Answer 9

if(!IsChanged) { 
     if (Status == "saving") // saving logic falls to here... 
     { 
      //  IsChanged = false; 
     } 
     else if (Status == "") // Executed when the close (x) button is pressed, as the Status string is not yet set to a real value... 
     { 
      CancelClose(); 
     } 
    } else { 
     return; 
    } 

Answer 10

if (IsChanged) 
    return; 

if (String.IsNullOrEmpty(Status)) // better use this unless you would like a 
    CancelClose();     // nullPointerException 

else if (Status.equals("Saving")) 
    // whatever you want for save 

Answer 11

Je ne suis pas familier avec C#, mais it supports the conditional operator

condition ? first_expression : second_expression; 

Depuis que je ne suis pas familier avec C#, Je ne vais pas essayer de réécrire votre code, mais dans tous les cas, l'opérateur ternaire peut conduire à une concision agréable dans un endroit s.