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