我天!xx.equals(null) 是什么骚操作??
问题背景
我的天,最近做 Code Review 看到一个同事的骚操作,他写了一个工具类,大概是这样的:
判断空,一般不是 null == object
就够了,object.equals(null)
是什么骚操作?
写程序这么多年,第一次看这样的写法,当时我就提出质疑了,同事拍着胸脯和我说,有个银行的请求参数必须得这么写,不然就验证不了。
我当时还在想,这是 JDK 出的什么新类型么,觉得还是不科学,考虑去跟下同事写的代码,然后用他所说的情况我亲自去验证一下。
看了下,这是个老业务系统,同事用了 json-lib
这个包,历史的江湖确实有这个包的存在,我之前也用过,不过后来这玩意就没怎么用了,现在都是 Gson
、Jackson
的天下了。
如下面 json-lib
例子所示:
我天!大家看到结果了吧,问题确实也如同事所说,一定要用 object.equals(null)
写法才行,不相信结果的大家也可以亲自验证一下。
纳了闷了,这样写,我传一个 null
值过去不是报空指针了么?这样写肯定有问题,继续深挖!
问题分析
从 fromObject
方法加载 JSON 串开始源码深入分析,找到了这个神奇解析 null
值的源码:
原来,JSON 串中的 null
值被解析成了它内部的 JSONNull
对象,然后再看下这个 JSONNull
的 equals 方法源码:
问题就出在他所用的 JSON 工具类了!!!
equals 方法被重写了……终于揭开了 object.equals(null)
的神秘面纱……
再来看下是否有新的更新包:
最新的版本停留在 2010 年 12 月,已经是被淘汰的东西了。
另外,json-lib
在 JDK 1.7+ 有性能影响。
解决方案
方法1:
换掉 object.equals(null)
,用 JSONNull
的实例去判断:
评论