A high number of imports can indicate a high degree of coupling within an object. The class 'Nmap' has a Cyclomatic Complexity of 7 (Highest = 45). This class has too many methods, consider refactoring it. Too many fields 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. 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 'mHelp' could be replaced by a local variable. Perhaps 'mShare' could be replaced by a local variable. Avoid excessively long variable names like installationVerified Avoid variables with short names like sh Avoid variables with short names like h Avoid variables with short names like pd Perhaps 'preferencesEditor' could be replaced by a local variable. The field name indicates a constant but its modifiers do not Variables should start with a lowercase character Avoid variables with short names like ne 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 45. The method onCreate() has an NPath complexity of 826200 Local variable 'adapterCommand' could be declared final Parameter 'arg0' is not assigned and could be declared final Parameter 'arg1' is not assigned and could be declared final Parameter 'arg2' is not assigned and could be declared final Parameter 'arg3' is not assigned and could be declared final Local variable 'adapterOutput' could be declared final Local variable 'target' could be declared final Local variable 'args' could be declared final Local variable 'result' could be declared final The String literal "installationVerified" appears 4 times in this file; the first occurrence is on line 258 The String literal "hasRunOneScan" appears 4 times in this file; the first occurrence is on line 260 The String literal "canGetRoot" appears 4 times in this file; the first occurrence is on line 262 Avoid variables with short names like ip Avoid using if statements without curly braces The method 'handleMessage' has a Cyclomatic Complexity of 15. Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). Found 'DU'-anomaly for variable 'alert' (lines '313'-'484'). An empty statement (semicolon) not part of a loop Avoid if (x != y) ..; else ..; Avoid using if...else statements without curly braces Avoid using if...else statements without curly braces Position literals first in String comparisons Avoid using if...else statements without curly braces Position literals first in String comparisons Avoid using if...else statements without curly braces Position literals first in String comparisons Avoid using if...else statements without curly braces Avoid assignments in operands Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Found 'DU'-anomaly for variable 'l' (lines '360'-'484'). Avoid using if statements without curly braces Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Found 'DU'-anomaly for variable 's' (lines '425'-'484'). Avoid instantiating String objects; this is usually unnecessary. Avoid instantiating String objects; this is usually unnecessary. Avoid using if statements without curly braces Avoid using if statements without curly braces Avoid excessively long variable names like savedInstanceState Avoid using if statements without curly braces Use bitwise inversion to invert boolean values Avoid using implementation types like 'ArrayList'; use the interface instead Avoid using implementation types like 'ArrayList'; use the interface instead Found non-transient, non-static member. Please mark as transient or provide accessors. Private field 'errors' could be made final; it is only initialized in the declaration or constructor. Avoid variables with short names like s Method names should not start with capital letters Parameter 's' is not assigned and could be declared final Avoid using if statements without curly braces Method names should not start with capital letters Avoid variables with short names like s Local variable 'stop' could be declared final Prefer StringBuffer over += for concatenating strings All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. Avoid variables with short names like c Parameter 'c' is not assigned and could be declared final Local variable 'tmpdir' could be declared final Local variable 'msg' could be declared final Local variable 'path' could be declared final Avoid variables with short names like st Local variable 'st' could be declared final Local variable 'token' could be declared final Avoid instantiating new objects inside loops A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses This statement may have some unnecessary parentheses Avoid variables with short names like c Parameter 'c' is not assigned and could be declared final A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses This statement may have some unnecessary parentheses Parameter 'myDataDir' is not assigned and could be declared final This statement may have some unnecessary parentheses An Interface should be used only to model a behaviour; consider converting this to a class. Avoid modifiers which are implied by the context Avoid modifiers which are implied by the context Avoid modifiers which are implied by the context Avoid modifiers which are implied by the context Avoid modifiers which are implied by the context 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 Class names should begin with an uppercase character The class 'scan' has a Cyclomatic Complexity of 10 (Highest = 20). Found non-transient, non-static member. Please mark as transient or provide accessors. Avoid excessively long variable names like executionParameters Found non-transient, non-static member. Please mark as transient or provide accessors. Perhaps 'executionParameters' could be replaced by a local variable. Avoid variables with short names like c Found non-transient, non-static member. Please mark as transient or provide accessors. Perhaps 'c' could be replaced by a local variable. Avoid variables with short names like os Found non-transient, non-static member. Please mark as transient or provide accessors. Perhaps 'os' could be replaced by a local variable. Avoid variables with short names like in Found non-transient, non-static member. Please mark as transient or provide accessors. 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 Switch statements should have a default label Avoid instantiating String objects; this is usually unnecessary. Avoid variables with short names like v Parameter 'v' is not assigned and could be declared final Avoid really long methods. Parameter 'params' is not assigned and could be declared final The method 'doInBackground' has a Cyclomatic Complexity of 20. The method doInBackground() has an NPath complexity of 59904 Avoid using if statements without curly braces 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 This statement may have some unnecessary parentheses Avoid using if...else statements without curly braces 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 This statement may have some unnecessary parentheses Avoid using if statements without curly braces Avoid using if statements without curly braces Avoid using if statements without curly braces 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 Avoid using if statements without curly braces Position literals first in String comparisons This statement may have some unnecessary parentheses Class names should begin with an uppercase character The class 'verifyInstallation' has a Cyclomatic Complexity of 8 (Highest = 20). Avoid excessively long variable names like installationResults 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. Private field 'filenames' could be made final; it is only initialized in the declaration or constructor. Found non-transient, non-static member. Please mark as transient or provide accessors. Private field 'requiredResources' could be made final; it is only initialized in the declaration or constructor. 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 'resources' could be replaced by a local variable. Avoid variables with short names like v Parameter 'v' is not assigned and could be declared final Avoid really long methods. Parameter 'params' is not assigned and could be declared final The method 'doInBackground' has a Cyclomatic Complexity of 20. The method doInBackground() has an NCSS line count of 102 The method doInBackground() has an NPath complexity of 43920 Found 'DU'-anomaly for variable 'buf' (lines '61'-'244'). Local variable 'buf' could be declared final Avoid if (x != y) ..; else ..; Avoid using if...else statements without curly braces A method should have only one exit point, and that should be the last statement in the method Avoid using if...else statements without curly braces Local variable 'in_nmap_a' could be declared final Local variable 'in_nmap_b' could be declared final Local variable 'in_nmap_c' could be declared final Local variable 'out_nmap' could be declared final A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses Avoid using while statements without curly braces Avoid using while statements without curly braces A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses Avoid instantiating new objects inside loops A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses Avoid variables with short names like p Avoid variables with short names like os Avoid variables with short names like in Avoid variables with short names like e Avoid assignments in operands Avoid assignments in operands A method should have only one exit point, and that should be the last statement in the method This statement may have some unnecessary parentheses This statement may have some unnecessary parentheses Parameter 'path' is not assigned and could be declared final