Помогите разобраться, почему валидатор не принимает запись в файл

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

Архивный

Помогите разобраться, почему валидатор не принимает по пункту: 2.3. В allFilesContent.txt последовательно записать содержимое всех файлов из п. 2.2.1. После каждого тела файла записать "\n".

1. На вход метода main() подаются два параметра.
Первый - path - путь к директории, второй - resultFileAbsolutePath - имя (полный путь) существующего файла, который будет содержать результат.
2. Переименовать resultFileAbsolutePath в allFilesContent.txt (используй метод FileUtils.renameFile(), и, если понадобится, FileUtils.isExist()).
3. Для каждого файла в директории path и в ее всех вложенных поддиректориях выполнить следующее:
Если у файла длина в байтах НЕ больше 50, то записать его содержимое в allFilesContent.txt.
После каждого тела файла записать "\n".
Все файлы имеют расширение txt.
В качестве разделителя пути используй "/".
Для создания файлов используй конструктор File(String pathname).

Требования:

Файл, который приходит вторым параметром в main, должен быть переименован в allFilesContent.txt.
Нужно создать поток для записи в переименованный файл.
Содержимое всех файлов, размер которых не превышает 50 байт, должно быть записано в файл allFilesContent.txt.
Поток для записи в файл нужно закрыть.
Не используй статические переменные.

package com.javarush.task.task31.task3101; import java.io.*; import java.util.*; /* Проход по дереву файлов */ public class Solution { public static void main(String[] args) throws IOException { String path = args[0]; String resultFileAbsolutePath = args[1]; File dir = new File(path); File result = new File(resultFileAbsolutePath); if (!FileUtils.isExist(dir)) { dir.mkdir(); } if (!FileUtils.isExist(result)) { result.createNewFile(); } File renamedFile = new File(result.getParentFile().getAbsolutePath() + "\\allFilesContent.txt"); if (FileUtils.isExist(renamedFile)) { renamedFile.delete(); } FileUtils.renameFile(result, renamedFile); List<File> filesList = getFilesListFromDir(dir); Collections.sort(filesList, Comparator.comparing(File::getName)); FileWriter writer = new FileWriter(renamedFile); filesList.stream() .filter(file -> file.length() <= 50) .forEach(file -> processFile(writer, file)); writer.flush(); writer.close(); } private static List<File> getFilesListFromDir(File dir) { List<File> filesList = new ArrayList<>(); for (File file : dir.listFiles()) { if (file.isFile()) { filesList.add(file); } else { filesList.addAll(getFilesListFromDir(file)); } } return filesList; } private static void processFile(FileWriter writer, File file) { StringBuilder builder = new StringBuilder(); try (FileReader reader = new FileReader(file)) { char x; while (reader.ready()) { x = (char) reader.read(); builder.append(x); } builder.append('\n'); writer.write(builder.toString()); } catch (IOException e) { e.printStackTrace(); } } }

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

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

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

Vadim

Уровень 35

22 апреля 2019, 05:49

У тебя в 45 строке fileList каждый раз заново создается при вызове метода. Тоже сначала на это напоролся, после дебага отловил. Надо его как то из метода вынести, только учти, что в классе Solution не должно быть статических методов по условию задачи))

Ola Backend Developer

22 апреля 2019, 08:31

1) а почему он не должен создаваться? 2) В условии сказано о статических переменных, ~~методах~~

Vadim

Уровень 35

22 апреля 2019, 09:25

1. Потому что предполагается, что добавляешь все файлы в список и в итоге метод его возвращает. А получается что добавляешь файлы в разные списки. Если потестишь, увидишь, что метод не добавляет в список файлы, находящиеся в поддиректориях.

Дмитрий

Уровень 41

12 марта 2019, 15:08

может с сортировкой проблемы? я решил через map и comparator и место "\\" в "\\allFilesContent.txt" использовал "/"

Ola Backend Developer

