1. JavaScript / Говнокод #23228

    −2

    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, 31 Июля 2017

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

    • ava j123123 31.07.2017 14:19 # +1

      иногда это даже быстрее
      https://lwn.net/Articles/414467/
      https://bugzilla.redhat.com/show_bug.cgi?id=638477#c31
      Ответить

    • ava warlord 02.08.2017 12:08 # +1

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

      • ava inho 02.08.2017 18:59 # 0

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

        • ava warlord 02.08.2017 19:25 # +1

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

          • ava inho 02.08.2017 19:55 # 0

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

            • ava warlord 02.08.2017 20:03 # 0

              Так чем плох обратный проход?
              Ответить

              • ava inho 02.08.2017 21:36 # 0

                Заставляет задуматься зачем он здесь.
                Ответить

            • ava sos 02.08.2017 22:21 # 0

              > Умнее борманда и кашыцына тут врядли кто есть.
              Я.
              Ответить

    • ava roman-kashitsyn 02.08.2017 12:17 # +1

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

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

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