PMD Results

The following document contains the results of PMD 5.2.3.

Files

jgloss/JDictionaryApp.java

Violation Line
Consider using varargs for methods or constructors which take an array the last parameter. 39
A method/constructor shouldnt explicitly throw java.lang.Exception 52
Consider using varargs for methods or constructors which take an array the last parameter. 52

jgloss/JGloss.java

Violation Line
Possible God class (WMC=58, ATFD=63, TCC=0.008658008658008658) 23–421
The class 'JGloss' has a Modified Cyclomatic Complexity of 3 (Highest = 13). 75–421
The class 'JGloss' has a Standard Cyclomatic Complexity of 3 (Highest = 13). 75–421
Consider using varargs for methods or constructors which take an array the last parameter. 119
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 132
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 150
A method/constructor shouldnt explicitly throw java.lang.Exception 173
Consider using varargs for methods or constructors which take an array the last parameter. 173
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 218
A method/constructor shouldnt explicitly throw java.lang.Exception 231
Potential violation of Law of Demeter (static property access) 235
Potential violation of Law of Demeter (static property access) 237
The method 'handleCommandLine' has a Standard Cyclomatic Complexity of 13. 244–305
A method/constructor shouldnt explicitly throw java.lang.Exception 244
Consider using varargs for methods or constructors which take an array the last parameter. 244
The method 'handleCommandLine' has a Modified Cyclomatic Complexity of 13. 244–305
Potential violation of Law of Demeter (method chain calls) 247
Potential violation of Law of Demeter (method chain calls) 247
Potential violation of Law of Demeter (method chain calls) 247
Potential violation of Law of Demeter (method chain calls) 247
Potential violation of Law of Demeter (method chain calls) 248
Potential violation of Law of Demeter (method chain calls) 248
System.out.println is used 249
Potential violation of Law of Demeter (method chain calls) 252
Potential violation of Law of Demeter (method chain calls) 252
Potential violation of Law of Demeter (method chain calls) 252
Potential violation of Law of Demeter (method chain calls) 252
Potential violation of Law of Demeter (method chain calls) 257
Potential violation of Law of Demeter (method chain calls) 259
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (method chain calls) 262
Potential violation of Law of Demeter (object not created locally) 266
Potential violation of Law of Demeter (object not created locally) 268
Potential violation of Law of Demeter (method chain calls) 270–271
Potential violation of Law of Demeter (method chain calls) 270–271
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 272
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 273
Potential violation of Law of Demeter (object not created locally) 274
Potential violation of Law of Demeter (method chain calls) 276–279
Potential violation of Law of Demeter (method chain calls) 276–279
Potential violation of Law of Demeter (method chain calls) 276–279
Potential violation of Law of Demeter (method chain calls) 276–279
Potential violation of Law of Demeter (object not created locally) 278
Potential violation of Law of Demeter (method chain calls) 278
Potential violation of Law of Demeter (object not created locally) 279
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 288
Potential violation of Law of Demeter (method chain calls) 289–290
Potential violation of Law of Demeter (method chain calls) 289–290
Potential violation of Law of Demeter (method chain calls) 292–293
Potential violation of Law of Demeter (method chain calls) 292–293
Potential violation of Law of Demeter (object not created locally) 293
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 298
Potential violation of Law of Demeter (method chain calls) 299
Potential violation of Law of Demeter (method chain calls) 299
Potential violation of Law of Demeter (method chain calls) 300
Potential violation of Law of Demeter (method chain calls) 300
Potential violation of Law of Demeter (method chain calls) 338
Potential violation of Law of Demeter (method chain calls) 338
Potential violation of Law of Demeter (method chain calls) 355–356
Potential violation of Law of Demeter (method chain calls) 356
Potential violation of Law of Demeter (method chain calls) 356
Potential violation of Law of Demeter (method chain calls) 365
Potential violation of Law of Demeter (object not created locally) 389
Potential violation of Law of Demeter (object not created locally) 392
Potential violation of Law of Demeter (object not created locally) 395–396
Potential violation of Law of Demeter (method chain calls) 395–396
Potential violation of Law of Demeter (method chain calls) 395–396
Potential violation of Law of Demeter (object not created locally) 398
Potential violation of Law of Demeter (object not created locally) 402
Potential violation of Law of Demeter (method chain calls) 406
Potential violation of Law of Demeter (object not created locally) 406
Potential violation of Law of Demeter (object not created locally) 407
Potential violation of Law of Demeter (method chain calls) 409
Potential violation of Law of Demeter (object not created locally) 409
Potential violation of Law of Demeter (object not created locally) 410
Potential violation of Law of Demeter (method chain calls) 412
Potential violation of Law of Demeter (object not created locally) 412
Potential violation of Law of Demeter (object not created locally) 413
Potential violation of Law of Demeter (method chain calls) 415
Potential violation of Law of Demeter (object not created locally) 415
Potential violation of Law of Demeter (object not created locally) 416

jgloss/JavaPreferences.java

Violation Line
Avoid catching NullPointerException; consider removing the cause of the NPE. 92
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 92
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 122
Avoid catching NullPointerException; consider removing the cause of the NPE. 122

jgloss/Messages.java

Violation Line
It is somewhat confusing to have a field name matching the declaring class name 13

jgloss/Preferences.java

Violation Line
Use block level rather than method level synchronization 288–290
Use block level rather than method level synchronization 300–318
Potential violation of Law of Demeter (object not created locally) 304
Potential violation of Law of Demeter (object not created locally) 306
Potential violation of Law of Demeter (object not created locally) 308
Potential violation of Law of Demeter (object not created locally) 308
Potential violation of Law of Demeter (object not created locally) 310
Potential violation of Law of Demeter (object not created locally) 312
Potential violation of Law of Demeter (object not created locally) 313
Potential violation of Law of Demeter (object not created locally) 313
Potential violation of Law of Demeter (object not created locally) 317

jgloss/PropertiesPreferences.java

Violation Line
Avoid using redundant field initializer for 'changed' 50
Potential violation of Law of Demeter (static property access) 69
Use block level rather than method level synchronization 76–83
Use block level rather than method level synchronization 92–100
Use block level rather than method level synchronization 111–120
Potential violation of Law of Demeter (object not created locally) 116
Use block level rather than method level synchronization 129–131
Use block level rather than method level synchronization 142–151
Potential violation of Law of Demeter (object not created locally) 147
Use block level rather than method level synchronization 160–162
Use block level rather than method level synchronization 172–181
Use block level rather than method level synchronization 190–192
Use block level rather than method level synchronization 200–210
Potential violation of Law of Demeter (object not created locally) 202
Potential violation of Law of Demeter (object not created locally) 208
Use block level rather than method level synchronization 218–223
Potential violation of Law of Demeter (object not created locally) 221
Use block level rather than method level synchronization 231–235
Potential violation of Law of Demeter (static property access) 233

jgloss/ui/AboutFrame.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 138
Potential violation of Law of Demeter (method chain calls) 173
Potential violation of Law of Demeter (object not created locally) 175
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 176
Potential violation of Law of Demeter (object not created locally) 176
Potential violation of Law of Demeter (object not created locally) 179
Potential violation of Law of Demeter (object not created locally) 183
Potential violation of Law of Demeter (object not created locally) 191
Potential violation of Law of Demeter (method chain calls) 198

jgloss/ui/AsynchronousLookupEngine.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 69
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 71
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 73
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_handler' is not final. 75
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_handler' is not final. 79
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_dictionaryEntryLimit' is not final. 79

jgloss/ui/AttributeLegend.java

