A modern programming language that makes developers happier.

kotlinlang

tried smth like this
```
target?.let { targetList.sortedWith(compareBy {it.priority == 1})}
```

Why do you need to call to `let`?
If you want to sort the list in-place by priority, you can use `targetList.sortBy { it.priority }`.

if i write smth like this
```
                    target.let { targetList.sortBy { it.priority == 1 } }
```
in logs i see also all targets

can write smth like this
```
if (target?.priority == 1) targetList.add(target)
```

I have no idea what you’re trying to do. Why would you want to sort by `it.priority == 1`?

I want to sort my list by desceding

now i sort my list if target priority is 1

There’s `targetList.sortByDescending { it.priority }`

Or is this what you need?
```
if (target?.priority == 1) {
    targetList.sortByDescending()
}```

i write smth like this now
```
fun getTargetsByPriority() {
        val valueEventListener = object : ValueEventListener {
            override fun onDataChange(dataSnapshot: DataSnapshot) {
                targetList.clear()
                for (targetSnapshot in dataSnapshot.children) {
                    val target = targetSnapshot.getValue(Target::class.java)
                    target?.let { targetList.add(it) }
                }
                targetList.sortedByDescending { it.priority }
                contract.updateViewContent()
            }

            override fun onCancelled(databaseError: DatabaseError) {
                Log.d("some", "Error trying to get targets for ${databaseError.message}")
            }
        }
        targetsRef?.addListenerForSingleValueEvent(valueEventListener)
    }
```
but get in log smth like this
```
2019-11-02 22:54:19.435 5489-5489/com.mandarine.target_list D/some: targetList: [Target(guid=-LrizWpFMU714rExtoit, name=huhy, description=hhhyy, date=0, priority=2), Target(guid=-LrizfGg0cKxlftLuL29, name=hhh, description=hhhh, date=0, priority=1), Target(guid=-Ls2Qr_H9jTbchvFBny4, name=hhh, description=hhhh, date=1572134400000, priority=0), Target(guid=-LshguMEgE6vPYMUxIQT, name=ооо, description=ооо, date=1574208000000, priority=1)]

```

Yes, `sortBy` sorts in-place, while `sortedBy` returns a new sorted list.

but can i simplify this method?
```
targetList.sortByDescending { it.priority }
targetList.reverse()
```

Just do `targetList.sortBy { it.priority }`. Why sort descending when you’re going to reverse it?