上一次我们讲过,关于代码覆盖率为0的原因,很大概率是因为我们相同的类,但是classId不同导致了这个问题,所以我们在上一篇文章里面提到过,通过类名再做一次合并的动作避免这种问题的出现。结果在改完代码逻辑以后,还是相同的同学,他又出现了一个一样的现象, 抱着一样的排查方式,我又重新排查了一次。
背景
先介绍下项目的背景: 出现问题的类名是 xxx.service.impl.MessageServiceImpl
,测试同学反馈的还是一样的问题,就是代码逻辑其实他通过手工测试已经触发了,但是在覆盖率报告里面却是没有覆盖,出现这个问题,其实我们刚才已经说过了,基本上是因为classId不一致导致了这个问题。所以我们重新查看下exec文件的情况。
我们分别找到了两个不同的服务覆盖率文件,发现了一个特殊的情况,竟然两个相同的类名探针的数据是不一样的。按理来说不应该会发生这种情况才对,因为相同的代码探针插入的情况应该是一样的才对,除非是代码不相同(开始我并没有往这块去思考)
分析
既然这样子我们就去分析下插桩后的代码的探针是啥样的,这里我们需要使用到阿里的一个神器了 arthas。 我们需要使用的是他一个反编译的功能,这里就不介绍arthas的使用方法了,有兴趣的同学可以百度了解下。
jad xxx.service.impl.MessageServiceImpl
通过以上的命令我们就得到了反编译后的代码
private static /* synthetic */ boolean[] $jacocoInit() {
boolean[] blArray = $jacocoData;
if ($jacocoData == null) {
Object[] objectArray = new Object[]{-8970768149164403672L, "com/seewoedu/train/service/impl/MessageServiceImpl", 539};
UnknownError.$jacocoAccess.equals(objectArray);
blArray = $jacocoData = (boolean[])objectArray[0];
}
return blArray;
}
我们可以看到大概这样子的效果,这里的blArray实际上就是探针的数组长度, 这里定义的就是539的长度,
我们再看下另外一个服务反编译后的代码情况
private static /* synthetic */ boolean[] $jacocoInit() {
boolean[] blArray = $jacocoData;
if ($jacocoData == null) {
Object[] objectArray = new Object[]{8280241422563924486L, "com/seewoedu/train/service/impl/MessageServiceImpl", 540};
UnknownError.$jacocoAccess.equals(objectArray);
blArray = $jacocoData = (boolean[])objectArray[0];
}
return blArray;
}
这里的长度确实是540。 所以这个就是为什么代码覆盖率不准确的原因了,因为探针长度不一样,导致覆盖率数据合并不成功导致的。
既然都已经能反编译出具体的代码代码内容了,那我们就看下具体不一样代码的地方吧。
通过以上截图我们终于知道为什么了,因为这两处的代码是不一样的,导致探针的数组就不一样了。
分析了一大波,结果发现问题的原因其实很简单,就是因为两个被测的服务其实使用的公共模块的代码并不是一致的,所以导致了这个问题。