``` override fun getItemViewType(position: Int): I...

# codereview

[ivano]

override fun getItemViewType(position: Int): Int {

        return if (hasHeader) {
            if (position == 0) HEADER
            else entryItem(position)
        } else if (position == 0) ITEM_WITH_DATE else entryItem(position + 1)
    }

[Mike]

return when {
    position == 0 -> if(hasHeader) HEADER else ITEM_WITH_DATE
   hasHeader -> entryItem(position)
   else -> entryItem(position + 1)
}

Marginally better, but I suspect still requires some documentation to make it a bit clearer what it's achieving.

[ivano]

is an improvement indeed

[ivano]

thank you very much