kotlin - 奇怪的kotlin checkNotNullParameter错误

Question

我们在 Firebase 上收到了一个 kotlin 方法的崩溃:

Fatal Exception: java.lang.NullPointerException: Parameter specified as non-null is null: method kotlin.jvm.internal.Intrinsics.checkNotNullParameter, parameter code
       at [redacted].DeliveryMethod.<init>(:2)
       at [redacted].DeliveryMethodsUpdater$addSingleDMInAd$clientCall$1.invokeSuspend(DeliveryMethodsUpdater.kt:121)
       at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
       at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
       at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:738)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)

型号是这个:

class DeliveryMethod() {
    lateinit var code: String
    lateinit var name: String
    lateinit var description: String

var isAddressRequired: Boolean? = null
var image: JSONObject? = null
var isDefault: Boolean = false

constructor(code: String) : this() {
    this.code = code
}

constructor(code: String, name: String, description: String, image: JSONObject? = null) : this() {
    this.code = code
    this.name = name
    this.description = description
    this.image = image
}
}

以及方法:

private suspend fun addSingleDMInAd(
        adId: Long,
        deliveryMethodCode: String
    ): JoinAdDeliveryMethod? {
        var addedDeliveryMethod: JoinAdDeliveryMethod? = null
        val clientCall = GlobalScope.async(Dispatchers.IO) {
            val cd = CountDownLatch(1)
            Client.getInstance().addDeliveryMethodInAd(
                adId,
                DeliveryMethod(deliveryMethodCode),
                object : NetworkCallback<JoinAdDeliveryMethod> {
                    override fun onSuccess(result: JoinAdDeliveryMethod) {
                        addedDeliveryMethod = result
                        cd.countDown()
                    }

                    override fun onFailure(err: NetworkError?) {
                        addedDeliveryMethod = null
                        cd.countDown()
                    }
                }
            )
            cd.await()
        }
        clientCall.await()
        return addedDeliveryMethod
    }

现在，我知道 DeliveryMethod 的构造函数正在使用 code 的 null 值调用，但我不明白为什么此时才出现异常。如您所见，方法参数也被标记为非空，之前的方法也是如此。在调用 DeliveryMethod 的构造函数之前不应该抛出异常吗？

编辑:这是 addSingleDMinAd() 的调用者:

 fun addDeliveryMethodsInAd(
        adId: Long,
        deliveryMethodCodesToAdd: List<String>,
        completionListener: (List<JoinAdDeliveryMethod?>) -> Unit
    ) {
        GlobalScope.launch {
            val updatedDms: MutableList<JoinAdDeliveryMethod?> = mutableListOf()
            for (deliveryCode in deliveryMethodCodesToAdd) {
                addSingleDMInAd(adId = adId, deliveryMethodCode = deliveryCode).run {
                    updatedDms.add(this)
                }
            }
            completionListener.invoke(updatedDms)
        }
    }

这是 addDeliveryMethodsInAd() 的 java 调用者(这是在 Android 服务中):

new DeliveryMethodsUpdater().addDeliveryMethodsInAd(
                            result.getId(),
                            deliveryMethodCodesToAdd,
                            updatedDMs -> {
                                // on failed delivery method request
                                for (JoinAdDeliveryMethod updatedDm : updatedDMs) {
                                    if (updatedDm == null) {
                                        //show error
                                        break;
                                    }
                                }
                                AdDetailUpdater
                                        .getInstance()
                                        .updateSubscribersWithDeliveryMethods(result.getId(), updatedDMs);
                                return null;
                            }
                    );

Answer 1

Shouldn't the exception be thrown way before getting to the constructor call for DeliveryMethod?

在 Kotlin 中，非空参数不可能在运行时意外地被赋予空值(因为代码一开始就不会编译)。但是，如果值是从 Java 传递的，则可能会发生这种情况。这就是为什么 Kotlin 编译器试图通过在某些方法的开头生成 null 检查来保护您免受 Java 的 null 不安全性(您在这里看到的内在 checkNotNullParameter 失败)。

但是，在私有(private)或挂起方法中这样做是没有意义的，因为它们只能从 Kotlin(通常)调用，并且会增加一些开销，这在性能敏感的代码中可能是 Not Acceptable 。这就是为什么这些检查仅针对非暂停的公共(public)/ protected /内部方法生成(因为它们的目标是防止 Java 滥用)。

这就是为什么，如果您设法调用 addSingleDMInAd使用 null 参数，它不会因此错误而失败。也就是说，看看你如何在这里得到 null 会很有趣，因为通常在公共(public) API 表面进行检查就足够了。这里是否涉及一些反射或不安全的类型转换？

编辑:添加调用代码后，问题就迎刃而解了。您正在调用一个采用 List<String> 的方法来自 Java，带有一个包含空值的列表。不幸的是，Kotlin 只检查参数本身(在这种情况下，它检查列表本身是否为空)，它不会迭代您的列表以检查内部是否存在空值。这就是在这种情况下它没有在公共(public) API 表面上失败的原因。

---

此外，您的模型设置方式也很奇怪。看来lateinit说谎是因为根据使用的构造函数，实际上可能根本没有设置属性。当该类的用户不设置这些属性的值时，将它们标记为可为空会更安全。这样做，你甚至不需要所有的辅助构造函数，你可以使用默认值:

class DeliveryMethod() {
    var code: String? = null,
    var name: String? = null,
    var description: String? = null,
    var image: JSONObject? = null,
) {
    var isAddressRequired: Boolean? = null
    var isDefault: Boolean = false
}

---

关于 addSingleDMInAd 的其他注意事项:

• 不要使用 GlobalScope在这种情况下。如果您需要运行短期协同程序，请为它们提供更小的范围，当不再需要工作时取消该范围 - 它确保没有协同程序泄漏。您可以阅读更多关于 GlobalScope 的潜在陷阱的信息。和可能的替代品in its own doc .也就是说，您可能根本不应该在这里启动协程，请参阅下一点。

• 不要使用 async {}如果您使用 await()紧接着 - 如果你在那里等待它，那么开始异步是没有意义的。如果要将上下文切换到 IO，请使用 withContext(Dispatchers.IO) { ... }反而。也就是说，您甚至不需要在此处使用 IO 调度程序，请参阅下一点。

• 不要使用 CountDownLatch以此目的。将异步 API 封装为 suspend 的正确方法协程的功能是使用 suspendCancellableCoroutine (查看其文档，它提供了如何使用它的示例)。一旦你使用它，就不再需要 Dispatchers.IO因为它不会再阻塞当前线程了。