Which is better and why? 1️⃣ This is broken

return binding.myField.getWidth() != 0 ? binding.myField.getWidth() : binding.otherField.getWidth()

2️⃣

return binding.myField.getWidth().let {
    if (it != 0) it else binding.otherField.getWidth()
}

3️⃣ by @Luke

return if (binding.myField.getWidth() != 0) binding.myField.getWidth() else binding.otherField.getWidth()

2️⃣ because 1️⃣ does not compile. There is no ternary operator in Kotlin. I would go with 3️⃣ :

return if (binding.myField.getWidth() != 0) binding.myField.getWidth() else binding.otherField.getWidth()

Whoops! I need to fix that. 😊

Another option is

return binding.myField.getWidth().takeUnless { it == 0 } ?: binding.otherField.getWidth()

@Luke Yeah, me too. It's not my code.

Option 3 may fail if another thread changes

myField

width between checking for 0 and returning it. This means it can return 0, something that this

if

statement is trying to prevent.