[jira] Commented: (IO-77) Add convenience moveFille() / moveDirectory methods to FileUtils

["Gary Gregory (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/IO-77?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12557056#action_12557056 ] 

Gary Gregory commented on IO-77:
--------------------------------

Note that in [lang] 'solution' for this type of issue has been around for quite a while: the class NullArgumentException. 

IMO, a method should fail on it's own with an NPE, an NPE should never be thrown as it is redundant. The only caveat is that throwing any exception lets the failure site provide a better error message. Hence:

throw new NullArgumentException("paramName"); 

or:

throw new IllegalArgumentException("The argument paramName cannot be null");

> Add convenience moveFille() / moveDirectory methods to FileUtils
> ----------------------------------------------------------------
>
>                 Key: IO-77
>                 URL: https://issues.apache.org/jira/browse/IO-77
>             Project: Commons IO
>          Issue Type: Improvement
>          Components: Utilities
>    Affects Versions: 1.3.2
>         Environment: Operating System: other
> Platform: Other
>            Reporter: nicolas de loof
>            Assignee: Niall Pemberton
>            Priority: Minor
>             Fix For: 1.4
>
>         Attachments: IO-77.patch, patch_io.txt
>
>
> I'm using FileUtils as it partially solves the missing "move" method for File,
> that is so simple to do in unix shell.
> A full implementation in FileUtils may be great :
> static boolean FileUtils.move(File src, File dest)
> throws IOException
> {
>     boolean rename = src.renameTo(dest);
>     if (!rename)
>     {
>         copyFile(file, dest);
>         file.delete();
>     }
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.