• ,

Говнокод #19

int[] indexes = list.getSelectedIndices();

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

5 комментариев

dev4j
да. Попахивает говнецом )
barban
тут не попахивает. тут уже вляпались и идем мыть руки
Cawich

int[] indexes;

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

как-то так наверное…
aiv
Не лучше, чем исходный вариант.
А если в getAllIndexes() будет запрос к удаленной базе? Зачем дважды делать запрос, сначала для того чтобы узнать, есть ли там что-то, а второй раз получить данные?
Archartem
На мой взгляд, здесь перепутана логика условий. «Ничего не выбрано! Ой, а там и выбирать-то не из чего!» Я бы так сделал:

int[] indexes;
if(getAllIndexes().length !=0){    //База не пустая
    if(list.getSelectedIndices().length !=0){    //Запрос не пустой
        indexes = list.getSelectedIndices();    //Отобрать по запросу
    }else indexes = getAllIndexes();    // Пустой запрос - берём всю базу
}else return false;    //База пустая - конец работе
Только зарегистрированные и авторизованные пользователи могут оставлять комментарии.