Checking if a directory exists in FTP without relying on exception handling

Question

I've written a method to check if a directory already exists on an FTP server. The only way I could see to make this work was to return a different result from the method if an exception is thrown - very bad practice in my opinion. Is there a better way to write this?

public bool DirectoryExists(string directory)
{
    if (String.IsNullOrEmpty(directory))
        throw new ArgumentException("No directory was specified to check for");

    // Ensure directory is ended with / to avoid false positives
    if (!directory.EndsWith("/"))
        directory += "/";

    try
    {
        var request = (FtpWebRequest)WebRequest.Create(directory);
        request.Method = WebRequestMethods.Ftp.ListDirectory;
        request.Credentials = new NetworkCredential(Username, Password);

        using (request.GetResponse())
        {
            return true;
        }
    }
    catch (WebException)
    {
        return false;
    }
}

The connection hostname, username and password referred to within this block of code are auto properties set when the class is initialized.

Heslacher · Accepted Answer · 2014-12-23 06:14:38Z

1

don't omit braces {} for single if statements. Using braces in this case will make your code less errorprone.

you want to return a state, which represents a specific states which shouldn't be restricted to only two states. This just calls for an enum.

public enum FtpResponse
{
    DirectoryExists, DirectoryNotFound, DirectoryNotSpecified
}


public FtpResponse DirectoryExists(string directory)  
{
    if (String.IsNullOrEmpty(directory))
    {
         return FtpResponse.DirectoryNotSpecified;
    } 
    // the rest of the code

}

edited Dec 23, 2014 at 6:14

answered Dec 23, 2014 at 5:03

Heslacher

51k5 gold badges83 silver badges178 bronze badges

6

\$\begingroup\$ I think it's better to throw an argument exception if the argument is invalid rather that returning another enum value. The single line if suggestion isn't helpful either, it's just one of the useless programmer holy wars. \$\endgroup\$

CodesInChaos
– CodesInChaos

2015-03-03 13:44:35 +00:00
Commented Mar 3, 2015 at 13:44

Add a comment |

dfhwze · Accepted Answer · 2019-08-29 16:35:31Z

(Missing) Specification

You are lacking a clear specification (comments). The way you implemented it, you consider not exists and don't know whether it exists as one and the same. I would expect a method called DirectoryExists to:

return true when the directory exists
return false when it does not exist
throw an exception when unabe to determine existence

Let consumers handle exceptions:

catch (WebException)
{
    //return false;
    throw;                  // if we don't log here, remove the catch altogether
}

It may very well be possible you cannot find a way to determine when to return false. In such case, the method either returns true or throws an exception. I would allow such behavior to be forward-compatible in case we ever do find a way to return false without impacting legacy consumer code.

If the purpose of this method is not to swallow exceptions (as your question title suggests), and only to return true or false, you should definately put in the specification that the method is a sand-box.

Valid paths

Isn't it possible that an empty path means the current directory?

if (String.IsNullOrEmpty(directory))
    throw new ArgumentException("No directory was specified to check for");

Not sure whether edge cases could escape your code below. But I would favour dispatching as much as we can to built-in functionality.

// Ensure directory is ended with / to avoid false positives
if (!directory.EndsWith("/"))
    directory += "/";

GetFullPath ensures at most one folder separator are appended. So I've turned the condition around. Always add the separator, and let GetFullPath clean it up.

directory = Path.GetFullPath(directory + "/");

Stack Exchange Network

Checking if a directory exists in FTP without relying on exception handling

2 Answers 2

(Missing) Specification

Valid paths

You must log in to answer this question.

Hot Network Questions

Checking if a directory exists in FTP without relying on exception handling

2 Answers 2

(Missing) Specification

Valid paths

You must log in to answer this question.

Related

Hot Network Questions