having else with return false everywhere is ugly

# getting-started

[ursus]

having else with return false everywhere is ugly

[Pavlo Liapota]

val foo1: () -> Boolean = foo@{
    if (flag1) {
        if (flag2) {
            return@foo true
        }
    }
    return@foo false
}

or

val foo2: () -> Boolean = fun (): Boolean {
    if (flag1) {
        if (flag2) {
            return true
        }
    }
    return false
}

[ursus]

thanks

[Sam Schilling]

Why not do

if (bar() && quax() && jazz()) {
    return true
}
return false

?

[Sam Schilling]

or better yet

[Sam Schilling]

return (bar() && quax() && jazz())

[ursus]

because there is code beween the ifs local to the branch, i ommited it for brewity