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

    +158

    1. 1
    ($property['valid'] == 'ошибка') !== TRUE

    вот и бери стажеров на работу

    Запостил: CRRaD, 15 Апреля 2013

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

    • ava neeedle 15.04.2013 17:16 # −3

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

      • ava eth0 15.04.2013 17:51 # 0

        Моветон сравнивать булево значение с константой.
        Ответить

        • ava bormand 16.04.2013 05:16 # +1

          > Моветон сравнивать булево значение с константой.
          Okay. Сравним с переменной:
          $true_val = TRUE;
($property['valid'] == 'ошибка') >= $true_val
          Ответить

          • ava neeedle 16.04.2013 13:00 # −2

            Я так и не понял, что тут не так.
            Ну может быть кривенькое логическое выражение, ну работает оно со статичной строкой, ну и что?
            Работает то оно ведь правильно.
            false - ошибка.
            true - все нормально.
            Похоже, что у кого-то перфекционизм.
            Ответить

            • ava CRRaD 16.04.2013 13:26 # 0

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

              • ava neeedle 16.04.2013 13:54 # −1

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

            • ava bormand 16.04.2013 17:42 # 0

              Да строка тут не при делах вообще. Говно в !== TRUE из-за которого этот код намного сложнее читать, чем если бы написали $property['valid'] != 'ошибка'. Это же классика boolshit'a.

              P.S. Но тред минуснул т.к. бульшиты уже наскучили.
              Ответить

              • ava neeedle 16.04.2013 17:53 # 0

                Я это и сам понял.
                Кривенько конечно же, но постить джуниоровский код на говнокодру, не комильфо.
                Я думал, что тут какая-то специфика php, что-то очень смешное, а так.
                Ничего особенного же. Просто условие сложно написано.
                Ответить

          • ava eth0 16.04.2013 17:53 # 0

            Ещё можно добавить кастов, немного битовой магии, переводов во что-нибудь.
            Ответить

      • ava scriptin 15.04.2013 20:07 # −1

        1. 
        $property['valid'] != 'ошибка'


        2.
        > $property['valid']
        > 'ошибка'
        Ответить

        • ava inkanus-gray 16.04.2013 00:14 # −2

          Нет, не то.

          1. Магическая константа в конце выражения. Избавляемся: TRUE → intval('')

          2. Магическая константа в квадратных скобках. Избавляемся: $property['valid'] → (object)$property->valid

          3. Слишком мало полосочек и неявное приведение типа. Исправляем: == → ===

          4. Слишком мало явных приведений типов. Итого получаем:
          strlen(strval((object)$property->valid === 'ошибка')) !== intval('')
          Ответить

          • ava anonimb84a2f6fd141 16.04.2013 01:05 # 0

            "Магические литералы в квадратных скобках" в питоне используются очень даже нередко.
            Ответить

          • ava scriptin 16.04.2013 20:21 # 0

            А 'ошибка' убирать не нужно?
            Ответить

          • ava guest 10.05.2013 03:07 # 0

            Как жестоко. Не видел больших извращений. Еще и с гайдом
            Ответить

    • ava anonimb84a2f6fd141 22.04.2013 07:55 # 0

      И действительно, не бери, нормально плати нормальным программистам.
      Ответить

    • ava guest 14.05.2013 21:57 # 0

      $property['valid'] = 'ошибка'
      это он тоже сам придумал или ему по-старше подсказали?

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

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