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

    +148.3

    1. 1
    2. 2
    3. 3
    4. 4
    $cnt2 = $db->get_rows("select count(*) from `".TABLE_PRODUCTS."` where `id`='".intval($s_id)."'");
if ($cnt2[0]['count(*)']) {
     //...
}

    Это писец.

    Запостил: aerowave, 09 Ноября 2009

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

    • ava interested 09.11.2009 18:01 # 0

      Писец -- это человек такой, пишет он. А здесь, видимо, "кодец" -- кодит, код набирает (наберец кода). =] (j/k)

      А в чём, собственно, плохость кода? Такое ощущение, что кое-какая дополнительная информация за кадром осталась.
      Ответить

      • ava aerowave 09.11.2009 18:03 # 0 

        $cnt2[0]['count(*)']

        есть еще num_rows.
        Ответить

        • ava guest 09.11.2009 18:22 # 0

          'count(*)' вполне себе легитимный ключ в хэш-таблице. Выглядит, конечно, ужасно, но ничего криминального в этом нет.
          Ответить

        • ava Yozheg 10.11.2009 05:08 # 0

          > есть еще num_rows

          Нипонил? То есть ты предлагаешь сделать
          SELECT * FROM `tablename`
          а потом посчитать строки? :))
          Ответить

          • ava aerowave 10.11.2009 07:19 # 0

            mysql_num_rows()
            Ответить

            • ava guest 12.11.2009 13:11 # 0

              Действительно, зачем использовать функционал СУБД, если есть великий и могучий пыхапэ со своим mysql_num_rows()
              Ответить

              • ava aerowave 12.11.2009 13:13 # 0

                Какая разница, "пыхапэ" это или перл? Библиотеку используют одну, вроде. с 5.3.0.
                Ответить

                • ava ivmaroz 13.11.2009 10:22 # 0

                  Жаль, что Вы так ничего и не поняли.
                  Для подсчета количество записей нужно использовать именно select count(*)
                  Ответить

          • ava interested 10.11.2009 08:09 # 0

            Надо было зайти с другой стороны...
            Намекните автору, что count(*) запрос агрегативный. =]
            Ответить

        • ava ivmaroz 10.11.2009 09:21 # 0

          > есть еще num_rows

          Не позорься
          Ответить

    • ava Oleg_quadro 09.11.2009 19:00 # 0

      Только так строки считать и нужно в таблице базы данных.
      Ответить

    • ava Oleg_quadro 09.11.2009 19:01 # 0

      Разве, что get_cell надо было вызвать, если есть такой метод.
      Ответить

    • ava guest 10.11.2009 09:35 # 0

      омг, что за база с энным количеством записей с одинаковым id?
      Ответить

      • ava guest 10.11.2009 09:37 # 0

        HAHAHA DISREGARD THAT, I SUCK COCKS

        блин, сам не понял, что спросил. Собственно да, можно было бы select id from table where id = 3... Но это не сильно круче
        Ответить

      • ava guest 25.11.2009 18:35 # 0

        ну а вдруг id это поле как parent для дерева
        Ответить

    • ava guest 30.11.2009 17:05 # 0

      Проверка на существование записи.
      Ответить

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