所以我有以下困境。这不是关于不起作用的问题,而是关于哪种方法更优雅/更好的做法以及原因。
因此,我们知道init
块,初始化逻辑就在其中。也许您打开文件或从配置文件中读取一些常量,也许您基于使用构造函数参数的更复杂算法来设置一些属性。
但是,如果您的某个/某些属性确实需要一点逻辑,琐碎的逻辑,例如验证参数或类似的东西,该怎么办?而且此逻辑仅与该特定属性有关,而与其他属性没有交互?
例如:
class MyCircularQueue(k: Int) {
private val arr = {
if (k < 1)
throw IllegalArgumentException("k must be at least 1")
else
Array(k) { 0 }
}()
private var head = 0
private var tail = 0
private var empty = true
}
这里
arr
需要初始化为0
的数组,但是如果k
小于1,显然会出现问题。需要非常短的初始化逻辑,只需执行简单的检查即可引发异常或初始化数组。没有与其他属性的交互,没有复杂的逻辑,只是极其琐碎的检查或逻辑。在这种情况下,我想知道将琐碎的初始化逻辑放在“临时”代码块(当场调用的lambda)中是否更好。这样做的好处是,声明和初始化逻辑将在同一位置,不需要时就不会分开,我认为这样更易读。
因此,在我写完这篇文章之后,我想知道是否还有其他人对于非常简单的初始化逻辑有此问题,但是我真的找不到任何有关它的信息。
底线是:这样可以吗?是好的做法,还是至少不是坏的做法?是否有意义?在Kotlin中,有没有更好的和/或官方的方法来做到这一点?
最佳答案
函数变通方法不是必需的,if
是一个表达式并返回一个值,因此可以简化:
private val arr =
if (k < 1)
throw IllegalArgumentException("k must be at least 1")
else
Array(k) { 0 }
另外,Kotlin在标准库中附带了许多功能性API,可帮助创建简洁的衬纸。在这种情况下,您可以执行以下操作:
private val arr =
k.takeIf { it > 0 }?.let { Array(k) { 0 } } ?: throw IllegalArgumentException("k must be at least 1")