奇怪的 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
    }

现在，我知道 for 的构造函数DeliveryMethod正在使用nullfor 的值进行调用code，但我不明白为什么仅在此时出现异常。正如你所看到的，方法参数也被标记为非空，之前的方法也是如此。在调用构造函数之前不应该抛出异常吗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)
        }
    }

这是 java 调用者addDeliveryMethodsInAd()（这是在 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

在调用 DeliveryMethod 的构造函数之前，不应该抛出异常吗？

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

但是，在私有或挂起方法中这样做是没有意义的，因为它们只能从 Kotlin 调用（通常），并且它会增加一些开销，这在性能敏感的代码中可能不可接受。这就是为什么这些检查仅针对非挂起公共/受保护/内部方法生成（因为它们的目标是防止 Java 误用）。

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

编辑：通过添加调用代码，这可以解决问题。您正在调用一个从 Java 获取 a 的方法List<String>，其中包含一个包含 null 的列表。不幸的是，Kotlin 只检查参数本身（在本例中，它检查列表本身不为空），它不会迭代列表来检查内部是否有空值。这就是为什么在这种情况下它在公共 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和可能的替代方案。也就是说，您可能根本不应该在这里启动协程，请参阅下一点。

• async {}如果您立即使用，则不要使用await()- 如果您在那里等待，那么启动异步操作是毫无意义的。如果你想将上下文切换到IO，请withContext(Dispatchers.IO) { ... }改为使用。也就是说，您甚至不需要在这里使用 IO 调度程序，请参阅下一点。

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