Говнокод #11

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;
 }

Комментарии (6)

ЧТОБЫ ПОСМОТРЕТЬ ВСЕ КОММЕНТАРИИ ИЛИ ОСТАВИТЬ КОММЕНТАРИЙ,
ПЕРЕЙДИТЕ В ПОЛНУЮ ВЕРСИЮ

SergeyKandalintsev Уровень 32

30 апреля 2014

flag = (su == null) || (su.getId().equals(id));

phashik Уровень 28

1 мая 2014

Добавлю, что изначальный говнокод не просто коряво написан, а вообще не будет работать: если su == null, то вылетит NullPointerException в первой строке.
Исправленный вариант лишен такого недостатка.

gnev Уровень 24

6 мая 2014

почему при su == null изначальный код не пойдет по ветке else?

SergeyKandalintsev Уровень 32

6 мая 2014

Какой результат будет при вычислении выражения в if если su равен null?

gnev Уровень 24

6 мая 2014

(su != null) не выполняется, переходим к else if, тот тоже не выполняется — к else.

SergeyKandalintsev Уровень 32

6 мая 2014

почитай про операторы "&&" и "&".