Hi this is horrible do you know a more elegant way...

# codereview

[ivano]

Hi this is horrible do you know a more elegant way to refactor that ? ( sorry cor the format i am on mobile the three format
``` are not picked up)
if (count() > 5) take(5).forEach { firstFiveSubsectionItems -> buildAlwaysVisibleLogic(alwaysVisibleView, firstFiveSubsectionItems)
} else {
forEach { itemsSubsection -> buildAlwaysVisibleLogic(alwaysVisibleView, itemsSubsection)
}
}

[sbyrne]

You can do take(5) even if there are fewer than 5, and you'll just get all of them.

[ivano]

Doh thanks!

[Astronaut4449]

In case take or any other function did not work for both cases you should write it like this:

val items = if (condition) a() else b()
items.forEach { commonAction(it) }

[ivano]

Thanks