hubert
41 уровень

Говнокод #11

Статья из группы Архив info.javarush
участников
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
почитай про операторы "&&" и "&".