更有效地删除使用IF语句C＃

Question

我想知道是否有更好的方法来使用if语句而不仅仅是长行

if(){
    //code
}else if(){
    //code
} else{
    //code
}

我在网站上做了一些研究，我发现了这个：

如果你只有2个值，我强烈建议使用你发布的代码，因为它可能是最易读，优雅和快速的代码（恕我直言）。

但是如果你有更多这样的案例并且更复杂，你可以考虑使用switch语句：

switch (el.type)
{
    case ElementType.Type1:
    case ElementType.Type2:
    case ElementType.Type3:
        //code here
        break;
    case ElementType.Type4:
    case ElementType.Type5:
        //code here
        break;
    case ElementType.Type6:
        //code here
        break;
}
that translated in if statements would be:

if (el.type == ElementType.Type1 ||
    el.type == ElementType.Type2 ||
    el.type == ElementType.Type3 )
{
    // code here
}else if(el.type == ElementType.Type4 ||
         el.type == ElementType.Type5)
{
    // code here
}else if(el.type == ElementType.Type6)
{
    // code here
}

它们完全等同于我，但是开关看起来更可读/更清晰，你需要输入更少（即代码长度方面的“更短”）:)

虽然我不太明白它告诉我的是什么，但它是否说switch语句更适用于long if语句或？

为了有一些关于我的问题的背景，我有一个带有一些单选按钮的Windows窗体应用程序 - 如果你愿意，我会问一个调查问卷 - 我想知道是否有更有效的方法可以减少重复的不必要的代码行并用短代码替换它们同样的工作。

Answer 1

切换语句以及许多if-else语句通常是臭设计的标志。 考虑重构为更面向对象的设计

If-else比switch-case更不可读，反之亦然。

Answer 2

目标应该是更可读/更清晰的代码，这并不总是意味着最短的陈述。

如果你有非常复杂的逻辑，有很多嵌套的if / else分支，试着把它拆分成更小，更简单的例程。

Answer 3

您可以应用CoR模式 。 在这里，您将完成.NET实现之一。 当您将其与IoC容器结合使用时，您可以实现非常灵活的架构......

Answer 4

switch语句不仅更容易阅读，而且编译器更容易优化，因为它们明确指定了一个触发所有代码的值。 编译器可以使用散列或跳转表而不是一系列比较来轻松优化它，使其更快（当它可以这样做时）。 从技术上讲，它可以检测到if语句是相同的，但可能会或可能不够复杂。 因此，如果你有256个单独的情况触发一个byte的值，可以将一个开关编译成代码偏移的硬编码数组（数组中的每个条目都是处理该情况的代码的偏移量），并且没有比较是完成所有因为它只能使用字节值作为硬编码数组的索引并跳转到正确的代码。 这比做255（或256）比较要快得多。

使用string值的开关也是专门处理的，使用哈希码可以获得更好的性能。 您可以明确地执行类似的操作，但是您的代码可读性会低得多。