refactoring - 将条件展平为重构

Question

考虑:

if (something) {
    // Code...
}

安装 CodeRush 后，建议执行以下操作:

if (!something) {
    return;
}
// Code...

有人能解释一下这有什么好处吗？肯定没有任何好处。

Answer 1

孤立，正如您所介绍的那样 - 没有任何好处。但是 mark4o 是正确的:它的嵌套更少，如果你看一下甚至会变得非常清楚，比如 4 级嵌套:

public void foo() {
    if (a)
        if (b)
            if (c)
                if (d)
                    doSomething();
}

对比

public void foo() {
    if (!a)
        return;
    if (!b)
        return;
    if (!c)
        return;
    if (!d)
        return;
    doSomething();
}

这样的早期返回提高了可读性。