FTPCommand conversion to use enum
---------------------------------

                 Key: NET-310
                 URL: https://issues.apache.org/jira/browse/NET-310
             Project: Commons Net
          Issue Type: Improvement
            Reporter: Sebb
         Attachments: FTPCommandEnum.patch

Seems to me it would be sensible to replace the integer values in the 
FTPCommand class with an enum.

I propose creating a new enum class (called FTPCommands) which contains the 
enum and aliases. The existing FTPCommand class is then rewritten using the 
enum ordinals; and the FTP class is changed to use the enums internally. 
Existing public methods which use an integer for the command id are deprecated.

If there are any further commands added to the set lter, they should only be 
added to the enum, and no corresponding ints should be defined.

See attached patch (to follow). I've not fixed the FTP subclasses yet as I 
wanted to get agreement on the approach first.

Note that the new class name FTPCommands is very similar to the old class 
FTPCommand; of course that can be changed.

Comments? Suggestions?

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

Reply via email to