- 1
- 2
- 3
- 4
- 5
- 6
protected void setPhones(ArrayList<MBPhone> phones) {
if(phones!=null)
this.phones=phones;
else
phones.clear();
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 2
+74
protected void setPhones(ArrayList<MBPhone> phones) {
if(phones!=null)
this.phones=phones;
else
phones.clear();
}
Код из одного западного вэб-сервиса для профессионалов в сфере недвижимости.
Мораль: не надо игнорировать подсказки IDE. Автор на самом деле хотел очистить this.phones. И Intelij Idea подсказывает, что в этом месте может быть брошен NullPointerException. Увидеть эту проблему можно было только используя аннотацию @SuppressWarnings. Потому что варнингов так много, что различить среди них опасные очень сложно.
+71.1
if (getHook() != null ? !getHook().equals(pmCounty.getHook()) : pmCounty.getHook() != null) return false;
Код западной enterprise системы для риэлторов. Написан русскими.