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

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

      А в чём, собственно, плохость кода? Такое ощущение, что кое-какая дополнительная информация за кадром осталась.
      Ответить
      • $cnt2[0]['count(*)']

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

          Нипонил? То есть ты предлагаешь сделать
          SELECT * FROM `tablename`
          а потом посчитать строки? :))
          Ответить
          • mysql_num_rows()
            Ответить
            • Действительно, зачем использовать функционал СУБД, если есть великий и могучий пыхапэ со своим mysql_num_rows()
              Ответить
              • Какая разница, "пыхапэ" это или перл? Библиотеку используют одну, вроде. с 5.3.0.
                Ответить
                • Жаль, что Вы так ничего и не поняли.
                  Для подсчета количество записей нужно использовать именно select count(*)
                  Ответить
          • Надо было зайти с другой стороны...
            Намекните автору, что count(*) запрос агрегативный. =]
            Ответить
        • > есть еще num_rows

          Не позорься
          Ответить
    • Только так строки считать и нужно в таблице базы данных.
      Ответить
    • Разве, что get_cell надо было вызвать, если есть такой метод.
      Ответить
    • омг, что за база с энным количеством записей с одинаковым id?
      Ответить
      • HAHAHA DISREGARD THAT, I SUCK COCKS

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

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