一个表示金额的数字是 100000000L，这是多少米（多少钱）？【代码可读性实践】

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

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

1. ...
2. bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
3. if (bankOrder.getAmt() < 100000000L) {
4.     bb1payopx1.setStlChn(CmbCloudSettlementTypeEnum.REAL_TIME.getCode());
5. } else {
6.     bb1payopx1.setStlChn(CmbCloudSettlementTypeEnum.EXPRESS.getCode());
7. }
8. ...

复制代码

有同学会说，不就是一个数字嘛，管它是多少呢？总之就是作为一个判断条件而已。
持这种想法的同学，估计这半辈子注定是一个码农。
作为应用开发者，我们要关注业务。这段代码所体现的业务逻辑是，转账交易金额低于 100w 时，银行会实时结算；而超过 100w 时，会走快速转账方式，结算时效会下降。
银行不同的结算通道，会直接影响到交易的时效。掌握了这一点，转账交易的业务层就可以调整相应的产品策略，以满足结算时效。

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

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

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

复制代码

或者定义一个常量。

1. public static final long ONE_MILLION_YUAN = 1000000L; // 或命名为 YUAN_100W
3. ...
4. bb1payopx1.setTrsAmt(bankOrder.getAmt() + "");
5. if (bankOrder.getAmt() < ONE_MILLION_YUAN * 100) {
6.     ...

复制代码

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

分享我的good-practice。
实践一：使用Java 7 开始引入的“语法糖”——数字分隔符，告别数零（数是数羊的数）。

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

复制代码

实践二：使用Money类，一目了然。

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

复制代码

来源：豆瓜网用户自行投稿发布，如果侵权，请联系站长删除