这个 if-else-else 逻辑可以重写为更简洁的东西吗？

Question

我能把下面的逻辑写得更简单、更容易阅读吗？下面满足我的需要，但非常混乱：

if (IsChanged == true)
{
    return;
}

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

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

谢谢

Answer 1

if (isChanged) return;

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

这已经是最简洁的了。请注意，因为如果 isChanged 为 true，则返回，所以您可以进一步始终假设 isChanged 为 false。

Answer 2

这更清晰一些：

if (IsChanged)
{

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

}
else if (Status == "")
{

}
else
{

}

我建议您使用 enum 来表示状态。这将使您的代码成为强类型的。

public enum Status
{
    Closing,
    Saving,
    Changed,
}

然后您可以使用一个很好的 switch 语句来决定要采取什么操作。

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

Answer 3

if(IsChanged)
  return;

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

Answer 4

由于您返回 if IsChanged==true，因此在其他 if 中不需要它。

    if (IsChanged == true)
        return;

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

Answer 5

是的：

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

Answer 6

• 将第一个 else if 剪切为 just if。如果 IsChanged 为真，则永远不会到达“else”。
• 从其他 if 中删除 IsChanged==false 因为它们始终为 true。
• 考虑使用枚举而不是字符串来表示您的状态。

我建议：

if (IsChanged)
{
      return;
}

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

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

Answer 7

    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 8

它可以简化为

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

您不需要在第一次检查中使用 == True，因为它已经是 true 或 false。您不需要检查其他选择是否为假，因为如果它不正确，那么它一定是错误的。

Answer 9

if (IsChanged) return;

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

我会避免以大写字母开头变量名称。

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

我不熟悉 c#，但 它支持条件运算符

condition ? first_expression : second_expression;

由于我不熟悉c#，我不会尝试重写你的代码，但无论如何，三元运算符可以在某些地方带来令人愉悦的简洁。