使用lateinit是最好的方法吗？

Question

使用cursors来获取android所保存的图像，需要对cursors进行外部存储授权。如果AndroidOS是6.0或更高版本，并且没有获得授权许可，申请将被终止。如果获得权限，则需要进行关闭游标的处理。

如果是Java，那么最好按照以下方式编写onStop，除了Kotlin，将会引发一个异常。

if (cursor != null )
  cursor.close();

kotlin.UninitializedPropertyAccessException: lateinit property cursor has not been initialized

cursor = contentResolver.query(
    MediaStore.Images.Media.INTERNAL_CONTENT_URI,
    null, null, null, null
)

如果你能给我建议如何写得最好，我会很高兴的。

下面是使用的代码：

private lateinit var cursor : Cursor

private val PERMISSIONS_REQUEST_CODE = 100

override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    setContentView(R.layout.activity_main)

    if(Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
        // Android Version 6.0 later
        if(checkSelfPermission(Manifest.permission.READ_EXTERNAL_STORAGE) == PackageManager.PERMISSION_GRANTED) {
            initCursor()
        } else {
            requestPermissions(arrayOf(Manifest.permission.READ_EXTERNAL_STORAGE), PERMISSIONS_REQUEST_CODE)
        }
    } else {
        // Android Version under 6.0
        initCursor()
    }
}

override fun onRequestPermissionsResult(requestCode: Int, permissions: Array<out String>, grantResults: IntArray) {
    when (requestCode) {
        PERMISSIONS_REQUEST_CODE ->
            if (grantResults[0] == PackageManager.PERMISSION_GRANTED) {
                initCursor()
            } else {
                finishApp()
            }
        else ->
            finishApp()
    }
}

private fun finishApp() {
    //  It prevents kotlin.UninitializedPropertyAccessException of cursor(lateinit valiant). It's last resort...
    cursor = contentResolver.query(
            MediaStore.Images.Media.INTERNAL_CONTENT_URI,
            null, null, null, null)
    /*
    if(cursor != null)
        cursor.close()
    */
    finish()
}

override fun onStop() {
    super.onStop()
    cursor.close()
}

private fun initCursor() {
    cursor = contentResolver.query(
            MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
            null, null, null, null)
    cursor.moveToFirst()
}

Answer 1

不，这不是lateInit的最佳用法。首先，lateInit的使用是为了一个在使用时应该可用的变量。使用nullable mean，变量可以为null，这与lateInit不同，这意味着它不应该为null。

在Android中，它作为oncreate而不是构造函数中的初始变量非常有用。它对依赖注入也很好，就像匕首一样。

它抛出uninitializedPropertyAccessException表示您的游标在访问时没有初始化。这意味着您的代码中存在一个逻辑错误，即光标在没有初始化的情况下被访问。

注意，保持游标不是一个好的做法。它保存到数据库的连接。您应该在使用它后关闭，并重新创建一个新的，而不是持有它。