专业的JAVA编程教程与资源

网站首页 > java教程 正文

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

temp10 2024-12-16 16:12:25 java教程 11 ℃ 0 评论

该篇文章参考了:

1. 阿里巴巴《java开发手册》v1.5.0 华山版

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

2. 《码出高效:Java开发手册》

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

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

为什么

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

高内聚、低耦合

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

魔数 3

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

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

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

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

重构、抽象、封装

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

  • 方便后面复用方法
  • 方便编写单测

而对于封装方法时,涉及到方法的入参,通常会纠结参数的处理,拆分的方法间需要参数传递数据。

限制参数列表过长的方式有很多, 比如包装成类、隐式传递或放在集合中等。

如果不抽取、封装,一个变量可从方法第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

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

设计模式

策略模式、状态模式

本文暂时没有评论,来添加一个吧(●'◡'●)

欢迎 发表评论:

最近发表
标签列表