使用这种方式将上下文传递给非上下文类是否安全

Question

我已经阅读了几篇关于将上下文传递给适配器或其他内容的文章，并且我制作了某种上下文容器来获取应用程序上下文：

import android.content.Context;

public class ContextHolder {
    private static ContextHolder ourInstance = new ContextHolder();
    private Context context;

    public static ContextHolder getInstance() {
        return ourInstance;
    }

    private ContextHolder() {
        context = null;
    }

    public void setContext(Context context){
        this.context = context;
    }

    public Context getApplicationContext(){
        return context;
    }
}

然后在 MainActivity 我创建 ContextHolder 对象并像这样设置上下文：

 protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        ContextHolder contextHolder = ContextHolder.getInstance();
        contextHolder.setContext(this.getApplicationContext());
    }

在其他一些类中，我需要使用 contex：

ContextHolder contextHolder = ContextHolder.getInstance();
Resources resources = contextHolder.getApplicationContext().getResources();

问题是，我做对了吗？ 它会导致内存泄漏或其他讨厌的东西吗？

Answer 1

在 MainActivity 我正在创建 ContextHolder

但是为了什么？ Activity 已经是 Context 的子类：

java.lang.Object
  ↳ android.content.Context
    ↳ android.content.ContextWrapper
      ↳ android.view.ContextThemeWrapper
        ↳ android.app.Activity

所以你可以使用this（或MainActivity.this）。更不用说 contextHolder 变量，您将持有者对象放在所示代码中是本地范围的，并且仅在 onCreate() 中可见。

它会导致内存泄漏或其他讨厌的东西吗？

我建议使用 LeakCanary 来捕获代码中的所有内存泄漏。见：https://github.com/square/leakcanary

在其他一些类中，我需要使用 contex：

ContextHolder contextHolder = ContextHolder.getInstance();
Resources resources = contextHolder.getApplicationContext().getResources();

这都是不必要的和过度设计的。如果您只需要获取应用程序上下文，子类Application 类：

class MyApplication extends Application {
    protected static Context mContext;

    @Override
    public void onCreate() {
       super.onCreate();

       mContext = this;
    }

    public static Context getAppContext() {
       return mContext;
    }
}

在 Manifest 中将其设置为您的应用应用程序类：

<application
    android:name=".MyApplication"
    ...

然后，当您需要上下文时，您无法通过其他方式获得，只需调用：

MyApplication.getAppContext();

Answer 2

我认为您保存对应用程序上下文的引用的方法很好。正如docs 所说：

通常不需要子类化应用程序。在大多数情况下，静态单例可以以更加模块化的方式提供相同的功能。

因此，只是为了获取对应用程序上下文的引用，您不应继承 Application 类。

在您的情况下，您应该在第一个活动的 onCreate 方法中初始化您的单例，以便您应该将 this.getApplicationContext() 传递给您的单例以保持。例如：

public class ContextHolder {
    private static ContextHolder ourInstance;
    private Context context;

    public static ContextHolder getInstance() {
        return ourInstance;
    }

    public static void init(Context context){
        ourInstance = new ContextHolder(context);
    }

    private ContextHolder(Context context) {
        this.context = context;
    }

    public Context getApplicationContext(){
        return context;
    }
}

在您的第一个 Activity 的 onCreate 中：

ContextHolder.init(getApplicationContext());