Estoy tratando de reducir la repetición de código usando un constructor privado para asignar un campo de clase, pero esto no parece ser posible.
Lo que estoy tratando de hacer es lo ejemplificado aquí:
class Foo { private val bar: Int private val baz: Int constructor(bar: Int, baz: Int) : this(baz) { this.bar = bar } constructor(bar: String, baz: Int) : this(baz) { this.bar = bar.toInt() } private constructor(baz: Int) { this.baz = baz } }La alternativa, que funciona, pero con la que no estoy satisfecho es hacer lo siguiente:
class Foo { private val bar: Int private val baz: Int constructor(bar: Int, baz: Int) { this.bar = bar this.baz = baz } constructor(bar: String, baz: Int) { this.bar = bar.toInt() this.baz = baz } } Para ser claros, no estoy satisfecho debido a la replicación de la asignación this.baz .
¿Esto simplemente no es posible en Kotlin, o me estoy perdiendo algo?
Deberías reescribir esto como:
class Foo( private val bar: Int, private val baz: Int, ) { constructor(bar: String, baz: Int) : this(bar.toInt(), baz) } El constructor (Int, Int) se ha convertido en un constructor primario, inicializando todos los valores. Y el segundo constructor delega al constructor principal.
Los primeros dos constructores secundarios que declaró en su código que no funciona no pueden reasignar los val s, porque delegan a otro constructor, y se supone que ese constructor inicializa todas las propiedades necesarias. Aunque en este caso, no es así, por lo que obtiene otro error.
Eso no puede funcionar porque su constructor privado no puede inicializar todas las propiedades. Lo estás tratando como una simple llamada de función.
¿Por qué no hacer esto?
class Foo { private val bar: Int private val baz: Int constructor(bar: Int, baz: Int) { this.bar = bar this.baz = baz } constructor(bar: String, baz: Int) : this(bar.toInt(), baz) }