设计模式之表达原则：如何让源代码成为一种逻辑线索？_源代码的业务逻辑

cName = InpList.get(0).replace(",", ".");

                    cCode = InpList.get(1).replace(",", ".");

                    cAlpha2 = InpList.get(2).replace(",", ".");

                    cAbreviation = InpList.get(3).replace(",", ".");

                    dYear = InpList.get(4).replace(",", ".");

                    dPoliticalCompatibility = InpList.get(5).replace(",", ".");

                    dRankPoliticalCompatibility = InpList.get(6).replace(",", ".");

                    dEconomicCompatibility = InpList.get(7).replace(",", ".");

                    dRankEconomicCompatibility = InpList.get(8).replace(",", ".");

                    dMilitaryCompatibility = InpList.get(9).replace(",", ".");

                    dRankMilitaryCompatibility = InpList.get(10).replace(",", ".");

                    dDemoScore = InpList.get(11).replace(",", ".");

                    dRankDemoScore = InpList.get(12).replace(",", ".");

                    dEnvironmentalCompatibility = InpList.get(13).replace(",", ".");

                    dRankEnvironmentalCompatibility = InpList.get(14).replace(",", ".");

                    dSumCompatibility = InpList.get(15).replace(",", ".");

                    dRankCompatibility = InpList.get(16).replace(",", ".");

                    dPoliticalUtility = InpList.get(17).replace(",", ".");

                    dRankPoliticalUtility = InpList.get(18).replace(",", ".");

                    dEconomicUtility = InpList.get(19).replace(",", ".");

                    dRankEconomicUtility = InpList.get(20).replace(",", ".");

                    dMilitaryUtility = InpList.get(21).replace(",", ".");

                    dRankMilitaryUtility = InpList.get(22).replace(",", ".");

                    dEnvironmentalUtility = InpList.get(23).replace(",", ".");

                    dRankEnvironmentalUtility = InpList.get(24).replace(",", ".");

                    dSumUtility = InpList.get(25).replace(",", ".");

                    dRankUtility = InpList.get(26).replace(",", ".");

                    dPoliticalScore = InpList.get(27).replace(",", ".");

                    dRankPoliticalScore = InpList.get(28).replace(",", ".");

                    dEconomicScore = InpList.get(29).replace(",", ".");

                    dRankEconomicScore = InpList.get(30).replace(",", ".");

                    dMilitaryScore = InpList.get(31).replace(",", ".");

                    dRankMilitaryScore = InpList.get(32).replace(",", ".");

                    dEnvironmentalScore = InpList.get(33).replace(",", ".");

                    dRankEnvironmentalScore = InpList.get(34).replace(",", ".");

                    dAggregate = InpList.get(35).replace(",", ".");

                    dRankAggregate = InpList.get(36).replace(",", ".");

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74

而是，希望看到这样的代码（HttpClient 的某个代码片段）：

/**

 * {@inheritDoc}

 */

@Override

public CloseableHttpResponse execute(

        final HttpUriRequest request,

        final HttpContext context) throws IOException, ClientProtocolException {

    Args.notNull(request, "HTTP request");

    return doExecute(determineTarget(request), request, context);

}

private static HttpHost determineTarget(final HttpUriRequest request) throws ClientProtocolException {

    // A null target may be acceptable if there is a default target.

    // Otherwise, the null target is detected in the director.

    HttpHost target = null;

    final URI requestURI = request.getURI();

    if (requestURI.isAbsolute()) {

        target = URIUtils.extractHost(requestURI);

        if (target == null) {

            throw new ClientProtocolException("URI does not specify a valid host name: "

                    + requestURI);

        }

    }

    return target;

}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

所以说，在开发代码时，应该更注重代码表达的意图是否清晰，考虑使用一些方法和技巧，虽然会耗费一点时间，但是从整体来看，你会节省很多沟通与解释的时间，做到在真正的提升编码效率。

如何写出有“逻辑线索”的源代码

要想写出可读性高的代码，你可以从三个方面来入手。

• 代码表现形式：在命名（变量名、方法名、类名）、代码格式、注释等方面的改进。

• 控制流和逻辑：尽量分离控制流和逻辑，让代码变得更容易理解。

• 惯性思维：找出常犯的一些惯性思考方式并逐一改进。

