c# - 这是双向 if-else 语句的最佳/标准做法

Question

我想知道在这些情况下更好的解决方案是什么，或者公司的编码标准对此有何看法？

        if (this.State == NetworkState.Server)
        {
            //...
        }
        else if (this.State == NetworkState.Client)
        {
            //...
        }
        else
        {
            //some error throwing or printing
        }

或

        if (this.State == NetworkState.Server)
        {
            //...
        }          
        else
        {
            //sure its client here
        }

我觉得第一个更具可读性，因为你可以清楚地看到 client 是另一个状态，但它也使代码更长，有时我真的不能做比打印更多的东西“这应该永远不会发生"在第三种情况下。是否有更好的解决方案？

Answer 1

在这里使用 switch 语句更有意义。

 switch(this.State)
 {
     case NetworkState.Server:
        //...
        break;
     case NetworkState.Client:
        //...
        break;
     default:
        //some error throwing or printing
        throw new NotSupportedException(string.Format("An unsupported enumeration value {0}.{1} was used.  Processing for the case is not supported because it has not been explicitly implemented.", typeof(NetworkState).Name, this.State), new NotImplementedException());
 }

它明确表明您正在考虑的唯一条件是枚举值。对于 if 语句，您必须阅读每个额外的“if”以确保没有其他条件影响逻辑。

编辑:我刚刚注意到您问题中询问如何处理不匹配情况的部分。我在那里添加了一个异常(exception)来解释这个问题。我喜欢这样做，因为在现实中——我唯一会遇到 default 子句的情况是: (1) 我忘记了一个选项，需要实现它；或者，(2) 自编写原始代码以来，已将一个值添加到枚举中。在任何一种情况下，都需要开发人员的关注，而不是处理，因此抛出一个错误来指示这种必要性似乎是唯一合理的回应。