Question

我有一个内部帮助器类，我想将它的每个新实例添加到包含类的地图中，如下所示：

public class SomeService {
    private final Map<Integer, ServiceTask> pendingTasksByKey;

    private class ServiceTask {
        private ServiceTask(int key) {
            // initialization...
            pendingTasksByKey.put(key, this);
        }
    }

    // the rest of the code follows
}

当我这样做时，NetBeans抱怨在构造函数中使用this。好吧，我明白了，在一般情况下这是一种危险的做法，因为有人可以扩展我的课程然后我会泄漏this引用一个未完全初始化的对象。我不想关掉这个警告，所以我想我可以上课final。这样，没有人能够扩展我的类，因此使用this应该是非常安全的，因为初始化已经完成了。但即使我将内部类标记为final，NetBeans仍会显示警告。

我是对的还是我错过了什么？它只是NetBeans过于挑剔吗？除了可能存在的多线程内存模型问题之外，我无法想到这种this使用的任何危险。

Answer 1

这是一个特定于Netbeans的警告。这是一个提醒，该对象尚未构建，因此可能是一个问题。如果您没有做任何会出现问题的事情，那么您可以忽略它。例如，看看这段代码：

class A {
    public Object obj;

    public A() {
        B b = new B();
        b.addMe(this);

        obj = new Object();
    }
}

class B {
    ArrayList<A> list = new ArrayList<A>(0);

    public void addMe(A a) {
        list.add(a);
        System.out.println(a.obj.toString());
    }
}

此代码存在问题，我不应忽略“在构造函数中泄漏”警告。

Answer 2

Leaking this in constructor warning讨论了这次Netbeans警告。

如果pendingTasksByKey在这些课程的生命周期中某处有remove，我会说你是对的，Netbeans很挑剔。

在最终类的构造函数中使用“this”

2 个答案: