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

    +71

    1. 1
    2. 2
    $res= (isset($r[0]) and count($r[0]) > 0) ? true : false;
    return $res;

    Классика

    Запостил: Мартин, 14 Мая 2012

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

    • А нивозможно была сразу написать return ( isset( $r[0] ) && count($r[0]) > 0); ?
      Ответить
      • так и переделал =)
        Ответить
      • а может !empty($r[0]), не ?

        хотя если $r[0] это не только массив, а объект реализующий Countable, то стоит подумать..
        Ответить
    • Слишком шаблонный пример неприятия логического типа. Я удивился, почему не так:
      if (isset($r[0])) {
          if (count($r[0]) > 0)) {
              $res = true;
          } else {
              $res = false;
          }
      }
      return $res;
      Специально сделал в коде ошибку, чтобы продемонстрировать ещё один шаблон.
      Ещё вариант (более похапешный):
      $res= (@count($r[0]) > 0) ? true : false;
      return $res;
      Ответить

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