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

    +147

    1. 1
    2. 2
    3. 3
    if ($result and mysql_num_rows($result) > 0) {
   /*здесь код*/
}

    бля..... ну это же php...

    Запостил: Sulik78, 28 Февраля 2011

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

    • ava Sulik78 28.02.2011 16:44 # −3

      за что минусуете?
      разве
      if ($result and mysql_num_rows($result))
      не одно и тоже?
      да еще если и запрос выше без ошибок был, так вообще
      if (mysql_num_rows($result))
      или может я заблуждаюсь?
      Ответить

      • ava Skelet0n 28.02.2011 16:50 # −1

        Ты сам ответил на свой вопрос:
        > бля..... ну это же php...
        Да, это PHP. Здесь может быть всё, что многим снится в кошмарах.
        Ответить

        • ava Sulik78 28.02.2011 16:58 # −1

          все да не все, если запрос кривой, так первое условие не сработает, а про php я написал именно по второму условию, в нем же вроде как если тип не задан, то ноль равно false и соответственно > 0 , это уже некая параноя. и условие точно такое же как
          mysql_num_rows($result) оно же либо 0 (false) либо > 0 (true)
          Ответить

          • ava Lure Of Chaos 28.02.2011 18:28 # +4

            ненене, ты не понял, в чем прикол. если $result был false (типа, ошибка), то mysql_num_rows кинет ворнинг. Если же он валидный, то дополнительно проверяем, не пустой ли набор.
            То есть, код будет выполняться только тогда, когда есть строки для обработки. Код совершенно правильный
            Ответить

            • ava gegMOPO4 28.02.2011 18:54 # −4

              if(@mysql_num_rows($result)), если кому больше нравится.
              Ответить

            • ava Sulik78 28.02.2011 19:57 # 0

              вот дошло теперь, только походу: > 0, все равно лишнее
              Ответить

              • ava Lure Of Chaos 28.02.2011 20:14 # 0

                для пхп да, в данном случае, т.к. отрицательным значение никак не может быть )
                Ответить

              • ava Int 28.02.2011 20:42 # 0

                Но и хуже от него не стало.
                Ответить

                • ava Lure Of Chaos 28.02.2011 21:03 # +2

                  и логика логичней. а то смотрится примерно так:
                  кол-во строк - да?
                  Ответить

    • ava SadKo 02.03.2011 12:18 # 0

      Даже не зная всех тонкостей PHP, отмечу, что для большинства интерпретируемых (и не только) языков это нормально.
      Ответить

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