Violation Line
Variables that are final and static should be all capitals, 'attributeComparator' is not all capitals. 57
Potential violation of Law of Demeter (method chain calls) 60–61
Variables that are final and static should be all capitals, 'categoryComparator' is not all capitals. 65
Potential violation of Law of Demeter (method chain calls) 68–69
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_dictionaryName' is not final. 77
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_text' is not final. 77
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 87
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
Consider using varargs for methods or constructors which take an array the last parameter. 114
StringBuffer constructor is initialized with size 16, but has at least 40 characters appended. 118
Potential violation of Law of Demeter (static property access) 129
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (static property access) 145–146
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (static property access) 148
Potential violation of Law of Demeter (object not created locally) 149
Potential violation of Law of Demeter (object not created locally) 150
Potential violation of Law of Demeter (method chain calls) 151–152
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 155
Potential violation of Law of Demeter (object not created locally) 158
Potential violation of Law of Demeter (static property access) 165
Potential violation of Law of Demeter (method chain calls) 173
Potential violation of Law of Demeter (method chain calls) 180
Potential violation of Law of Demeter (static property access) 191
Potential violation of Law of Demeter (object not created locally) 209

jgloss/ui/AttributeResultFilter.java

Violation Line
The class 'AttributeResultFilter' has a Standard Cyclomatic Complexity of 4 (Highest = 18). 31–126
The class 'AttributeResultFilter' has a Modified Cyclomatic Complexity of 4 (Highest = 18). 31–126
Field acceptIfSet has the same name as a method 35
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_name' is not final. 37
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_description' is not final. 37
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_attribute' is not final. 38
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_acceptIfSet' is not final. 38
The method 'acceptIfSet' has a Modified Cyclomatic Complexity of 18. 58–113
The method 'acceptIfSet' has a Standard Cyclomatic Complexity of 18. 58–113
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (method chain calls) 65
Potential violation of Law of Demeter (method chain calls) 70
Potential violation of Law of Demeter (method chain calls) 77
Potential violation of Law of Demeter (method chain calls) 82
Potential violation of Law of Demeter (method chain calls) 89
Potential violation of Law of Demeter (method chain calls) 94
Potential violation of Law of Demeter (method chain calls) 99
Potential violation of Law of Demeter (method chain calls) 104
Potential violation of Law of Demeter (method chain calls) 117

jgloss/ui/BrowseHyperlinkListener.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 27
Potential violation of Law of Demeter (method chain calls) 33
Potential violation of Law of Demeter (method chain calls) 33
Potential violation of Law of Demeter (method chain calls) 33

jgloss/ui/CustomFileView.java

Violation Line
Variables should start with a lowercase character, 'FILE_VIEW' starts with uppercase character. 39
The field name indicates a constant but its modifiers do not 39
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'FILE_VIEW' is not final. 39
Singleton is not thread safe 46–48
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
Possible unsafe assignment to a non-final static field in a constructor. 52
Variables should start with a lowercase character, 'JGLOSS_DESCRIPTION' starts with uppercase character. 52
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'JGLOSS_DESCRIPTION' is not final. 52
The field name indicates a constant but its modifiers do not 52
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
The field name indicates a constant but its modifiers do not 53
Possible unsafe assignment to a non-final static field in a constructor. 53
Variables should start with a lowercase character, 'XML_DESCRIPTION' starts with uppercase character. 53
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'XML_DESCRIPTION' is not final. 53
The field name indicates a constant but its modifiers do not 54
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'HTML_DESCRIPTION' is not final. 54
Variables should start with a lowercase character, 'HTML_DESCRIPTION' starts with uppercase character. 54
Possible unsafe assignment to a non-final static field in a constructor. 54
Variables should start with a lowercase character, 'TEX_DESCRIPTION' starts with uppercase character. 55
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55
Possible unsafe assignment to a non-final static field in a constructor. 55
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEX_DESCRIPTION' is not final. 55
The field name indicates a constant but its modifiers do not 55
Possible unsafe assignment to a non-final static field in a constructor. 56
The field name indicates a constant but its modifiers do not 56
Variables should start with a lowercase character, 'TEXT_DESCRIPTION' starts with uppercase character. 56
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEXT_DESCRIPTION' is not final. 56
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 56
The field name indicates a constant but its modifiers do not 57
Variables should start with a lowercase character, 'TEMPLATE_DESCRIPTION' starts with uppercase character. 57
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 57
Possible unsafe assignment to a non-final static field in a constructor. 57
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEMPLATE_DESCRIPTION' is not final. 57
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 59
Possible unsafe assignment to a non-final static field in a constructor. 59
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'JGLOSS_ICON' is not final. 59
Variables should start with a lowercase character, 'JGLOSS_ICON' starts with uppercase character. 59
The field name indicates a constant but its modifiers do not 59
Variables should start with a lowercase character, 'XML_ICON' starts with uppercase character. 60
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
The field name indicates a constant but its modifiers do not 60
Possible unsafe assignment to a non-final static field in a constructor. 60
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'XML_ICON' is not final. 60
Possible unsafe assignment to a non-final static field in a constructor. 61
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 61
Variables should start with a lowercase character, 'HTML_ICON' starts with uppercase character. 61
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'HTML_ICON' is not final. 61
The field name indicates a constant but its modifiers do not 61
Variables should start with a lowercase character, 'TEX_ICON' starts with uppercase character. 62
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 62
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEX_ICON' is not final. 62
The field name indicates a constant but its modifiers do not 62
Possible unsafe assignment to a non-final static field in a constructor. 62
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEXT_ICON' is not final. 63
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 63
Possible unsafe assignment to a non-final static field in a constructor. 63
The field name indicates a constant but its modifiers do not 63
Variables should start with a lowercase character, 'TEXT_ICON' starts with uppercase character. 63
Variables should start with a lowercase character, 'TEMPLATE_ICON' starts with uppercase character. 64
The field name indicates a constant but its modifiers do not 64
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 64
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), 'TEMPLATE_ICON' is not final. 64
Possible unsafe assignment to a non-final static field in a constructor. 64
Potential violation of Law of Demeter (method chain calls) 87
Potential violation of Law of Demeter (object not created locally) 89
Potential violation of Law of Demeter (object not created locally) 91
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 93
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (object not created locally) 99
Potential violation of Law of Demeter (method chain calls) 113
Potential violation of Law of Demeter (object not created locally) 115
Potential violation of Law of Demeter (object not created locally) 117
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 123
Potential violation of Law of Demeter (object not created locally) 125

jgloss/ui/DescriptorDictionaryWrapper.java

Violation Line
The class 'DescriptorDictionaryWrapper' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 9–66
The class 'DescriptorDictionaryWrapper' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 9–66
Avoid using final local variables, turn them into fields 38
The method 'equals' has a Standard Cyclomatic Complexity of 10. 46–65
The method 'equals' has a Modified Cyclomatic Complexity of 10. 46–65
Avoid using if statements without curly braces 47–48
Avoid using if statements without curly braces 49–50
Avoid using if statements without curly braces 51–52
Avoid using if statements without curly braces 55–56
Avoid using if statements without curly braces 57–58
Avoid using if statements without curly braces 60–61
Avoid using if statements without curly braces 62–63

jgloss/ui/Dictionaries.java

