陈旧语法密度之三——用直接删除else的方法消除if-else if-else

最新推荐文章于 2026-08-27 23:14:14 发布

原创最新推荐文章于 2026-08-27 23:14:14 发布 · 1.9k 阅读

0 ·

本内容遵循CC 4.0 BY-SA版权协议

本文探讨了如何通过直接删除else来简化if-else if-else结构，指出在某些情况下，这样的简化是可行的，并强调不应盲目认为旧代码是良好的实践。此外，针对特殊情况，建议封装方法以避免代码重复并提高可维护性。

用直接删除else法消除if-else if-else

这几节先说一些简单的情况，在基本语法的层面，即使是仍然保持面向过程的编码，也能消除。

请看下面的代码：

public String getReversedOperator3() {

    if (operator.equals("=")) {
        return "!=";
    } else if (operator.equals("==")) {
        return "!=";
    } else if (operator.equals("!=")) {
        return "=";
    } else if (operator.equals("<")) {
        return "!<";
    } else if (operator.equals(">")) {
        return "!>";
    } else if (operator.equals("<=")) {
        return "!<=";
    } else if (operator.equals(">=")) {
        return "!>=";
    }
    return "?";
}

实际上，居然可以直接删除所有else，变成：

public String getReversedOperator4() {
    if (operator.equals("=")) {
        return "!=";
    }
    if (operator.equals("==")) {
        return "!=";
    }
    if (operator.equals("!=")) {
        return "=";
    }
    if (operator.equals("<")) {
        return "!<";
    }
    if (operator.equals(">")) {
        return "!>";
    }
    if (operator.equals("<=")) {
        return "!<=";
    }
    if (operator.equals(">=")) {
        return "!>=";
    }
    return "?";
}

有人可能会说：“我怎么可能犯这种低级错误！”且慢，请先在自己项目的代码中搜索一下else if再说吧。这种情况在我所看的代码中数量庞大，大约10%~20%的else（包括下面的变形）都可以如此兵不血刃地删除掉。

下面则是一种类似的变形：

public String getReversedOperator1() {
    String reversedOperator = null;
    if (operator.equals("=")) {
        reversedOperator = "!=";
    } else if (operator.equals("==")) {
        reversedOperator = "!=";
    } else if (operator.equals("!=")) {
        reversedOperator = "=";
    } else if (operator.equals("<")) {
        reversedOperator = "!<";
    } else if (operator.equals(">")) {
        reversedOperator = "!>";
    } else if (operator.equals("<=")) {
        reversedOperator = "!<=";
    } else if (operator.equals(">=")) {
        reversedOperator = "!>=";
    } else {
        reversedOperator = "?";
    };
    return reversedOperator;
}