Почему я получаю kotlin .UninitializedPropertyAccessException, даже если свойство lateinit инициализировано (вероятно) - PullRequest
0 голосов
/ 01 мая 2020

Хорошо, поэтому я объявил переменную lateinit job в виджете, как показано ниже.

class TempHumidDisplayWidget : AppWidgetProvider(), CoroutineScope {
    private lateinit var job: Job
    override val coroutineContext: CoroutineContext
        get() = Dispatchers.Main + job
    ...

В функции onEnabled я инициализировал ее:

override fun onEnabled(context: Context) {
    job = Job()
    Log.i("com.github.animeshz", "On enabled")
    ...

И внутри onUpdate Я собираюсь использовать запуск, который вызывает функцию get из coroutineContext для отправки сопрограммы, которая использует переменную задания.

override fun onUpdate(
    context: Context,
    appWidgetManager: AppWidgetManager,
    appWidgetIds: IntArray
) {
    // There may be multiple widgets active, so update all of them
    for (appWidgetId in appWidgetIds) {
        launch { updateAppWidget(context, appWidgetManager, appWidgetId) }
    }
}

А внутри logcat я получаю следующую ошибку:

4425-4425/? I/com.github.animeshz: On enabled
...
2020-05-01 11:06:06.639 4425-4425/? E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.github.animeshz.shivamwidget, PID: 4425
    java.lang.RuntimeException: Unable to start receiver com.github.animeshz.shivamwidget.TempHumidDisplayWidget: kotlin.UninitializedPropertyAccessException: lateinit property job has not been initialized
        at android.app.ActivityThread.handleReceiver(ActivityThread.java:3183)
        at android.app.ActivityThread.-wrap18(ActivityThread.java)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1636)
        at android.os.Handler.dispatchMessage(Handler.java:102)
        at android.os.Looper.loop(Looper.java:154)
        at android.app.ActivityThread.main(ActivityThread.java:6334)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)
     Caused by: kotlin.UninitializedPropertyAccessException: lateinit property job has not been initialized
        at com.github.animeshz.shivamwidget.TempHumidDisplayWidget.getCoroutineContext(TempHumidDisplayWidget.kt:26)
        at kotlinx.coroutines.CoroutineContextKt.newCoroutineContext(CoroutineContext.kt:33)
        at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch(Builders.common.kt:50)
        at kotlinx.coroutines.BuildersKt.launch(Unknown Source)
        at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch$default(Builders.common.kt:47)
        at kotlinx.coroutines.BuildersKt.launch$default(Unknown Source)
        at com.github.animeshz.shivamwidget.TempHumidDisplayWidget.onUpdate(TempHumidDisplayWidget.kt:38)
        at android.appwidget.AppWidgetProvider.onReceive(AppWidgetProvider.java:66)
        at android.app.ActivityThread.handleReceiver(ActivityThread.java:3171)
        at android.app.ActivityThread.-wrap18(ActivityThread.java) 
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1636) 
        at android.os.Handler.dispatchMessage(Handler.java:102) 
        at android.os.Looper.loop(Looper.java:154) 
        at android.app.ActivityThread.main(ActivityThread.java:6334) 
        at java.lang.reflect.Method.invoke(Native Method) 
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886) 
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776) 

Ясно, что onEnabled вызывается перед методом onUpdate, но переменная job не инициализируется каким-либо образом, я не знаю, как это исправить или отладить дальше

Как это возможно, что переменная задания, инициализированная в onEnabled, фактически не инициализируется в функции onUpdate? Это ошибка или что-то в этом роде?

Буду признателен за любую помощь, заранее спасибо!

1 Ответ

1 голос
/ 01 мая 2020

Это просто плевок, так как я не знаю деталей виджетов в Android, но что-то вроде этого может сработать:

class TempHumidDisplayWidget : AppWidgetProvider() {
    companion object {
        val jobs = mutableMapOf<Context, Job>()

        fun getScope(context: Context): CoroutineScope {
            val job = jobs[context] ?: throw IllegalStateException("Context $context not enabled currently")
            return object : CoroutineScope { 
                override val coroutineContext: CoroutineContext = Dispatchers.Main + job
                ...
            }
        }
    }

    override fun onEnabled(context: Context) {
        jobs[context] = Job()
        ...
    }

    override fun onDisabled(context: Context) {
        jobs.remove(context)?.cancel()
        ...
    }

    override fun onUpdate(
        context: Context,
        appWidgetManager: AppWidgetManager,
        appWidgetIds: IntArray
    ) {
        val scope = getScope(context)
        for (appWidgetId in appWidgetIds) {
            scope.launch { updateAppWidget(context, appWidgetManager, appWidgetId) }
        }
    }
}

Обычно плохая идея хранить ссылки на Context s "статически", но это может быть хорошо, потому что они удалены в onDisabled. Или вместо этого используйте WeakHashMap.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...