Any way to write this in funcitonal fashion? ``` controls.mapValues { val ans = EnumSet.noneOf(N...

# getting-started

[miha-x64]

Any way to write this in funcitonal fashion?

controls.mapValues {
    val ans = EnumSet.noneOf(NotificationTarget::class.java)
    if (it.value.pushSwitch.isChecked) ans += NotificationTarget.PUSH
    if (it.value.emailSwitch.isChecked) ans += NotificationTarget.EMAIL
    ans
}

[okkero]

controls.mapValues {
    EnumSet.copyOf(
            listOf(
                    it.value.pushSwitch to NotificationTarget.PUSH,
                    it.value.emailSwitch to NotificationTarget.EMAIL
            )
                    .filter { it.first.isChecked }
                    .map { it.second }
    )
}

[okkero]

how about that ^ ?

[miha-x64]

Thank you, that’s an interesting approach, but too allocationful. There are many of switch pairs, and it’s not great to create extra map for each pair.

[okkero]

If you're worried about the performance of the functional approach, then it might just be better to do it imperatively

[okkero]

Couldn't you just store the mappings in a top level immutable Map?

[okkero]

I guess you'd have to do an extra filter, then...hmm

[miha-x64]

I thought about a global immutable map at the very first moment, but then I must tag actual switches somehow. Maybe I’d better stick with the original solution.