Не могу понять в чем проблема

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

Обсуждается

Всем привет! Не могу понять в чем проблема. Вроде все делаю согласно условию. Заранее спасибо!

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.File; public class FileUtils { public static void deleteFile(File file) { if (!file.delete()) System.out.println("Can not delete file with name " + file.getName()); } public static void renameFile(File source, File destination) { if (!source.renameTo(destination)) System.out.println("Can not rename file with name " + source.getName()); } public static boolean isExist(File file) { return file.exists(); } }

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

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

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

Денис Enterprise Java Developer

11 апреля 2023, 11:17

До этого херню я написал ) Но всё равно есть странное поведение. Например зачем создавать объекты Path если ты ими не планируешь пользоваться, создай сразу File. Это и в условии написано кстати Для создания файлов используй конструктор File(String pathname). Зачем тебе ifExists на первом параметре? Это же папка в которой файлы лежат, и вовсе не должно оно быть связано с результирующим файлом. Если уж и применять метод то на втором параметре - на собственно файле результате. И уже потом переименовывать именно его. Насчёт потока для записи... тебе класс MyFileVisitor уже дали или ты сам написал? Я бы передал ему сразу поток записи, а не путь. Пусть бы им и пользовался, а создать поток в мейне, думаю что валидатор именно этого хочет. К тому же в текущей реализации ты просто перетираешь содержимое файла постоянно открывая поток записи в один и тот же файл. И не флашишься совсем, так что могут быть проблемы с полной записью контента в файл. Ну и от себя еще замечание: Вот такие штуки при работе с реальными файлами тебя очень быстро приведут к ООМ

byte[] content = Files.readAllBytes(file);

Вот попадётся тебе там какой ни будь "xxx.avi" на 10 гигабайт и ты его попытаешься всосать в память весь). Есть же у InuptStream прекрасный метод available() - почему бы не воспользоваться им? :) А в том же Files есть отличный метода Files.copy(InputStream is, OutputStream os) Вот и вышло бы у тебя

@Override
public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
    InputStream is = Files.newInputStream(file);
    if (is.available() <= 50) {
        Files.copy(is, os);
        os.write("\n".getBytes(StandardCharsets.UTF_8));
    }
    os.flush();
    return FileVisitResult.CONTINUE;
}

Но я бы потестил еще.