C#：嵌套条件与 continue 语句

Question

在最近使用 ReSharper 时，建议我通过反转 if 条件并使用 continue 语句来减少某些地方的嵌套。

嵌套条件：

foreach(....)
{
    if(SomeCondition)
    {
        //do some things

        if(SomeOtherNestedCondition)
        {
            //do some further things
        }
    }
}

继续声明：

foreach(....)
{
    if(!SomeCondition) continue;

    //do some things

    if(!SomeOtherNestedCondition) continue;

    //do some further things
}

我了解您为什么要减少嵌套以解决性能和内存问题以及两个 sn-ps 如何相互等同的一些逻辑，但是从我的开发背景来看，之前 em> 示例在阅读代码时更容易理解。

您更喜欢哪种方法，为什么？您是否在日常代码中使用 continue 而不是嵌套的 if？

Answer 1

作为一项规则，我发现最好始终使用除 out 之外的任何条件启动语句块，因为它降低了复杂性，但更重要的是在它们进一步执行之前抛出不兼容的情况，这可以提高代码和内存性能.这还可以通过维护确保您的条件在一段时间内的安全性，从而不太可能将无效场景传递到它们不属于的代码中。

另外，我认为两者中的第二个个人更具可读性，因为您没有范围层混淆可用的内容，很容易在以后的一层中创建一个变量，而不会意识到它在另一层中不可用，或必须管理它们以进行适当的修改等。

这不仅仅是循环中的继续，而是指方法应该返回的条件；而不是有一个方法开始

if (valid)
{
    do stuff;
}

它应该总是开始

if (notValid)
{
    return;
}

Answer 2

应该没有显着的性能差异，这都是关于可读性的。我个人认为后者更容易阅读。嵌套更少，更易于阅读。

Answer 3

简答：

我倾向于使用缩进，这样它意味着发生了一些真正的决策逻辑，即。预期不止一个可能的执行路径的逻辑。

更长的答案：

我通常更喜欢缩进块而不是前面的“中断”语句（continue、break、return 或 throw）。

原因： 在我看来，中断语句通常会使代码更难阅读。如果您缩进代码，很容易找到某些决策逻辑发生的位置：这将是缩进较少的代码的第一行。如果您使用带倒置条件的中断语句，则必须做更多工作来了解代码如何分支以及在哪些情况下跳过某些代码。

对我来说一个值得注意的例外，即在方法的开头验证参数。我将这段代码格式化如下：

if (thisArgument == null) throw new NullArgumentException("thisArgument");
if (thatArgument < 0) throw new ArgumentOutOfRangeException("thatArgument");
...

原因：由于我不期望这些异常被实际抛出（也就是说，我期望方法被正确调用；调用函数负责检测无效输入，IMO），我不想为不应该发生的事情缩进所有其余代码。

Answer 4

带有continue 样式代码；您将如何处理不依赖于 SomeOtherNestedCondition 的第三个操作，这使得代码的顺序很重要，而 IMO 使其更难维护。

例如：

foreach(....) 
{ 
    if(!SomeCondition) continue; 

    //do some things 

    if(!SomeOtherNestedCondition) continue; 

    //do some further things 

    if(!SomeSecondNonNestedCondition) continue;

    // do more stuff
}

如果 SomeOtherNestedCondition 导致 continue; 发生，但 SomeSecondNonNestedCondition 仍应执行，会发生什么情况？

我会重构出每一段“代码”并使用嵌套的if()s 来调用每个重构的方法，并且我会保留嵌套结构。

Answer 5

简单的答案。哪一个更容易阅读和理解。

Answer 6

结合使用两者。我在循环顶部使用if(condition) continue; 和if(condition) return; 来验证当前状态，并进一步向下嵌套if 语句来控制我想要完成的任何事情的流程。

Answer 7

在内存或性能方面没有区别。底层的 IL 只是分支/跳转指令的选择，因此它们是跳转回循环顶部还是跳转到“else”语句并不重要。

你应该选择更容易阅读的那个。我个人更喜欢避免“继续”，但如果您有许多级别的嵌套，那么您的第二个示例可能更容易遵循。

Answer 8

使用continues 使大部分代码处于常规程序代码级别。 否则，如果有五次检查，您将根据缩进大小将方法的“肉”缩进 10 或 20 个字符，而这些是 10/20 字符，您必须滚动才能看到较长的行。

Answer 9

原文continue含义：任何条件为假时，代码不会被处理。下面是一个例子：

class Program
{
    static bool SomeCondition = false;
    static bool SomeOtherNestedCondition = false;
    static void Main(string[] args)
    {    
        for (int i = 0; i < 2; i++)
        {
            if (SomeCondition)
            {
                //do some things

                if (SomeOtherNestedCondition)
                {
                    //do some further things
                }
            }
            Console.WriteLine("This text appeared from the first loop.");
        }
        for (int i = 0; i < 2; i++)
        {
            if (!SomeCondition) continue;

            //do some things

            if (!SomeOtherNestedCondition) continue;

            //do some further things
            Console.WriteLine("This text appeared from the second loop.");
        }
        Console.ReadLine(); 
    }
}

输出将是：

Answer 10

但是从我的开发背景来看，在阅读代码时，前面的示例更容易理解。

嗯 - 我认为，这是您的个人意见。我 - 例如 - 尽量避免这种嵌套，因为在我看来它会使代码更难阅读。

如果您更喜欢“之前”版本而不是“之后”版本，请使用它。
只需配置 ReSharper，让它建议您真正想要它做什么。

始终牢记：ReSharper 是一个相当“愚蠢”的重构工具 - 它无法取代开发人员。它只能通过做一些其他愚蠢的复制和粘贴工作来帮助他。
即使在 ReSharper 建议相反的重构的情况下，ReSharper 进行的一些重构也会产生结果。

因此，不要将 ReSharper 的建议视为最佳实践，而是您可以做的可能性。

顺便说一句：您应该在这里考虑性能 - 如果性能方面确实存在显着差异，我会感到惊讶。