static 全局变量滥用，引发线上故障

很多时候，并不是真的不懂，只是一时疏忽，产生了线上问题。

踩坑经验使你在第二次犯相同错误时及时发现。—— 琼斯

如何在避免自己踩坑的情况下获得丰富的踩坑经验，最好的办法就是窥视别人的踩坑记录。

故障描述

订单退款/改价，金额计算有误。

          

故障原因

费项枚举定义了不同的费项配置，费项列表是全部费项枚举的集合，是 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声明的变量为静态成员变量，为该类的公用变量，被该类的所有实例共享，在类加载时被显式初始化。

          

实际开发过程中，不建议对静态成员变量进行修改；

在编写方法返回值时，也不建议直接返回静态成员变量，因为调用方不知道返回的是静态变量，可能会对其进行修改。