问题背景

我的天，最近做 Code Review 看到一个同事的骚操作，他写了一个工具类，大概是这样的：

publicstaticbooleanisNull(Objectobject){

returnnull==object||object.equals(null);

}

判断空，一般不是 null == object 就够了，object.equals(null) 是什么骚操作？

写程序这么多年，第一次看这样的写法，当时我就提出质疑了，同事拍着胸脯和我说，有个银行的请求参数必须得这么写，不然就验证不了。

我当时还在想，这是 JDK 出的什么新类型么，觉得还是不科学，考虑去跟下同事写的代码，然后用他所说的情况我亲自去验证一下。

看了下，这是个老业务系统，同事用了 json-lib 这个包，历史的江湖确实有这个包的存在，栈长之前也用过，不过后来这玩意就没怎么用了，现在都是 Gson、Jackson 的天下了。

如下面 json-lib 例子所示：

public static void main(String[] args) {

String jsonString ="{\"name\": \"hi\",\"sex\": \"boy\", \"age\": null}";

JSONObject jsonObject = net.sf.json.JSONObject.fromObject(jsonString);

Object age = jsonObject.get("age");

//输出：null

System.out.println("age: "+ age);

//输出：false

System.out.println("age == null: "+ (age ==null));

//输出：true

System.out.println("age.equals(null): "+ (age.equals(null)));

}

我天！大家看到结果了吧，问题确实也如同事所说，一定要用 object.equals(null) 写法才行，不相信结果的大家也可以亲自验证一下。

纳了闷了，这样写，我传一个 null 值过去不是报空指针了么？这样写肯定有问题，继续深挖！

问题分析

从 fromObject 方法加载 JSON 串开始源码深入分析，找到了这个神奇解析 null 值的源码：

原来，JSON 串中的 null 值被解析成了它内部的 JSONNull 对象，然后再看下这个JSONNull 的 equals 方法源码：

publicbooleanequals(Objectobject){

returnobject==null||

object==this||

object== instance ||

objectinstanceof JSONObject &&

((JSONObject)object).isNullObject() ||

"null".equals(object);

}

问题就出在他所用的 JSON 工具类了！！！

equals 方法被重写了……终于揭开了 object.equals(null) 的神秘面纱……

再来看下是否有新的更新包：

最新的版本停留在 2010 年 12 月，已经是被淘汰的东西了。

另外，json-lib 在 JDK 1.7+ 有性能影响。推荐阅读：请不要在 JDK 7+ 中使用这个 JSON 包了

解决方案

方法1：

换掉 object.equals(null)，用 JSONNull 的实例去判断：

publicstaticbooleanisNull(Objectobject){

returnnull==object|| JSONNull.getInstance().equals(object);

}

方法2：

换掉 json-lib 库，用主流的 Gson、Jackson，具体看下这篇：Java常用的几个Json库，性能强势对比，Json-lib 真不建议用了，另外 FastJson 也不建议用，漏洞比较多。

这个由于是老系统，太多业务使用了这个库，换掉的开发、测试成本和风险比较大，暂时考虑先用方案1先解决这个问题。