Violation Line
Possible God class (WMC=63, ATFD=119, TCC=0.01904761904761905) 23–583
The class 'Dictionaries' has a Standard Cyclomatic Complexity of 3 (Highest = 13). 85–583
The class 'Dictionaries' has a Modified Cyclomatic Complexity of 3 (Highest = 13). 85–583
It is somewhat confusing to have a field name matching the declaring class name 98
Singleton is not thread safe 118–120
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 158
The constructor 'Dictionaries' has a Standard Cyclomatic Complexity of 13. 177–329
The constructor 'Dictionaries' has a Modified Cyclomatic Complexity of 13. 177–329
Potential violation of Law of Demeter (object not created locally) 191
Potential violation of Law of Demeter (object not created locally) 192
Potential violation of Law of Demeter (object not created locally) 205
Potential violation of Law of Demeter (object not created locally) 206
Potential violation of Law of Demeter (method chain calls) 223
Potential violation of Law of Demeter (method chain calls) 225
Potential violation of Law of Demeter (object not created locally) 253
Potential violation of Law of Demeter (object not created locally) 254
Potential violation of Law of Demeter (object not created locally) 256
Potential violation of Law of Demeter (object not created locally) 257
Potential violation of Law of Demeter (method chain calls) 281
Potential violation of Law of Demeter (static property access) 332
Potential violation of Law of Demeter (static property access) 345
Potential violation of Law of Demeter (static property access) 347–348
Potential violation of Law of Demeter (static property access) 356
String.trim().length()==0 is an inefficient way to validate an empty String. 357
Potential violation of Law of Demeter (object not created locally) 357
Potential violation of Law of Demeter (method chain calls) 357
Potential violation of Law of Demeter (static property access) 364
Consider using varargs for methods or constructors which take an array the last parameter. 372
Potential violation of Law of Demeter (object not created locally) 395
Potential violation of Law of Demeter (method chain calls) 396
Potential violation of Law of Demeter (method chain calls) 396
Potential violation of Law of Demeter (object not created locally) 396
Potential violation of Law of Demeter (method chain calls) 408
Potential violation of Law of Demeter (static property access) 416
Potential violation of Law of Demeter (static property access) 417
Potential violation of Law of Demeter (static property access) 419
Potential violation of Law of Demeter (static property access) 419
Potential violation of Law of Demeter (static property access) 422
Potential violation of Law of Demeter (static property access) 429
Potential violation of Law of Demeter (method chain calls) 452
Potential violation of Law of Demeter (static property access) 457–458
Potential violation of Law of Demeter (static property access) 459
Potential violation of Law of Demeter (object not created locally) 473
Potential violation of Law of Demeter (object not created locally) 474
Potential violation of Law of Demeter (object not created locally) 476
Potential violation of Law of Demeter (object not created locally) 477
Potential violation of Law of Demeter (static property access) 488
Potential violation of Law of Demeter (static property access) 494
Potential violation of Law of Demeter (static property access) 503
These nested if statements could be combined 511–515
Potential violation of Law of Demeter (method chain calls) 511
There is log block not surrounded by if 512
Potential violation of Law of Demeter (method chain calls) 514
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 519
Potential violation of Law of Demeter (object not created locally) 558
Potential violation of Law of Demeter (object not created locally) 559
Potential violation of Law of Demeter (static property access) 561

jgloss/ui/DictionaryEntryFormat.java

Violation Line
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 46–249
The class 'DictionaryEntryFormat' has a Modified Cyclomatic Complexity of 2 (Highest = 11). 46–249
The class 'DictionaryEntryFormat' has a Standard Cyclomatic Complexity of 2 (Highest = 11). 46–249
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 89
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 90
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 91
Variables that are final and static should be all capitals, 'word' is not all capitals. 93
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 93–94
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95–96
Variables that are final and static should be all capitals, 'reading' is not all capitals. 95
Variables that are final and static should be all capitals, 'rom' is not all capitals. 97
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97–98
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99–100
Variables that are final and static should be all capitals, 'crm' is not all capitals. 99
Variables that are final and static should be all capitals, 'syn' is not all capitals. 101
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 101–102
The method 'getAttributeFormatter' has a Standard Cyclomatic Complexity of 11. 123–171
The method 'getAttributeFormatter' has a Modified Cyclomatic Complexity of 11. 123–171
Potential violation of Law of Demeter (static property access) 138
Potential violation of Law of Demeter (static property access) 139
Potential violation of Law of Demeter (static property access) 143
Potential violation of Law of Demeter (static property access) 144
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 200

jgloss/ui/DictionaryLoader.java

Violation Line
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 64
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 87
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (object not created locally) 105

jgloss/ui/DictionaryStateWrapper.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_obj' is not final. 8
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_enabled' is not final. 12
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_selected' is not final. 12
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_obj' is not final. 12

jgloss/ui/ExclusionList.java

Violation Line
Possible God class (WMC=48, ATFD=77, TCC=0.11695906432748537) 23–491
The class 'ExclusionList' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 74–491
The class 'ExclusionList' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 74–491
It is somewhat confusing to have a field name matching the declaring class name 93
Singleton is not thread safe 105–107
Potential violation of Law of Demeter (method chain calls) 119
Potential violation of Law of Demeter (method chain calls) 126
The constructor 'ExclusionList' has a Standard Cyclomatic Complexity of 10. 132–240
The constructor 'ExclusionList' has a Modified Cyclomatic Complexity of 10. 132–240
Potential violation of Law of Demeter (method chain calls) 144
Potential violation of Law of Demeter (static property access) 243
Potential violation of Law of Demeter (object not created locally) 256
Potential violation of Law of Demeter (object not created locally) 257
Potential violation of Law of Demeter (object not created locally) 271
Potential violation of Law of Demeter (object not created locally) 276
Potential violation of Law of Demeter (static property access) 288–289
Potential violation of Law of Demeter (static property access) 289–290
Potential violation of Law of Demeter (object not created locally) 291
Potential violation of Law of Demeter (object not created locally) 293
Potential violation of Law of Demeter (object not created locally) 294
Potential violation of Law of Demeter (static property access) 306
Prefer StringBuffer over += for concatenating strings 308
Potential violation of Law of Demeter (static property access) 322
Potential violation of Law of Demeter (object not created locally) 325
Potential violation of Law of Demeter (object not created locally) 326
Potential violation of Law of Demeter (object not created locally) 327
Potential violation of Law of Demeter (object not created locally) 331
Potential violation of Law of Demeter (method chain calls) 333–337
Potential violation of Law of Demeter (method chain calls) 334–335
Potential violation of Law of Demeter (static property access) 334–335
Potential violation of Law of Demeter (object not created locally) 335
Potential violation of Law of Demeter (object not created locally) 335
Potential violation of Law of Demeter (method chain calls) 335
Potential violation of Law of Demeter (static property access) 336
Potential violation of Law of Demeter (static property access) 350
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 356
Potential violation of Law of Demeter (method chain calls) 358–362
Potential violation of Law of Demeter (static property access) 359–360
Potential violation of Law of Demeter (method chain calls) 359–360
Potential violation of Law of Demeter (method chain calls) 360
Potential violation of Law of Demeter (static property access) 361
Potential violation of Law of Demeter (method chain calls) 409
Potential violation of Law of Demeter (static property access) 410
Potential violation of Law of Demeter (object not created locally) 410
Potential violation of Law of Demeter (object not created locally) 411
Potential violation of Law of Demeter (object not created locally) 412
Potential violation of Law of Demeter (object not created locally) 413
Potential violation of Law of Demeter (object not created locally) 416
Potential violation of Law of Demeter (method chain calls) 416
Potential violation of Law of Demeter (method chain calls) 416
Potential violation of Law of Demeter (method chain calls) 416
Potential violation of Law of Demeter (method chain calls) 420
Potential violation of Law of Demeter (object not created locally) 420
Potential violation of Law of Demeter (object not created locally) 422
Potential violation of Law of Demeter (object not created locally) 423
Potential violation of Law of Demeter (object not created locally) 424
Potential violation of Law of Demeter (object not created locally) 427
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 431
Potential violation of Law of Demeter (object not created locally) 432
Potential violation of Law of Demeter (object not created locally) 436
Potential violation of Law of Demeter (object not created locally) 436
Potential violation of Law of Demeter (method chain calls) 436
Potential violation of Law of Demeter (object not created locally) 437
Potential violation of Law of Demeter (static property access) 438
Potential violation of Law of Demeter (static property access) 457
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 461
Potential violation of Law of Demeter (method chain calls) 462–466
Potential violation of Law of Demeter (static property access) 463–464
Potential violation of Law of Demeter (method chain calls) 463–464
Potential violation of Law of Demeter (method chain calls) 464
Potential violation of Law of Demeter (static property access) 465
Potential violation of Law of Demeter (method chain calls) 468

