1. PHP / Говнокод #12743

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    $threadusers = $db->query_read("
  SELECT user.username, user.usergroupid, user.membergroupids,
   session.userid, session.inthread, session.lastactivity, session.badlocation,
   IF(user.displaygroupid = 0, user.usergroupid, user.displaygroupid) AS displaygroupid, infractiongroupid,
   IF(user.options & " . $vbulletin->bf_misc_useroptions['invisible'] . ", 1, 0) AS invisible
  FROM " . TABLE_PREFIX . "session AS session
  LEFT JOIN " . TABLE_PREFIX . "user AS user ON(user.userid = session.userid)
  WHERE  session.lastactivity > $datecut
 ");

...

while ($loggedin = $db->fetch_array($threadusers))

    Я уже несколько раз писал о гениальности ребят которые пишут форум vBulletin. Продолжу эту традицию.
    Выше приведен огрызок кода, которым ребятки собирают список активных пользователей в текущей теме. Думаю тут все понятно, но все же объясню на примере того форума, где довелось увидеть это чудо. В таблице session около 7к записей, активных записей в среднем 4к. Т.е. на выходе мы имеем запрос, который кладет сервак при 64 гигах оперативы + последовательный перебор 4к записей для того что бы выбрать парочку юзеров, которые активны в этой теме.
    Ну в общем ребятки поставили себе прижизненный памятник, который я спешу предоставить на лицезрение сообщества.

    Запостил: vagrand, 13 Марта 2013

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

    • ava eth0 13.03.2013 20:01 # 0

      Если активность в треде выбирают не запросом, то да, довольно упорото. Запрос сам по себе типовой.
      Ответить

      • ava vagrand 13.03.2013 20:05 # 0

        Дело не столько в запросе а в том сколько данных он выбирает в результате своей работы. Если на форуме 4к активных пользователей то каждый из них выберет 4к записей.
        Ответить

        • ava eth0 14.03.2013 17:08 # 0

          Поэтому и надо дополнить запрос условием.
          Ответить

    • ava хуита 13.03.2013 20:16 # −2

      Commet/Websockets
      Использование ORM, вместо сырого SQL
      Пиздец, эта параша же ещё и платная.
      Ответить

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