今天下午,我在CR时,注意到下面代码中的100000000L
。谁能看出来100000000L
是多少呢?
bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
if (bankOrder.getAmt()
有同学会说,管它是多少呢?总之就是一个分支判断而已。
持这种想法的同学,估计这半辈子注定是一个码农。
作为应用开发者,我们要关注业务。这段代码所体现的业务逻辑是,转账交易金额低于 100w 时,银行会实时结算;而超过 100w 时,会走快速转账方式,结算时效会下降。
银行不同的结算通道,会直接影响到交易的时效。掌握了这一点,转账交易的业务层就可以调整相应的产品策略,以满足结算时效。
那么,从代码可读性的角度来看,怎么能让人一眼就能看出来100000000L
表示的是 1百万 呢?
有同学说了,加个注释。
bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
// 100 万以下走 R-实时转账,≥100 万走 Q-快速转账
if (bankOrder.getAmt()
当然, 除了javadoc注释,你还有什么更好的实践?
欢迎评论区留言。我将在今晚放大招~提供一个最佳实践。
当看到一些不好的代码时,会发现我还算优秀;当看到优秀的代码时,也才意识到持续学习的重要!--buguge
本文来自博客园,转载请注明原文链接:https://www.cnblogs.com/buguge/p/19034109
参与评论
手机查看
返回顶部