hubert
41 уровень

Говнокод #11

Пост из группы Архив info.javarush.ru
3681 участников
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, 20:06
flag = (su == null) || (su.getId().equals(id));
phashik 28 уровень
1 мая 2014, 16:40
Добавлю, что изначальный говнокод не просто коряво написан, а вообще не будет работать: если su == null, то вылетит NullPointerException в первой строке.
Исправленный вариант лишен такого недостатка.
gnev 24 уровень
6 мая 2014, 10:53
почему при su == null изначальный код не пойдет по ветке else?
SergeyKandalintsev 32 уровень, Днепр
6 мая 2014, 11:17
Какой результат будет при вычислении выражения в if если su равен null?
gnev 24 уровень
6 мая 2014, 11:35
(su != null) не выполняется, переходим к else if, тот тоже не выполняется — к else.
SergeyKandalintsev 32 уровень, Днепр
6 мая 2014, 12:40
почитай про операторы "&&" и "&".