• 简化条件表达式之分解条件表达式(Decompose Conditional)


    你有一个复杂的条件语句。从if、then、else三个段落中分别提炼出独立函数

              动机:

           程序之中,复杂的条件逻辑是最常导致复杂度上升的地点之一。你必须编写代码来检查不同的条件分支、根据不同的分支做不同的事,然后,你很快就会得到一个相当长的函数。大型函数自身就会使代码的可读性下降,而条件逻辑则会使代码更难阅读。在带有复杂条件逻辑的函数中,代码(包括检查条件分支的代码和真正实现功能的代码)会告诉你发生的事,当常常让你弄不清为什么会发生这样的事,这就说明代码的可读性的确大大降低了。

           和任何大块头代码一样,你可以将它分解为多个独立函数,根据每个小块代码的用途,为分解的新函数命名,并将原函数中对应的代码改为调用新建函数,从而更清楚的表达自己的意图。对于条件逻辑,将每个分支条件分解为新函数还可以给你带来更多好处:可以突出条件逻辑,更清楚地表明每个分支的作用,并且突出每个分支的原因。

              做法:

           1、将if段落提炼出来,构成一个独立函数。

           2、将then和else段落提炼出来,各自构成一个独立函数。

           如果发现嵌套的条件逻辑,先观察是否可以使用Replace Nested Conditional with Guard Clauses (以卫语句取代嵌套条件表达式)。如果不行,才开始分解其中的每个条件

    if (username == null || username.equals("") || password == null || password.equals("")) {
            //todo
     }

    这段条件代码要做的事情就是判断用户名和密码是否正确,但是它用了三个“||”运算符,整个条件判断显得十分冗长。那么,怎么简化呢?我们可以分别把它们提出来,就像下面这样

    if (userNameIsValid(username) && passwordIsValid(password)) return true;
        //todo
    }
     
     private boolean userNameIsValid(Object userName) {
        if (userName == null || userName.equals("")) return false;
        return true;
    }
     
    private boolean passwordIsValid(Object password) {
        if (password == null || password.equals("")) return true;
        return true;
    }

     虽然条件语句很短 但是不要觉得短就没有提炼的必要,我们要关注的是代码意图和它自身的差距。提炼之后可读性会更高,它看起来就像一条注释一样简单明了。


    参考 https://blog.csdn.net/CountryWisdom/article/details/41286837

  • 相关阅读:
    WPF Layout & Image异步加载
    WPF Binding Validation 数据验证
    推荐一个.NET 命令行参数Parser 库
    Windows 下 命令行增强工具
    Windbg 离线调试.Net 程序入门
    拼写检查算法 Golang 版
    新Blog
    WPF 实现Loading效果
    struct结构体的变长特性
    第2章 构造函数语意学
  • 原文地址:https://www.cnblogs.com/newbee0101/p/11981676.html
Copyright © 2020-2023  润新知