Option type is now set to String by default (Fixes CLI-215)

The type is now explicitly a Class instance instead of an unspecified Object

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1091485 13f79535-47bb-0310-9956-ffa450edef68

Author: ebourg

src/main/java/org/apache/commons/cli/CommandLine.java

@@ -113,16 +113,14 @@ public Object getOptionObject(String opt)
     public Object getParsedOptionValue(String opt) throws ParseException
     {
         String res = getOptionValue(opt);
-
         Option option = resolveOption(opt);
-        if (option == null)
+        
+        if (option == null || res == null)
         {
             return null;
         }
-
-        Object type = option.getType();
-
-        return (res == null) ? null : TypeHandler.createValue(res, type);
+        
+        return TypeHandler.createValue(res, option.getType());
     }
 
     /**

src/main/java/org/apache/commons/cli/Option.java

@@ -69,7 +69,7 @@ public class Option implements Cloneable, Serializable
     private int numberOfArgs = UNINITIALIZED;
 
     /** the type of this Option */
-    private Object type;
+    private Class type = String.class;
 
     /** the list of argument values **/
     private List values = new ArrayList();
@@ -193,7 +193,7 @@ public Object getType()
      *
      * @param type the type of this Option
      */
-    public void setType(Object type)
+    public void setType(Class type)
     {
         this.type = type;
     }

src/main/java/org/apache/commons/cli/OptionBuilder.java

@@ -48,7 +48,7 @@ public final class OptionBuilder
     private static int numberOfArgs = Option.UNINITIALIZED;
 
     /** option type */
-    private static Object type;
+    private static Class type;
 
     /** option can have an optional argument value */
     private static boolean optionalArg;
@@ -81,7 +81,7 @@ private static void reset()
         description = null;
         argName = null;
         longopt = null;
-        type = null;
+        type = String.class;
         required = false;
         numberOfArgs = Option.UNINITIALIZED;
         optionalArg = false;
@@ -287,7 +287,7 @@ public static OptionBuilder hasOptionalArgs(int numArgs)
      * @param newType the type of the Options argument value
      * @return the OptionBuilder instance
      */
-    public static OptionBuilder withType(Object newType)
+    public static OptionBuilder withType(Class newType)
     {
         OptionBuilder.type = newType;
 

src/main/java/org/apache/commons/cli/PatternOptionBuilder.java

@@ -91,7 +91,7 @@ public class PatternOptionBuilder
      * @param ch the specified character
      * @return The class that <code>ch</code> represents
      */
-    public static Object getValueClass(char ch)
+    public static Class getValueClass(char ch)
     {
         switch (ch)
         {
@@ -149,7 +149,7 @@ public static Options parsePattern(String pattern)
     {
         char opt = ' ';
         boolean required = false;
-        Object type = null;
+        Class type = null;
 
         Options options = new Options();
 

src/test/java/org/apache/commons/cli/CommandLineTest.java

@@ -61,4 +61,16 @@ public void testGetOptions()
 
         assertEquals(3, cmd.getOptions().length);
     }
+
+    public void testGetParsedOptionValue() throws Exception {
+        Options options = new Options();
+        options.addOption(OptionBuilder.hasArg().withType(Number.class).create("i"));
+        options.addOption(OptionBuilder.hasArg().create("f"));
+        
+        CommandLineParser parser = new DefaultParser();
+        CommandLine cmd = parser.parse(options, new String[] { "-i", "123", "-f", "foo" });
+        
+        assertEquals(123, ((Number) cmd.getParsedOptionValue("i")).intValue());
+        assertEquals("foo", cmd.getParsedOptionValue("f"));
+    }
 }

src/test/java/org/apache/commons/cli/OptionBuilderTest.java

@@ -26,14 +26,14 @@ public void testCompleteOption( ) {
                                      .hasArg( )
                                      .isRequired( )
                                      .hasArgs( )
-                                     .withType( new Float( 10 ) )
+                                     .withType( Float.class )
                                      .withDescription( "this is a simple option" )
                                      .create( 's' );
 
         assertEquals( "s", simple.getOpt() );
         assertEquals( "simple option", simple.getLongOpt() );
         assertEquals( "this is a simple option", simple.getDescription() );
-        assertEquals( simple.getType().getClass(), Float.class );
+        assertEquals( simple.getType(), Float.class );
         assertTrue( simple.hasArg() );
         assertTrue( simple.isRequired() );
         assertTrue( simple.hasArgs() );
@@ -44,14 +44,14 @@ public void testTwoCompleteOptions( ) {
                                      .hasArg( )
                                      .isRequired( )
                                      .hasArgs( )
-                                     .withType( new Float( 10 ) )
+                                     .withType( Float.class )
                                      .withDescription( "this is a simple option" )
                                      .create( 's' );
 
         assertEquals( "s", simple.getOpt() );
         assertEquals( "simple option", simple.getLongOpt() );
         assertEquals( "this is a simple option", simple.getDescription() );
-        assertEquals( simple.getType().getClass(), Float.class );
+        assertEquals( simple.getType(), Float.class );
         assertTrue( simple.hasArg() );
         assertTrue( simple.isRequired() );
         assertTrue( simple.hasArgs() );
@@ -64,7 +64,7 @@ public void testTwoCompleteOptions( ) {
         assertEquals( "d", simple.getOpt() );
         assertEquals( "dimple option", simple.getLongOpt() );
         assertEquals( "this is a dimple option", simple.getDescription() );
-        assertNull( simple.getType() );
+        assertEquals( String.class, simple.getType() );
         assertTrue( simple.hasArg() );
         assertTrue( !simple.isRequired() );
         assertTrue( !simple.hasArgs() );