访问Kotlin属性时的`java.lang.StackOverflowError`

问题描述 投票:0回答:2

我使用带有继承的辅助构造函数从Packt的“Programming Kotlin”获得了这个(人为的)样本。

编辑:从答案很清楚,问题是关于backing field。但这本书没有引入这个想法,只是用错误的例子。

open class Payment(val amount: Int) 

class ChequePayment : Payment { 
    constructor(amount: Int, name: String, bankId: String) :  super(amount) { 
        this.name = name
        this.bankId = bankId 
    }

    var name: String
        get() = this.name
    var bankId: String
        get()  = this.bankId
} 


val c = ChequePayment(3, "me", "ABC")    
println("${c} ${c.amount} ${c.name}")

当我运行它时会显示此错误。

$ kotlinc -script class.kts 2>&1 | more
java.lang.StackOverflowError
    at Class$ChequePayment.getName(class.kts:10)
    at Class$ChequePayment.getName(class.kts:10)
    at Class$ChequePayment.getName(class.kts:10)

10行似乎确实是无限递归,如何解决呢?

kotlin
2个回答
3
投票

要访问支持字段,您必须使用关键字field而不是this.name,请参阅https://kotlinlang.org/docs/reference/properties.html#backing-fields

this.name引用了getter,它引用了this.name,这是一个无限递归,正如你已经注意到的那样。在代码中:

var name: String
    get() = field
var bankId: String
    get()  = field

附注:Android Studio和Idea会正确地抱怨你在这种情况下不需要getter。所以你可以进一步简化:

var name: String
var bankId: String

6
投票

你的代码中有一个递归:

class ChequePayment : Payment { 
    constructor(amount: Int, name: String, bankId: String) :  super(amount) { 
        this.name = name
        this.bankId = bankId 
    }

    var name: String
        get() = this.name // recursion: will invoke getter of name (itself)
    var bankId: String
        get()  = this.bankId // recursion: will invoke getter of bankId (itself)
} 

如果您不需要为getter定制逻辑,只需将您的属性保留为:

var name: String
var bankId: String

它们将具有默认的getter,它只会返回支持字段的值。

注意:代码可以/应该重构为:

class ChequePayment(amount: Int, var name: String, var bankId: String) : Payment(amount) {
    // ...
}

这使用主构造函数,而且冗余要少得多。

热门问题
推荐问题
最新问题