Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
//это метод
public Integer getNoDeletedSubscribers() {
Integer count = 0;
for (int i = 0, n = subscribers.size(); i < n; count += (!subscribers.get(i++).isDeleted() ? 1 : 0));
return count;
}
//а вот пример использования этого метода в коде))
private void actionValidateInput() {
saveTaskBtn.setEnabled(!(((taskName.getText().equals("") ||
subscriberBox.getSelectedItem()==null ||
f51Box.getSelectedItem()==null || getTask().getNoDeletedSubscribers()==0 ) && getTask().getTaskType()==TaskType.SEND_MAIL) ||
((taskName.getText().equals("") && getTask().getTaskType()==TaskType.RECIVE_MAIL))));
}
Ну вообще, что считать говнокодом. Определение такавоха я не видал на сайте Ы
По мне дык и 1 метод гавнокод, бо раком написан и делает лишние действия.
И вообще метод еквивалентен следующему, что говорит что он скорее всего не пашет.
public Integer getNoDeletedSubscribers() {
if (subscribers.size() > 0) {
return (!subscribers.get(0).isDeleted() ? 1 : 0);
}
}
Исходя из чего следут что метод тупо проверяет 1ого сабскрайбера.
Вследствие чего, если сабскрайбер (колекшин скорее всего) имел всего 1 обжект внутри себя, то всё ок, и афтор был счастлив, что метод работал. 😀
Но судя по названию тоже куита, бо предпалагается subscriberS, т.е. многа, а в результате Integer 🙂
Guest ты написал какую то куйню public Integer getNoDeletedSubscribers() {
if (subscribers.size() > 0) {
return (!subscribers.get(0).isDeleted() ? 1 : 0);
}
}
вот это точно говнокод
в комментарии ко второму методу надо было написать
//а вот пример использования этого метода в говнокоде))
По мне дык и 1 метод гавнокод, бо раком написан и делает лишние действия.
И вообще метод еквивалентен следующему, что говорит что он скорее всего не пашет.
public Integer getNoDeletedSubscribers() {
if (subscribers.size() > 0) {
return (!subscribers.get(0).isDeleted() ? 1 : 0);
}
}
Исходя из чего следут что метод тупо проверяет 1ого сабскрайбера.
Вследствие чего, если сабскрайбер (колекшин скорее всего) имел всего 1 обжект внутри себя, то всё ок, и афтор был счастлив, что метод работал. 😀
Но судя по названию тоже куита, бо предпалагается subscriberS, т.е. многа, а в результате Integer 🙂
Гавнокод 100%!! 😀
public Integer getNoDeletedSubscribers() {
if (subscribers.size() > 0) {
return (!subscribers.get(0).isDeleted() ? 1 : 0);
}
}
вот это точно говнокод
я отписал позжей 🙂 что просмотрел ";" в конце фора, а та куйня это аналог, если бы после фора не стояла ";".
всё генитальнае проста! Ы
Метод работает и правильно, единственное что без аннотации выдается warning;
а ";" там как раз необходима.
иди синтаксис подучи.
@SuppressWarnings("empty-statement")
public Integer getNoDeletedSubscribers() {
Integer count = 0;
for (int i = 0, n = subscribers.size(); i < n; count += (!subscribers.get(i++).isDeleted() ? 1 : 0));
return count;
}