- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
//это метод
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))));
}
Follow us!
в комментарии ко второму методу надо было написать
//а вот пример использования этого метода в говнокоде))
По мне дык и 1 метод гавнокод, бо раком написан и делает лишние действия.
И вообще метод еквивалентен следующему, что говорит что он скорее всего не пашет.
public Integer getNoDeletedSubscribers() {
if (subscribers.size() > 0) {
return (!subscribers.get(0).isDeleted() ? 1 : 0);
}
}
Исходя из чего следут что метод тупо проверяет 1ого сабскрайбера.
Вследствие чего, если сабскрайбер (колекшин скорее всего) имел всего 1 обжект внутри себя, то всё ок, и афтор был счастлив, что метод работал. :D
Но судя по названию тоже куита, бо предпалагается subscriberS, т.е. многа, а в результате Integer :)
Гавнокод 100%!! :D
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;
}