Uma opção bacana que teria para reduzir um pouco o Boilerplate daqueles IF/ELSE seria colocar todas as alterações dentro da mesma validação, reduziria algumas linhas de código, ou talvez a utilização do When se for aplicável nesse cenário xD
if (transaction.type == Types.REVENUE) {
viewCreated.transaction_value.setTextColor(ContextCompat.getColor(
context,
R.color.revenue_color))
viewCreated.transaction_icon.setBackgroundResource(R.drawable.icone_transacao_item_receita)
} else {
viewCreated.transaction_value.setTextColor(ContextCompat.getColor(
context,
R.color.expense_color))
viewCreated.transaction_icon.setBackgroundResource(R.drawable.icone_transacao_item_despesa)
}