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

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

// 可能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个问题,就这么多行了。

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