buguge - Keep it simple,stupid

知识就是力量,但更重要的,是运用知识的能力why buguge?

导航

【注意】当同一个Class的多个对象出现在同一段代码内的时候,极易出错。

同一个方法里,多个BankOrderFlow对象...这像“shǐ”一样的代码

下面的 paymentQuery 方法坐落于我们lijianjin系统的LiJianJinPaymentQueryBizService。

这两个bankOrderFlow、orderFlow,... 乱花渐欲迷人眼的赶脚,但其实,这是像“shǐ”一样的代码。

 

 

 

 

无独有偶,bug来了

在我们的zhongtai-channel-provider, 上周出现并紧急fix了一个bug。竟然也是 此种情况 引发的bug。

先上截图。下图中这段代码,有 3 个 PayPaymentFlow 对象实例:payPaymentFlow、paymentFlow、payFlow。 这命名,与隔壁老王家的 大狗、二狗、三狗一样。

 

来复习一下我们紧急fix的bug。↓

我们在做内部重构时,不慎...,直接下面的改动吧。

通过上面的diff,你能看出来发生了什么bug吗? 看不出来的话也没关系,毕竟代码比较乱。那么,接着看下面fix后的diff,相信你能看出来。

 

 

bug是fix掉了。那么,谁又能保证未来不会再出现类似的bug呢?

 

因此,如何规避这样的bug?也就是说,如何重构呢?

 

重构要点

一个要点:将复杂的方法拆分为多个更小单元的方法,通过方法的封装来限制变量的作用域。

 

posted on 2024-04-28 19:49  buguge  阅读(79)  评论(1编辑  收藏  举报