下面我就来具体解释下。

1. 优化代码表现形式

命名在编程中至关重要，无论是变量名、类名还是方法名，好的名字能快速准确地传达要表达的含义，而缩写、自定义名称会让代码变得难以理解。我们先来看一段代码：

public class T {

    private Set<String> pns = new HashSet();

    private int s = 0;

    private Boolean f(String n) {return pns.contains(n);}

    int getS() {return s;}

    int s(List<T> ts, String n) {

        for (T t :ts) 

            if (t.f(n)) 

                return t.getS();

        return 0;

    }

}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

这段代码到底实现了什么功能？估计没有人能回答出来。如果编写者不是我，我肯定也无法理解这段代码。光凭看代码，几乎是无法理解这段代码的真实含义到底是什么的。

实际上，这个类是获取球队比赛得分的，除了通过球队直接获得比赛得分外，还可以通过球队里的某个球员来查找对应得分，具体修改如下：

/**

 * 获取球队比赛得分

 **/

public class Team {

    private Set<String> playerNames = new HashSet(); //保证名字不重复

    private int score = 0; //默认为零

    

    /**

     * 判断是否包含球员

     * @param playerName

     * @return

     */

    private Boolean containsPlayer(String playerName) {

        return playerNames.contains(playerName);

    }

    

    /**

     * 知道队伍，直接获取分数

     * @return

     */

    public int getScore() {

        return score;

    }

    

    /**

     * 通过队员名字查找所属队伍分数

     * @param teams 支持多个队伍

     * @param playerName 

     * @return 兜底为0分，不出现负分

     */

    public int getTeamScoreForPlayer(List<Team> teams, String playerName) {

        for (Team team :teams) {

            if (team.containsPlayer(playerName)) {

                return team.getScore();

            }

        }

        return 0;

    }

}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78

从优化后的代码中，你就能直观地看到，“命名的优化加上注释的说明”一下子就让源代码的逻辑变得清晰起来，即便你没有学过编程，也能大致了解这段代码的逻辑和作用。

2. 改进控制流和逻辑

这里我们还是直接从一个例子开始，具体代码如下：

public List<User> getUsers(int id) {

    List<User> result = new ArrayList<>();

    User user = getUserById(id);

    if (null != user) {

        Manager manager = user.getManager();

        if (null != manager) {

            List<User> users = manager.getUsers();

            if (null != users && users.size() > 0) {

                for (User user1 : users) {

                    if (user1.getAge() >= 35 && "MALE".equals(user1.getSex())) {

                        result.add(user1);

                    }

                }

            } else {

                System.out.println("获取员工列表失败");

            }

        } else {

            System.out.println("获取领导信息失败");

        }

    } else {

        System.out.println("获取员工信息失败");

    }

    return result;

}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

这段代码的含义是：想要通过 id 来查询员工的信息，如果 id找不到，就查询员工的领导，然后通过他领导下的员工信息来寻找，这时还需要判断员工年龄大于 35 岁且为男性。

这是我们最常使用的逻辑实现方式，俗称箭头型代码，但是随着判断条件逐渐增多，嵌套就会增多。代码逻辑越多，你就越容易搞不清楚逻辑是什么，因为看到最内层的代码时，你已经忘记前面每一层的条件判断是什么了。

那么，我们该如何去优化呢？其实很简单，就是改变控制流，先判断会出现失败的条件，一旦出现优先推出。优化后的代码如下：

public List<User> getStudents(int uid) {

    List<User> result = new ArrayList<>();

    User user = getUserByUid(uid);

    if (null == user) {

        System.out.println("获取员工信息失败");

        return result;

    }

    Manager manager = user.getManager();

    if (null == manager) {

        System.out.println("获取领导信息失败");

        return result;

    }

    List<User> users = manager.getUsers();

    if (null == users || users.size() == 0) {

        System.out.println("获取员工列表失败");

        return result;

    }

    for (User user1 : users) {

        if (user1.getAge() > 35 && "MALE".equals(user1.getSex())) {

            result.add(user1);

        }

    }

    return result;

}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

现在，代码逻辑是不是很清晰了？虽然这个快速失败方法很简单，但是非常有效。实际上，快速失败就是 KISS 原则一个很好的实践，这样能保证条件判断的逻辑简单清晰。只要 if 的嵌套超过三层，你就可以应用这个原则来改进控制流，让逻辑更清晰易懂。

