1. Java / Говнокод #7137

    +79

    1. 1
    2. 2
    boolean direct = !Boolean.FALSE.equals(directParam);
    boolean demoAgency = !direct;

    Запостил: yvu, 03 Июля 2011

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

    • ava SmackMyBitchUp 03.07.2011 16:34 # +3

      boolean direct = directParam;
      boolean demoAgency = !directParam;

      Еще один, который имеет необъятную фантазию, а так дак вообще хер заткнешь.
      Смахивает на выдуманный ГК.
      Ответить

      • ava absolut 03.07.2011 18:00 # 0

        boolen TRUE = !Boolean.FALSE;
        boolen FALSE = !TRUE;
        Ответить

      • ava gegMOPO4 06.07.2011 19:27 # 0

        Внезапно directParam = null.
        Ответить

        • ava SmackMyBitchUp 06.07.2011 19:55 # 0

          Если внезапно directParam = null (что говорит нам о том, что directParam - объект класса Boolean), то NullPointerException, если же directParam имеет тип boolean, то уже никак не может принимать значение null.
          Ответить

          • ava gegMOPO4 06.07.2011 20:21 # 0

            Не будет NullPointerException, equals безопасен насчёт null. Будет false, именно поэтому там !Boolean.FALSE.equals(directParam), а не Boolean.TRUE.equals(directParam).

            Если у автора есть хоть капля мозгов (а нет оснований предполагать обратное), то всё правильно написал, и это не говнокод.
            Ответить

            • ava SmackMyBitchUp 06.07.2011 21:55 # 0

              дедушка мороз. вы комментарий писали к моему комменту? да, к моему! ну дак я свой код и рассматривал на наличие нулла.
              Ежели рассматривать код автора, то да, вы правы.
              Ответить

              • ava gegMOPO4 07.07.2011 09:35 # 0

                Ну вот потому ваш код и не эквивалентен исходному.
                Ответить

                • ava SmackMyBitchUp 07.07.2011 10:35 # 0

                  И слава богу.
                  Ответить

                  • ava gegMOPO4 07.07.2011 11:00 # 0

                    Благодарить будете, когда после такого левшацкого «улучшения» поймаете NullPointerException.
                    Ответить

                    • ava SmackMyBitchUp 07.07.2011 11:55 # 0

                      Я же уже расписал вам 2 варианта:
                      1) boolean
                      2) Boolean

                      Причем 2 раза расписал, а вы все продолжаете какую-то ересь городить.
                      Ответить

                      • ava gegMOPO4 07.07.2011 12:26 # 0

                        1) Есть ли основания подозревать boolean, а не Boolean?
                        2) Для Boolean ваш код — ересь и левшизм.

                        Давайте говорить здесь о реальном говнокоде, а не воображаемом.
                        Ответить

                        • ava SmackMyBitchUp 07.07.2011 20:38 # 0

                          1) А есть ли основания подозревать Boolean?
                          2) Заебали

                          Было же сказано - информации мало.
                          Даже говоря о реальном гк вы не можете утверждать что-либо.
                          Ответить

                          • ava gegMOPO4 07.07.2011 20:57 # 0

                            1) Да, есть. Boolean в тексте.
                            2) Думайте об Англии.

                            Информации мало, чтобы решить, то ли автор кода сговнокодил, то ли постер неосилил. Собственно, отсутствует самая главная информация, о типе и возможных значениях directParam, — только это может служить основанием в говнокодстве. А это даёт основания подозревать постера в неосиляторстве.
                            Ответить

                            • ava SmackMyBitchUp 08.07.2011 00:15 # 0

                              1) Странно, но обнаружено присутствие не одного только Boolean
                              Ответить

    • ava SmackMyBitchUp 03.07.2011 20:29 # 0

      Хотя, если честно, как романтику мне код очень даже понравился, в заметки кину :D
      Ответить

    • ava jQuery 04.07.2011 02:50 # +2

      Опять java, опять boolean :/
      Ответить

    • ava guest 04.07.2011 13:56 # +3

      PHP - даты, C++ - тернистый оператор, Java - boolean, продолжите ряд...
      Ответить

    • ava gegMOPO4 06.07.2011 19:31 # 0

      Без контекста не говнокод. directParam может быть null (если он Boolean) или вообще Object.
      Ответить

      • ava SmackMyBitchUp 06.07.2011 19:59 # 0

        А ну вот пока писал не заметил ваш пост. Да, мало информации.
        Ответить

    • ava guest8 09.04.2019 11:07 # −999

      показать все, что скрытоvanished
      Ответить

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