常说的代码整洁到底是什么？

发表于 2年以前 | 总阅读数：392 次

代码整洁之道

写在前面

这个文章本来没打算写，直到经历了几次代码评审会议之后，我意识到自己编码方式还不成系统，仍然需要进行系统化的学习，掌握前辈们总结出的最适用的规律无疑是一种好的方式。恰好很早之前就收藏了这本代码整洁之道，便决定趁着闲暇之际阅读总结一下，如果想系统学习的话建议还是读书，本文档只是作为自己的记录用。

一个人的职业素养体现在解决问题的方式、步骤以及反思的程度，而不在于这个问题本身的难度。思考一个问题：一个技术人员要具备哪些素质可以被认为是专业人员呢？如果还不具备需要如何改变才能被视为专业人士呢？

整洁代码

一、为什么要写糟糕的代码？

每个人都有自己的原因，相信很多人都会想着等有时间的话再进行代码优化，但是要记住一句话：稍后等于永不。

二、混乱代码的代价？

后续难以维护和修改，生产力和时间呈现负相关。

三、什么是整洁的代码？

整洁的代码只做好一件事：每个类、每个函数、每个模块都专注于一事，完全不受四周细节的干扰和污染。

更全面的概括是：减少重复代码、提高表达力、提早构建简单抽象。

更具体的实现：请接着往下看吧！

更好的变量命名

一、见名知意

二、抽象工厂：接口不要命名为IShapeFactory，前导字母对于用户来说其实是干扰，用户只需要知道那是个抽象工厂，建议使用CShapeFactory或许体验更好

三、类名要用名词，方法要用动词，词性相近的get、fetch这种词不应出现在一起，可以添加后缀getNumber、fetchData实现相同的效果

四、别害怕长名字：使用描述性的名称，哪怕比较长也比短而令人费解的名称好

更好的函数

一、函数的结构本质上要短小、再短小，以不容纳if/else if/else嵌套结构为目标

二、只做一件事：正如前面所说，函数只做好一件事就足够了，标志就是“看是否还能拆出一个函数，该函数不仅只是单纯地重新诠释其实现”

三、每个函数一个抽象层级：代码一般是“自顶向下”的阅读顺序，每个函数后面跟着的应该是下一抽象层级的函数

[抽象层级：getHtml函数位于较高抽象层，pagePathName = pathParser.render(pagePath)位于中间抽象层，.append("\n")则位于较低抽象层]

四、switch语句：天生就需要做N件事，但是可以将其放置在较低抽象层级，但是当出现新的类型时会违反“单一权责原则、开放闭合原则”，此时最好创建多态对象

//原文中：对于每个case分支进行单独处理，添加新类型不必修改原来的代码增加新的处理类即可
function getName(name){
    switch(name){
        case 'ming':
            return new ClassMing(name);
        case 'hu':
            return new ClassHu(name);
        case 'uzi':
            return new ClassUzi(name);
        default:
            throw new ClassCommon(name);
    }
}

//我更喜欢用另一种方法：修改只需要在对象里修改即可，且提高了函数的简洁性
const nameCollectionUtils = {
    ming：new ClassMing('ming');
    hu: new ClassHu('hu');
    uzi: new ClassUzi('uzi');
}
function getName(name){
    return nameCollectionUtils.hasOwn(name) ? nameCollectionUtils[name] : new ClassCommon(name)
}
复制代码

五、函数参数最多不多于两个：包括输入参数和输出参数

六、无副作用：函数内部不要做出未能预期的改动，不要对外部产生影响

七、使用异常替代返回错误码：使用try...catch替代多层级的if嵌套，永远走在主路上，不要过多考虑边界，这样可以让你一直保持思维连贯

八、错误处理单独抽出：这一条我认为可以视情况而定，毕竟抽出仅仅是为了美观

九、别重复：多个函数使用的相同逻辑的代码一定要抽出，可以参考面向对象的基类，前端开发中的面向组件编程、面向模块编程也是这种思想

注释&格式

每个人有每个人的习惯，采取一些通用准则即可，毕竟如何太过离谱在公司是会挨打的~

错误处理

也没有什么固定的章程，最好采取try...catch优先的原则

边界

总结而言，使用自己可以控制的代码

单元测试

现在的互联网企业绝大多数都是敏捷式开发，很少有能遵守测试驱动原则的公司，而且为了保证进度很少会有技术团队会去要求单元测试，所以这一条仁者见仁吧，个人认为这一项的实际实现只能是一个比较美好的愿景。

类

一、类的组织：按照下面的顺序，不要暴露出内部属性，利用方法达到同样的目的

class DemoOrganization{
  static sname = 'sname'
  private pname = 'pname'
  private _pname = '_pname'
  protected tname = 'tname'

  public getPublicName(){
    return this.pname
  }     
  private _getPrivateName(){
    return this._pname
  }
}
复制代码

二、单一权责原则（SPR）：类或模块应有且只有一条加以修改的理由，实现了这个原则的类更易得到复用

三、保持内聚性：类中定义的变量应被尽可能多的方法使用到，如果不能满足的话就把使用到变量的函数拆分成小类

四、开放封闭原则（OCP）：类应当对扩展开放，对修改封闭，通过子类化手段可以实现新功能的添加的同时不触及其他类

五、依赖倒置原则（DIP）：类应当依赖于抽象而不是依赖于具体细节

六、解耦：不同方法和模块间不要互相产生影响，即“分而治之”、“化整为零”