113

I've got code as below:

class ListPageXMLFiles implements FileFilter { @Override public boolean accept(File pathname) { DebugLog.i("ListPageXMLFiles", "pathname is " + pathname); String regex = ".*page_\\d{2}\\.xml"; if(pathname.getAbsolutePath().matches(regex)) { return true; } return false; } } public void loadPageTrees(String xml_dir_path) { ListPageXMLFiles filter_xml_files = new ListPageXMLFiles(); File XMLDirectory = new File(xml_dir_path); for(File _xml_file : XMLDirectory.listFiles(filter_xml_files)) { loadPageTree(_xml_file); } }

The FileFilter is working nicely, but listFiles() seems to be listing the files in reverse alphabetical order. Is there some quick way of telling listFile() to list the files in alphabetical order?

Improve this question
1
  • 1
    Instead of using regex, use .endsWith(".xml") instead. Commented Oct 7, 2015 at 21:44

4 Answers 4

Reset to default
255

The listFiles method, with or without a filter does not guarantee any order.

It does, however, return an array, which you can sort with Arrays.sort().

File[] files = XMLDirectory.listFiles(filter_xml_files); Arrays.sort(files); for(File _xml_file : files) { ... }

This works because File is a comparable class, which by default sorts pathnames lexicographically. If you want to sort them differently, you can define your own comparator.

If you prefer using Streams:

A more modern approach is the following. To print the names of all files in a given directory, in alphabetical order, do:

Files.list(Paths.get(dirName)).sorted().forEach(System.out::println)

Replace the System.out::println with whatever you want to do with the file names. If you want only filenames that end with "xml" just do:

Files.list(Paths.get(dirName)) .filter(s -> s.toString().endsWith(".xml")) .sorted() .forEach(System.out::println)

Again, replace the printing with whichever processing operation you would like.

Improve this answer
Sign up to request clarification or add additional context in comments.

4 Comments

This is one beautiful line of code, but Arrays.sort() seems to return void, instead of an Iterable. I'll poke around a bit.
@Thunder Rabbit, My Apologies!! So sorry, you are right. I should have tested it. I edited my answer.
It seems to be working fine in Windows, but in Ubuntu folders with capital letters appear first than the rest.
That's the way I would expect it to work. Capital letters come before lower case letters in Unicode. Unixes have case-sensitive filenames, so capital letters come first. If you are using case-insensitive filenames on Windows, the capitals and lowercase names will be mixed together. This is completely as expected. If you want the Windows way on a Unix, supply a comparator to sort.
5

In Java 8:

Arrays.sort(files, (a, b) -> a.getName().compareTo(b.getName()));

Reverse order:

Arrays.sort(files, (a, b) -> -a.getName().compareTo(b.getName()));
Improve this answer

3 Comments

You can use this code too: Arrays.sort(fList, Comparator.comparing(File::getName));
Be careful with this. This code is non-deterministic and should not be used as is.
Why not, and whose code?
2

I think the previous answer is the best way to do it here is another simple way. just to print the sorted results.

 String path="/tmp"; String[] dirListing = null; File dir = new File(path); dirListing = dir.list(); Arrays.sort(dirListing); System.out.println(Arrays.deepToString(dirListing));
Improve this answer

5 Comments

would you please provide comment as why you think this is not a correct answer. So, I can revise it accordingly.
-1 I really don't understand the advantage of converting the file array to a string array and then sorting rather than just sorting the file array as the accepted answer does.
@zelanix thanks for providing the feedback. I don't have all the answers and I make mistakes sometimes answering the questions. I have updated the answer, would you please review it and see if this is a better solution.I would appreciate it if you could reconsider your vote. Thanks in advance.
Ok, now you justify why it's simpler if you just want to print the results so I will remove my downvote.
@CPU100 i believe that by using the list() instead of listFiles() gives the advantage of having only the file names without the parent directory paths, resulting in shorter strings and lesser cpu time to sort/compare.
0

This is my code:

 try { String folderPath = "../" + filePath.trim() + "/"; logger.info("Path: " + folderPath); File folder = new File(folderPath); File[] listOfFiles = folder.listFiles(); int length = listOfFiles.length; logger.info("So luong files: " + length); ArrayList<CdrFileBO> lstFile = new ArrayList< CdrFileBO>(); if (listOfFiles != null && length > 0) { int count = 0; for (int i = 0; i < length; i++) { if (listOfFiles[i].isFile()) { lstFile.add(new CdrFileBO(listOfFiles[i])); } } Collections.sort(lstFile); for (CdrFileBO bo : lstFile) { //String newName = START_NAME + "_" + getSeq(SEQ_START) + "_" + DateSTR + ".s"; String newName = START_NAME + DateSTR + getSeq(SEQ_START) + ".DAT"; SEQ_START = SEQ_START + 1; bo.getFile().renameTo(new File(folderPath + newName)); logger.info("newName: " + newName); logger.info("Next file: " + getSeq(SEQ_START)); } } } catch (Exception ex) { logger.error(ex); ex.printStackTrace(); }

Improve this answer

Comments

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.