我不知道如何开始这个。我有一个荒谬的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代理。
@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
答案 0 :(得分:1)
我可以通过一个小程序重现它。这个问题确实与AOP有关。在调试的帮助下,我发现最终方法不适用于AspectJ代理。
事实证明,AspectJ中的代理是由运行时子类化完成的。因此,它创建了一个包装器子类,它将所有方法委托给代理对象。 它适用于非最终方法。
简化示例:
class WrapperProxy extends MyClass {
private MyClass delegate = new MyClass();
@Override
public void run() {
delegate.run();
}
}
这种技术在最终方法中存在问题,因为它们无法在子类中重写。 AspectJ以一种奇怪的方式处理它:它创建MyClass的另一个委托实例,并将所有最终方法调用重定向到它。这个委托甚至没有初始化,在我的情况下,它有最后的字段。
要修复我的特定问题,我只是从方法中删除了最后一个关键字。