jgloss/ui/ExtensionFileFilter.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (method chain calls) 100

jgloss/ui/FontTester.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 64
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (static property access) 90
Potential violation of Law of Demeter (static property access) 91
Potential violation of Law of Demeter (object not created locally) 95
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 97

jgloss/ui/GeneralDialog.java

Violation Line
Use block level rather than method level synchronization 81–86
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 88
Private field 'startFrame' could be made final; it is only initialized in the declaration or constructor. 90
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 90
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 91
Private field 'startWordLookup' could be made final; it is only initialized in the declaration or constructor. 91
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 93
Private field 'clickTooltip' could be made final; it is only initialized in the declaration or constructor. 93
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
Private field 'clickSelect' could be made final; it is only initialized in the declaration or constructor. 94
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
Private field 'chasenLocation' could be made final; it is only initialized in the declaration or constructor. 96
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 98
Private field 'importClipboardParserSelector' could be made final; it is only initialized in the declaration or constructor. 98
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 99
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 100
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 101
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 102
Potential violation of Law of Demeter (static property access) 196
Potential violation of Law of Demeter (static property access) 206
Potential violation of Law of Demeter (static property access) 211
Potential violation of Law of Demeter (static property access) 218
Potential violation of Law of Demeter (static property access) 220
Potential violation of Law of Demeter (static property access) 225
Potential violation of Law of Demeter (method chain calls) 226
Potential violation of Law of Demeter (static property access) 234
Potential violation of Law of Demeter (static property access) 236
Potential violation of Law of Demeter (static property access) 237
Potential violation of Law of Demeter (object not created locally) 238
Avoid using Literals in Conditional Statements 238
Potential violation of Law of Demeter (object not created locally) 240
Potential violation of Law of Demeter (object not created locally) 240
Potential violation of Law of Demeter (static property access) 251
Potential violation of Law of Demeter (static property access) 252
Potential violation of Law of Demeter (static property access) 254
Potential violation of Law of Demeter (static property access) 256
Potential violation of Law of Demeter (static property access) 257–258
Potential violation of Law of Demeter (method chain calls) 257–258
Potential violation of Law of Demeter (method chain calls) 258
Potential violation of Law of Demeter (static property access) 259–260
Potential violation of Law of Demeter (static property access) 261–262
Potential violation of Law of Demeter (static property access) 264
Potential violation of Law of Demeter (method chain calls) 297
Potential violation of Law of Demeter (static property access) 298
Potential violation of Law of Demeter (method chain calls) 306
Potential violation of Law of Demeter (object not created locally) 307
Potential violation of Law of Demeter (object not created locally) 307
Position literals first in String comparisons 307
Position literals first in String comparisons 307
Potential violation of Law of Demeter (static property access) 315
Potential violation of Law of Demeter (method chain calls) 317
Potential violation of Law of Demeter (method chain calls) 317
Potential violation of Law of Demeter (static property access) 324–325
Potential violation of Law of Demeter (static property access) 325–326

jgloss/ui/Highlighter.java

Violation Line
Avoid modifiers which are implied by the context 32

jgloss/ui/HistoryItem.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_lookupModel' is not final. 14
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_resultState' is not final. 15

jgloss/ui/HyperlinkAttributeFormatter.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_values' is not final. 17
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_attribute' is not final. 17
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_value' is not final. 18
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 29
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 30
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 31
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 32
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_references' is not final. 34
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_protocol' is not final. 34
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_linkValues' is not final. 34
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_parent' is not final. 35

jgloss/ui/HyperlinkKeyNavigator.java

Violation Line
Possible God class (WMC=58, ATFD=89, TCC=0.225) 23–378
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_editor' is not final. 92
Use equals() to compare object references. 93
Potential violation of Law of Demeter (method chain calls) 99
Potential violation of Law of Demeter (method chain calls) 99
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (method chain calls) 100
Potential violation of Law of Demeter (object not created locally) 110–111
Potential violation of Law of Demeter (object not created locally) 112–113
Potential violation of Law of Demeter (object not created locally) 114–115
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 119
Potential violation of Law of Demeter (object not created locally) 120
Potential violation of Law of Demeter (object not created locally) 121
Potential violation of Law of Demeter (object not created locally) 122
Potential violation of Law of Demeter (method chain calls) 152
Potential violation of Law of Demeter (method chain calls) 155–156
Potential violation of Law of Demeter (object not created locally) 159
Potential violation of Law of Demeter (method chain calls) 165
Potential violation of Law of Demeter (method chain calls) 172
Potential violation of Law of Demeter (object not created locally) 229
Potential violation of Law of Demeter (object not created locally) 230–231
Potential violation of Law of Demeter (method chain calls) 230–231
Potential violation of Law of Demeter (object not created locally) 230
Potential violation of Law of Demeter (object not created locally) 238
Potential violation of Law of Demeter (object not created locally) 260
Potential violation of Law of Demeter (object not created locally) 261–262
Potential violation of Law of Demeter (object not created locally) 261
Potential violation of Law of Demeter (method chain calls) 261–262
Potential violation of Law of Demeter (object not created locally) 269
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_backwards' is not final. 287
Potential violation of Law of Demeter (method chain calls) 368–369
Potential violation of Law of Demeter (method chain calls) 368–369
Potential violation of Law of Demeter (method chain calls) 368–369

jgloss/ui/HyperlinkListFormatter.java

Violation Line
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 12
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_protocol' is not final. 14
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_references' is not final. 14
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_parent' is not final. 15
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_buffer' is not final. 22
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_length' is not final. 22

jgloss/ui/ImportDialog.java

Violation Line
The class 'ImportDialog' has a Modified Cyclomatic Complexity of 3 (Highest = 13). 67–345
The class 'ImportDialog' has a Standard Cyclomatic Complexity of 3 (Highest = 13). 67–345
The constructor 'ImportDialog' has a Modified Cyclomatic Complexity of 13. 100–279
The constructor 'ImportDialog' has a Standard Cyclomatic Complexity of 13. 100–279
The constructor with 1 parameters has an NCSS line count of 112 100–279
Potential violation of Law of Demeter (method chain calls) 112
Potential violation of Law of Demeter (method chain calls) 114
Potential violation of Law of Demeter (method chain calls) 114
Potential violation of Law of Demeter (static property access) 115–116
Potential violation of Law of Demeter (method chain calls) 115–116
Potential violation of Law of Demeter (method chain calls) 116
Potential violation of Law of Demeter (static property access) 117–118
Potential violation of Law of Demeter (static property access) 119–120
Potential violation of Law of Demeter (static property access) 122
Potential violation of Law of Demeter (method chain calls) 142
Potential violation of Law of Demeter (method chain calls) 142
Potential violation of Law of Demeter (method chain calls) 187
Potential violation of Law of Demeter (method chain calls) 190
Potential violation of Law of Demeter (method chain calls) 192
Potential violation of Law of Demeter (method chain calls) 192
Potential violation of Law of Demeter (method chain calls) 193
Potential violation of Law of Demeter (method chain calls) 193
Potential violation of Law of Demeter (method chain calls) 193
Use ArrayList instead of Vector 203
Avoid using Literals in Conditional Statements 251
Potential violation of Law of Demeter (method chain calls) 261

jgloss/ui/KeystrokeForwarder.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 34
Potential violation of Law of Demeter (object not created locally) 71

jgloss/ui/LookupConfigPanel.java

