这个涉及到一个重构的问题.
也是我没有注意到.
在修改一个方法的时候一定要注意调用者的情况, 以及这个方法的本意.
比如有两个方法: getOrder
和 getHistoryOrder
, 很多地方都调用了 getOrder
这个方法获取用户正在进行的订单.
但是在解决某个 bug 的时候你发现, 需要查询用户所有的订单, 但是在那个 bug 里面调用的是 getOrder
方法, 然后你 (也就是我) 傻不拉的在 getOrder
中去查询历史订单.
导致的结果就是每个调用 getOrder
的方法都会去查询历史订单, 不仅会引发新的 bug 还会导致性能问题.
真的是防不胜防.
文章评论