• ,

Говнокод #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
flag = (su == null) || (su.getId().equals(id));
phashik
Добавлю, что изначальный говнокод не просто коряво написан, а вообще не будет работать: если su == null, то вылетит NullPointerException в первой строке.
Исправленный вариант лишен такого недостатка.
gnev
почему при su == null изначальный код не пойдет по ветке else?
SergeyKandalintsev
Какой результат будет при вычислении выражения в if если su равен null?
gnev
  • gnev
  • 0
  • Комментарий отредактирован 2014-05-06 11:35:48 пользователем gnev
(su != null) не выполняется, переходим к else if, тот тоже не выполняется — к else.
SergeyKandalintsev
почитай про операторы "&&" и "&".
Только зарегистрированные и авторизованные пользователи могут оставлять комментарии.