Правильный ли у меня здесь код?

Question

[Alexey Trigubin]

let rank = allUsers.sort((a, b) => (b.cash - a.cash) + (b.bank - a.bank)).findIndex(e => e.user_id === target.id) + 1

мне нужно найти индекс пользователя в этом отсортированном массиве пользователей БД.Подскажите,можно ли как нибудь сделать этот код короче?

Comments

[Aetae]

Если это именно в бд - делайте это собственно запросом в бд, а не гоняйте всех юзеров на клиент.
Ну и сортировка странная, хз что она должна символизировать.

Answer 1

[Rsa97]

Короче делать смысла нет, а быстрее можно.

const targetUser = allUsers
  .find(user => user.user_id == target.id)
const targetUserMoney = targetUser.cash + targetUser.bank
let rank = 1;
for (let i = 0; i < allUsers.length; i += 1) {
  if (allUsers[i].cash + allUsers[i].bank > targetUserMoney) {
    rank += 1;
  }
{

Comments

[Alexey Trigubin]

это точно будет быстрее?

[Rsa97]

kirick kirick, sort() имеет сложность O(n*log₂n)
В моём варианте сложность O(n).