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的另一个委托实例，并将所有最终方法调用重定向到它。这个委托甚至没有初始化，在我的情况下，它有最后的字段。

要修复我的特定问题，我只是从方法中删除了最后一个关键字。

访问最终字段

1 个答案: