Exception handling in Files.walk and Files.find
    Peter Levart 
    peter.levart at gmail.com
       
    Thu Jul  2 13:56:03 UTC 2015
    
    
  
Hi,
Recently I wanted to use Files.walk method returning a Stream<Path> to 
scan a directory of files. I quickly learned that it is of limited use. 
For example, the following code:
         long pngFilesCount = Files.walk(Paths.get("/usr"))
                 .filter(path -> path.toString().endsWith(".png"))
                 .count();
Throws exception on my computer:
Exception in thread "main" java.io.UncheckedIOException: 
java.nio.file.AccessDeniedException: /usr/share/polkit-1/rules.d
This is a consequence of the fact that some files and/or directories in 
the tree being scanned are not accessible to my user account on the 
computer. If I wanted to skip those files/directories I would be forced 
to use different API (Files.walkFileTree using FileVisitor which can 
handle exceptions too).
I propose the addition of overloads to Files.walk and Files.find that 
take an additional parameter of type Consumer<IOException> to be 
responsible for handling IOExceptions that are thrown while walking the 
file tree:
http://cr.openjdk.java.net/~plevart/jdk9-dev/Files.walkIOException/webrev.01/
Ignoring AccessDeniedException(s) in above code can therefore be coded as:
         long pngFilesCount = Files.walk(Paths.get("/usr"), 
Integer.MAX_VALUE,
                 ioe -> {
                     if (!(ioe instanceof AccessDeniedException)) {
                         throw new UncheckedIOException(ioe);
                     }
                 })
                 .filter(path -> path.toString().endsWith(".png"))
                 .count();
What do you think?
Regards, Peter
    
    
More information about the core-libs-dev
mailing list