CppCoreGuidelines F.56 避免不必要的嵌套条件判断

20 July 2022

F.56: Avoid unnecessary condition nesting

理由

浅层嵌套的代码更容易理解；代码意图更清晰。尽量把代码放在逻辑判断的最外层。不到万不得已，不用多层嵌套逻辑判断。

例子

利用保卫语句（ guard-clause ），让代码早返回或早退出。

// Bad: Deep nesting
void foo() {
    ...
    if (x) {
        computeImportantThings(x);
    }
}

// Bad: Still a redundant else.
void foo() {
    ...
    if (!x) {
        return;
    }
    else {
        computeImportantThings(x);
    }
}

例子

// Good: Early return, no redundant else
void foo() {
    ...
    if (!x)
        return;

    computeImportantThings(x);
}

例子

// Bad: Unnecessary nesting of conditions
void foo() {
    ...
    if (x) {
        if (y) {
            computeImportantThings(x);
        }
    }
}

例子

// Good: Merge conditions + return early
void foo() {
    ...
    if (!(x && y))
        return;

    computeImportantThings(x);
}

强化

标记多余的 else。
标记函数体只是简单的通过条件语句包含了代码块。

programming 164

CppCoreGuidelines 475