when using a private constructor then a public constructor, it displays the public one as unused. Sh...

# serialization

[zt]

when using a private constructor then a public constructor, it displays the public one as unused. Shouldn't kotlinx serialization somehow indicate that it will be used for serialization?

[ephemient]

kotlinx.serialization uses the primary constructor. if you mean something like

@Serializable
class Foo private constructor(
    ...
) {
    constructor(...) : this(..)
}

then it's using the private constructor, not the public constructor

[zt]

That doesn't seem right I think. On the docs it looks to be the other way around https://github.com/Kotlin/kotlinx.serialization/blob/master/docs/basic-serialization.md#constructor-properties-requirement

[ephemient]

in that example, "Serialization works with a private primary constructor"

[ephemient]

what is the other way around about it?

[zt]

@Serializable
internal class ApiText private constructor(private val text: String) {
    constructor(runs: List<TextRun>) : this(
        text = runs.joinToString(separator = "", transform = TextRun::text)
    )

    @Serializable
    data class TextRun(val text: String)
}

Ok so this is what I have, but how would I fix it?

[zt]

the json has a list of objects containing a text key

[zt]

Would this be the correct way?

@Serializable
internal class ApiText private constructor(private val runs: List<TextRun>) {
    val text = runs.joinToString(separator = "", transform = TextRun::text)

    @Serializable
    data class TextRun(val text: String)

    override fun toString() = text
}

android studio is saying Constructor parameter is never used as a property on the runs param

[ephemient]

if that's how you want to use it then yes, that's the right way