Normally I prefer `when` to `switch`, but I create...

# announcements

[Ellen Spertus]

Normally I prefer when to switch, but I created this monstrosity in order to combine multiple is cases. Is there a better way without changing the sealed class ConnectionState (which I might end up doing)?

if (savedConnectionState is ConnectionState.Isolated) {
    view.initializeButtons(true, false)
} else if (savedConnectionState is ConnectionState.Advertising || 
    savedConnectionState is ConnectionState.Discovering ||
    savedConnectionState is ConnectionState.Initiating ||
    savedConnectionState is ConnectionState.Authenticating ||
    savedConnectionState is ConnectionState.Connecting) {
    view.initializeButtons(false, false)
} else if (savedConnectionState is ConnectionState.ReadyToSend) {
    view.initializeButtons(false, true)
} else if (savedConnectionState is ConnectionState.Failure) {
    view.initializeButtons(false, false)
}

[pt]

when can group cases together, is that what you’re asking?

[Ruckus]

when (savedConnectionState) {
    is ConnectionState.Isolated -> view.initializeButtons(true, false)
    is ConnectionState.Advertising,
    is ConnectionState.Discovering,
    is ConnectionState.Initiating,
    is ConnectionState.Authenticating,
    is ConnectionState.Connecting -> view.initializeButtons(false, false)
    is ConnectionState.ReadyToSend -> view.initializeButtons(false, true)
    is ConnectionState.Failure -> view.initializeButtons(false, false)
}

[Ellen Spertus]

@pt I hadn’t known I could group cases together. Thank you and @Ruckus

[Ellen Spertus]

It looks even nicer on wider lines in my editor. @Ruckus

[Ruckus]

Indeed. Slack does not make for a good IDE.