- 1
- 2
if (name != null ? !name.equals(module.name) : module.name != null) return false;
return true;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+90
if (name != null ? !name.equals(module.name) : module.name != null) return false;
return true;
artureg 20.01.2011 14:37 # 0
gegMOPO4 20.01.2011 14:50 # +1
artureg 20.01.2011 15:00 # +1
gegMOPO4 20.01.2011 15:22 # 0
artureg 20.01.2011 15:03 # 0
gegMOPO4 20.01.2011 15:26 # 0
Как ни крути, а нужны либо тернарный оператор, либо вспомогательная переменная, либо четыре сравнения.
Анонимус 20.01.2011 21:00 # +4
никогда так не делайте
на самом деле сама ситуация говно
gegMOPO4 21.01.2011 00:15 # −2
Анонимус 21.01.2011 00:18 # 0
gegMOPO4 21.01.2011 12:08 # +1
Рассмотрите случаи:
1) module.name == null
2) module == null
absolut 20.01.2011 15:09 # +1
Jk 20.01.2011 15:57 # +3
return (name != null) ? name.equals(useCase.name) : useCase.name == null;
tr00_gr1m_doomster 20.01.2011 19:39 # +2
gegMOPO4 21.01.2011 00:16 # −1
tir 21.01.2011 20:33 # +1
return name == useCase.name || name != null && name.equals(useCase.name)
tr00_gr1m_doomster 21.01.2011 21:37 # 0
gegMOPO4 21.01.2011 22:10 # 0
Кстати, в некоторых случаях это может оказаться даже эффективнее. Только мозги некоторым свернёт.
bugmenot15 22.01.2011 04:02 # −6
bugmenot15 22.01.2011 04:11 # −6
tr00_gr1m_doomster 22.01.2011 15:56 # 0
gegMOPO4 22.01.2011 17:05 # 0
tr00_gr1m_doomster 22.01.2011 19:58 # 0
gegMOPO4 22.01.2011 20:34 # 0
tir 23.01.2011 21:23 # 0
tr00_gr1m_doomster 23.01.2011 22:06 # −1
tir 23.01.2011 23:36 # +1
tr00_gr1m_doomster 24.01.2011 04:52 # −1
tr00_gr1m_doomster 24.01.2011 05:08 # 0
gegMOPO4 21.01.2011 22:05 # 0
TarasB 25.01.2011 14:45 # 0
А то получается обёртка над обёрткой, которая вызывает обёртку, сравнивающую именно непустые строки.
guest 25.01.2011 17:10 # 0
Что Вам не нравится? Если это всё превратится в один вызов, то какие проблемы? Главное код, что-б был красивый.
expl 21.01.2011 00:21 # 0
if (name == null) {
return useCase.name == null;
}
return name.equals(useCase.name);
Но, похоже, доходчивее этот код уже не сделаешь
Анонимус 21.01.2011 00:39 # +1
tr00_gr1m_doomster 21.01.2011 15:57 # 0
Vasiliy 21.01.2011 16:50 # −2
Анонимус 21.01.2011 21:03 # +2