为什么阿里巴巴Java开发手册编码规范里推荐单个方法的总行数不超过80行

x33g5p2x  于2021-12-17 转载在 Java  
字(1.1k)|赞(0)|评价(0)|浏览(389)
该篇文章参考了:
1. 阿里巴巴《java开发手册》v1.5.0 华山版
2. 《码出高效:Java开发手册》

单个方法的总行数不超过80行

在阿里巴巴《java开发手册》v1.5.0 华山版中第十一条
推荐单个方法的总行数不超过80行

为什么

推荐80行呢?还是除了注释之外的?80行随便写几个if、else 不就快80行了吗?

高内聚、低耦合

把相关的功能强内聚,把弱相关的功能拆解开来, 重新抽象、重新封装。是每个方法都都遵守单一职责原则。这样也有利于方法的复用。

魔数 3

心理学认为人对事物的印象通常不能超过3 这个魔法数, 三屏是人类短期记忆的极限,而80 行在一般显示器上是两屏半的代码量。

通过对阿里代码抽样调查显示, 只有不到5% 的方法才会超过80 行, 而这些方法通
常都有明显的优化空间。

如何重构、优化代码(压缩代码到80行)

虽然开发手册只是推荐级别,但不可否认的是:谁也不能忍受方法过长,比如1000行。假设现在公司确定下来规范,单方法不能超过80行或者100行或者120行,那么如何将长行数方法重构到这个数呢?

重构、抽象、封装

按照方法单一职责,一个方法只做一件事,封装单独的方法,把相关的功能强内聚,把弱相关的功能拆解开来,这样做有两个好处:

  1. 方便后面复用方法
  2. 方便编写单测

而对于封装方法时,涉及到方法的入参,通常会纠结参数的处理,拆分的方法间需要参数传递数据。
限制参数列表过长的方式有很多, 比如包装成类、隐式传递或放在集合中等。

如果不抽取、封装,一个变量可从方法第1行使用到第1000行,虽然不用考虑方法间用参数传递数据,
 但是超过3屏,你大概率会忘了这个变量声明的意义

条件取反,减少代码缩进

方案一
// idList 可能为方法入参,也可为从数据库查询
if(!CollectionUtils.isEmpty(idList)){
     for(String id : idList){
         // do something
     }
 }
方案二
Assert.notEmpty(idList,"idList should not be empty");
if(CollectionUtils.isEmpty(idList)){
    //throw new IllegalArgumentException("idList should not be empty");
    return;
}
for(String id : idList){
    // do something
}

对比两方案,方案二比方案一减少了一层代码缩进,同时在方案二中,若入参较多,且入参校验较多时,可单独提取公共方法。

卫语句 Guard clauses

如果条件语句极其复杂,就应该将条件语句拆解开,然后逐个检查,并在条件为真时立刻从函数中返回,这样的单独检查通常被称之为卫语句

设计模式

策略模式、状态模式

相关文章