用匿名内部类初始化的静态字段有什么问题吗？

Question

我在旧的遗留系统中找到了这段代码，如果没有任何问题，我不想去碰它。

但是我觉得这段代码有一些缺陷，我找不到它。这是不是常见的模式，还是有一些隐藏的陷阱或内存泄漏？

private static final Foo action =  new Foo() {

    @Override
    public void onAction(MyDialogFragment fragment) {
        if (fragment.getContext() != null) {
            fragment.getActivity().finish();
        }
    }
};

此代码用于片段

更新：我怀疑匿名内部类的对象引用了父片段，由于该类的对象保存在静态字段中，因此永远不会收集，因此也永远不会收集父片段。我的推理错了吗？

Answer 1

最好的解释如下：https://stackoverflow.com/a/27739694/5868421

静态上下文中的匿名类不包含对外部对象的引用。

Answer 2

是的，有。最好这样得到片段：

MyDialogFragment fragment = MyDialogFragment.instance();
fragment.setNavigator(this);
getSupportFragmentManager().beginTransaction().replace(R.id.activity_myDialog_layout, fragment).commit();

还有更好的方法。您可以创建3个类和一个由3个类实现的接口。活动、片段和演示者。接口将是合同。这是更先进的。

Answer 3

这种方法没有任何错误，因为逻辑只处理作为参数传递的值。使用静态可以简单地避免创建Foo类的多个实例。