Kotlin代码“不要将Android上下文类放在静态字段中”投诉

Question

我有一个用Java编写的基于Android Studio的应用程序。我工作正常，不会导致Android Studio抱怨任何事情：

public class MainActivity extends AppCompatActivity {

  static Context maincActivityContext;

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    mainActivityContext = getApplicationContext();
    ...
  }

  /* JNI function called from c++ */
  private void updateStatus(String event, final String call) {
    ...
    runOnUiThread(new Runnable() {
      @Override
      public void run() {
            TextView caller_uri = new TextView(mainActivityContext);
            ...
      }
    });
  }
  ...
}

然后我去要求Android Studio将其从Java转换为Kotlin。转换后，MainActivity如下所示：

class MainActivity : AppCompatActivity() {

  override fun onCreate(savedInstanceState: Bundle?) {

    super.onCreate(savedInstanceState)
    mainActivityContext = applicationContext
    ...
  }

  /* JNI function called from C++ */
  private void updateStatus(String event, final String call) {
    ...
    runOnUiThread(new Runnable() {
        @Override
        public void run() {
            TextView caller_uri = new TextView(mainActivityContext);
            ...
        }
    });
  }

  companion object {
    internal lateinit var mainActivityContext: Context
    ...
  }
}

否则，与 Java 版本一样，但类变量主活动上下文已从主活动的顶部移动到伴随对象。现在，该应用程序再次运行良好，但Android Studio抱怨主要活动上下文变量声明：

Do not place Android context classes in static fields

然后，我将声明移回Java版本中的位置：

class MainActivity : AppCompatActivity() {

  internal lateinit var mainActivityContext: Context

  override fun onCreate(savedInstanceState: Bundle?) {
    mainActivityContext = applicationContext
    ...

Android Studio很满意。但是，当我尝试运行该应用程序时，它像这样崩溃了：

03-18 16：08：52.788 7467 7467 E Android运行时间： 科特林.未初始化属性访问异常： lateinit 属性 mainActivity上下文尚未初始化

03-18 16:08:52.788 7467 7467 E AndroidRuntime： atcom.foo.bar.MainActivity.updateStatus（MainActivity.kt:362）

可以肯定的是，变量是（像以前一样）在onCreate函数中初始化的。

问题：

为什么初始化在runOnUIThread代码中不可用，如何正确地在其中初始化变量？

我检查了关于这个主题的早期线程，找不到（或理解）适用于这里的答案。

Answer 1

我能够通过将活动上下文作为参数传递给启动本机应用程序的函数，然后将其作为updateState函数的参数传递回MainActive，从而完全摆脱main ActivityContext变量：

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        ...
        Thread(Runnable { nativeAppStart(applicationContext) }).start()
    }
}

原生应用程序如下所示：

static jobject *appContext;

JNIEXPORT void JNICALL
Java_com_tutpro_baresip_MainActivity_baresipStart(JNIEnv *env, jobject thiz, jobject *context)
{
    appContext = context;
    ...
}

它的updateStatus调用如下所示:

jmethodID statusId = (*env)->GetMethodID(env, pctx->jniHelperClz,
                                         "updateStatus",
                                         "(Landroid/content/Context;Ljava/lang/String)V");
...
(*env)->CallVoidMethod(env, pctx->jniHelperObj, statusId, appContext, status);

现在我在updateStatus函数中有了可用的工作应用程序上下文，接下来的问题是如何使用它来设置内容视图。我会再提一个问题。

感谢所有做出贡献的人。

Answer 2

永远不要将任何android上下文放在静态字段中（在静态编程语言中，该字段进入伴随对象），这是一种不好的做法，因为这样会导致内存泄漏。

如果要访问类中的上下文，请在其构造函数中将上下文传递给它，或者如果类类似于活动，请使用getApplicationContext（）访问上下文。

在您的示例中，您根本不需要存储上下文，因为您的活动实际上是一个context本身（您可以使用thisas context）。