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

    +162.1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if ( !empty($a1) ||
     !empty($a1) && !empty($a2) ||
     !empty($a1) && !empty($a2) && !empty($a3) ||
     !empty($a1) && !empty($a2) && !empty($a3) || && !empty($a4) ||
     !empty($a1) && !empty($a2) && !empty($a3) || && !empty($a4) || && !empty($a5) ||
...
     !empty($a1) && !empty($a2) && !empty($a3) || && !empty($a4) || && !empty($a5) || && ... !empty($a8) )
{
 // что-то делать...
}

    Уже и не припомню где именно, но всретил я такую проверочку на рабочем сайте:
    После небольшой оптимизации получилось нормально условие... кто скажет какое ???

    Запостил: Death, 24 Февраля 2010

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

    • ava guest 24.02.2010 21:57 # −2

      if ( !empty($a1) ||
      !(empty($a1) || empty($a2)) ||
      !(empty($a1) || empty($a2) || empty($a3)) ||
      ......
      Так?
      Ответить

    • ava HyperGeek 24.02.2010 22:01 # 0 

      || &&

      Это что за хрень?
      Ответить

    • ava guest 24.02.2010 22:30 # +1

      !empty($a1)
      Уже обращает выражение в true.
      Ответить

    • ava guest 24.02.2010 22:32 # +0.2

      на рабочем сайте это могло быть только картинкой
      Ответить

    • ava TarasB 24.02.2010 23:33 # 0

      Каждое следующее условие - подусловие предыдущего.
      Так что !empty($a1).

      > !empty($a1) Уже обращает выражение в true.

      Ну я пых не знаю, поэтому чисто с точки зрения логики сказал.
      Ответить

    • ava guest 25.02.2010 00:50 # 0

      if ( !empty($a1) )
      {
      // что-то делать...
      }

      :)
      Ответить

      • ava guest 25.02.2010 00:51 # +1

        кстати сказать, "|| &&" выдает подделку :)
        автор, выпей кой-чего
        Ответить

        • ava zxm 25.02.2010 11:14 # −1

          А что с ними не так?
          Ответить

        • ava guest 25.02.2010 13:20 # −1

          Кэп каг-бэ намекает, что это результат сжатия кода.
          Ответить

        • ava Death 25.02.2010 14:07 # −1

          С чего такие мысли???
          || === or
          && === and
          Ответить

          • ava murz 25.02.2010 14:22 # +0.8

            А где Вы видели, чтобы эти два оператора шли друг за другом?
            Ответить

          • ava guest 25.02.2010 17:25 # +0.8

            Скорее ==, потому что у них приоритет разный всё-таки. У or и and он едва ли не самый низкий среди всех операторов, допустимых в выражениях.
            Ответить

    • ava mrbig66 25.02.2010 09:21 # −0.2

      Если обязательно наличие элемента а1, то достаточно...
      if(!empty($a1))
  return true;
return false;


      Если должно быть наличие хотя бы одного элемента, то...
      if(empty($a1) AND empty($a2) AND ... empty($a8))
  return false;
return true;
      Ответить

    • ava Death 25.02.2010 14:02 # −2.2

      Правильный ответ это

      guest 16 часов назад # +1
      !empty($a1)
      Уже обращает выражение в true.

      при этом !empty($a1) можно упрощённо написать просто как if ($a1)
      Спасибо всем, кто пытался поливать гразью в комментах...
      Кстати, операторы && и || такие-же php-ый как и сишные...
      Ответить

      • ava murz 25.02.2010 14:32 # 0

        Может в Си и можно писать подряд два логических оператора, а в PHP такого нет
        Ответить

        • ava Death 25.02.2010 15:02 # 0

          Уверены??? проверьте.

          zxm
          http://www.php.net/manual/en/language.operators.logical.php
          Ответить

          • ava murz 25.02.2010 15:16 # 0

            Если у Вас выполнится этот код
            <?
