intellij is telling me to remove that true because...

# announcements

[sreich]

intellij is telling me to remove that true because it is superfluous

[jk]

it’s probably telling you it’s superfluous because it’s unnecessary to write an if statement when bar() returns a boolean - the when could be written this to achieve the same result:

when(blah) {
    Thing.Value -> bar()
}

[sreich]

nope. that isn't it

[sreich]

i still don't understand this.

fun bizz(a: Int) = when (a) {
            1 -> {
                bar()
                true
            }

            2 -> {
                if (foo()) {
                   doStuff()
                    true //"this statement is unused"..but why?
                }
                    false
            }
           else -> true

[jk]

it’s because there’s no else here:

if (foo()) {
    doStuff()
     true //"this statement is unused"..but why?
}
false

[sreich]

but why?

[sreich]

it's because the last boolean is considered the return, right?

[sreich]

regardless of whichever ones come before

[jk]

yeah that’s right. if you put the else in, the if expression becomes the last statement

[sreich]

ok, makes sense. too bad there wasn't a way to specify you want it to be the last one. but return isn't allowed, so..

[jk]

is it not possible to write it with the else?

if (foo()) {
    doStuff()
    true
} else {
    false
}

[sreich]

yeah it is, just more verbose, ya know?

[kingsley]

@sreich the last statement is considered return. Yes. However, if you need to return explicitly. You should be able to return@when. Not sure if that works currently. Or you could just put a label at the when and return@label