无数次 一次又一次的 事实证明,只要不把 代码 写清楚,就容易出现各种问题。
今天是 复制粘贴,漏了一处 改名的。
// 可能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个问题,就这么多行了。
如果面条代码,后来的人就痛苦了