Здравствуйте. Нужен ответ людей разобравшихся в этой теме. Внимание - спойлеры!

Вопрос по задаче

Решен

Задачу то я решил, вот ощущение того, что решил костылями не покидает меня. Был создан конструктор без параметров, в котором инициировались переменные для последующей удобной работы.

if (attrs.size() < maxSize && attrs.size() > minSize && file.getFileName().toString().contains(partOfName) && new String(content).contains(partOfContent)) {
        foundFiles.add(file);
}

Что скажете, господа? Костыльный способ решения?

Комментарии (4)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Сергеев Виктор

Уровень 40

Master

9 мая 2019, 09:10

решение

нормальный, наверно, но я бы вместо && использовал несколько if

if(!1){
    return;
}
if(!2){
    return;
}
if(!3){
    return;
}
if(!4){
    return;
}

так как-то привычней. Либо выкинул бы в отдельный метод, что-то типо checkFile, который возвращал boolean подходит файл под параметры или нет.

Андрей Старжинский

Уровень 25

9 мая 2019, 09:21

решение

Да, в принципе это приятнее для глаз было бы. А вот возвращать то что? Я просто не совсем разобрался с этой темой.

Сергеев Виктор

Уровень 40

Master

9 мая 2019, 11:12

return подразумевает выход из условия, чтобы не идти дальше. Т.е. там теже действия, что у вас за пределами вашего if в случае с отдельным методом возвращать boolean

Андрей Старжинский

Уровень 25

9 мая 2019, 11:54

Точно, вечно забываю, что пустой return можно юзать как выход из метода. Посмотрел решения остальных ребят, то в принципе у всех подобное решение. Спасибо!