ЙажаСценарий / Говнокод #23228 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
for (var i = pos_Song.length - 1; i >= 0; i--) {
      pos_Song[i].sing = Singer.findOne({_id : pos_Song[i].singer}).name;
    }

Один ёбнутый товарищ не научился думать и ходит по массиву с конца. Классика

Запостил: timblaer timblaer, (Updated )

Комментарии (9) RSS

  • ava warlord # Ссылка на оригинал 0

    Классика у него блять. У бати твоего классика.
    Ебнутый товарищ знает, что обращение к переменной быстрее, чем обращение к свойству объекта.
    Еще лучше так: for (var i = pos_Song.length; i--;) {}
    Можно и так: var i = pos_Song.length; while (i--) {}
    Да, это экономия на спичках. Но каждый дрочит так, как хочет.
    Автор, если тебе нечем заняться на каникулах - почитай то, что тебе задали по литературе. Или маме помоги по дому.
    И съеби уже из интернета.
    Ответить

    • ava inho # Ссылка на оригинал 0 ⇈

      Ты блядь ещё предложи i-- на --i заменить, он же быстрее. Это всё хуйня, по сравнению с тем что в теле цикла.
      Ответить

      • ava warlord # Ссылка на оригинал 0 ⇈

        Я и не смотрел, что в теле цикла. В посте претензии были к самому циклу. На эту тему я и высказался. Насчет того, что это все хуйня я уже написал: "Да, это экономия на спичках. Но каждый дрочит так, как хочет." roman-kashitsyn еще 7 часов назад откомментил по поводу тела. Поэтому, либо ты тормознул, либо решил сумничать, прочитав коммент внимательного человека. В любом случае твой коммент уже не к месту. Если бы было написано "Обращение к базе в цикле. Вот лох. Классика" или вообще с абстракным "Вот это говно!" - я бы начал смотреть в чем говно и увидел бы только дерьмовое тело.
        Ответить

        • ava inho # Ссылка на оригинал 0 ⇈

          Да ты не ссы. Умнее борманда и кашыцына тут врядли кто есть. Просто я считаю что доёбки автора тут по делу.
          Ответить

  • ava roman-kashitsyn # Ссылка на оригинал 0

    Кмк, самое сочное здесь не в цикле, а в N запросах к базе данных (если я правильно понял, это MongoDB, и мы в цикле подгружаем из неё документы по _id). А можно было бы просто выгрести всю пачку за один запрос.
    Т.е. горе-оптимизатор заоптимизировал копеечный цикл, а вот как заоптимизировать реально узкое место — чтение из базы — не подумал.

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

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