Violation Line
Possible God class (WMC=52, ATFD=131, TCC=0.005847953216374269) 23–491
The class 'LookupConfigPanel' has a Modified Cyclomatic Complexity of 3 (Highest = 11). 68–491
The class 'LookupConfigPanel' has a Standard Cyclomatic Complexity of 3 (Highest = 11). 68–491
Potential violation of Law of Demeter (object not created locally) 78
Potential violation of Law of Demeter (object not created locally) 79
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_parent' is not final. 89
Potential violation of Law of Demeter (object not created locally) 102
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 108
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 110
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 112
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 114
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 116
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 117
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 119
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 121
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 122
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 123
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 125
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 126
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 128
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_model' is not final. 132
The constructor with 1 parameters has an NCSS line count of 119 132–316
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_searchModes' is not final. 136
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_filters' is not final. 237
Unnecessary use of fully qualified name 'java.beans.PropertyChangeListener' due to existing import 'java.beans.PropertyChangeListener' 306
Potential violation of Law of Demeter (method chain calls) 309
Potential violation of Law of Demeter (static property access) 326
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_model' is not final. 340
Potential violation of Law of Demeter (method chain calls) 373
Potential violation of Law of Demeter (method chain calls) 373
Potential violation of Law of Demeter (object not created locally) 373
Potential violation of Law of Demeter (method chain calls) 374
Potential violation of Law of Demeter (object not created locally) 374
Potential violation of Law of Demeter (method chain calls) 374
Potential violation of Law of Demeter (method chain calls) 375
Potential violation of Law of Demeter (method chain calls) 375
Potential violation of Law of Demeter (object not created locally) 375
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (object not created locally) 377
Potential violation of Law of Demeter (method chain calls) 377
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (method chain calls) 378
Potential violation of Law of Demeter (object not created locally) 378
Potential violation of Law of Demeter (method chain calls) 384
Potential violation of Law of Demeter (method chain calls) 384
Potential violation of Law of Demeter (method chain calls) 390
Potential violation of Law of Demeter (method chain calls) 390
Potential violation of Law of Demeter (object not created locally) 403
Potential violation of Law of Demeter (object not created locally) 406
Potential violation of Law of Demeter (object not created locally) 416
Potential violation of Law of Demeter (method chain calls) 420
Potential violation of Law of Demeter (method chain calls) 422
Potential violation of Law of Demeter (object not created locally) 426
Potential violation of Law of Demeter (method chain calls) 427
Potential violation of Law of Demeter (method chain calls) 429
The method 'updateStateLater' has a Standard Cyclomatic Complexity of 11. 448–482
The method 'updateStateLater' has a Modified Cyclomatic Complexity of 11. 448–482

jgloss/ui/LookupEngine.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_handler' is not final. 44
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_dictionaryEntryLimit' is not final. 48
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_handler' is not final. 48
Potential violation of Law of Demeter (method chain calls) 60
Potential violation of Law of Demeter (object not created locally) 60
Potential violation of Law of Demeter (object not created locally) 62
Potential violation of Law of Demeter (method chain calls) 62
Consider using varargs for methods or constructors which take an array the last parameter. 83
Potential violation of Law of Demeter (method chain calls) 86
Potential violation of Law of Demeter (method chain calls) 107
Potential violation of Law of Demeter (method chain calls) 107

jgloss/ui/LookupFrame.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_model' is not final. 85
Potential violation of Law of Demeter (method chain calls) 121
Unnecessary use of fully qualified name 'UIUtilities.createMenuItem' due to existing import 'jgloss.ui.util.UIUtilities' 144
Potential violation of Law of Demeter (static property access) 169
Potential violation of Law of Demeter (static property access) 170
Overridable method 'getPreferredSize' called during object construction 194
Potential violation of Law of Demeter (method chain calls) 215
Unnecessary use of fully qualified name 'UIUtilities.createMenuItem' due to existing import 'jgloss.ui.util.UIUtilities' 219
Potential violation of Law of Demeter (method chain calls) 240
Potential violation of Law of Demeter (method chain calls) 258
Potential violation of Law of Demeter (method chain calls) 259
Potential violation of Law of Demeter (method chain calls) 260
Potential violation of Law of Demeter (static property access) 266
Potential violation of Law of Demeter (method chain calls) 268
Potential violation of Law of Demeter (static property access) 282
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 284
Potential violation of Law of Demeter (method chain calls) 285
Potential violation of Law of Demeter (method chain calls) 286
Potential violation of Law of Demeter (method chain calls) 295
Potential violation of Law of Demeter (method chain calls) 295
Potential violation of Law of Demeter (method chain calls) 301

jgloss/ui/LookupHistory.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 109

jgloss/ui/LookupModel.java

Violation Line
Possible God class (WMC=178, ATFD=128, TCC=0.1383838383838384) 23–741
The class 'LookupModel' has a Modified Cyclomatic Complexity of 3 (Highest = 14). 48–741
The class 'LookupModel' has a Standard Cyclomatic Complexity of 3 (Highest = 14). 48–741
Avoid using redundant field initializer for 'allDictionariesSelected' 52
Avoid using redundant field initializer for 'multiDictionaryMode' 53
Avoid using redundant field initializer for 'multiDictionarySelection' 54
Avoid using redundant field initializer for 'searchExpressionEnabled' 63
Avoid using redundant field initializer for 'distanceEnabled' 65
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_dictionaries' is not final. 71
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_searchModes' is not final. 71
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_filters' is not final. 71
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 115
Potential violation of Law of Demeter (method chain calls) 116
Potential violation of Law of Demeter (object not created locally) 135
Potential violation of Law of Demeter (object not created locally) 136
Potential violation of Law of Demeter (object not created locally) 149
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (method chain calls) 163
Potential violation of Law of Demeter (method chain calls) 168
Potential violation of Law of Demeter (method chain calls) 178
The method 'setMultiDictionarySelectionMode' has a Modified Cyclomatic Complexity of 11. 222–266
The method 'setMultiDictionarySelectionMode' has a Standard Cyclomatic Complexity of 11. 222–266
Potential violation of Law of Demeter (object not created locally) 231
Potential violation of Law of Demeter (object not created locally) 235
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_newDictionaries' is not final. 293
The method 'setDictionaries' has a Standard Cyclomatic Complexity of 12. 293–346
The method 'setDictionaries' has a Modified Cyclomatic Complexity of 12. 293–346
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 295
Potential violation of Law of Demeter (object not created locally) 297
Potential violation of Law of Demeter (object not created locally) 297
Potential violation of Law of Demeter (object not created locally) 303
Potential violation of Law of Demeter (object not created locally) 305
Potential violation of Law of Demeter (object not created locally) 309
Potential violation of Law of Demeter (object not created locally) 309
Potential violation of Law of Demeter (object not created locally) 318
Potential violation of Law of Demeter (object not created locally) 321
Potential violation of Law of Demeter (method chain calls) 326
Potential violation of Law of Demeter (method chain calls) 327
Potential violation of Law of Demeter (object not created locally) 351
The method 'selectDictionary' has a Modified Cyclomatic Complexity of 14. 365–415
The method 'selectDictionary' has a Standard Cyclomatic Complexity of 14. 365–415
Potential violation of Law of Demeter (object not created locally) 376
Potential violation of Law of Demeter (object not created locally) 377
Potential violation of Law of Demeter (object not created locally) 390
Potential violation of Law of Demeter (method chain calls) 418
Potential violation of Law of Demeter (method chain calls) 422
Potential violation of Law of Demeter (method chain calls) 432
Potential violation of Law of Demeter (method chain calls) 433
Potential violation of Law of Demeter (method chain calls) 483
Potential violation of Law of Demeter (method chain calls) 511
Potential violation of Law of Demeter (method chain calls) 515
Potential violation of Law of Demeter (method chain calls) 520
Potential violation of Law of Demeter (method chain calls) 531
Potential violation of Law of Demeter (method chain calls) 532
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_searchExpression' is not final. 550
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_distance' is not final. 561
Potential violation of Law of Demeter (object not created locally) 575
Potential violation of Law of Demeter (object not created locally) 577
Potential violation of Law of Demeter (object not created locally) 578
Potential violation of Law of Demeter (object not created locally) 578
Potential violation of Law of Demeter (object not created locally) 579
Potential violation of Law of Demeter (object not created locally) 580
Potential violation of Law of Demeter (object not created locally) 581
Potential violation of Law of Demeter (object not created locally) 587
Potential violation of Law of Demeter (object not created locally) 588
Potential violation of Law of Demeter (object not created locally) 601
Potential violation of Law of Demeter (object not created locally) 603
Potential violation of Law of Demeter (object not created locally) 604
Potential violation of Law of Demeter (object not created locally) 606
Potential violation of Law of Demeter (object not created locally) 618
Potential violation of Law of Demeter (object not created locally) 620
Potential violation of Law of Demeter (object not created locally) 621
Potential violation of Law of Demeter (object not created locally) 621
Potential violation of Law of Demeter (object not created locally) 622
Potential violation of Law of Demeter (object not created locally) 623
Potential violation of Law of Demeter (object not created locally) 624
Potential violation of Law of Demeter (object not created locally) 631
Potential violation of Law of Demeter (object not created locally) 632
Potential violation of Law of Demeter (object not created locally) 645
Potential violation of Law of Demeter (object not created locally) 651
Potential violation of Law of Demeter (method chain calls) 671
Potential violation of Law of Demeter (object not created locally) 673
Potential violation of Law of Demeter (object not created locally) 674
Potential violation of Law of Demeter (object not created locally) 674
Potential violation of Law of Demeter (object not created locally) 675–676
Potential violation of Law of Demeter (object not created locally) 677
Potential violation of Law of Demeter (object not created locally) 677
Potential violation of Law of Demeter (object not created locally) 682
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 724

