Можно ли ускорить функцию составления топа?

Question

[DeFaNJI]

Я создал функцию, которая сортирует юзеров по уменьшению значения в поле, а также ищет позицию человека в топе.

const getTop = async (name, userId) => {
    const top = await Users.find({ admin: false }, { [name]: 1, id: 1 }).sort({
        [`${name}`]: -1,
    });

    let userPosition = 0;

    for (let i = 0; i < top.length; i++) {
        const { id } = top[i];

        if (id === userId) {
            userPosition = i;
            break;
        }
    }

    const topUsers = top.slice(0, 10);

    return [topUsers, userPosition + 1];
}

Возможно ли изменить ее, что бы она работала быстрее? Мне кажется, что при большом кол-ве человек топ будет составляться большое кол-во времени.

Сразу говорю, что искал в интернете, как узнать номер документа при сортировке, но не нашел никакой информации по этому поводу

Answer 1

[kapp1]

Постараюсь быть менее токсичным и не писать про 10+ лет опыта)

Тебе надо почитать статьи ну или ролики про high performance в монго или optimize query performance.

Если коротко, то не раздувай коллекцию лишними данными и делай индексы.

Comments

[DeFaNJI]

Спасибо, посмотрю и почитаю!)

Answer 2

[Алексей Дубровин]

По поводу оптимизации.

сортирует юзеров по уменьшению значения в поле, а также

Я бы подумал по поводу разделения кода, а не скорости.
Сейчас вы думаете, а как же оптимизировать. Да не как. Вы не настолько разбираетесь в языке что-бы это делать.

Когда у вас реально будет "Влиять на скорость". Тогда и начнёте думать.

Сразу говорю, что искал в интернете, как узнать номер документа при сортировке, но не нашел никакой информации по этому поводу

Вот эта информация вообще к чему.
Какой документ, какая сортировка?

Comments

[DeFaNJI]

Обычно я сортировал просто создавал запрос вида: .sort({ [value]: -1 }).limit(10) и это было быстрее, чем циклом проходить по всему массиву пользователей и сортировать по уменьшению баланса, пробовал искать похожие решения, но не нашел, поэтому и написал

По поводу оптимизации я думаю лишь потому, что бы человек не ждал в будущем по 10-15 секунд, на мой взгляд лучше сразу написать хорошо, что бы потом не исправлять и не думать над оптимизацией?

[Алексей Дубровин]

DeFaNJI,
Можно мне рабочий пример где у вас будет это делаться по 10-15 секунд.
Когда вы увидете сколько пользователей для этого необходимо задайте себе следующий вопрос.
Через сколько лет у вас будет столько пользователей.

запрос вида: .sort({ [value]: -1 }).limit(10)

Мне этот код вообще ни о чём не говорит. я не знаю что там внутри и как работает.

Чтобы сразу писать хорошо. Нужно иметь 10+ лет опыта.
Выбор решения "оптимального в данном случае не понятен" тк как там множество факторов и по одному коду это сделать нельзя.
Есть решения которые мы можем делать. Но это не такой случай)
Мб лучшее вообще создать структуру данных дерево?
На текущем уровне думать об этом особо бесполезно.
Технический долг это не создаёт.
Значит нужно отложить на как можно поздний срок.
Когда действительно этот код создаст нагрузку у вас уже будут другие знания.

На текущем уровне старайтесь решать задачи и понимать как и что происходит.
Это всё что мы можем.

Answer 3

[sasmoney]

ограничить количество и разделять на отдельные страницы с получением 1001 строки, 2001 и тд