```private fun getActiveFilters(categoryId: Int): ...

# getting-started

[oday]

private fun getActiveFilters(categoryId: Int): String? {
        val activeFilters =  mutableFilterItems.value?.mapNotNull {
            it.takeIf { filterItem ->
                filterItem.categoryId == categoryId && filterItem.toggled
            }
        }?.joinToString { it.itemValue.capitalize() }
        
        return activeFilters
    }

[spand]

return activeFilters?.takeIf { it.isNotEmpty() } ?

[oday]

yea true, was looking for something like ternary or some kung fu stuff

[StavFX]

Most glaring thing I see is the use of mapNotNull + takeIf could be replaced with a simple filter

[StavFX]

So combined with Johannes/Hakob’s suggestion

private fun getActiveFilters(categoryId: Int): String? {
   return mutableFilterItems.value
      ?.filter { it.categoryId == categoryId && it.toggled }
      ?.takeIf { it.isNotEmpty() }
      ?.joinToString { it.itemValue.capitalize() }
}

But I would definitely reconsider the use of nulls here, maybe return an empty String like @llsouder suggested