JavaRush/Помощь по Java задачам/Что ему не нравится?!

34 уровень

19.11.2019
908просмотров
8комментариев

Что ему не нравится?!

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

Решен

Вместо того, что бы городить сложные конструкции - пошел по пути наименьшего сопротивления (ИМХО, самый верный путь!), а именно задал значения по умолчанию для всех полей и добавляю файлы в список, ТОЛЬКО если ВСЕ условия true; На моей тачке все работает исправно! Ни один пункт не проходит, кроме первых двух... Ну мало ли, может им не нравится что я сразу инициализирую поля? Окей. Делаю в конструкторе по умолчанию инициализацию и тоже шиш! Хорошо, пусть так. Тогда может быть заполнить поля непосредственно в методе visitFile? Неа, говорит мне валик. Я честно говоря бросил JR потому что устал с ним бороться, осталось попробовать рефлексию, но вряд ли поможет... Уже работаю, но не понимаю зачем такое порно? Что надо изменить что бы пройти валик? Нагородить нечитаемую, неподдерживаемую конструкцию из if/else? Вопрос в первую очередь к авторам курса, может я чего-то не понимаю, тогда поясните пожалуйста, чем огород плохих практик лучше, чем предложенное мной решение?

Давай реализуем настраиваемый поиск файлов в директории.
Просмотри интерфейс java.nio.file.FileVisitor и его базовую реализацию java.nio.file.SimpleFileVisitor.
Во время поиска по дереву файлов с помощью метода Files.walkFileTree(Path start, FileVisitor<? super Path> visitor)
мы используем объект FileVisitor для выполнения необходимых операций над найденными объектами Path.

Наш класс для поиска будет называться SearchFileVisitor и расширять SimpleFileVisitor.

Поиск можно будет выполнять по таким критериям:
- выражение, встречающееся в названии файла (String);
- выражение, встречающееся в содержимом файла (String);
- максимальный и минимальный размер файла (int).
Можно задавать либо один, либо сразу несколько критериев для поиска.

Я в main написал код, который использует готовый SearchFileVisitor для поиска файлов, тебе осталась совсем легкая задача - выполнить его реализацию.
Подсказка 1: методы get... , set... - это геттеры и сеттеры полей. Основная логика класса по поиску выполняется в методе visitFile(Path file, BasicFileAttributes attrs).
Подсказка 2: для работы с файлами используй только классы из пакета java.nio.

Требования:

В классе SearchFileVisitor нужно создать поля partOfName, partOfContent, minSize, maxSize и сеттеры для них.
В классе SearchFileVisitor нужно создать поле foundFiles и геттер для него. Поле должно быть сразу инициализировано.
Если в SearchFileVisitor задан критерий поиска partOfName, метод visitFile должен добавить файл в foundFiles, если в названии содержится строка partOfName.
Если в SearchFileVisitor задан критерий поиска partOfContent, метод visitFile должен добавить файл в foundFiles, если в содержимом встречается строка partOfContent.
Если в SearchFileVisitor задан критерий поиска maxSize, метод visitFile должен добавить файл в foundFiles, если размер файла меньше maxSize.
Если в SearchFileVisitor задан критерий поиска minSize, метод visitFile должен добавить файл в foundFiles, если размер файла больше minSize.
Метод visitFile должен быть реализован так, чтобы учитывать сразу несколько критериев поиска.

package com.javarush.task.task31.task3111; import java.io.IOException; import java.nio.file.Files; import java.nio.file.Path; import java.nio.file.Paths; import java.util.List; /* Продвинутый поиск файлов */ public class Solution { public static void main(String[] args) throws IOException { SearchFileVisitor searchFileVisitor = new SearchFileVisitor(); searchFileVisitor.setPartOfName("amigo"); searchFileVisitor.setPartOfContent("programmer"); searchFileVisitor.setMinSize(500); searchFileVisitor.setMaxSize(10000); Files.walkFileTree(Paths.get("D:/SecretFolder"), searchFileVisitor); List<Path> foundFiles = searchFileVisitor.getFoundFiles(); for (Path file : foundFiles) { System.out.println(file); } } }

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

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

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

Жоский Тип

Уровень 34

19 ноября 2019, 14:15