3. 避免惯性思维

除了改进表层和逻辑外，我们更应该尽量避免设计代码时的一些惯性思维，这里我总结出了“五个避免”，下面我们就来具体分析一下。

第一，要避免一次性代码。一次性编码最大的坏处在于，一旦需要修改，多处就得跟着修改，而多次修改又可能会出现遗漏的风险。一次性代码在越来越多的软件代码中出现，一个本质的原因就是多人协作开发的情况越来越多。由于编程是一件非标准化的事情，不同程序员可能对同一个逻辑的理解完全不同，而一旦每个人都只从自己的角度出发写一次性代码，那么同一个系统里的代码很快就会变得冗余与混乱。

第二，要避免复制粘贴代码。一方面，不同的人编码风格可能会有所不同，这会给阅读者在理解上造成一定的认知负担（需要来回切换判断标准）。另一方面，还会带来未知 Bug的风险。复制过来的代码，更多是关注输入和输出，一旦代码正常运行后，很少会去关注代码的内部逻辑，但是等出现问题后，再想去梳理逻辑反而变得更加困难（因为不知道详细的实现逻辑）。

第三，避免写超长代码。超长代码带来的最大问题是：在阅读代码时，函数方法之间的跳转过多，思维很容易发生混乱，尤其对于一些命名相同但参数不同的方法，很容易出现修改错误的情况。从编写者的角度来看，你写超长代码，可能是觉得在一个文件里维护代码比较方便；但对于阅读者来说，他可能并不知道你是如何对代码进行职责划分的，更多时候他都会以为一个类里都是一个职责，但实际上一旦出现多个职责，加上逻辑跳转很多，阅读者基本上是会放弃阅读的。

第四，避免过度简化命名和表达式。在开发任务重的时候，我们通常会选用一些简化命名的方法，比如，num1、num2、num3 这类变量命名形式。虽然在写代码的时候，我们可能记得这些变量的含义，但是过一段时间后，如果没有注释或说明，几乎是不可能直接通过名字知道它们的作用的，还得借助上下文代码，这样不仅费时，而且还可能会出现理解错误的情况。

第五，避免写“是什么”的注释。代码的命名和结构如果能直接反映出来“是什么”的话，我们就不应该用注释去表达，因为看代码一眼就能明白，比如，获取用户信息的方法名——get 和 getFromUserInfo。

我们应该多写“为什么”的注释，比如，为什么要多加一个适配的方法，原因可能是线上 xxx 问题引起，或临时修复的Bug，后续可能随 xxx 接口调整而废弃，等等。在很多优秀的开源框架中，我们都能看到作者会在 interface 接口上写很多“为什么”的说明，就是为了帮助我们快速抓住代码的逻辑线索。

另外，写“为什么”的注释还有一个好处：尤其在早期快速迭代过程中，能给后来的维护者提供一个优化的切入点，而不至于交接代码后让维护代码的人看不懂、不敢动。

总结

表达原则的核心思想在于：通过代码清晰地表达我们的真实意图。

虽然软件开发过程中会有诸多文档，比如，架构设计文档、流程文档、PRD 文档等，但这些文档并不足以帮助我们正确理解代码是如何运行和组织的，很多时候我们只能通过阅读代码来掌握软件的真实运行情况。

我们之所以应该把提高代码可读性作为第一要务，就是因为读代码的次数远比写代码的次数多，包括你正在写的代码也是如此。

今天我们学习了三种主要的改进办法：一是表层的改进，从命名、表达式、变量和注释入手去提升代码可读性；二是改进代码逻辑，比如用策略模式替换嵌套过多的if-else 代码；三是改进思维习惯，习惯的改善更多是需要日积月累的坚持。

课后思考

学习完表达原则后，你一定意识到编程中表达意图的重要性了，那是不是就应该一味地只重视表达而忽略算法和数据结构呢？还有哪些你认为很重要却在编程中常常被忽略的地方呢？

欢迎留言分享，我会第一时间给你回复。

在下一讲，我会接着与你分享“职责原则：如何在代码设计中实现职责分离？”的相关内容，记得按时来听课！