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

    +82

    1. 1
    Boolean hasRefId = !node.getAttributes().getNamedItem("refid").equals(null);

    equals(null)

    Запостил: roman-kashitsyn, 18 Сентября 2014

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

    • ava bormand 18.09.2014 11:52 # 0

      Но ведь работает ;)
      Ответить

      • ava someone 18.09.2014 12:06 # 0

        Ну, если ловить NullPointerException, то да.

        Он ещё и присваивается почему-то Boolean'у с большой буквы. Вот сборщик копается в мусоре потом, а разработчики ему даже спасибо не говорят.
        Ответить

        • ava roman-kashitsyn 18.09.2014 12:26 # +2

          > Вот сборщик копается в мусоре потом

          Сомневаюсь. Автобоксинг работает через Type.valueOf(type), а не через вызов конструктора. Поэтому использоваться всегда будет один объект Boolean.FALSE.
          Ответить

          • ava someone 18.09.2014 12:27 # +1

            Да, забыла. Но всё равно без необходимости не надо использовать обёртки вместо примитивных типов.
            Ответить

        • ava tirinox 18.09.2014 12:45 # +4

          Я бы остерегался языков, где есть два типа, различающихся регистром одной из букв. Спасибо, что не b00lean.
          Ответить

          • ava someone 18.09.2014 12:47 # +1

            Да, особый статус примитивов и необходимость классов-обёрток - это давняя болячка Java, тянущаяся ещё с 1.0. Наученные горьким опытом разработчики .NET всё сразу сделали правильно.
            Ответить

          • ava 3.14159265 18.09.2014 13:11 # +3

            >Я бы остерегался языков, где есть два типа,
            Расскажите, пожалуйста, сколько логических типов в языке Обжептив С?
            Ответить

          • ava 3.14159265 18.09.2014 13:17 # +5

            Ладно. Можете не отвечать.
            http://nshipster.com/bool/
            #12177
            BOOL / bool / Boolean / NSCFBoolean
            Так что в жаве еще всё нормально с типами.
            Ответить

            • ava 1024-- 18.09.2014 14:23 # 0

              >> где есть два типа, различающихся регистром одной из букв.
              > BOOL / bool / Boolean / NSCFBoolean
              Одну бУкву можно проПустить случайно, а четыре - с гораздо меньшей вероятностью. ПомехоустОйчивость.
              Ответить

            • ava inkanus-gray 18.09.2014 17:01 # +2

              А теперь подключим <windef.h>, и питушни будет гораздо больше!
              Ответить

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