89无数次的事实证明，单一职责，结构清楚，bug最少

Posted 2022-02-01 小雷FansUnion

无数次 一次又一次的 事实证明，只要不把 代码 写清楚，就容易出现各种问题。 

今天是 复制粘贴，漏了一处 改名的。

// 可能1个拆成2个  private List<ResultRequest> resultRequest(ResultVo resultVo, Map<Integer, ProductVo> productMap,      Map<Integer, CrmCustomerVo> customerMap)      ResultRequest resultRequest = buildBaseResultRequest(resultVo, productMap, customerMap);      List<ResultRequest> resultRequestList = Lists.newArrayList();      resultRequestAll(resultVo, resultRequest, resultRequestList);      resultRequestSub(resultVo, resultRequest, resultRequestList);      return resultRequestList;

resultRequestAll和resultRequestSub，代码结构基本一致。

所以sub复制了all的，但是有一处忘记改名了。

因为在同一个大方法里，上下文找得到变量，语法不会报错。

如果拆分成2个独立的方法，复制粘贴，忘记改名，就会报错了。

及时发现错误，解决只要30秒。

不及时发现，可能得10分钟。

一行代码 一个方法 解决1个问题，就这么多行了。

如果面条代码，后来的人就痛苦了