hubert
41 уровень

Говнокод #19

Пост из группы Архив info.javarush.ru
3710 участников
int[] indexes = list.getSelectedIndices(); if (indexes.length == 0) { indexes = getAllIndexes(); } if (indexes.length == 0) { return false; }
Комментарии (5)
  • популярные
  • новые
  • старые
Для того, что бы оставить комментарий вы должны авторизоваться
Cawich18 уровень, Самара
24 декабря 2015, 04:36
int[] indexes;

if (0 == getSelectedIndices().length) {
    if (0 == getAllIndexes().length)
        return false;
    indexes = getAllIndexes();
}

как-то так наверное…
aiv27 уровень, Vologda
12 апреля 2016, 22:20
Не лучше, чем исходный вариант.
А если в getAllIndexes() будет запрос к удаленной базе? Зачем дважды делать запрос, сначала для того чтобы узнать, есть ли там что-то, а второй раз получить данные?
Archartem33 уровень
29 сентября 2016, 13:28
На мой взгляд, здесь перепутана логика условий. «Ничего не выбрано! Ой, а там и выбирать-то не из чего!» Я бы так сделал:
int[] indexes;
if(getAllIndexes().length !=0){    //База не пустая
    if(list.getSelectedIndices().length !=0){    //Запрос не пустой
        indexes = list.getSelectedIndices();    //Отобрать по запросу
    }else indexes = getAllIndexes();    // Пустой запрос - берём всю базу
}else return false;    //База пустая - конец работе
barban15 уровень, Одесса
6 ноября 2014, 22:41
тут не попахивает. тут уже вляпались и идем мыть руки
dev4j7 уровень
23 мая 2014, 19:16
да. Попахивает говнецом )