is there a cleaner way to do this? ` fun isVali...

# codingconventions

[liminal]

is there a cleaner way to do this? fun isValid(): Boolean = noteText.isNotEmpty() && noteText.isNotBlank() && noteText.length > 3 to avoid repeating noteText several times? thanks

[bdawg.io]

if you’re just trying to get rid of noteText from being repeated, there’s always

noteText.run {
    isNotEmpty() && isNotBlank() && length > 3
}

[kirillrakhman]

Doesn't isNotBlank imply isNotEmpty?

[skennedy]

yes, it does

[skennedy]

isNotEmpty() is literally length > 0, so it makes no sense to have a length check as well anyway

[bdawg.io]

yeah. That trim() solution is as literal in a single expression as it gets

[kirillrakhman]

trim() is allocating which is suboptimal