oday
04/01/2020, 1:59 PMprivate 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
04/01/2020, 2:02 PMreturn activeFilters?.takeIf { it.isNotEmpty() }
?oday
04/01/2020, 2:09 PMStavFX
04/06/2020, 9:20 PMmapNotNull + takeIf
could be replaced with a simple filter
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