Kotlin lateinit 属性，NPE 危险？

Question

我正在使用 lateinit 属性以避免使用 ?操作员。我有很多 View 属性是在 getViews() 函数中第一次分配的。如果该函数不存在，我的应用程序将因来自 Kotlin 代码的 NPE 而崩溃。

在我看来，lateinit 属性基本上破坏了该语言良好的空安全特性。我知道它们在 M13 中引入是因为更好的框架支持，但这值得吗？

还是我遗漏了什么？

代码如下：

package com.attilapalfi.exceptional.ui

import android.os.Bundle
import android.support.v7.app.AppCompatActivity
import android.view.View
import android.widget.Button
import android.widget.ImageView
import android.widget.TextView
import com.attilapalfi.exceptional.R
import com.attilapalfi.exceptional.dependency_injection.Injector
import com.attilapalfi.exceptional.model.Exception
import com.attilapalfi.exceptional.model.ExceptionType
import com.attilapalfi.exceptional.model.Friend
import com.attilapalfi.exceptional.persistence.*
import com.attilapalfi.exceptional.rest.ExceptionRestConnector
import com.attilapalfi.exceptional.ui.helpers.ViewHelper
import com.attilapalfi.exceptional.ui.question_views.QuestionYesNoClickListener
import com.google.android.gms.maps.MapView
import java.math.BigInteger
import javax.inject.Inject

public class ShowNotificationActivity : AppCompatActivity(), QuestionChangeListener {
    @Inject
    lateinit val exceptionTypeStore: ExceptionTypeStore
    @Inject
    lateinit val friendStore: FriendStore
    @Inject
    lateinit val imageCache: ImageCache
    @Inject
    lateinit val metadataStore: MetadataStore
    @Inject
    lateinit val viewHelper: ViewHelper
    @Inject
    lateinit val exceptionInstanceStore: ExceptionInstanceStore
    @Inject
    lateinit val exceptionRestConnector: ExceptionRestConnector
    @Inject
    lateinit val questionStore: QuestionStore
    private lateinit var sender: Friend
    private lateinit var exception: Exception
    private lateinit var exceptionType: ExceptionType
    private lateinit var exceptionNameView: TextView
    private lateinit var exceptionDescView: TextView
    private lateinit var senderImageView: ImageView
    private lateinit var senderNameView: TextView
    private lateinit var sendDateView: TextView
    private lateinit var mapView: MapView
    private lateinit var questionText: TextView
    private lateinit var noButton: Button
    private lateinit var yesButton: Button

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_show_notification)
        Injector.INSTANCE.applicationComponent.inject(this)
        questionStore.addChangeListener(this)
        getModelFromBundle()
        getViews()
        loadViewsWithData()
    }

    override fun onDestroy() {
        super.onDestroy()
        questionStore.removeChangeListener(this)
    }

    private fun getModelFromBundle() {
        val bundle = intent.extras
        val instanceId = BigInteger(bundle.getString("instanceId"))
        exception = exceptionInstanceStore.findById(instanceId)
        sender = friendStore.findById(exception.fromWho)
        exceptionType = exceptionTypeStore.findById(exception.exceptionTypeId)
    }

    private fun getViews() {
        exceptionNameView = findViewById(R.id.notif_full_exc_name) as TextView
        exceptionDescView = findViewById(R.id.notif_exc_desc) as TextView
        senderImageView = findViewById(R.id.notif_sender_image) as ImageView
        senderNameView = findViewById(R.id.notif_sender_name) as TextView
        sendDateView = findViewById(R.id.notif_sent_date) as TextView
        mapView = findViewById(R.id.notif_map) as MapView
        questionText = findViewById(R.id.notif_question_text) as TextView
        noButton = findViewById(R.id.notif_question_no) as Button
        yesButton = findViewById(R.id.notif_question_yes) as Button
    }

    private fun loadViewsWithData() {
        exceptionNameView.text = exceptionType.prefix + "\n" + exceptionType.shortName
        exceptionDescView.text = exceptionType.description
        imageCache.setImageToView(sender, senderImageView)
        senderNameView.text = viewHelper.getNameAndCity(exception, sender)
        sendDateView.text = exception.date.toString()
        loadQuestionToViews()
    }

    private fun loadQuestionToViews() {
        if (exception.question.hasQuestion) {
            showQuestionViews()
        } else {
            hideQuestionViews()
        }
    }

    private fun showQuestionViews() {
        questionText.text = exception.question.text
        val listener = QuestionYesNoClickListener(exception, exceptionRestConnector, noButton, yesButton)
        noButton.setOnClickListener(listener)
        yesButton.setOnClickListener(listener)
    }

    private fun hideQuestionViews() {
        questionText.visibility = View.INVISIBLE
        noButton.visibility = View.INVISIBLE
        yesButton.visibility = View.INVISIBLE
    }

    override fun onQuestionsChanged() {
        onBackPressed()
    }
}

Answer 1

lateinit 的相同基本特征在 M13 之前的 Delegates.notNull 实际上是可能的。

还有其他功能也可以让您绕过可空性强制执行。 !! 运算符会将可空值转换为非空值。

关键不是要严格要求可空性约束，而是要使可空性成为语言的一个非常明确的部分。每次使用 lateinit 或 !! 时，您都是在有意识地决定离开可空性约束的安全性，希望有充分的理由。

根据经验，最好避免 lateinit、!!，甚至 ?（可为空）尽可能多。

很多时候你可以使用 lazy 委托来避免 lateinit 这会让你陷入困境非空值（M14 现在禁止将值与 lateinit 一起使用）。

您链接的代码包含很多 lateinit 视图。您可以通过执行以下操作将它们保留为非空值：

private val mapView: MapView by lazy { findViewById(R.id.notif_map) as MapView }

这将在第一次使用 mapView 时初始化值，然后使用之前初始化的值。需要注意的是，如果您在调用 setContentView 之前尝试使用 mapView，这可能会中断。然而，这似乎没什么大不了的，而且您已经获得了初始化就在声明旁边的好处。

这是kotterknife library用来实现视图注入的。

Answer 2

Kotlin 的惰性委托在很多情况下都适用，尽管您在重新加载已保存在 FragmentManager 中的 Fragment 时会运行遇到麻烦。当 Android 系统重建片段时，它实际上会重新创建视图，导致 view?.findViewById(R.id.notif_map) 实际上 return 无效视图。

在这些情况下，您将不得不使用只读属性:

private val mapView: MapView
    get() = view?.findViewById(R.id.notif_map) as MapView

Kotlin lateinit 属性，NPE 危险？

Kotlin lateinit properties, NPE danger?

android

kotlin