[Title] Code enhancement as static analysis

[Type]      Enhancement
[Module]    Sub
[Priority]  Minor
[CQ#]       // CQ Issue Number
[Redmine#]  // Redmine Isuue Number
[Problem]   // Problem Description
[Cause]     // Cause Description
[Solution]  // Solution Description
[TestCase]  // Executed the test-target (How to)

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/ILaunchOptions.java b/org.tizen.cli/src/org/tizen/cli/exec/ILaunchOptions.java
new file mode 100755 (executable)
index 0000000..db295ef
--- /dev/null
+++ b/org.tizen.cli/src/org/tizen/cli/exec/ILaunchOptions.java
@@ -0,0 +1,142 @@
+package org.tizen.cli.exec;\r
+\r
+public interface ILaunchOptions\r
+{\r
+    \r
+    /**\r
+     * <p>\r
+     * Description for target widget id\r
+     * \r
+     * This is printed out in usage\r
+     * </p>\r
+     * \r
+     * @see #OPT_WIDGETID\r
+     */\r
+    String DESC_WIDGETID = "Command is performed of running widget kill in target, widget id is user written id in config.xml";\r
+\r
+    /**\r
+     * <p>\r
+     * Option for target widget file\r
+     * </p>\r
+     */\r
+    String OPT_WIDGETFILE = "widget";\r
+\r
+    /**\r
+     * <p>\r
+     * Description for target widget file\r
+     * \r
+     * This is printed out in usage\r
+     * </p>\r
+     * \r
+     * @see #OPT_WIDGETFILE\r
+     */\r
+    String DESC_WIDGETFILE = "Command is performed of widget file install in target";\r
+\r
+    /**\r
+     * <p>\r
+     * Option for target widget id\r
+     * </p>\r
+     */\r
+    String OPT_WIDGETID = "id";\r
+\r
+    /**\r
+     * <p>\r
+     * Option for project name\r
+     * </p>\r
+     */\r
+    String OPT_NAME = "name";\r
+\r
+    /**\r
+     * <p>\r
+     * Description for project name\r
+     * \r
+     * This is printed out in usage\r
+     * </p>\r
+     * \r
+     * @see #OPT_WIDGETFILE\r
+     */\r
+    String DESC_NAME = "generation project name";\r
+\r
+    /**\r
+     * <p>\r
+     * Option for project name\r
+     * </p>\r
+     */\r
+    String OPT_PATH = "path";\r
+\r
+    /**\r
+     * <p>\r
+     * Description for project name\r
+     * \r
+     * This is printed out in usage\r
+     * </p>\r
+     * \r
+     * @see #OPT_WIDGETFILE\r
+     */\r
+    String DESC_PATH = "generation project path";\r
+\r
+    \r
+    /**\r
+     * <p>\r
+     * Option for include file pattern\r
+     * </p>\r
+     */\r
+    String OPT_INCLUDE = "include";\r
+    \r
+    /**\r
+     * <p>\r
+     * Description for include file pattern option\r
+     * \r
+     * It is printed out in usage\r
+     * </p>\r
+     */\r
+    String DESC_INCLUDE = "file name pattern to include";\r
+    \r
+    /**\r
+     * <p>\r
+     * Option for exclude file pattern\r
+     * </p>\r
+     */\r
+    String OPT_EXCLUDE = "exclude";\r
+    \r
+    /**\r
+     * <p>\r
+     * Description for exclude file pattern option\r
+     * \r
+     * It is printed out in usage\r
+     * </p>\r
+     */\r
+    String DESC_EXCLUDE = "file name pattern to exclude";\r
+    \r
+    /**\r
+     * <p>\r
+     * Option for no check of validation\r
+     * </p>\r
+     */\r
+    String OPT_NOCHECK = "nocheck";\r
+    \r
+    /**\r
+     * <p>\r
+     * Description for no check option\r
+     * \r
+     * It is printed out in usage\r
+     * </p>\r
+     */\r
+    String DESC_NOCHECK = "skip validation check";\r
+    \r
+    /**\r
+     * <p>\r
+     * Option for over-writing of output file\r
+     * </p>\r
+     */\r
+    String OPT_OVERWRITE = "overwrite";\r
+    \r
+    /**\r
+     * <p>\r
+     * Description for overwrite option\r
+     * \r
+     * It is printed out in usage\r
+     * </p>\r
+     */\r
+    String DESC_OVERWRITE = "overwrite output file if it exists";\r
+}\r

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/debug/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/debug/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index bd10fb0..7436287
--- a/org.tizen.cli/src/org/tizen/cli/exec/debug/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/debug/Main.java
@@ -28,6 +28,7 @@ package org.tizen.cli.exec.debug;
 import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.cli.exec.WRTLauncher;
 import org.tizen.common.core.command.Executor;
 import org.tizen.common.core.command.Prompter;
@@ -51,31 +52,8 @@ import org.tizen.web.launch.cli.command.ModeCommnad;
 public class
 Main
 extends WRTLauncher
+implements ILaunchOptions
 {
-    /**
-     * <p>
-     * Option for target widget file
-     * </p>
-     */
-    protected static final String OPT_WIDGETFILE = "widget";
-
-    /**
-     * <p>
-     * Description for target widget file
-     * 
-     * This is printed out in usage
-     * </p>
-     * 
-     * @see #OPT_WIDGETFILE
-     */
-    protected static final String DESC_WIDGETFILE = "Command is performed of widget file install in target";
-
-    /**
-     * <p>
-     * Option for target widget id
-     * </p>
-     */
-    protected static final String OPT_WIDGETID = "id";
 
     /**
      * <p>

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/gen/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/gen/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index f4e65ed..977cf0f
--- a/org.tizen.cli/src/org/tizen/cli/exec/gen/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/gen/Main.java
@@ -33,6 +33,7 @@ import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
 import org.tizen.cli.exec.AbstractLauncher;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.common.core.command.Prompter;
 import org.tizen.common.core.command.prompter.ChoiceOption;
 import org.tizen.common.core.command.prompter.Option;
@@ -49,43 +50,10 @@ import org.tizen.web.project.wizard.cli.command.ProjectGenCommand;
  */
 public class
 Main
-extends AbstractLauncher {
+extends AbstractLauncher
+implements ILaunchOptions
+{
 
-       /**
-        * <p>
-        * Option for project name
-        * </p>
-        */
-       protected static final String OPT_NAME = "name";
-
-       /**
-        * <p>
-        * Description for project name
-        * 
-        * This is printed out in usage
-        * </p>
-        * 
-        * @see #OPT_WIDGETFILE
-        */
-       protected static final String DESC_NAME = "generation project name";
-
-       /**
-        * <p>
-        * Option for project name
-        * </p>
-        */
-       protected static final String OPT_PATH = "path";
-
-       /**
-        * <p>
-        * Description for project name
-        * 
-        * This is printed out in usage
-        * </p>
-        * 
-        * @see #OPT_WIDGETFILE
-        */
-       protected static final String DESC_PATH = "generation project path";
 
        /**
         * Entry point for cli main

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/install/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/install/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index f63f1b8..8df3719
--- a/org.tizen.cli/src/org/tizen/cli/exec/install/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/install/Main.java
@@ -29,6 +29,7 @@ import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
 
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.cli.exec.WRTLauncher;
 import org.tizen.common.core.command.Executor;
 import org.tizen.common.core.command.Prompter;
@@ -51,31 +52,8 @@ import org.tizen.web.launch.cli.command.ModeCommnad;
 public class
 Main
 extends WRTLauncher
+implements ILaunchOptions
 {
-    /**
-     * <p>
-     * Option for target widget file
-     * </p>
-     */
-    protected static final String OPT_WIDGETFILE = "widget";
-
-    /**
-     * <p>
-     * Description for target widget file
-     * 
-     * This is printed out in usage
-     * </p>
-     * 
-     * @see #OPT_WIDGETFILE
-     */
-    protected static final String DESC_WIDGETFILE = "Command is performed of widget file install in target";
-
-    /**
-     * <p>
-     * Option for target widget id
-     * </p>
-     */
-    protected static final String OPT_WIDGETID = "id";
 
     /**
      * <p>

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/run/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/run/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index 1637fed..d404f17
--- a/org.tizen.cli/src/org/tizen/cli/exec/run/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/run/Main.java
@@ -28,6 +28,7 @@ package org.tizen.cli.exec.run;
 import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.cli.exec.WRTLauncher;
 import org.tizen.common.core.command.Executor;
 import org.tizen.common.core.command.Prompter;
@@ -51,34 +52,10 @@ import org.tizen.web.launch.cli.command.RunCommand;
 public class
 Main
 extends WRTLauncher
+implements ILaunchOptions
 {
     /**
      * <p>
-     * Option for target widget file
-     * </p>
-     */
-    protected static final String OPT_WIDGETFILE = "widget";
-
-    /**
-     * <p>
-     * Description for target widget file
-     * 
-     * This is printed out in usage
-     * </p>
-     * 
-     * @see #OPT_WIDGETFILE
-     */
-    protected static final String DESC_WIDGETFILE = "Command is performed of widget file install in target";
-
-    /**
-     * <p>
-     * Option for target widget id
-     * </p>
-     */
-    protected static final String OPT_WIDGETID = "id";
-
-    /**
-     * <p>
      * Description for target widget id
      * 
      * This is printed out in usage

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/sign/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/sign/Main.java
index 4283cff..d5119db 100755 (executable)
--- a/org.tizen.cli/src/org/tizen/cli/exec/sign/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/sign/Main.java
@@ -34,6 +34,7 @@ import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
 import org.tizen.cli.exec.AbstractLauncher;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.common.core.command.Prompter;
 import org.tizen.common.core.command.prompter.ChoiceOption;
 import org.tizen.common.core.command.prompter.Option;
@@ -51,64 +52,11 @@ import org.tizen.web.sign.command.SignCommand;
 public class
 Main
 extends AbstractLauncher
+implements ILaunchOptions
 {
        
        /**
         * <p>
-        * Option for include file pattern
-        * </p>
-        */
-       protected static final String OPT_INCLUDE = "include";
-       
-       /**
-        * <p>
-        * Description for include file pattern option
-        * 
-        * This is printed out in usage
-        * </p>
-        * 
-        * @see #OPT_INCLUDE
-        */
-       protected static final String DESC_INCLUDE = "file name pattern to include";
-       
-       /**
-        * <p>
-        * Option for exclude file pattern
-        * </p>
-        */
-       protected static final String OPT_EXCLUDE = "exclude";
-       
-       /**
-        * <p>
-        * Description for exclude file pattern option
-        * 
-        * This is printed out in usage
-        * </p>
-        * 
-        * @see #OPT_EXCLUDE
-        */
-       protected static final String DESC_EXCLUDE = "file name pattern to exclude";
-       
-       /**
-        * <p>
-        * Option for no check of validation
-        * </p>
-        */
-       protected static final String OPT_NOCHECK = "nocheck";
-       
-       /**
-        * <p>
-        * Description for no check option
-        * 
-        * This is printed out in usage
-        * </p>
-        * 
-        * @see #OPT_NOCHECK
-        */
-       protected static final String DESC_NOCHECK = "skip validation check";
-       
-       /**
-        * <p>
         * Option for profile with/without profiles file
         * 
         * Examples are following

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/uninstall/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/uninstall/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index b0e86fc..dc9fe90
--- a/org.tizen.cli/src/org/tizen/cli/exec/uninstall/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/uninstall/Main.java
@@ -30,6 +30,7 @@ import java.text.MessageFormat;
 import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.cli.exec.WRTLauncher;
 import org.tizen.common.core.command.Executor;
 import org.tizen.common.core.command.Prompter;
@@ -46,16 +47,10 @@ import org.tizen.web.launch.cli.command.UninstallCommand;
 public class
 Main
 extends WRTLauncher
+implements ILaunchOptions
 {
     /**
      * <p>
-     * Option for target widget id
-     * </p>
-     */
-    protected static final String OPT_WIDGETID = "id";
-
-    /**
-     * <p>
      * Description for target widget id
      * 
      * This is printed out in usage

diff --git a/org.tizen.cli/src/org/tizen/cli/exec/wgt/Main.java b/org.tizen.cli/src/org/tizen/cli/exec/wgt/Main.java
old mode 100644 (file)
new mode 100755 (executable)
index b4c2c2c..fd3b6d3
--- a/org.tizen.cli/src/org/tizen/cli/exec/wgt/Main.java
+++ b/org.tizen.cli/src/org/tizen/cli/exec/wgt/Main.java
@@ -33,6 +33,7 @@ import org.apache.commons.cli.CommandLine;
 import org.apache.commons.cli.OptionBuilder;
 import org.apache.commons.cli.Options;
 import org.tizen.cli.exec.AbstractLauncher;
+import org.tizen.cli.exec.ILaunchOptions;
 import org.tizen.common.core.command.Executor;
 import org.tizen.common.core.command.Prompter;
 import org.tizen.common.core.command.prompter.ChoiceOption;
@@ -52,71 +53,10 @@ import org.tizen.web.common.WebConstant;
 public class
 Main
 extends AbstractLauncher
+implements ILaunchOptions
 {
        
-       /**
-        * <p>
-        * Option for include file pattern
-        * </p>
-        */
-       protected static final String OPT_INCLUDE = "include";
-       
-       /**
-        * <p>
-        * Description for include file pattern option
-        * 
-        * It is printed out in usage
-        * </p>
-        */
-       protected static final String DESC_INCLUDE = "file name pattern to include";
-       
-       /**
-        * <p>
-        * Option for exclude file pattern
-        * </p>
-        */
-       protected static final String OPT_EXCLUDE = "exclude";
-       
-       /**
-        * <p>
-        * Description for exclude file pattern option
-        * 
-        * It is printed out in usage
-        * </p>
-        */
-       protected static final String DESC_EXCLUDE = "file name pattern to exclude";
-       
-       /**
-        * <p>
-        * Option for no check of validation
-        * </p>
-        */
-       protected static final String OPT_NOCHECK = "nocheck";
-       
-       /**
-        * <p>
-        * Description for no check option
-        * 
-        * It is printed out in usage
-        * </p>
-        */
-       protected static final String DESC_NOCHECK = "skip validation check";
-       
-       /**
-        * <p>
-        * Option for over-writing of output file
-        * </p>
-        */
-       protected static final String OPT_OVERWRITE = "overwrite";
-       
-       /**
-        * <p>
-        * Description for overwrite option
-        * 
-        * It is printed out in usage
-        * </p>
-        */
-       protected static final String DESC_OVERWRITE = "overwrite output file if it exists";
+
        
        /**
         * Entry point for cli main