一个NullPointerException，竟然有这么多花样！原创
案发现场
我们先看一下给出的异常栈
java.lang.NullPointerException
at org.springframework.data.redis.cache.RedisCache.get(RedisCache.java:180)
at org.springframework.data.redis.cache.RedisCache.get(RedisCache.java:133)
at org.springframework.cache.transaction.TransactionAwareCacheDecorator.get(TransactionAwareCacheDecorator.java:69)
at org.springframework.cache.interceptor.AbstractCacheInvoker.doGet(AbstractCacheInvoker.java:71)
at org.springframework.cache.interceptor.CacheAspectSupport.findInCaches(CacheAspectSupport.java:537)
at org.springframework.cache.interceptor.CacheAspectSupport.findCachedItem(CacheAspectSupport.java:503)
at org.springframework.cache.interceptor.CacheAspectSupport.execute(CacheAspectSupport.java:389)
at org.springframework.cache.interceptor.CacheAspectSupport.execute(CacheAspectSupport.java:327)
at org.springframework.cache.interceptor.CacheInterceptor.invoke(CacheInterceptor.java:61)
根据异常栈我们很轻松就能定位到源码位置
很明显exists为null，就会出现本篇的NullPointerException。当然如果这样三分钟没到本篇就草率结束，明显有失肥朝的男儿本色！另外若当真如此草率，那么关注肥朝公众号的意义何在？肥朝公众号的老粉丝们都知道，肥朝的海量源码实战类文章，都有三个特点
-
从源码原理角度分析，为什么会出现这个问题？
-
如何解决掉这个问题？
-
我们如何深度思考，不断从这次经历中压榨出最大价值。（非常重点！）
比如就拿这个问题来说，按照我们正常的思维惯性，我们看到176行有个return，又知道exists是null，那么我们点进connection.exists(cacheKey.getKeyBytes())这个方法一探究竟。
结果发现果然如我们所料，这里竟然有两个return null的情况,这个时候就有粉丝把持不住，要九浅一深直入源码分析，看这两个条件什么时候满足。但是肥朝想说的是，且慢动手！
你注意看这段代码
Boolean exists = (Boolean) redisOperations.execute(new RedisCallback<Boolean>() {
@Override
public Boolean doInRedis(RedisConnection connection) throws DataAccessException {
return connection.exists(cacheKey.getKeyBytes());
}
});
public interface RedisOperations<K, V> {
<T> T execute(RedisCallback<T> action);
}
我们现在已知execute的返回值T（exists）为null，T和action的返回值的关系，要看execute代码里面的具体实现的！这点很重要，也很容易被疏忽！由于文中提到了随机出现，那么这里就涉及到一个排查的技巧。一般和随机出现有关的，根据经验，主要从两个方向入手
1.根据多个异常的情况的入参数据来分析，看看多个异常情况的入参都有什么特点。
2.模拟并发量，因为很多问题本地重现不了是因为并发量不够。
该粉丝根据方式二，将自己的项目代码中的Redis代码抽取了一个最简模型，将问题重现。但是由于该模型是ssm项目，为了方便大家都能重现并参与其中,我用springboot抽取了相关模型，如下图:
真相大白
将上面的代码运行，果不其然出现了我们想要找的异常栈。
我们把断点打在了上面说的那两个可能返回null的地方，可能你却惊奇的发现，断点根本没有进去。那就只剩下一个可能了。那就是redisOperations.execute()方法。我们发现，demo中继承RedisTemplate自定义了RedisOperations，名为RedisCustomTemplate。然后我们在catch处打上断点。
从这里我们就知道了。连接池参数blockWhenExhausted = true(默认),如果连接池没有可用Jedis连接，会等待maxWaitMillis(毫秒)，依然没有获取到可用Jedis连接，会抛出这个异常。
根据沟通了解到，该同学生产代码中，catch住后是没有任何日志输出，直接返回了null，自然就导致了NullPointerException。
如何解决
肥朝认为解决问题要从两个方面入手。
1.连接池为什么不够？连接数不够时，盲目调大连接数是最常见的错误做法，根本解决办法还是要挖掘背后不够的原因，一般连接数不够，根据《Redis开发与运维》作者付磊在书中总结出无非是以下几点。
-
1.1 连接泄漏（较为常见）
-
1.2 业务并发量大，maxTotal确实设置小了。
-
1.3 Jedis连接还的太慢（Redis发生了阻塞，例如慢查询等原因）。
-
1.4 其他问题（例如丢包、DNS、客户端TCP参数配置）。
具体是属于上述哪个情况，自己对症下药。
2.自定义RedisCache。
我们知道RedisCallback确实会存在两个返回null的情况，根据
if (!exists) {
return null;
}
这样的判断方式，存在很大的空指针异常隐患，我们可以继承RedisCache，然后重写该方法，将这个判断的bug改掉，也可以去最大同性交友网站github上，查看最新版本的bug修复情况，当然很多公司更换jar版本都要走流程，所以具体处理方式，酌情处理。
问题复盘
在真相大白和给出解决方案后，那么我们就将整件事进行复盘。
提问的艺术
我们再回顾一下该粉丝的提问：“你们有没有遇到xxx问题”，其实显而易见，这样的提问方式，相信和他一样因为这个不规范，并且公司还有一定流量触发出问题的概率，小之又小。如果把“你们有没有遇到过xxx问题”换成“我遇到了一个xxx问题”，这样可能回复率还会多一丢丢。当然，问人毕竟是最低效的解决方式，最重要的还是，要掌握分析问题的技巧，以及源码实战的经验。当然很多同学反馈，公司根本没有源码实战经验的机会，因此，关注肥朝公众号，积累源码实战经验就显得格外重要了。
编码规范
如果同学没有吞掉异常，日志输出了真实异常
org.springframework.data.redis.RedisConnectionFailureException: Cannot get Jedis connection;
nested exception is redis.clients.jedis.exceptions.JedisException: Could not get a resource from the pool
那么这个问题简直是随便搜索一下都秒解决。但是关键就在于吞掉了真实异常，并且返回null，非常碰巧的是
public Boolean exists(byte[] key) {
try {
if (isPipelined()) {
pipeline(new JedisResult(pipeline.exists(key)));
return null;
}
if (isQueueing()) {
transaction(new JedisResult(transaction.exists(key)));
return null;
}
return jedis.exists(key);
} catch (Exception ex) {
throw convertJedisAccessException(ex);
}
}
该方法还有两个返回null的情况，给排查时造成了极大的迷惑性，容易让我们把精力放在了这里。
命名不规范。
在沟通中，我发现demo中RedisCustomTemplate这个成为问题最关键突破口的地方，在他们公司，竟然被起了一个欺骗性的命名。
异常输出。
从该粉丝的demo中，我们看到了这样的代码。
别以为这只是个demo无所谓，细节往往能看出编码的意识! 大家可以检查一下自己的项目。
ex.printStackTrace();
的方式。这样的方式存在两个非常大的隐患。
4.1 该方式用到了synchronized，当然这个还是小问题，毕竟synchronized在jdk 1.6做了很多优化，性能提升了很大。（这个很多优化到底是啥优化，后续肥朝再讲解）
public void println(String x) {
synchronized (this) {
print(x);
newLine();
}
}
4.2 这样的输出方式，无法将异常输出到日志文件。因为这个是jdk的方法，人家怎么可能输出到你要的文件路径。然后我们生产分析问题，都是要看日志，你通过这种方式输出，自然就会导致关键信息丢失！
4.3 ex.getMessage()的方式，是无法输出异常栈信息的。我们来看一下阿里规范手册提到的输出方式：