if ($a1 || && $a2) {
}
?>
            я Вам медальку дам

            zxm
            А да, протупил.
            Ответить

            • ava Death 25.02.2010 15:20 # 0

              Готов выдать ответный орден за выполнение кода
              if ($a1 or and $a2) :)

              Вы наверно про опечатку в примере... говорите... не увидел, то я ошибся гогда набирал... Sorry
              Ответить

              • ava murz 25.02.2010 15:24 # 0

                Мне то за что? Это у Вас в коде написано два подряд логических оператора и утверждаете, что такая конструкция есть в PHP
                Ответить

                • ava Death 25.02.2010 15:32 # 0

                  Уже понял про что вы говорите.

                  не увидел, то я ошибся когда набирал... Sorry.
                  хотя это уже не принципиально...
                  Ответить

      • ava guest 25.02.2010 14:33 # +1

        >>!empty($a1) можно упрощённо написать просто как if ($a1)
        Нельзя так писать.
        1) 0, пустая строка и false это не емпти
        2) warning
        Ответить

        • ava murz 25.02.2010 14:39 # 0

          > 1) 0, пустая строка и false это не емпти
          А что?
          Ответить

          • ava guest 25.02.2010 16:38 # 0

            протупил сорри, это же не isset()
            но if() сгенерит warning, если переменной нет, тут я не ошибся.
            но мне собственно || &&
            Ответить

            • ava Death 25.02.2010 16:51 # 0

              про "|| &&" уже ответил: это просто опечатка у меня возникла при наборе... копипастил и не заметил...
              там просто было много конструкций типа:
              !empty($a1) && !empty($a2) && !empty($a3) ||... и так до $a8
              Ответить

            • ava murz 25.02.2010 16:58 # 0

              если нет переменной, то notice
              Ответить

        • ava Death 25.02.2010 15:02 # 0

          0, пустая строка, false, пустой массив это емпти!!!

          начиная в php 4
          Ответить

      • ava HyperGeek 25.02.2010 14:56 # 0

        КО деанонимизировался
        Ответить

    • ava guest 25.02.2010 17:45 # 0

      Чем отличаются if (isset($a)) , if (!empty($a)) и if ($a) ? также if (not_null($a)) от if ($a != 0) ?
      Ответить

      • ava Death 25.02.2010 18:16 # 0

        очень многим, если хотите, расскажу

        if (!empty($a)) от if ($a) не сильно отличается, а вот остальное...
        Ответить

        • ava Death 25.02.2010 18:22 # 0

          isset - даёт true вообще если есть переменная, даже если это будет /index.php?var >> (isset('var')) будет true

          empty($x) == true для:
          "", 0 (0 integer), "0" (0 string), NULL, FALSE, array(), var $var

          единственная разница от $x, это кажется, empty($x) когда $x - это класс без полей и методов в php4 давало false, а в php5 - true.

          not_null и ($a != 0) -- тут по аналогии и так понятно...
          Ответить

    • ava JloJle4Ka 26.07.2021 06:30 # 0

      Спасибо вам Борис огромное за :empty. Никогда не видел интересный пример его практического употребления, моя вина - не искал. Прям разинул рот от удивления. Сколько лишнего 'ternary' в коде! :) Бывает, однако, что мы от мусора не можем избавится. Делаем первую ревизию K.I.S.S. и YAGNI, но заказчик требует добавить/изменить/ дополнить/убрать... и мы делаем это. Потом опять, и опять. В какой-то момент ты осознаёшь что всё надо снести до фундаментa и сделать с начала... но заказчик за такое платить не будет... Потом получается вместо "keep it simple, stupid"... "keep it at least working"...
      Ответить

      • ava Fike 26.07.2021 13:10 # 0

        Коллега, кажется у нас с Вами один и тот же заказчик...
        Ответить

      • ava Desktop 26.07.2021 13:11 # 0

        я было подумал, что один другого благодарит за смайл :empty:
        Ответить

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