PMD Results

The following document contains the results of PMD 5.2.3.

Files

jgloss/ui/im/KanaInputMethod.java

Violation Line
Avoid using redundant field initializer for 'active' 42
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 44
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 48
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_context' is not final. 70
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_locale' is not final. 75
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_subsets' is not final. 86
Consider using varargs for methods or constructors which take an array the last parameter. 86
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_event' is not final. 106
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 117
Potential violation of Law of Demeter (object not created locally) 121
Avoid using Literals in Conditional Statements 124
Potential violation of Law of Demeter (object not created locally) 127
Potential violation of Law of Demeter (object not created locally) 141
Potential violation of Law of Demeter (object not created locally) 147

jgloss/ui/im/KanaInputMethodDescriptor.java

Violation Line
Returning 'LOCALES' may expose an internal array. 49

jgloss/ui/im/RomajiTranslator.java

Violation Line
The class 'RomajiTranslator' has a Modified Cyclomatic Complexity of 4 (Highest = 12). 30–520
The class 'RomajiTranslator' has a Standard Cyclomatic Complexity of 4 (Highest = 12). 30–520
Avoid instantiation through private constructors from outside of the constructors class. 32
Avoid instantiation through private constructors from outside of the constructors class. 33
The method 'translate' has a Modified Cyclomatic Complexity of 12. 44–119
The method 'translate' has a Standard Cyclomatic Complexity of 12. 44–119
Avoid using Literals in Conditional Statements 63
Potential violation of Law of Demeter (method chain calls) 149
This final field could be made static 172
The constructor with 0 parameters has an NCSS line count of 118 174–330
This final field could be made static 347
The constructor with 0 parameters has an NCSS line count of 118 349–507