Question

我正在使用lateinit属性以避免连续的null检查？运营商。我有很多在getViews（）函数中第一次分配的View属性。如果那个函数不存在，我的应用程序将从一个NPE，从Kotlin代码崩溃。

在我看来，lateinit属性基本上破坏了该语言的漂亮的null安全功能。我知道它们是在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

在M13之前，Delegates.notNull实际上可以实现 lateinit 的相同基本功能。

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

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

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

很多时候你可以使用 lazy 委托来避免 lateinit ，它可以让你保持在非null值的范围内（M14现在禁止在 lateinit 中使用val。）

您链接的代码包含许多晚期观看次数。您可以通过执行以下操作将这些保留为非null值：

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

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

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

Answer 2

Kotlin的懒惰委托将适用于许多情况，尽管在重新加载已保存在FragmentManager中的碎片时会遇到麻烦。当Android系统重建片段时，它实际上会重新创建视图，导致view?.findViewById(R.id.notif_map)实际返回无效的视图。

在这些情况下，您必须使用只读属性：

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

Kotlin lateinit属性，NPE危险吗？

2 个答案: