implemented the parser factory, added new unit tests for the GnuParser, a good deal of refactoring for both parsers

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

Author: John Keyes

src/java/org/apache/commons/cli/CommandLineParser.java

@@ -1,7 +1,7 @@
 /*
- * $Header: /home/jerenkrantz/tmp/commons/commons-convert/cvs/home/cvs/jakarta-commons//cli/src/java/org/apache/commons/cli/CommandLineParser.java,v 1.1 2002/06/16 23:00:57 jkeyes Exp $
- * $Revision: 1.1 $
- * $Date: 2002/06/16 23:00:57 $
+ * $Header: /home/jerenkrantz/tmp/commons/commons-convert/cvs/home/cvs/jakarta-commons//cli/src/java/org/apache/commons/cli/CommandLineParser.java,v 1.2 2002/07/04 22:32:12 jkeyes Exp $
+ * $Revision: 1.2 $
+ * $Date: 2002/07/04 22:32:12 $
  *
  * ====================================================================
  *
@@ -60,28 +60,34 @@
  */
 package org.apache.commons.cli;
 
-import java.util.List;
-
 /**
  * @author John Keyes (jbjk at mac.com)
  */
 public interface CommandLineParser {
 
     /**
      * Parse the arguments according to the specified options.
+     *
      * @param options the specified Options
      * @param arguments the command line arguments
      * @return the list of atomic option and value tokens
+     * @throws ParseException if there are any problems encountered
+     * while parsing the command line tokens.
      */
-    public List parse( Options options, List arguments );
-    
+    public CommandLine parse( Options options, String[] arguments )
+    throws ParseException;
+
     /**
      * Parse the arguments according to the specified options.
+     *
      * @param options the specified Options
      * @param arguments the command line arguments
      * @param stopAtNonOption specifies whether to continue parsing the
      * arguments if a non option is encountered.
      * @return the list of atomic option and value tokens
+     * @throws ParseException if there are any problems encountered
+     * while parsing the command line tokens.
      */
-    public List parse( Options options, List arguments, boolean stopAtNonOption );
+    public CommandLine parse( Options options, String[] arguments, boolean stopAtNonOption )
+    throws ParseException;
 }

src/java/org/apache/commons/cli/CommandLineParserFactory.java

@@ -0,0 +1,95 @@
+/*
+ * $Header: /home/jerenkrantz/tmp/commons/commons-convert/cvs/home/cvs/jakarta-commons//cli/src/java/org/apache/commons/cli/Attic/CommandLineParserFactory.java,v 1.1 2002/07/04 22:32:12 jkeyes Exp $
+ * $Revision: 1.1 $
+ * $Date: 2002/07/04 22:32:12 $
+ *
+ * ====================================================================
+ *
+ * The Apache Software License, Version 1.1
+ *
+ * Copyright (c) 1999-2001 The Apache Software Foundation.  All rights
+ * reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ *
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ *
+ * 3. The end-user documentation included with the redistribution, if
+ *    any, must include the following acknowlegement:
+ *       "This product includes software developed by the
+ *        Apache Software Foundation (http://www.apache.org/)."
+ *    Alternately, this acknowlegement may appear in the software itself,
+ *    if and wherever such third-party acknowlegements normally appear.
+ *
+ * 4. The names "The Jakarta Project", "Commons", and "Apache Software
+ *    Foundation" must not be used to endorse or promote products derived
+ *    from this software without prior written permission. For written
+ *    permission, please contact apache@apache.org.
+ *
+ * 5. Products derived from this software may not be called "Apache"
+ *    nor may "Apache" appear in their names without prior written
+ *    permission of the Apache Group.
+ *
+ * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESSED OR IMPLIED
+ * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED.  IN NO EVENT SHALL THE APACHE SOFTWARE FOUNDATION OR
+ * ITS CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF
+ * USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
+ * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ * ====================================================================
+ *
+ * This software consists of voluntary contributions made by many
+ * individuals on behalf of the Apache Software Foundation.  For more
+ * information on the Apache Software Foundation, please see
+ * <http://www.apache.org/>.
+ *
+ */
+package org.apache.commons.cli;
+
+/**
+ * Constructs CommandLineParser instances.  The implementation class
+ * is specified by the <code>org.apache.commons.cli.parser</code>
+ * system property.
+ *
+ * @author John Keyes (jbjk at mac.com)
+ */
+public class CommandLineParserFactory {
+
+    /** The PosixParser is the default parser implementation */
+    private static String DEFAULT_PARSER = "org.apache.commons.cli.PosixParser";
+
+    /**
+     * @return the CommandLineParser
+     */
+    public static CommandLineParser newParser() {
+        String parserImpl = System.getProperty( "org.apache.commons.cli.parser" );
+        try {
+            return (CommandLineParser)Class.forName( parserImpl ).newInstance();
+        }
+        catch( Exception exp ) {
+            // could not create according to parserImpl so default to
+            // PosixParser
+            try {
+                return (CommandLineParser)Class.forName( DEFAULT_PARSER ).newInstance();
+            }
+            catch( Exception exp2 ) {
+                // this will not happen ?
+            }
+        }
+        return null;
+    }
+}