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

    +161

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    reset($queue);
          $query = 'SELECT COUNT(*)'
          .' FROM #__datsogallery'
          .' WHERE (0 != 0';
          while (list($key, $cat) = each($queue)) {
            $query .= ' OR catid = '.$cat;
          }
          $query = $query
          .') AND published = 1'
          .' AND approved = 1';
          $db->setQuery($query);
          $result = $db->query();

    Очень возбуждает строка 4.
    Это в официальном релизе компонента com_datsogallery к Joomla 1.5

    Запостил: airrussia, 20 Октября 2011

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

    • показать все, что скрыто> Очень возбуждает строка 4.
      а меня - женщины
      Ответить
    • Joomla такая Joomla
      Ответить
      • Да причем тут Joomla. Как framework очень даже. А вот кривые руки, пишущие такие компоненты. Это да.
        Ответить
        • к сожалению как фреймворк она убога. но это моё мнение, как человека с ней работавшего.
          Ответить
          • какой нах framework?!
            Ответить
          • ну помоему.... хорошая штука, когда правильно пользовать.... тут спецом хотел сравнить JF и ZF пока выиграл JF, по самому главному параметру ВРЕМЯ
            Ответить
    • Ну всё понятно, собственно. Авторы поленились городить дополнительный код и влепили условие 0 != 0, чтобы AND после него всегда сработал, а не выдал синтаксическую ошибку в случае нулевого размера queue.
      Ответить
      • очень на то похоже.
        Ответить
      • А помоему они не знаю о существовании такой функции как implode
        Ответить
        • Один хрен sql-запросы с inline-вставкой параметров говно по определению.
          Ответить
    • показать все, что скрытоvanished
      Ответить

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