我有两种排序方法。按优先级和截止日期排序。
fun getTargetsByPriority() {
val valueEventListener = object : ValueEventListener {
override fun onDataChange(dataSnapshot: DataSnapshot) {
targetList.clear()
dataSnapshot.children
.mapNotNull { it.getValue(Goal::class.java) }
.sortedBy { it.priority }
.toCollection(targetList)
contract.updateViewContent()
}
override fun onCancelled(databaseError: DatabaseError) {
Log.d("some", "Error trying to get targets for ${databaseError.message}")
}
}
targetsRef?.addListenerForSingleValueEvent(valueEventListener)
}
fun getTargetsByDeadline() {
val valueEventListener = object : ValueEventListener {
override fun onDataChange(dataSnapshot: DataSnapshot) {
targetList.clear()
dataSnapshot.children
.mapNotNull { it.getValue(Goal::class.java) }
.sortedBy { it.deadline }
.toCollection(targetList)
contract.updateViewContent()
}
override fun onCancelled(databaseError: DatabaseError) {
Log.d("some", "Error trying to get targets for ${databaseError.message}")
}
}
targetsRef?.addListenerForSingleValueEvent(valueEventListener)
}但它们彼此非常相似。如何优化这段代码?
发布于 2019-11-27 06:32:52
你可以这样做:
inline fun <reified T : Any, R : Comparable<R>> DataSnapshot.sortChildrenTo(
targetList: MutableList<T>,
crossinline sortCriteria: (T) -> R?
) {
children.mapNotNull { it.getValue(T::class.java) }
.sortedBy(sortCriteria)
.toCollection(targetList)
}这样你就可以写下:
targetList.clear()
dataSnapshot.sortChildrenTo(targetList, Goal::priority)发布于 2019-11-27 01:30:21
您可以使用sortedWith并将比较器作为参数传递,而不是使用sortedBy。因此,将两个函数合并为一个函数,并将比较器作为参数。例如,下面的函数需要一个字符串比较器。
fun getTargets(comparator: Comparator<String>) {
}现在,您可以按如下方式调用此函数
getTargets(Comparator{ o1, o2 -> o1.compareTo(o2)})现在在您的sortedWith函数调用中使用这个比较器,您可以在其中对列表进行排序。
您的呼叫将如下所示。
getTargets(Comparator{ o1, o2 -> o1.priority.compareTo(o2.priority)}) // For priority based sorting
getTargets(Comparator{ o1, o2 -> o1.deadline.compareTo(o2.deadline)}) // For deadline based sortinghttps://stackoverflow.com/questions/59055795
复制相似问题