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

    +79.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if ((su != null) & (!su.getId().equals(id))) {
                flag = false;
 } else if ((su != null) & (su.getId().equals(id))) {
                flag = true;//true- not exist
 } else {
                flag = true;
 }

    не пишите код по ночам... а то такое выходит..))

    Запостил: maki, 24 Ноября 2009

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

    • ava guest 24.11.2009 01:18 # +1

      особенно доставил комент
      //true- not exist
      Ответить

    • ava guest 24.11.2009 08:22 # 0

      NullPointer на NullPointer'е
      Зачот.
      Ответить

    • ava guest 24.11.2009 12:58 # −1

      я бы написал так:
      flag = true;
if(su)
     flag = su.getId().equals(id);
      Ответить

      • ava TarasB 24.11.2009 14:50 # 0

        Длинно, уродливо смотрится.
        flag = !su || su.getId().equals(id);
        Ответить

        • ava guest 24.11.2009 17:36 # +2

          а где указано. что su - boolean?
          Ответить

          • ava TarasB 24.11.2009 17:53 # 0

            Виноват.
            flag = (su == null) || su.getId().equals(id);
            Ответить

            • ava guest 24.11.2009 20:19 # 0

              Честно говоря, за такой код, как руководитель проекта, вставил бы "пистон". Корректен, да. Но его трудночитаемость сводит на нет возможную элегантность решения. Не желаю никому разбираться в коде, пестрящим подобными однострочниками.
              Ответить

              • ava TarasB 24.11.2009 21:20 # 0

                Это трудночитаемо?! Дааа...
                Ответить

                • ava guest 24.11.2009 23:15 # 0

                  Алсо, если su==null то не факт что не будет проверенно второе условие
                  А если оно будет проверено, то фейл
                  Ответить

                  • ava TarasB 25.11.2009 00:11 # 0

                    А что, где-то ещё используется полная булева проверка?
                    Кстати, если su.getId().equals(id) делает какую-то операцию ещё, то в исходном говнокоде она делается дважды для ненулевого su, а у нас только по одному разу.
                    Ответить

                  • ava guest 25.11.2009 09:47 # 0

                    || - второе условие не будет проверено. факт. rtfm
                    Ответить

                    • ava guest 29.01.2010 14:43 # 0

                      +1
                      Ответить

                    • ava guest 04.02.2010 20:18 # 0

                      А вот и никуя :) по дефолту мб и не проверит, но не факт.
                      Если выставить опцию у жавы, чтобы делала полные проверки, то будет куй :) 100% :) Ы

                      Учите аспуку :D

                      Но конечно полную проверку бесмыслено ставить, и никто не ставит :) но я ваще пишу к тому, что это не "факт", и не надо тарапицо с вывадами :) Ы
                      Ответить

                      • ava guest 16.02.2010 13:38 # 0

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

            • ava guest 25.11.2009 09:50 # 0

              Неплохо, но su.getId() тоже может null вернуть. В apache lib для этих случаев есть ObjectUtils.equals
              Ответить

    • ava Billy the Kidd 26.11.2009 07:19 # +1

      Есть мнение, что самое большое говно в этом коде - наличие флага flag.
      Там тремя строками ниже будет что-то вроде:

      if (flag) {
      flag1 = true;
      } else {
      flag1 = false;
      }

      [КС]
      Ответить

      • ava guest 26.11.2009 14:35 # 0

        Пипец. А не легче сделать flag1 = flag;
        воистину кладезь говнокода
        Ответить

      • ava maki 26.11.2009 23:14 # 0

        ну просто гениально....)
        Ответить

    • ava guest 27.11.2009 05:15 # 0

      пишу код ночами, ничего такого не бывает

      мозги надо иметь
      Ответить

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