访问final字段时出现奇怪的NullPointerException

Question

我不知道如何开始这个.NullPointerException在一个不应该的地方我很荒谬.我对答案的期望不高,因为情况看起来像是异常,但问题和答案(如果我最终找到答案)可能对教育有用.如果不是,我可能会删除这个问题.

Caused by: java.lang.NullPointerException
        at com.ah.dao.hbase.Snapshotable.lock(Snapshotable.java:17)
        at com.ah.pipeline.dump.DumpController.dump(DumpController.java:78)

---

07: public abstract class Snapshotable {
08:    private final AtomicBoolean readonly = new AtomicBoolean(false);
09:
10:    abstract public TableSuit getTableInfo();
11:
12:    public boolean locked() {
13:       return readonly.get();
14:    }
15:
16:    public final void lock() {
17:        readonly.set(true); <-- happens here
18:    }
19:
20:    public final void release() {
21:        readonly.set(false);
22:    }
23: }

起初readonly变量不是最终的,所以我虽然可能是不安全的出版效果,但现在我没有想法.在我们的代码中没有使用这些变量的反射技巧,但是这个类后代的一些方法用aspectj代理.

---

使用aop详细信息更新

@Service
public class HDao extends Snapshotable {

   @PerformanceMonitoring
   public void save(PatchEvent patchEvent) {
       if (locked()) {
            throw new DumpException(tableName);
       }

@Aspect
@Component
public class PMAdvice {

   @Around(value = "@annotation(performanceMonitoring)", argNames = "jp, p")
   public Object saveEvent(ProceedingJoinPoint jp, PerformanceMonitoring p) throws Throwable {
      // basic stuff

Answer 1

我可以用一个小程序重现它。这个问题确实与AOP有关。在调试的帮助下，我发现最终方法不适用于 AspectJ 代理。

事实证明，AspectJ 中的代理是通过运行时子类化完成的。因此它创建了一个包装子类，它将所有方法委托给代理对象。它对于非最终方法效果很好。

简化示例：

class WrapperProxy extends MyClass {
    private MyClass delegate = new MyClass();

    @Override
    public void run() {
       delegate.run();
    }
}

该技术对最终方法有一个问题，因为它们不能在子类中重写。AspectJ 以一种奇怪的方式处理它：它创建 MyClass 的另一个委托实例并将所有最终方法调用重定向到它。该委托甚至没有初始化，在我的例子中，它的最终字段为空。

为了解决我的特定问题，我刚刚从方法中删除了 Final 关键字。