都是运算符优先级的锅

向右看齐 2022-10-28 10:11 130阅读 0赞

*记一次生产环境出现的问题。*

最近生产环境出现了一个问题，用户登录自己公司的账号，下载公司员工的数据，结果EXCEL中不是本公司的数据，而是其他公司的数据。这种出现数据混乱的问题也是一件比较严重的事情了，你都可以看见别的公司的数据，数据的安全性没得到保证。出现问题的原因在于代码中使用了缓存，缓存将用户数据放进本地缓存中，当用户下载的时候，首先从缓存中获取，如果缓存中有数据，则取出数据，并从缓存中移除。这样看似没有什么问题，问题就在于对于缓存key的设置上。缓存key没有正确设置成针对用户的唯一key，而是所有用户使用了同样的key，于是出现了当两个用户同时下载数据，会出现数据错误的现象。

下面来看看设置缓存key的代码：

public static void main(String[] args) {
       String value = "java";
       String token = "212434234234234423423";
       String phone = "11111111111";
       String key = token + phone + value!=null?value:"";
       System.out.println(key);
    }

对于上面的代码，最终的结果是什么呢？结果是java，你答对了吗？

正如上面代码所示，token+phone的组合对于每一个用户来说都是唯一的，问题在于计算key的代码token + phone + value!=null?value:"";上。

在Java中，+运算符的优先级要高于三目运算符，所以，这行代码，要先运行token + phone + value，然后再将计算结果执行三目运算符。即token + phone + value!=null?value:"";等价于

（token + phone + value)!=null?value:"";因为token + phone + value的值是永远不会为null的，所以每次的结果都会是java，这样就造成了所有用户的缓存key都相同了。而造成这样的错误的原因。

就是我没有认识到运算符的优先级，错误的认为会先执行三目运算符的逻辑，然后再执行+拼接字符串。也就是token + phone + (value!=null?value:"")。所以要想得到正确的结果，需要使用小括号来调整代码执行的优先级。