hi , any idea why this code snippet doesn't work for buildString , only currency is evaluated , amou...

# android

[althaf]

hi , any idea why this code snippet doesn't work for buildString , only currency is evaluated , amount and ratepercent are skipped

spannableAmount = SpannableString(buildString {
    currency?.takeIf { it.isNotEmpty() }?.let { currency ->
        currencyLength1 = currency.length
        append("$currency")
    }
    amount?.takeIf { it.isNotEmpty() }?.let { amount ->
        amountLength1 = currencyLength1 + amount.length
        append(" $amount")
    }
    ratePercent?.takeIf { it.isNotEmpty() }?.let { rate ->
        rateLength1 = amountLength1 + rate.length
        append(" $ratePercent")
    }
})

[ildar.i [Android]]

Attach a debugger and place some breakpoints

[althaf]

it is skipping amount and rate percent, both are non null

[ildar.i [Android]]

then break down your call chain to see where it breaks

[ildar.i [Android]]

this code runs correctly online at https://play.kotlinlang.org/

fun main() {
    val currency = "eur"
    val amount = "1000"
    val ratePercent = "10"
    var currencyLength1 = 0
    var amountLength1 = 0
    var rateLength1 = 0
    val spannableAmount = buildString {
        currency?.takeIf { it.isNotEmpty() }?.let { currency ->
            currencyLength1 = currency.length
            append("$currency")
        }
        amount?.takeIf { it.isNotEmpty() }?.let { amount ->
            amountLength1 = currencyLength1 + amount.length
            append(" $amount")
        }
        ratePercent?.takeIf { it.isNotEmpty() }?.let { rate ->
            rateLength1 = amountLength1 + rate.length
            append(" $ratePercent")
        }
    }
    println(spannableAmount.toString())
    println(rateLength1)
}

[Adrijan Rogan]

1. Use a simple if instead of takeIf + let

if (currency != null && currency.isNotEmpty()) {
    currencyLength1 = currency.length
    append(currency)
}

2. Nit: instead of using append(" $amount") , use two appends -- append(" ") and append(amount) -- to avoid additional string construction