VolleyInstance 中的 Android 上下文泄漏

Question

>我最近使用了作为单例模式实现的 Volley 库。我在谷歌的开发人员培训页面上提到了这个实现：

public class MySingleton {
private static MySingleton mInstance;
private RequestQueue mRequestQueue;
private ImageLoader mImageLoader;
private static Context mCtx;
private MySingleton(Context context) {
mCtx = context;
mRequestQueue = getRequestQueue();
mImageLoader = new ImageLoader(mRequestQueue,
new ImageLoader.ImageCache() {
private final LruCache<String, Bitmap>
cache = new LruCache<String, Bitmap>(20);
@Override
public Bitmap getBitmap(String url) {
return cache.get(url);
}
@Override
public void putBitmap(String url, Bitmap bitmap) {
cache.put(url, bitmap);
}
});
}
public static synchronized MySingleton getInstance(Context context) {
if (mInstance == null) {
mInstance = new MySingleton(context);
}
return mInstance;
}
public RequestQueue getRequestQueue() {
if (mRequestQueue == null) {
// getApplicationContext() is key, it keeps you from leaking the
// Activity or BroadcastReceiver if someone passes one in.
mRequestQueue = Volley.newRequestQueue(mCtx.getApplicationContext());
}
return mRequestQueue;
}
public <T> void addToRequestQueue(Request<T> req) {
getRequestQueue().add(req);
}
public ImageLoader getImageLoader() {
return mImageLoader;
}

}

如果有人将上述单例实例化为

RequestQueue queue = MySingleton.getInstance(this). getRequestQueue();

它仍然会导致mCtx泄漏，因为它被声明为静态的。

他们不应该只是将 Singleton 的实现更改为：

public class MySingleton {
private static MySingleton mInstance;
private RequestQueue mRequestQueue;
private ImageLoader mImageLoader;
private static Context mCtx;
private MySingleton(Context context) {
mCtx = context;
mRequestQueue = getRequestQueue();
mImageLoader = new ImageLoader(mRequestQueue,
new ImageLoader.ImageCache() {
private final LruCache<String, Bitmap>
cache = new LruCache<String, Bitmap>(20);
@Override
public Bitmap getBitmap(String url) {
return cache.get(url);
}
@Override
public void putBitmap(String url, Bitmap bitmap) {
cache.put(url, bitmap);
}
});
}
public static synchronized MySingleton getInstance(Context context) {
if (mInstance == null) {
mInstance = new MySingleton(context.getApplicationContext()); // <--Already using application context
}
return mInstance;
}
public RequestQueue getRequestQueue() {
if (mRequestQueue == null) {
// getApplicationContext() is key, it keeps you from leaking the
// Activity or BroadcastReceiver if someone passes one in.
mRequestQueue = Volley.newRequestQueue(mCtx);
}
return mRequestQueue;
}
public <T> void addToRequestQueue(Request<T> req) {
getRequestQueue().add(req);
}
public ImageLoader getImageLoader() {
return mImageLoader;
}

}

在这里，我已将mInstance = new MySingleton(context);更改为mInstance = new MySingleton(context.getApplicationContext());

我的问题是：为什么要让用户发送应用程序上下文？为什么不在这个类中保护？有什么想法吗？

Answer 1

重要的是使用 APPLICATION 上下文创建它：

RequestQueue queue = MySingleton.getInstance(this.getApplicationContext()).
getRequestQueue();

由于应用程序上下文在应用程序生命周期的整个时间都存在，因此它不会导致内存泄漏。

Answer 2

好吧，您可以使用应用程序上下文，但除此之外，没有什么可做的，请记住，只要应用程序正在运行，Context就存在。所以你不能真正泄漏上下文。您可以尝试从应用中的ApplicationController获取上下文，但上下文仍然相同：

private MySingleton() {
mCtx = ApplicationController.getContext();
mRequestQueue = getRequestQueue();
...

在您的应用程序中class像这样定义getContext()：

public class ApplicationController extends Application {
public static Context getContext(){
return mContext;
}
...