弱引用无效

Question

我在静态处理程序中使用弱引用来避免内存泄漏，然而，有时这个引用会被取消，我不明白为什么。

静态处理程序在存储库类中定义，该存储库类有一个在后台执行操作的方法，接收回调以在操作完成时通知调用方：

public class MyRepository {

    public void performOperation(ContentResolver cr, RepositoryCallback callback) {
        MyHandler handler = new MyHandler(cr, callback);
        handler.startQuery(...)
    }

    interface RepositoryCallback {
        void onSuccess(MyModel model);
    }

    // Handler class code here
}

处理程序的代码如下所示：

private static class MyHandler extends AsyncQueryHandler {

    private final WeakReference<RepositoryCallback> weakCallback;

    public MyHandler(ContentResolver cr, RepositoryCallback callback) {
        super(cr);
        this.weakCallback = new WeakReference<>(callback);
    }

    @Override
    protected void onQueryComplete(int token, Object cookie, Cursor cursor) {
        RepositoryCallback callback = this.weakCallback.get();
        if (callback != null) { // --> Here sometimes it is null
            // Do some stuff with the cursor to create MyModel
            callback.onSuccess(model);
        }
    }

}

出于某种原因，这样做。weakCallback。get（）有时是空的，我试图理解为什么。

活动代码如下所示：

public class MyActivity extends AppCompatActivity {
    public void loadModel() {
        showLoadingView();
        myRepository.performOperation(context.getContentResolver(), new RepositoryCallback() {
            @Override
            public void onSuccess(MyModel model) {
                hideLoadingView();
                // Do something with model
            }
        });
    }
}

正如您所见，我正在为回调创建一个匿名类，但没有人持有对它的引用。

这就是弱引用无效的原因吗？

谢谢。

Answer 1

这是与弱引用相关的“经典”错误。

如果观察者持有对观察者的唯一引用，并且该引用很弱，则可以清除该引用，并对观察者进行垃圾收集。

由于您使用的是匿名类，可观察将保存对它的唯一引用，因此它将被清除。

作为旁注——在我的整个Android开发经验中，每当我看到开发人员使用弱引用时，总是有一种代码味道。通常，这表明开发人员不了解弱引用是如何工作的，或者他们不信任自己的代码。

一个很好的经验法则是，你永远不应该使用弱引用。

编辑：

我认为Handler通常是一个反模式。你可以在这个Reddit线程中读到更多关于这一点的信息。还有一个线程，我帮助一个开发人员看看他如何在他的代码库中摆脱HandlerThread。

另一方面，杰克·沃顿不同意我的说法。

从那里拿走你想要的，但是，总的来说，我会说拥有一个静态的处理程序肯定是反模式的。

如果你担心Android Studion的警告，那么请记住Google负责AsyncWork和Loaders。这个警告不仅仅是无用的，而且实际上很糟糕。他们应该把它变成你不应该使用静态Hadler。

如果您只需要将工作转移到BG线程，然后在UI线程上获得回调，那么使用RxJava之类的工具会更好。甚至是邪恶的异步任务。

我猜您正在使用AsyncQueryHandler来访问ContentProvider。这也是一个非常有争议的方法。如果您不需要与其他应用程序共享数据，那么最好使用一些ORM来为您处理多线程。