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

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    $qs = $this->_generateQuery();
// obtain count of items
$number_of_items = 0;
if ($qs['count'] == 'SELECT 1'): // что за ужасть??
  $number_of_items = 1;
else:
  $query = $qs['count'];
  $res = mysql__query ( $query );
 //......................................
endif;

    Кусок самописной цмски интернет-магазина. Оптимизация подсчета товаров в выборке.

    Запостил: Silentium, 19 Июля 2011

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

    • ava Lure Of Chaos 19.07.2011 10:32 # 0

      оптимизация???
      Ответить

      • ava Silentium 19.07.2011 10:35 # −2

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

        • ava Lure Of Chaos 19.07.2011 10:37 # 0

          а если select 1 то все ок ^^
          Ответить

          • ava Silentium 19.07.2011 10:38 # −2

            Так в том всё и дело, что туда сначала специально ставится "SELECT 1", а потом проверяется, вместо того, чтобы подумать головой и прописать количество единицу.
            Ответить

            • ava Lure Of Chaos 19.07.2011 10:40 # 0

              sql запрос должен быть запросом
              Ответить

              • ava Silentium 19.07.2011 10:42 # −2

                Так он не будет запросом, если он не выполняется. И кто вообще сказал, что там обязательно нужен запрос?
                Ответить

                • ava bugmenot 19.07.2011 10:46 # 0

                  это - балласт, когда потребуется увеличить скорость - закомментируют
                  Ответить

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