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

    +147

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

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

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

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

    • за что минусуете?
      разве
      if ($result and mysql_num_rows($result))
      не одно и тоже?
      да еще если и запрос выше без ошибок был, так вообще
      if (mysql_num_rows($result))
      или может я заблуждаюсь?
      Ответить
      • Ты сам ответил на свой вопрос:
        > бля..... ну это же php...
        Да, это PHP. Здесь может быть всё, что многим снится в кошмарах.
        Ответить
        • все да не все, если запрос кривой, так первое условие не сработает, а про php я написал именно по второму условию, в нем же вроде как если тип не задан, то ноль равно false и соответственно > 0 , это уже некая параноя. и условие точно такое же как
          mysql_num_rows($result) оно же либо 0 (false) либо > 0 (true)
          Ответить
          • ненене, ты не понял, в чем прикол. если $result был false (типа, ошибка), то mysql_num_rows кинет ворнинг. Если же он валидный, то дополнительно проверяем, не пустой ли набор.
            То есть, код будет выполняться только тогда, когда есть строки для обработки. Код совершенно правильный
            Ответить
            • if(@mysql_num_rows($result)), если кому больше нравится.
              Ответить
            • вот дошло теперь, только походу: > 0, все равно лишнее
              Ответить
              • для пхп да, в данном случае, т.к. отрицательным значение никак не может быть )
                Ответить
              • Но и хуже от него не стало.
                Ответить
                • и логика логичней. а то смотрится примерно так:
                  кол-во строк - да?
                  Ответить
    • Даже не зная всех тонкостей PHP, отмечу, что для большинства интерпретируемых (и не только) языков это нормально.
      Ответить

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