jgloss/ui/LookupResultHyperlinker.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_words' is not final. 38
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_translations' is not final. 38
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_readings' is not final. 38
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_references' is not final. 39
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_allAttributes' is not final. 39
Potential violation of Law of Demeter (method chain calls) 66–67

jgloss/ui/LookupResultList.java

Violation Line
Possible God class (WMC=51, ATFD=42, TCC=0.03333333333333333) 23–420
Too many fields 62–420
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 101
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 103
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 105
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 107
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 112
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 113
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 115
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 116
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 118
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 119
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 120
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 121
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 123
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 124
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 125
StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 125
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 126
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 127
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 128
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 129
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 130
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_entryLimit' is not final. 136
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_showAllDictionaries' is not final. 140
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_styleSheet' is not final. 140
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_entryLimit' is not final. 140
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_hyperlinker' is not final. 141
Potential violation of Law of Demeter (method chain calls) 177
Potential violation of Law of Demeter (method chain calls) 178
Potential violation of Law of Demeter (static property access) 179
Potential violation of Law of Demeter (static property access) 180
Potential violation of Law of Demeter (method chain calls) 182–185
Potential violation of Law of Demeter (method chain calls) 182–185
Potential violation of Law of Demeter (method chain calls) 228
Potential violation of Law of Demeter (static property access) 281
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 282
The String literal "\n\n" appears 5 times in this file; the first occurrence is on line 283 283
Potential violation of Law of Demeter (static property access) 292–293
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 294
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 312
Unnecessary use of fully qualified name 'Level.SEVERE' due to existing import 'java.util.logging.Level' 318
Potential violation of Law of Demeter (static property access) 322
Potential violation of Law of Demeter (method chain calls) 327
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 330
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 337
Potential violation of Law of Demeter (static property access) 353
Prefer StringBuffer over += for concatenating strings 355
Potential violation of Law of Demeter (static property access) 355
Potential violation of Law of Demeter (object not created locally) 381
Potential violation of Law of Demeter (object not created locally) 386
Potential violation of Law of Demeter (object not created locally) 386
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 387
Avoid instantiation through private constructors from outside of the constructors class. 401–402
Potential violation of Law of Demeter (method chain calls) 402
Potential violation of Law of Demeter (method chain calls) 410

jgloss/ui/LookupResultMarker.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_child' is not final. 18

jgloss/ui/LookupResultProxy.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_forwardTo' is not final. 57
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_handlers' is not final. 65
Avoid throwing raw exception types. 137

jgloss/ui/MarkerListFormatter.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textBefore' is not final. 52
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textAfter' is not final. 52
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_markedText' is not final. 68
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textBefore' is not final. 75
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textAfter' is not final. 82
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 94
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 96
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 97
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 98
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_parent' is not final. 100
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_markedText' is not final. 100
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textBefore' is not final. 101
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textAfter' is not final. 101
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_group' is not final. 108
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_parent' is not final. 108
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_markedText' is not final. 113
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textBefore' is not final. 117
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_textAfter' is not final. 121
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_buffer' is not final. 130
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_length' is not final. 130
Potential violation of Law of Demeter (object not created locally) 146
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (object not created locally) 148
Potential violation of Law of Demeter (object not created locally) 149
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 164

jgloss/ui/MatchHighlighter.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 61
Use one line for each declaration, it enhances code readability. 63
Potential violation of Law of Demeter (object not created locally) 76
Avoid modifying an outer loop incrementer in an inner loop for update expression 81–83
Potential violation of Law of Demeter (object not created locally) 89

jgloss/ui/OpenRecentMenu.java

Violation Line
The class 'OpenRecentMenu' has a Modified Cyclomatic Complexity of 4 (Highest = 10). 42–210
The class 'OpenRecentMenu' has a Standard Cyclomatic Complexity of 4 (Highest = 10). 42–210
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 77
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 81
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 85
Potential violation of Law of Demeter (static property access) 115
Potential violation of Law of Demeter (object not created locally) 135
The method 'addDocument' has a Modified Cyclomatic Complexity of 10. 149–198
The method 'addDocument' has a Standard Cyclomatic Complexity of 10. 149–198
Potential violation of Law of Demeter (object not created locally) 164
Potential violation of Law of Demeter (object not created locally) 165
Potential violation of Law of Demeter (object not created locally) 166
Potential violation of Law of Demeter (object not created locally) 167
Potential violation of Law of Demeter (method chain calls) 176
Potential violation of Law of Demeter (object not created locally) 176
Potential violation of Law of Demeter (object not created locally) 181
Potential violation of Law of Demeter (object not created locally) 182
Potential violation of Law of Demeter (object not created locally) 182
Potential violation of Law of Demeter (object not created locally) 183
Potential violation of Law of Demeter (object not created locally) 193
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 195
Potential violation of Law of Demeter (object not created locally) 197
Potential violation of Law of Demeter (static property access) 197

jgloss/ui/ParserSelector.java

Violation Line
Possible God class (WMC=55, ATFD=49, TCC=0.0196078431372549) 23–410
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 79
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 81
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 86
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 103
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 105
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 106
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 107
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 111
Use ArrayList instead of Vector 164
Potential violation of Law of Demeter (method chain calls) 178
Potential violation of Law of Demeter (method chain calls) 216
Potential violation of Law of Demeter (object not created locally) 217
Potential violation of Law of Demeter (method chain calls) 217
Potential violation of Law of Demeter (object not created locally) 218
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 222
Potential violation of Law of Demeter (method chain calls) 281
Potential violation of Law of Demeter (method chain calls) 281
Potential violation of Law of Demeter (method chain calls) 281
Potential violation of Law of Demeter (method chain calls) 295
Potential violation of Law of Demeter (method chain calls) 295
Potential violation of Law of Demeter (method chain calls) 295
Potential violation of Law of Demeter (method chain calls) 310
Potential violation of Law of Demeter (method chain calls) 310
Potential violation of Law of Demeter (method chain calls) 311
Potential violation of Law of Demeter (method chain calls) 311
Potential violation of Law of Demeter (method chain calls) 328
Potential violation of Law of Demeter (method chain calls) 343
Potential violation of Law of Demeter (object not created locally) 354
Potential violation of Law of Demeter (object not created locally) 354
Potential violation of Law of Demeter (object not created locally) 357
Potential violation of Law of Demeter (object not created locally) 362
Potential violation of Law of Demeter (object not created locally) 362
Potential violation of Law of Demeter (object not created locally) 365

