很多时候,并不是真的不懂,只是一时疏忽,产生了线上问题。
踩坑经验使你在第二次犯相同错误时及时发现。—— 琼斯
如何在避免自己踩坑的情况下获得丰富的踩坑经验,最好的办法就是窥视别人的踩坑记录。
故障描述
订单退款/改价,金额计算有误。
故障原因
费项枚举定义了不同的费项配置,费项列表是全部费项枚举的集合,是 static 修饰的全局变量。
这两元素都定义在 API 包中,由调用方直接引用。
某次改价需求,对部分费项做特殊的处理时,直接操作了全局的费项列表,费项列表数据不完整,
导致后续的退款/改价,费项金额计算有误。
以下是一段简单的代码演示:
/**
* 费项列表
*/
public static ListFEE_ITEM_LIST = Lists.newArrayList(FeeItem.values());
public static void main(String[] args) {
// 初始打印
System.out.println(FEE_ITEM_LIST.toString());
// 模拟请求一 移除 FEE_ITEM_LIST 中的某个费项
FEE_ITEM_LIST.removeIf(feeItem -> TRANSPORTATION_FEE.equals(feeItem));
// 模拟请求二访问
System.out.println(FEE_ITEM_LIST.toString());
输出结果如下:
[BASE_FEE, TRANSPORTATION_FEE, EXTRA_FEE]
[BASE_FEE, EXTRA_FEE]
上述代码只是演示,比较简单,一般都能够非常直观的看出其中的问题;
实际应用中的代码逻辑比这个复杂的多,如果不注意就这么改了,部署环境后由于通常都是多台机器,可能测试时不一定能够立马发现问题。
故障总结
在 Java 中,static 表示全局或者静态的意思,在类中用 static声明的变量为静态成员变量,为该类的公用变量,被该类的所有实例共享,在类加载时被显式初始化。
实际开发过程中,不建议对静态成员变量进行修改;
在编写方法返回值时,也不建议直接返回静态成员变量,因为调用方不知道返回的是静态变量,可能会对其进行修改。