具有许多条件返回语句的重构方法

Question

我有一个包含许多条件语句的验证方法。基本上，

If Check1 = false 
  return false
If Check2 = false
  return false
etc

FxCop 抱怨圈复杂度太高。我知道在函数中间使用 return 语句并不是最佳实践，但同时我看到的唯一替代方案是丑陋的 If-else 语句列表。解决这个问题的最佳方法是什么？

提前致谢。

Answer 1

我不同意您的说法，即在方法中间使用 return 语句不是最佳实践。有些人为了得到一个单一的 return 语句而付出的努力是疯狂的——选择任何能产生最具可读性的代码。有时这将是单点返回，但我经常发现有一个“提前退出” - 并且最好有该返回，而不是使用 if 作为替代路径在代码中引入更多嵌套。根据经验，我喜欢最终不会缩进太远的方法:)

话虽如此，该方法除了检查之外真的什么都没有吗？支票是否独立？他们需要什么变量？您能否将它们分组为较小的方法来代表您正在执行的检查的“区域”？

Answer 2

使用你的例子：

return (Check1 == false)   ||   (Check2 == false)   [ || etc ]

Answer 3

对于您的特定情况，似乎您可以使用循环...假设这来自表单或其他内容中的事件处理程序：

For Each ctrl As Control In Me.Controls

    Dim check As CheckBox = TryCast(ctrl, CheckBox)

    If check IsNot Nothing AndAlso check.Checked = False Then
        Return False
    End If

Next

Return True

编辑：经过进一步审查，我意识到这是基于伪代码，而您不是实际上使用复选框。 但是，我认为同样的方法适用。您可以非常轻松地将规则重构为实现自定义接口（IRule 或类似接口）的单独类，或者拥有一个返回 true/false 的委托列表，您可以将其作为检查模式的一部分进行迭代。

Answer 4

如果您连续进行多项检查并且不需要保留短路，您可以尝试如下操作：

        bool isValid = true;
        isValid &= Condition1;
        isValid &= Condition2;
        isValid &= Condition3;
        if (!isValid)
            return false;

如果确实需要保留短路，则可以将条件合并到一个大的 if 语句中。但是，如果有很多条件，我更喜欢具有多个返回的原始代码，因为单个大的“if”可能会变得有点难看。

请注意，在后一种情况下，您只是回避了指标，因为逻辑分支实际上是相同的。如果条件检查没有副作用（我真的希望它们没有），那么我相信高圈复杂度是一个误报，因为多重返回实际上只是表达复杂布尔验证逻辑的更易读的方式。

然而，只有当条件全部串联时，这才是正确的。如果它们分布在整个代码中，并且在检查之间发生有意义的处理，那么该指标就表明了真正的复杂性（必须测试更多分支）。在这种情况下，您可能会考虑该方法是否可以在逻辑上分解为更小的、可单独测试的部分。