Может кому пригодится: вметсо

file.toFile().getName().contains(partOfName)

надо

file.getFileName().toString().contains(partOfName)

Алексей Java Developer

Expert

19 ноября 2019, 13:30

Насчет того, что выше, заметил две ошибки: if(maxSize==0) maxSize=Integer.MAX_VALUE; // А если мы ищем файлы нулевого размера? content.length>minSize&&content.length<maxSize) // а там должно быть меньшеравно большеравно

Жоский Тип

Уровень 34

19 ноября 2019, 14:03

if(minSize!=0) {
    if(content.length>minSize) {
        flag = true;
    } else {
        return super.visitFile(file, attrs);
    }
}
if(maxSize!=0) {
    if(content.length<maxSize){
        flag=true;
    } else {
        return super.visitFile(file, attrs);
    }
}

проходит валидацию по размеру

Жоский Тип

Уровень 34

19 ноября 2019, 13:26

Окей. Я-таки нашел ошибочку)) каюсь. Нагородил херни уже лишь бы пройти валик. Признаю что мое решение выше не учитывает ту часть условия, где сказано "либо один, либо несколько". Тут что не так?

@Override
    public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
        boolean flag=false;

        byte[] content = Files.readAllBytes(file); // размер файла: content.length
        if(partOfName!=null) {
            if(file.toFile().getName().toUpperCase().contains(partOfName.toUpperCase())) {
                flag = true;
            } else {
                return super.visitFile(file, attrs);
            }
        }
        if(partOfContent!=null) {
            if(new String(content).contains(partOfContent)) {
                flag = true;
            } else {
                return super.visitFile(file, attrs);
            }
        }
        if(minSize!=0) {
            if(content.length>minSize) {
                flag = true;
            } else {
                return super.visitFile(file, attrs);
            }
        }
        if(maxSize!=0) {
            if(content.length<maxSize){
                flag=true;
            } else {
                return super.visitFile(file, attrs);
            }
        }
        if (flag) {
            foundFiles.add(file);
        }
        return super.visitFile(file, attrs);
    }

IceBerg

Уровень 41

Master

19 ноября 2019, 13:45

Думаю файл должен попадать в список, если критерии вообще не указаны. И почему toUpperCase?

Жоский Тип

Уровень 34

19 ноября 2019, 14:07

убрал флаг (т.е. нет проверки на наличие хотя бы 1 условия) - ничего не поменялось, не принимает проверку вхождения в имя...

@Override
public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
    byte[] content = Files.readAllBytes(file); // размер файла: content.length
    if(partOfName!=null) {
        if(!file.toFile().getName().contains(partOfName)) {
            return super.visitFile(file, attrs);
        }
    }
    if(partOfContent!=null) {
        if(!new String(content).contains(partOfContent)) {
            return super.visitFile(file, attrs);
        }
    }
    if(minSize!=0) {
        if(content.length<minSize) {
            return super.visitFile(file, attrs);
        }
    }
    if(maxSize!=0) {
        if(content.length>maxSize){
            return super.visitFile(file, attrs);
        }
    }
        foundFiles.add(file);
    return super.visitFile(file, attrs);
}

toUpperCase - думал может надо игнорить регистр, ни в какую...

Василий

Уровень 27

24 ноября 2019, 14:48

Мы можем преобразовать байтовый массив в строку многими способами. Преобразуем байтовый массив в String , используя кодировку платформы по умолчанию. Созданием нового объекта. ... byte[] content = Files.readAllBytes(file) String contentFile = new String (content) Так у меня нахождение в имени получилось - contentFile.contains(partOfContent)

Василий

Уровень 27

24 ноября 2019, 15:26

7. Метод visitFile должен быть реализован так, чтобы учитывать сразу несколько критериев поиска. - Эта формулировка, в переводе на нормальный русский язык означает, что если один или несколько критериев поиска не заданы, то они не участвуют в фильтрации никак. А если указаны, то ВСЕ СРАЗУ УСЛОВИЯ должны удовлетворять - иначе не соответствуют условию. У тебя в верхнем примере - не проверяются отсутствующие критерии, в нижнем вообще не проверяются критерии поиска при добавлении файла в список...