1

我在一个不应该为空的变量上有一个奇怪的 java.lang.NullPointerException。

Object[] params = new Object[10];
if (param1 != null)
    params[0] = param1;

param1 作为方法参数给出，它可以为 null。通常，当影响 param1 到 params[0] 时，它不为空（否则它不会通过 if 语句）。

这是错误（第 144 行是“params[0] = param1;”）：

Exception in thread "Jalen Agent" java.lang.NullPointerException
    at jalen.MethodStats.onMethodEntry(MethodStats.java:144)
    at hanoi.TowersOfHanoi.solveHanoi(TowersOfHanoi.java)
    at hanoi.TowersOfHanoi.main(TowersOfHanoi.java:29)

以下是发生异常的方法的完整代码：

public static void onMethodEntry(int id, String className, String methodName, Object param1, Object param2, Object param3, Object param4, Object param5) {
    synchronized (globalLock) {
        Object[] params = new Object[10];
        if (param1 != null)
            params[0] = param1;
        if (param2 != null)
            params[1] = param2;
        if (param3 != null)
            params[2] = param3;
        if (param4 != null)
            params[3] = param4;
        if (param5 != null)
            params[4] = param5;
        MethodStats.onMethodEntry(id, className, methodName, params);
    }
}

编辑：

为了澄清我的例子，我处于以下情况：

  1. 使用 ASM 检测 Java 应用程序
  2. 在使用 Java 代理时运行新的检测类
  3. 代理将使用检测代码收集的信息来运行一些测量
  4. 代理还收集应用程序的方法参数值。

为此，在每个方法运行时都会执行 onMethodEntry。我有几种具有不同签名的方法。尤其是：

public static void onMethodEntry(int id, String className, String methodName, Object[] params) {
    synchronized (globalLock) {
        StringBuilder fullMethodName = new StringBuilder(className).append('.').append(methodName).append('-').append(Thread.currentThread().getId());
        MethodStats.addNewMethod(id, fullMethodName.toString());
        System.out.println(fullMethodName.toString() + " -- " + id);
        for (Object param : params) {
            if (param != null)
                System.out.println("Param: " + param.toString());
        }
        startTimes[depth] = System.nanoTime();
        stack[depth++] = MethodStats.getMethodInfo(id);
    }
}

public static void onMethodEntry(int id, String className, String methodName, Object param1) {
    synchronized (globalLock) {
        Object[] params = new Object[10];
        if (param1 != null)
            params[0] = param1;
        MethodStats.onMethodEntry(id, className, methodName, params);
    }
}

public static void onMethodEntry(int id, String className, String methodName, Object param1, Object param2) {
    synchronized (globalLock) {
        Object[] params = new Object[10];
        if (param1 != null)
            params[0] = param1;
        if (param2 != null)
            params[1] = param2;
        MethodStats.onMethodEntry(id, className, methodName, params);
    }
}

这是我用于检测程序类的代码（即河内塔）：

public void visitCode() {
    mv.visitLdcInsn(new Integer(this.methodID));
    this.visitLdcInsn(this.className);
    this.visitLdcInsn(this.methodName);
    String paramCall = "";
    if (this.numParam > 0) {
        // Load parameters
        for (int i=1; i<=this.numParam; i++) {
            this.visitVarInsn(Opcodes.ALOAD, i);
            paramCall += "Ljava/lang/Object;";
        }
    }
    System.out.println(paramCall);
    mv.visitMethodInsn(Opcodes.INVOKESTATIC,
            "jalen/MethodStats",
            "onMethodEntry",
            "(ILjava/lang/String;Ljava/lang/String;" + paramCall + ")V");
    super.visitCode();
}
4

2 回答 2

5

首先，错误不在行params[0] = param1，因为堆栈跟踪显示你的代码进入了onMethodEntry方法。

if添加语句绝对没有意义：如果param1是null，则不会有null赋值，但params[0]会保留 null，因为 in 中的所有未赋值位置new Object[10]最初都设置为null。

为了解决这个问题，首先要弄清楚哪些对象必须放在数组中而不是null参数中，然后修改条件如下：

if (param1 != null) {
    params[0] = param1;
} else {
    params[0] = // something else
}
于 2012-11-23T14:32:35.427 回答
0

你为什么不更换你的

onMethodEntry(int id, String className, String methodName, Object[] params) {

可变参数版本的方法：

onMethodEntry(int id, String className, String methodName, Object... params) {

这样，您可以摆脱其他方法，它们都是相同的，只是参数数量不同。包含您的错误的方法也会消失。如果错误仍然存​​在，您离找到它又近了一步。此外，您不必更改调用该方法的任何代码，因为它的签名是相同的。

于 2012-11-23T15:51:41.493 回答