Please what's wrong with this code

Question about the task

Iterating through a file tree

Java Collections, Level 1, Lesson 2

Archived

I don't know where is the mistake

1. The main method is passed two arguments as inputs.
The first argument is path, which is the path to the directory; the second is resultFileAbsolutePath, which is the name (full path) of an existing file that will store the result.
2. For each file in the path directory and in all of its subdirectories, do the following:
For each file whose size in bytes is NOT greater than 50, do the following:
2.1. Sort them by file name in ascending order. Don't include the path when sorting.
2.2. Rename resultFileAbsolutePath to "allFilesContent.txt" (use the FileUtils.renameFile method and FileUtils.isExist if necessary).
2.3. Sequentially write the contents of each file from step 2.1 to allFilesContent.txt. After the body of each file, write "\n".
All files have the TXT extension.
Use "/" as the path separator.

Requirements:

The file passed as the second argument to main must be renamed to allFilesContent.txt.
You need to create a stream to write to the renamed file.
The contents of every file that is 50 bytes or smaller must be written to the allFilesContent.txt file.
The output stream to the file must be closed.
Don't use static variables.

package com.codegym.task.task31.task3101; import java.io.File; import java.io.*; import java.io.IOException; import java.util.*; /* Iterating through a file tree */ public class Solution { public static void main(String[] args) throws IOException{ File path = new File(args[0]); File resultFileAbsolutePath = new File(args[1]); resultFileAbsolutePath.createNewFile(); File renamingFile = new File(resultFileAbsolutePath.getParent() + "\\allFilesContent.txt"); FileUtils.renameFile(resultFileAbsolutePath,renamingFile); List<File> listFiles = findInnerFiles(path); Collections.sort(listFiles, new Comparator<File>(){ @Override public int compare(File file1, File file2) { // TODO Auto-generated method stub return file1.getName().compareTo(file2.getName()); } }); writeFilesContent(renamingFile, listFiles); } public static List<File> findInnerFiles(File parentDir){ List<File> files = new ArrayList<File>(); File[] list = parentDir.listFiles(); for(File fileItem : list) { if(fileItem.isFile() && fileItem.length()<=50 ) files.add(fileItem); if(fileItem.isDirectory()) { files.addAll(findInnerFiles(fileItem)); } } return files; } public static void writeFilesContent(File container , List<File> files) throws IOException{ BufferedWriter writer = null; BufferedReader reader = null; try{ writer =new BufferedWriter( new FileWriter(container)); for(File file : files) { reader = new BufferedReader(new FileReader(file)); while(reader.ready()) { writer.write(reader.read()); writer.flush(); } writer.write((int) '\n'); } }finally{ if(writer != null) try{ writer.close(); }catch(Exception e){ } if(reader != null) try{ reader.close(); }catch(Exception e){ } } } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

15 May 2020, 10:28

Thanks a lot

Tony Roy

Level 41 , Victorville, United States

14 May 2020, 19:50

useful

I just finished this task (after 35!! attempts). I stopped getting this error when I removed your (14). Our write methods are different, but this may help?

Ivan

Level 22 , Nope, Bulgaria

30 April 2020, 14:42

At first glance, I think line 54 should be reader.readLine() and line 57 should be writer.write("\n")

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

30 April 2020, 23:34

It still doesn't work !