查看: 37|回复: 0

一个表示金额的数字是 100000000L,这是多少米(多少钱)?【代码可读性实践】

[复制链接]

4

主题

0

回帖

0

积分

热心网友

金币
0
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2011-5-16
发表于 2025-8-12 16:14:00 | 显示全部楼层 |阅读模式

易读的代码(Readable Code)是高质量软件开发的核心要素之一


今天下午,我在CR时,注意到下面的银行付款程序中的100000000L。如果不数零,谁能直观看出来这个庞大的数字代表多少钱呢?

...
bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
if (bankOrder.getAmt() < 100000000L) {
    bb1payopx1.setStlChn(CmbCloudSettlementTypeEnum.REAL_TIME.getCode());
} else {
    bb1payopx1.setStlChn(CmbCloudSettlementTypeEnum.EXPRESS.getCode());
}
...

有同学会说,不就是一个数字嘛,管它是多少呢?总之就是作为一个判断条件而已。

持这种想法的同学,估计这半辈子注定是一个码农。

作为应用开发者,我们要关注业务。这段代码所体现的业务逻辑是,转账交易金额低于 100w 时,银行会实时结算;而超过 100w 时,会走快速转账方式,结算时效会下降。

银行不同的结算通道,会直接影响到交易的时效。掌握了这一点,转账交易的业务层就可以调整相应的产品策略,以满足结算时效。


那么,从代码可读性的角度来看,怎么能让人一眼就能看出来100000000L表示的是 1百万 呢?


有同学说了,使用javadoc加个注释。

bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
// 100 万以下走 R-实时转账,≥100 万走 Q-快速转账
if (bankOrder.getAmt() < 100000000L) {
    ...

或者定义一个常量。

public static final long ONE_MILLION_YUAN = 1000000L; // 或命名为 YUAN_100W

...
bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
if (bankOrder.getAmt() < ONE_MILLION_YUAN * 100) {
    ...

当然, 除此之外,你还有什么更好的实践?

image





分享我的good-practice。

实践一:使用Java 7 开始引入的“语法糖”——数字分隔符,告别数零(数是数羊的数)。

bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
if (bankOrder.getAmt() < 100 * 1_000_000) {
    ...

实践二:使用Money类,一目了然。

bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
if (bankOrder.getAmt() < Money.ofYuan(1_000_000).fen()) {
    ...
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

相关侵权、举报、投诉及建议等,请发 E-mail:qiongdian@foxmail.com

Powered by Discuz! X5.0 © 2001-2026 Discuz! Team.

在本版发帖返回顶部