jgloss/ui/PartialHighlighter.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 55
Use one line for each declaration, it enhances code readability. 60
Potential violation of Law of Demeter (object not created locally) 65
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (object not created locally) 83

jgloss/ui/PerformSearchPreconditions.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 42

jgloss/ui/PreferencesFrame.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 78
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 83
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 85
Consider using varargs for methods or constructors which take an array the last parameter. 87
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_panels' is not final. 104
Consider using varargs for methods or constructors which take an array the last parameter. 104

jgloss/ui/ReplaceableItemsComboBoxModel.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_items' is not final. 48
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_selection' is not final. 59
Potential violation of Law of Demeter (object not created locally) 75
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 76
Potential violation of Law of Demeter (object not created locally) 79
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 82
Potential violation of Law of Demeter (object not created locally) 82
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 85
Potential violation of Law of Demeter (object not created locally) 86

jgloss/ui/SaveFileChooser.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 55
Potential violation of Law of Demeter (object not created locally) 68
Potential violation of Law of Demeter (static property access) 69
Potential violation of Law of Demeter (static property access) 70
Potential violation of Law of Demeter (static property access) 72–73
Potential violation of Law of Demeter (object not created locally) 73
Potential violation of Law of Demeter (static property access) 74

jgloss/ui/SelectAllDictionariesAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 20

jgloss/ui/SelectFilterAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 23
Potential violation of Law of Demeter (method chain calls) 23
Potential violation of Law of Demeter (method chain calls) 23

jgloss/ui/SelectMatchModeAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (object not created locally) 34

jgloss/ui/SelectSearchFieldAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (method chain calls) 26
Potential violation of Law of Demeter (method chain calls) 33
Potential violation of Law of Demeter (object not created locally) 34

jgloss/ui/SelectSearchModeAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 24

jgloss/ui/SetDistanceListener.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 18
Potential violation of Law of Demeter (method chain calls) 27
Potential violation of Law of Demeter (method chain calls) 27
Potential violation of Law of Demeter (static property access) 29

jgloss/ui/SimpleLookup.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 73–74
Potential violation of Law of Demeter (method chain calls) 73–74
Potential violation of Law of Demeter (object not created locally) 85–86
Potential violation of Law of Demeter (method chain calls) 85–86
Potential violation of Law of Demeter (method chain calls) 86
Potential violation of Law of Demeter (method chain calls) 86
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 117
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 119
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 121
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 122
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 124
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 125
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 126
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 128
Potential violation of Law of Demeter (object not created locally) 208
Potential violation of Law of Demeter (method chain calls) 214
Potential violation of Law of Demeter (object not created locally) 214
Potential violation of Law of Demeter (object not created locally) 215
Potential violation of Law of Demeter (method chain calls) 248
Potential violation of Law of Demeter (method chain calls) 249
Potential violation of Law of Demeter (method chain calls) 250
Potential violation of Law of Demeter (static property access) 263
Potential violation of Law of Demeter (method chain calls) 265
Potential violation of Law of Demeter (object not created locally) 269
Potential violation of Law of Demeter (object not created locally) 270
Potential violation of Law of Demeter (object not created locally) 271

jgloss/ui/SplashScreen.java

Violation Line
Perhaps 'version' could be replaced by a local variable. 55

jgloss/ui/SplitPaneManager.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_prefsPrefix' is not final. 46
Potential violation of Law of Demeter (static property access) 61
Potential violation of Law of Demeter (static property access) 71
Potential violation of Law of Demeter (object not created locally) 72
Potential violation of Law of Demeter (method chain calls) 73
Potential violation of Law of Demeter (object not created locally) 75
Potential violation of Law of Demeter (object not created locally) 77
Potential violation of Law of Demeter (object not created locally) 78
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 79
Potential violation of Law of Demeter (object not created locally) 83
Potential violation of Law of Demeter (object not created locally) 83–84
Potential violation of Law of Demeter (object not created locally) 84
Potential violation of Law of Demeter (object not created locally) 84
Potential violation of Law of Demeter (static property access) 86

jgloss/ui/StateWrapper.java

Violation Line
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_obj' is not final. 36
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_enabled' is not final. 40
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_selected' is not final. 40
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_obj' is not final. 40
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_selected' is not final. 50
Only variables that are final should contain underscores (except for underscores in standard prefix/suffix), '_enabled' is not final. 51

jgloss/ui/StopableReader.java

Violation Line
Avoid using redundant field initializer for 'stopped' 40
Use of modifier volatile is not recommended. 40
Avoid using redundant field initializer for 'charsRead' 41

jgloss/ui/StyleDialog.java

Violation Line
Possible God class (WMC=60, ATFD=65, TCC=0.03463203463203463) 23–548
The class 'StyleDialog' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 69–548
The class 'StyleDialog' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 69–548
Singleton is not thread safe 85–87
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 95
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 100
Potential violation of Law of Demeter (static property access) 103
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 107
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 109
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 110
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 112
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 113
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 115
Overridable method 'loadPreferences' called during object construction 282
Consider using varargs for methods or constructors which take an array the last parameter. 288
Potential violation of Law of Demeter (static property access) 295
Potential violation of Law of Demeter (static property access) 300
Potential violation of Law of Demeter (static property access) 302
Potential violation of Law of Demeter (static property access) 303
Potential violation of Law of Demeter (static property access) 306
Potential violation of Law of Demeter (static property access) 316
Potential violation of Law of Demeter (static property access) 318
Potential violation of Law of Demeter (static property access) 322
Potential violation of Law of Demeter (static property access) 328
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 329
Potential violation of Law of Demeter (object not created locally) 330
Potential violation of Law of Demeter (static property access) 331
Use block level rather than method level synchronization 347–372
Potential violation of Law of Demeter (static property access) 348
Potential violation of Law of Demeter (method chain calls) 360
The method 'autodetectFontsAction' has a Modified Cyclomatic Complexity of 10. 383–441
The method 'autodetectFontsAction' has a Standard Cyclomatic Complexity of 10. 383–441
Consider using varargs for methods or constructors which take an array the last parameter. 383
Potential violation of Law of Demeter (object not created locally) 394
Potential violation of Law of Demeter (static property access) 404
Potential violation of Law of Demeter (static property access) 405
Potential violation of Law of Demeter (object not created locally) 425
Potential violation of Law of Demeter (static property access) 459–460
Potential violation of Law of Demeter (static property access) 461
Potential violation of Law of Demeter (object not created locally) 474
Potential violation of Law of Demeter (object not created locally) 475
Potential violation of Law of Demeter (static property access) 496
Potential violation of Law of Demeter (static property access) 498
Potential violation of Law of Demeter (static property access) 499
Potential violation of Law of Demeter (static property access) 501
Potential violation of Law of Demeter (static property access) 502
Potential violation of Law of Demeter (static property access) 504
Potential violation of Law of Demeter (static property access) 505
Potential violation of Law of Demeter (static property access) 507
Potential violation of Law of Demeter (static property access) 508
Potential violation of Law of Demeter (static property access) 510
Potential violation of Law of Demeter (static property access) 511
Potential violation of Law of Demeter (static property access) 525
Potential violation of Law of Demeter (static property access) 544
Potential violation of Law of Demeter (static property access) 546

jgloss/ui/TextChangeDocumentListener.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 36
Potential violation of Law of Demeter (object not created locally) 36

jgloss/ui/View.java

Violation Line
Avoid short class names like View 5–11

jgloss/ui/download/ChooseDownloadDirectoryAction.java

Violation Line
Potential violation of Law of Demeter (static property access) 55
Potential violation of Law of Demeter (static property access) 56

