Wilf Middleton created IO-340:
---------------------------------

             Summary: The use of file.exists() is failure prone. Where 
file.exists() is checked, it should be replaced with the following dual check:  
exists = ( file.exists() || file.length()>0 )
                 Key: IO-340
                 URL: https://issues.apache.org/jira/browse/IO-340
             Project: Commons IO
          Issue Type: Bug
          Components: Utilities
    Affects Versions: 2.5, 3.x
         Environment: All
            Reporter: Wilf Middleton
             Fix For: 2.5, 3.x


The use of file.exists() is failure prone. In many cases the jvm can report 
that a file does not exist even when it does. This is due to the caching in the 
operating system.
Since file.exists() is prone to report false values, a second check can and 
should be made to file.length()>0.

Where file.exists() is checked, it should be replaced with the following dual 
check:  

A file more reliably exists if:  file.exists() || file.length()>0

Then if the file does not exist, file.length will be 0.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to