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

    • Если активность в треде выбирают не запросом, то да, довольно упорото. Запрос сам по себе типовой.
      Ответить
      • Дело не столько в запросе а в том сколько данных он выбирает в результате своей работы. Если на форуме 4к активных пользователей то каждый из них выберет 4к записей.
        Ответить
        • Поэтому и надо дополнить запрос условием.
          Ответить
    • Commet/Websockets
      Использование ORM, вместо сырого SQL
      Пиздец, эта параша же ещё и платная.
      Ответить

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