An Interface should be used only to model a behaviour; consider converting this to a class. Avoid excessively long variable names like MENU_ITEM_DEBUG_ENABLE Avoid excessively long variable names like MENU_ITEM_DEBUG_EMAIL Avoid excessively long variable names like MENU_ITEM_MORE_HELP Avoid excessively long variable names like MENU_ITEM_REINSTALL Avoid excessively long variable names like MENU_GROUP_EXTRA_OPTIONS Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. Local variable 'msg' could be declared final A method should have only one exit point, and that should be the last statement in the method Local variable 'msg' could be declared final Local variable 'msg' could be declared final Local variable 'buf' could be declared final Local variable 'in_nmap_a' could be declared final Avoid excessively long variable names like installationResults Avoid variables with short names like os Avoid variables with short names like in Avoid assignments in operands Found 'DU'-anomaly for variable 'input' (lines '240'-'265'). Avoid assignments in operands Found 'DU'-anomaly for variable 'error' (lines '244'-'265'). A high number of imports can indicate a high degree of coupling within an object. The class 'NmapMain' has a Cyclomatic Complexity of 5 (Highest = 20). This class has too many methods, consider refactoring it. Too many fields Avoid variables with short names like sh Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. Avoid variables with short names like pd Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. Perhaps 'preferencesEditor' could be replaced by a local variable. Avoid variables with short names like ne Found non-transient, non-static member. Please mark as transient or provide accessors. Perhaps 'wifiManager' could be replaced by a local variable. Perhaps 'wifiInfo' could be replaced by a local variable. Avoid excessively long variable names like savedInstanceState Avoid really long methods. Parameter 'savedInstanceState' is not assigned and could be declared final The method 'onCreate' has a Cyclomatic Complexity of 20. The method onCreate() has an NPath complexity of 900 Local variable 'target' could be declared final Local variable 'args' could be declared final The String literal "installVerified" appears 4 times in this file; the first occurrence is on line 200 The String literal "hasRunOneScan" appears 4 times in this file; the first occurrence is on line 202 The String literal "canGetRoot" appears 4 times in this file; the first occurrence is on line 204 Local variable 'curVersion' could be declared final Avoid variables with short names like v Parameter 'v' is not assigned and could be declared final Avoid excessively long variable names like targetAndArguments Found 'DU'-anomaly for variable 'targetAndArguments' (lines '257'-'318'). Found 'DU'-anomaly for variable 'targetAndArguments' (lines '257'-'318'). Found 'DU'-anomaly for variable 'targetAndArguments' (lines '257'-'318'). Found 'DU'-anomaly for variable 'targetAndArguments' (lines '257'-'318'). Avoid variables with short names like v Parameter 'v' is not assigned and could be declared final Parameter 'v' is not assigned and could be declared final Avoid instantiating String objects; this is usually unnecessary. Avoid instantiating String objects; this is usually unnecessary. Avoid excessively long variable names like savedInstanceState Parameter 'savedInstanceState' is not assigned and could be declared final Avoid using if statements without curly braces Avoid using if statements without curly braces Local variable 'uri' could be declared final Local variable 'surveyActivity' could be declared final Avoid using if statements without curly braces Avoid really long methods. The method 'initHandler' has a Cyclomatic Complexity of 18. Avoid really long methods. The method 'handleMessage' has a Cyclomatic Complexity of 17. Found 'DU'-anomaly for variable 'alert' (lines '539'-'647'). Found 'DU'-anomaly for variable 'alert' (lines '539'-'647'). Found 'DU'-anomaly for variable 'alert' (lines '539'-'647'). An empty statement (semicolon) not part of a loop Position literals first in String comparisons Position literals first in String comparisons Position literals first in String comparisons Avoid assignments in operands Found 'DU'-anomaly for variable 'line' (lines '603'-'647'). Found 'DU'-anomaly for variable 'line' (lines '603'-'647'). Found 'DU'-anomaly for variable 'line' (lines '603'-'647'). The class 'NmapScan' has a Cyclomatic Complexity of 10 (Highest = 21). Avoid variables with short names like os Perhaps 'os' could be replaced by a local variable. Avoid variables with short names like in Perhaps 'in' could be replaced by a local variable. Perhaps 'err' could be replaced by a local variable. Avoid variables with short names like p Perhaps 'p' could be replaced by a local variable. Avoid really long methods. The method 'doInBackground' has a Cyclomatic Complexity of 21. The method doInBackground() has an NPath complexity of 79872 Found 'DU'-anomaly for variable 'exeParams' (lines '80'-'233'). Found 'DU'-anomaly for variable 'temporaryFolder' (lines '88'-'233'). A method should have only one exit point, and that should be the last statement in the method A method should have only one exit point, and that should be the last statement in the method Position literals first in String comparisons Avoid using if...else statements without curly braces Avoid assignments in operands Prefer StringBuffer over += for concatenating strings Local variable 'msg1' could be declared final Local variable 'msg' could be declared final Avoid assignments in operands Local variable 'msg' could be declared final Local variable 'msg' could be declared final Local variable 'msg' could be declared final Position literals first in String comparisons Avoid excessively long variable names like savedInstanceState Parameter 'savedInstanceState' is not assigned and could be declared final Avoid variables with short names like v Parameter 'v' is not assigned and could be declared final Local variable 'myUploadData' could be declared final To be compliant to J2EE, a webapp should not use any thread. Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. Found non-transient, non-static member. Please mark as transient or provide accessors. It is a good practice to call super() in a constructor Local variable 'client' could be declared final Local variable 'url' could be declared final Local variable 'uri' could be declared final Local variable 'httppost' could be declared final Avoid instantiating new objects inside loops Found 'DD'-anomaly for variable 'dataDir' (lines '108'-'110').