java - 减少圈复杂度

Question

我有一段代码在降低圈复杂度方面遇到了问题。由于必须匹配多个条件，我不确定进一步分解它的最佳方法。使事情复杂化的是，在其中两种情况下创建了一个新对象，但在第三种情况下却没有(它调用了另一个方法)。这是伪代码:

    if (!cond3 && !cond1 && cond2 && cond4) {
        // actions to perform
        calculateValues();
        return result;
    } else if (!cond1 && cond2 && cond3) {
        // actions to perform
        Object result = new Result();
        return result;
    } else if (!cond4 && cond3 && cond1 && cond5) {
        // actions to perform
        Object result = new Result();
        return result;
    } else {
        // throw error because inputs are invalid
    }

Answer 1

您应该重构该代码，使用方法来抽象那些条件，高圈表示代码需要重构。例如，假设:!cond4 && cond3 && cond1 && cond5 测试登录用户是否有车，那么您应该将该条件组合重构为一个方法:

private boolean loggedUserHasCar() {
    return !cond4 && cond3 && cond1 && cond5;
}

对其他条件做同样的事情。带有 4 个条件的 if 语句很难阅读。提取这些语句将降低您的方法圈复杂度并使您的代码更具可读性