13 марта 2019, 07:01

Следующее не принимается валидатором, если я правильно понял твою идею.

psvm {
        Map<String, File> filesMap = getMapFilesFromDir(dir);
        FileWriter writer = new FileWriter(renamedFile);

        for (Map.Entry<String, File> pair : filesMap.entrySet()) {
            File file = pair.getValue();
            if (file.length() <= 50) {
                processFile(writer, file);
            }
        }
}


private static Map<String, File> getMapFilesFromDir(File dir) {
        Map<String, File> filesMap = new TreeMap<>();
        for (File file : dir.listFiles()) {
            if (file.isFile()) {
                filesMap.put(file.getName(), file);
            } else if (file.isDirectory()) {
                filesMap.putAll(getMapFilesFromDir(file));
            }
        }
        return filesMap;
    }

private static void processFile(FileWriter writer, File file) {
        StringBuilder builder = new StringBuilder();
        try (FileReader reader = new FileReader(file)) {
            char x;
            while (reader.ready()) {
                x = (char) reader.read();
                builder.append(x);
            }
            builder.append('\n');
            writer.write(builder.toString());
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

Дмитрий

Уровень 41

13 марта 2019, 07:44

я когда делал у меня была 1.TreeMap<String, File> в классе Solution со следующим компаратором

public int compare(String o1, String o2) {
            return o1.compareTo(o2);
        }

2. метод который рекурсивно заполнял эту мапу файлами из директории у которых длина в байтах НЕ больше 50 3. и в методе main через BufferedWriter и BufferedReader все записывал

BufferedWriter writer = new BufferedWriter(new FileWriter(newFile));

        for(Map.Entry<String, File> pair: solution.result.entrySet()){
            BufferedReader reader = new BufferedReader(new FileReader(pair.getValue()));

            while (reader.ready()) {
                writer.write(reader.readLine());
            }
            writer.write("\n");

            reader.close();
        }
        writer.close();

    }

Дмитрий

Уровень 41

13 марта 2019, 07:46

а ну и само собой

File file = new File(args[1]);//"C:/result.txt"
        File newFile = new File(file.getParent()+"/allFilesContent.txt");
        FileUtils.renameFile(file,newFile);

Ola Backend Developer

13 марта 2019, 09:23

к сожалению, все это не принимает валидатор у меня =(

Дмитрий

Уровень 41

13 марта 2019, 09:54

полезный

public class Solution {
    private  Map<String, File> result = new TreeMap<>(new Comparator<String>() {
        @Override
        public int compare(String o1, String o2) {
            return o1.compareTo(o2);
        }
    });

    public static void main(String[] args) throws IOException {
        Solution solution = new Solution();

        File my = new File(args[0]); //"D:/Учеба/Java/JavaRush"
        solution.allFile(my);

        File file = new File(args[1]);//"C:/result.txt"
        File newFile = new File(file.getParent()+"/allFilesContent.txt");
        FileUtils.renameFile(file,newFile);

        BufferedWriter writer = new BufferedWriter(new FileWriter(newFile));

        for(Map.Entry<String, File> pair: solution.result.entrySet()){
            BufferedReader reader = new BufferedReader(new FileReader(pair.getValue()));

            while (reader.ready()) {
                writer.write(reader.readLine());
            }
            writer.write("\n");

            reader.close();
        }
        writer.close();

    }

    private void allFile(File file) throws IOException {
        for (File f : file.listFiles()) {
            if (f.isFile()) {
                if (f.length() <= 50) {
                    FileInputStream inputStream = new FileInputStream(f); // не знаю зачем но без этого не работает
                    result.put(f.getName(), f);
                    inputStream.close();
                }
            } else allFile(f);
        }
    }
}

Ola Backend Developer

14 марта 2019, 05:50

Нехорошо поступил: сдал твое решение. Но оставлю вопрос открытым, может кто-нибудь даст ответ почему мое решение не принимается валидатором.