`stream.filter { condition == null || it == condit...

# announcements

[nil2l]

stream.filter { condition == null || it == condition }

[Andreas Sinz]

With this code, the list is always "filtered", which is unnecessary when condition == null

[nil2l]

c?.let { s.filter.(it::equals) } ?: s

[Andreas Sinz]

thats better, but IMO the ìf/else version is more readable. Just because you can, doesn't mean you should 😄

[nil2l]

It depends. If it is a part of a chain then single-statement is ok.