- 1
isRegKeyRequired = distKey == null ? false : distKey.equals("yes") ? true : false;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−37.2
isRegKeyRequired = distKey == null ? false : distKey.equals("yes") ? true : false;
Уматный ХавнАкоД коллеги из Индии. Ну просто няйсь. Смотрится куда зло*буче чем желаемый вариант (isRegKeyRequired = "yes".equals(distKey));
guest 23.04.2009 15:28 # 0
Вообще
>> "yes".equals(distKey))
говнокод ещё тот. Ибо человек его читает как: а равно ли "yes" тому-то? Правильней писать:
>> distKey.equals("yes"))
вот почему null-ы сосут.
guest 19.07.2009 17:30 # 0
"yes".equals(distKey) - наиболее правильный вариант, т.к. точно не упаде NullPointerException.
Это стандартная практика, и я знаю, что во многих конторах е%ут, если ты сравниваешь контанту, ставя ее в аргумент (distKey.equals("yes")).
Поэтому сравнения типа "".equals(someVal) или "MaZaFaKa".equalsIgnoreCase(someVal) - нормалоьныя распространенная практика.
UNV 13.08.2009 09:13 # 0