jgloss/ui/download/DictionaryDownloadDialog.java

Violation Line
Non-static initializers are confusing 49–51
Potential violation of Law of Demeter (method chain calls) 55
Potential violation of Law of Demeter (method chain calls) 55

jgloss/ui/download/DictionaryDownloadPanel.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 109
Potential violation of Law of Demeter (method chain calls) 109

jgloss/ui/download/DictionaryDownloader.java

Violation Line
A method/constructor shouldnt explicitly throw java.lang.Exception 83
Potential violation of Law of Demeter (object not created locally) 94
Potential violation of Law of Demeter (object not created locally) 116
Potential violation of Law of Demeter (object not created locally) 131
Potential violation of Law of Demeter (method chain calls) 167
Potential violation of Law of Demeter (method chain calls) 167

jgloss/ui/download/DictionaryListLoader.java

Violation Line
A method/constructor shouldnt explicitly throw java.lang.Exception 49
Potential violation of Law of Demeter (object not created locally) 55
Potential violation of Law of Demeter (object not created locally) 56

jgloss/ui/download/DictionaryPanel.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 96
Potential violation of Law of Demeter (static property access) 97
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (method chain calls) 101
Potential violation of Law of Demeter (object not created locally) 103
Potential violation of Law of Demeter (static property access) 104
There is log block not surrounded by if 122

jgloss/ui/download/DictionarySchemaUtils.java

Violation Line
Potential violation of Law of Demeter (object not created locally) 48
Potential violation of Law of Demeter (method chain calls) 48
Potential violation of Law of Demeter (object not created locally) 49
Potential violation of Law of Demeter (object not created locally) 51
Potential violation of Law of Demeter (object not created locally) 52

jgloss/ui/download/DictionaryUnpacker.java

Violation Line
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 45

jgloss/ui/download/DownloadPanel.java

Violation Line
Switch statements should have a default label 88–100
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 112
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 114
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 116
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 118
Potential violation of Law of Demeter (static property access) 150
Potential violation of Law of Demeter (method chain calls) 159
Potential violation of Law of Demeter (method chain calls) 161
Potential violation of Law of Demeter (method chain calls) 202
Potential violation of Law of Demeter (method chain calls) 202
Potential violation of Law of Demeter (method chain calls) 208
Potential violation of Law of Demeter (method chain calls) 208

jgloss/ui/download/GZipUnpacker.java

Violation Line
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 38

jgloss/ui/download/ZipUnpacker.java

Violation Line
Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. 40
Potential violation of Law of Demeter (method chain calls) 41
Potential violation of Law of Demeter (object not created locally) 46
Potential violation of Law of Demeter (object not created locally) 47

jgloss/ui/util/HyperlinkAction.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 63
Potential violation of Law of Demeter (method chain calls) 73

jgloss/ui/util/HyperlinkLabel.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 52
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 60
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 62
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 64

jgloss/ui/util/Icons.java

Violation Line
It is somewhat confusing to have a field name matching the declaring class name 35
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 35
Avoid throwing null pointer exceptions. 46
Potential violation of Law of Demeter (method chain calls) 59

jgloss/ui/util/ShowingChangedAdapter.java

Violation Line
Potential violation of Law of Demeter (method chain calls) 40
An empty method in an abstract class should be abstract instead 51–52
An empty method in an abstract class should be abstract instead 57–58

jgloss/ui/util/SwingWorkerProgressFeedback.java

Violation Line
Potential violation of Law of Demeter (static property access) 161

jgloss/ui/util/UIUtilities.java

Violation Line
Potential violation of Law of Demeter (static property access) 80
Potential violation of Law of Demeter (static property access) 85
Potential violation of Law of Demeter (object not created locally) 87
Potential violation of Law of Demeter (static property access) 93
Potential violation of Law of Demeter (object not created locally) 95
Potential violation of Law of Demeter (method chain calls) 97–98
Potential violation of Law of Demeter (method chain calls) 97–98
Potential violation of Law of Demeter (method chain calls) 97
Potential violation of Law of Demeter (method chain calls) 97–98
Potential violation of Law of Demeter (method chain calls) 97–98
Potential violation of Law of Demeter (object not created locally) 97
Potential violation of Law of Demeter (method chain calls) 97–98
Potential violation of Law of Demeter (method chain calls) 97–98
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 99
There is log block not surrounded by if 100
Potential violation of Law of Demeter (object not created locally) 101
Potential violation of Law of Demeter (static property access) 107
Potential violation of Law of Demeter (object not created locally) 109
Potential violation of Law of Demeter (static property access) 126
Potential violation of Law of Demeter (static property access) 131
Potential violation of Law of Demeter (object not created locally) 133
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (object not created locally) 135
Potential violation of Law of Demeter (method chain calls) 135
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (method chain calls) 135–136
Potential violation of Law of Demeter (method chain calls) 135–136
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 137
There is log block not surrounded by if 138
Potential violation of Law of Demeter (object not created locally) 139
Potential violation of Law of Demeter (static property access) 145
Potential violation of Law of Demeter (object not created locally) 147
Potential violation of Law of Demeter (object not created locally) 188
Potential violation of Law of Demeter (object not created locally) 202
Potential violation of Law of Demeter (method chain calls) 230–232
Potential violation of Law of Demeter (method chain calls) 230–232
Potential violation of Law of Demeter (method chain calls) 233–234
Potential violation of Law of Demeter (method chain calls) 233–234
Potential violation of Law of Demeter (method chain calls) 244–245
Potential violation of Law of Demeter (method chain calls) 244–245
Potential violation of Law of Demeter (object not created locally) 248
Potential violation of Law of Demeter (method chain calls) 250
Potential violation of Law of Demeter (method chain calls) 250
Potential violation of Law of Demeter (object not created locally) 250
Potential violation of Law of Demeter (object not created locally) 254
Potential violation of Law of Demeter (object not created locally) 279
Potential violation of Law of Demeter (method chain calls) 279
Potential violation of Law of Demeter (method chain calls) 282
Potential violation of Law of Demeter (method chain calls) 301
Potential violation of Law of Demeter (method chain calls) 302

jgloss/ui/util/XCVManager.java

Violation Line
The class 'XCVManager' has a Modified Cyclomatic Complexity of 2 (Highest = 10). 55–266
The class 'XCVManager' has a Standard Cyclomatic Complexity of 2 (Highest = 10). 55–266
The constructor 'XCVManager' has a Modified Cyclomatic Complexity of 10. 98–156
The constructor 'XCVManager' has a Standard Cyclomatic Complexity of 10. 98–156
Potential violation of Law of Demeter (method chain calls) 170
Potential violation of Law of Demeter (method chain calls) 170
Potential violation of Law of Demeter (method chain calls) 184
Potential violation of Law of Demeter (method chain calls) 184
Use block level rather than method level synchronization 202–221
Potential violation of Law of Demeter (object not created locally) 206
Potential violation of Law of Demeter (object not created locally) 207
Potential violation of Law of Demeter (object not created locally) 209
Potential violation of Law of Demeter (object not created locally) 211
Use equals() to compare object references. 217
Use block level rather than method level synchronization 228–238
Use block level rather than method level synchronization 240–265
Potential violation of Law of Demeter (static property access) 254–255
Potential violation of Law of Demeter (method chain calls) 254–255
Potential violation of Law of Demeter (method chain calls) 254–255
Potential violation of Law of Demeter (object not created locally) 258
Potential violation of Law of Demeter (object not created locally) 258–259
Potential violation of Law of Demeter (object not created locally) 259

jgloss/util/ConfigurationProperties.java

Violation Line
Variables should start with a lowercase character, 'CONFIGURATION' starts with uppercase character. 34
The field name indicates a constant but its modifiers do not 34

jgloss/ui/download/schema/ObjectFactory.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 36–37