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

    +163

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if($user_key !== '') {
   .... 
} elseif ($user_key == '') {
  .....
} else {
   exit('error');
}

    Еще один шедевр

    Запостил: revis0r, 09 Декабря 2010

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

    • ava wmmorgun 09.12.2010 11:38 # +4

      Как часто срабатывает exit? :-D
      Ответить

      • ava revis0r 09.12.2010 11:44 # +5

        Чувак, видимо, думал, что с вероятностью 33.33% :)
        Ответить

      • ava Oleg_quadro 09.12.2010 12:30 # 0

        нежданчик
        http://govnokod.ru/4058#comment43817
        Ответить

      • ava TarasB 09.12.2010 13:27 # −6

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

        • ava revis0r 09.12.2010 13:57 # 0

          Вообще да, но exit все равно никогда не сработает. Если, например, передать 0, то сработает первое условие :)
          Ответить

        • ava wmmorgun 09.12.2010 13:58 # 0

          Что плохого в динамической типизации?
          Плюс там однозначно до exit дело не дойдет, ибо делается строгое неравенство к '', то есть сначала проверяется тип поля, а потом значение, так что в elseif попадет только в одном случае, когда $user_key === '', т. е. только в случае пустой строки. В случай null, 0 попадем в "then"
          Ответить

          • ava Анонимус 09.12.2010 14:23 # +1

            >>Что плохого в динамической типизации?
            ничего. Плохо когда нет статической.
            Вернее это тоже хорошо, когда у Вас программа из ста строк кода.
            Если строк больше, то без статической типизации жизнь превращается в ад. А именно такой и становится жизнь любого пыхапешника, если он пытается программировать а не говнокодить
            Ответить

          • ava KirAmp 09.12.2010 19:12 # 0

            >>Что плохого в динамической типизации?
            Если вы захотите сложить овец с баранами то при динамики они сложутся и ошибку будите искать не один час.
            А при статике у вас компилятор (или IDE) выдаст ошибку типов
            Ответить

            • ava Анонимус 09.12.2010 19:20 # −1

              а еще если Вы решите переименовать метод getName у класса User, то при статической типизации у Вас IDE само переименует все его вызовы

              а без статической типизации надо будет руками менять везде
              Ответить

              • ava wmmorgun 09.12.2010 19:33 # 0

                Методы задокументированы, пока проблем с переименовыванием в phpStorm'е не возникало, правда и делать это приходилось пару раз.
                Ответить

                • ava Анонимус 09.12.2010 19:36 # −1

                  >>Методы задокументированы
                  т.е. документация (доки) лечит косяки отсутствия статической типизации?
                  ничего не скажешь -- красивое решение)))

                  >>равда и делать это приходилось пару раз.
                  в крупных проектах рефакторинг делается часто. Меняются классы, методы, сигнатуры, методы подымаются выше/ниже по иерархии.. Или в мире пхп делать рефакторинг не принято?)
                  Ответить

                  • ava bugmenot 09.12.2010 21:06 # −1

                    вспоминается php manual где говенные особенности язычка стыдливо не упоминают
                    Ответить

                  • ava Lure Of Chaos 10.12.2010 17:14 # +1

                    > Или в мире пхп делать рефакторинг не принято?)

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

        • ava paranoid 09.12.2010 15:27 # −2

          разве в школе начались каникулы?

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

    • ava dwinner 09.12.2010 11:56 # +4

      Клёво! Всё без ошибок! ))) Что ещё web-developer'у для счастья надо?!
      Ответить

    • ava Lure Of Chaos 09.12.2010 17:34 # +1

      зато обработаны все случаи жизни! если бы так каждый ретурн-флаг проверять ))
      типа
      $h=fopen($fn);
if($h) { ... }
elseif(!$h) { ... }
else { ... }

      и далее по коду = )
      Ответить

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