The following document contains the results of PMD 5.3.2.

Files

premon - eu/fbk/dkm/premon/premonitor/BankConverter.java

Violation Line
Possible God class (WMC=114, ATFD=104, TCC=0.010526315789473684) 1783
The class 'BankConverter' has a Cyclomatic Complexity of 5 (Highest = 52). 40783
The class 'BankConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 49). 40783
The class 'BankConverter' has a Standard Cyclomatic Complexity of 5 (Highest = 49). 40783
Avoid using redundant field initializer for 'nonVerbsToo' 47
Avoid using redundant field initializer for 'isOntoNotes' 48
Avoid using redundant field initializer for 'noDef' 49
Avoid using redundant field initializer for 'mapArgLabel' 70
Parameter 'properties' is not assigned and could be declared final 91
The constructor 'BankConverter' has a Cyclomatic Complexity of 10. 91137
Parameter 'path' is not assigned and could be declared final 91
Parameter 'language' is not assigned and could be declared final 91
Parameter 'resource' is not assigned and could be declared final 91
Parameter 'sink' is not assigned and could be declared final 91
Parameter 'wnInfo' is not assigned and could be declared final 92
Local variable 'vnPath' could be declared final 97
Local variable 'vnFile' could be declared final 100
Local variable 'thisClass' could be declared final 113
Local variable 'id' could be declared final 114
Local variable 'mID' could be declared final 115
Deeply nested if..then statements are hard to read 116120
Parameter 'file' is not assigned and could be declared final 139
Parameter 'isOntoNotes' is not assigned and could be declared final 139
Parameter 'onlyVerbs' is not assigned and could be declared final 139
These nested if statements could be combined 151154
Avoid really long methods. 161571
The method 'convert' has a Modified Cyclomatic Complexity of 49. 161571
The method 'convert' has a Standard Cyclomatic Complexity of 49. 161571
The method 'convert' has a Cyclomatic Complexity of 52. 161571
The method convert() has an NPath complexity of 1377860802 161571
The method convert() has an NCSS line count of 231 161571
Local variable 'jaxbContext' could be declared final 171
Local variable 'jaxbUnmarshaller' could be declared final 172
Local variable 'file' could be declared final 174
Avoid instantiating new objects inside loops 182
Local variable 'mainType' could be declared final 206
Local variable 'origLemma' could be declared final 207
Local variable 'uriOrigLemma' could be declared final 208
Local variable 'noteOrPredicate' could be declared final 210
Local variable 'predicate' could be declared final 212
Local variable 'replacedLemma' could be declared final 219
Local variable 'uLemma' could be declared final 220
Local variable 'goodLemma' could be declared final 221
Local variable 'tokens' could be declared final 223
Avoid instantiating new objects inside loops 223
Local variable 'pos' could be declared final 224
Avoid instantiating new objects inside loops 224
Local variable 'leURI' could be declared final 228
Avoid instantiating new objects inside loops 230
Local variable 'noteOrRoleset' could be declared final 236
Local variable 'roleset' could be declared final 252
Avoid instantiating new objects inside loops 257
Local variable 'aliases' could be declared final 258
Avoid instantiating new objects inside loops 260
Local variable 'alias' could be declared final 261
Deeply nested if..then statements are hard to read 262280
Local variable 'aliasLemma' could be declared final 263
Local variable 'aliasULemma' could be declared final 264
Local variable 'aliasSinglePos' could be declared final 265
Local variable 'aliasTokens' could be declared final 266
Avoid instantiating new objects inside loops 266
Avoid instantiating new objects inside loops 267
Local variable 'aliasPos' could be declared final 267
Local variable 'aliasLexicalEntry' could be declared final 270271
Avoid instantiating new objects inside loops 272273
Local variable 'aliasComplexLemma' could be declared final 272273
Avoid instantiating new objects inside loops 274
Local variable 'complexLemmaWithMappings' could be declared final 274
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 284
Local variable 'complexLemmaWithMappings' could be declared final 285
Avoid instantiating new objects inside loops 285
Avoid using if statements without curly braces 298300
Position literals first in String comparisons 299
Avoid using if statements without curly braces 299300
Deeply nested if..then statements are hard to read 299300
These nested if statements could be combined 299300
Prefer StringBuffer over += for concatenating strings 300
Local variable 'rolesetURI' could be declared final 302
Local variable 'lemma' could be declared final 311
Local variable 'lexicalEntryURI' could be declared final 313
Local variable 'clOLemma' could be declared final 314
Local variable 'uriLemma' could be declared final 315
Local variable 'mainPos' could be declared final 316
Local variable 'conceptualizationURI' could be declared final 321
Local variable 'functionMap' could be declared final 328
Local variable 'key' could be declared final 329
Local variable 'argumentURI' could be declared final 330
Avoid instantiating new objects inside loops 337
Local variable 'examples' could be declared final 337
Local variable 'rolesOrExample' could be declared final 339
Local variable 'rOrE' could be declared final 340
Local variable 'noteOrRole' could be declared final 342
Local variable 'role' could be declared final 343
Deeply nested if..then statements are hard to read 344388
Local variable 'n' could be declared final 345
Local variable 'f' could be declared final 346
Local variable 'descr' could be declared final 347
Local variable 'nf' could be declared final 349
Avoid instantiating new objects inside loops 349
Local variable 'argumentURI' could be declared final 371
Local variable 'lemma' could be declared final 379
Local variable 'example' could be declared final 404
Local variable 'exName' could be declared final 408
Local variable 'exSrc' could be declared final 409
Local variable 'myRels' could be declared final 411
Avoid instantiating new objects inside loops 411
Local variable 'myArgs' could be declared final 412
Avoid instantiating new objects inside loops 412
Local variable 'exThings' could be declared final 414415
Local variable 'thing' could be declared final 416
Local variable 'exampleURI' could be declared final 438
Local variable 'annotationSetURI' could be declared final 439
Deeply nested if..then statements are hard to read 446448
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 452
Local variable 'rel' could be declared final 457
Local variable 'origValue' could be declared final 459460
Local variable 'start' could be declared final 463
Deeply nested if..then statements are hard to read 464469
Local variable 'end' could be declared final 470
Local variable 'markableURI' could be declared final 472
Local variable 'annotationURI' could be declared final 473
Deeply nested if..then statements are hard to read 482486
Local variable 'conceptualizationURI' could be declared final 483484
Avoid instantiating new objects inside loops 495
Local variable 'nf' could be declared final 495
Local variable 'argName' could be declared final 496
Local variable 'argType' could be declared final 497
Local variable 'arg' could be declared final 503
Local variable 'value' could be declared final 504505
Local variable 'start' could be declared final 507
Deeply nested if..then statements are hard to read 508512
Local variable 'end' could be declared final 513
Local variable 'markableURI' could be declared final 515
Local variable 'annotationURI' could be declared final 516
Avoid instantiating new objects inside loops 532
Local variable 'nf' could be declared final 532
Deeply nested if..then statements are hard to read 535538
Deeply nested if..then statements are hard to read 541543
Local variable 'argumentURI' could be declared final 553554
Parameter 'source' is not assigned and could be declared final 576
Local variable 'ret' could be declared final 578
Local variable 'parts' could be declared final 582
Local variable 'part' could be declared final 583
String.trim().length()==0 is an inefficient way to validate an empty String. 584
Local variable 'matcher' could be declared final 588
Avoid reassigning parameters such as 'vnList' 600
Avoid reassigning parameters such as 'vnList' 600
Local variable 'vnClasses' could be declared final 602
Local variable 'tmpClasses' could be declared final 608
Position literals first in String comparisons 614
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 622
Local variable 'matcher' could be declared final 623
Parameter 'type' is not assigned and could be declared final 639
Parameter 'role' is not assigned and could be declared final 639
Parameter 'argumentURI' is not assigned and could be declared final 639
Parameter 'rolesetID' is not assigned and could be declared final 639
Parameter 'vnLemmas' is not assigned and could be declared final 639
Parameter 'uriLemma' is not assigned and could be declared final 639
Local variable 'rolesetURI' could be declared final 641
Local variable 'conceptualizationURI' could be declared final 642
Local variable 'vnroleList' could be declared final 644
Local variable 'vnrole' could be declared final 645
Local variable 'vnClasses' could be declared final 646
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 652
Local variable 'vnClass' could be declared final 654
Local variable 'vnLink' could be declared final 655
Local variable 'vnLemma' could be declared final 656
Local variable 'vnClassURI' could be declared final 660
Local variable 'vnConceptualizationURI' could be declared final 661662
Local variable 'vnArgumentURI' could be declared final 663
Parameter 'lemmaFromPredicate' is not assigned and could be declared final 678
Parameter 'key' is not assigned and could be declared final 687
Parameter 'argumentURI' is not assigned and could be declared final 687
Parameter 'keyURI' is not assigned and could be declared final 687
Parameter 'lemma' is not assigned and could be declared final 687
Parameter 'role' is not assigned and could be declared final 688
Parameter 'type' is not assigned and could be declared final 688
Parameter 'lexicalEntryURI' is not assigned and could be declared final 688
Parameter 'vnLemmas' is not assigned and could be declared final 688
Parameter 'rolesetID' is not assigned and could be declared final 688
Parameter 'rolesetID' is not assigned and could be declared final 704
Parameter 'exampleText' is not assigned and could be declared final 704
Avoid long parameter lists. 746747
Parameter 'textualPOS' is not assigned and could be declared final 756
Switch statements should have a default label 761771

premon - eu/fbk/dkm/premon/premonitor/ComplexLemma.java

Violation Line
Parameter 'pos' is not assigned and could be declared final 23
Parameter 'tokens' is not assigned and could be declared final 23
Parameter 'mainPos' is not assigned and could be declared final 23
Parameter 'lexiconURI' is not assigned and could be declared final 23
Parameter 'uriLemma' is not assigned and could be declared final 23
Parameter 'goodLemma' is not assigned and could be declared final 23
Parameter 'lexicalEntryURI' is not assigned and could be declared final 24

premon - eu/fbk/dkm/premon/premonitor/ComplexLemmaWithMappings.java

Violation Line
Parameter 'pbSource' is not assigned and could be declared final 19
Parameter 'lemma' is not assigned and could be declared final 23
Parameter 'rolesetID' is not assigned and could be declared final 31
Parameter 'framenet' is not assigned and could be declared final 39
Parameter 'vn' is not assigned and could be declared final 47

premon - eu/fbk/dkm/premon/premonitor/Converter.java

Violation Line
Possible God class (WMC=104, ATFD=55, TCC=0.11260504201680673) 1686
Avoid unused imports such as 'org.openrdf.query.algebra.evaluation.function.hash.MD5' 23
Too many fields 39686
Avoid using redundant field initializer for 'notadded' 51
Avoid using redundant field initializer for 'total' 51
Avoid using redundant field initializer for 'added' 51
Avoid using redundant field initializer for 'nclass' 52
Avoid using redundant field initializer for 'nrole' 52
Avoid using redundant field initializer for 'ncon' 52
Avoid using redundant field initializer for 'website' 62
Avoid using redundant field initializer for 'baseResource' 63
Avoid using redundant field initializer for 'extractExamples' 71
Avoid instantiating new objects inside loops 88
Avoid using redundant field initializer for 'onlyOne' 114
Parameter 'onlyOne' is not assigned and could be declared final 120
Parameter 'resource' is not assigned and could be declared final 124
Parameter 'wnInfo' is not assigned and could be declared final 129
Parameter 'newSink' is not assigned and could be declared final 156
Parameter 'linkList' is not assigned and could be declared final 162
Parameter 'linkString' is not assigned and could be declared final 162
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 167
Parameter 'linkString' is not assigned and could be declared final 173
Local variable 'builder' could be declared final 174
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 179
Parameter 'subject' is not assigned and could be declared final 188
Parameter 'object' is not assigned and could be declared final 188
Parameter 'predicate' is not assigned and could be declared final 188
Parameter 'statement' is not assigned and could be declared final 192
Parameter 'subject' is not assigned and could be declared final 200
Parameter 'object' is not assigned and could be declared final 200
Parameter 'graph' is not assigned and could be declared final 200
Parameter 'predicate' is not assigned and could be declared final 200
Ensure that resources like this Statement object are closed after use 208
Local variable 'statement' could be declared final 208
Parameter 'subject' is not assigned and could be declared final 216
Parameter 'objectValue' is not assigned and could be declared final 216
Parameter 'predicate' is not assigned and could be declared final 216
Parameter 'objectValue' is not assigned and could be declared final 220
Parameter 'subject' is not assigned and could be declared final 220
Parameter 'predicate' is not assigned and could be declared final 220
Parameter 'graph' is not assigned and could be declared final 221
Parameter 'subject' is not assigned and could be declared final 225
Parameter 'predicate' is not assigned and could be declared final 225
Parameter 'objectValue' is not assigned and could be declared final 225
Parameter 'useLanguage' is not assigned and could be declared final 226
Parameter 'objectValue' is not assigned and could be declared final 230
Parameter 'subject' is not assigned and could be declared final 230
Parameter 'predicate' is not assigned and could be declared final 230
Parameter 'graph' is not assigned and could be declared final 231
Parameter 'useLanguage' is not assigned and could be declared final 231
Parameter 'objectValue' is not assigned and could be declared final 248
Parameter 'subject' is not assigned and could be declared final 248
Parameter 'predicate' is not assigned and could be declared final 248
Local variable 'object' could be declared final 249
Parameter 'objectValue' is not assigned and could be declared final 253
Parameter 'predicate' is not assigned and could be declared final 253
Parameter 'subject' is not assigned and could be declared final 253
Local variable 'object' could be declared final 254
Parameter 'subject' is not assigned and could be declared final 258
Parameter 'predicate' is not assigned and could be declared final 258
Parameter 'objectValue' is not assigned and could be declared final 258
Local variable 'object' could be declared final 259
Parameter 'predicate' is not assigned and could be declared final 263
Parameter 'objectValue' is not assigned and could be declared final 263
Parameter 'subject' is not assigned and could be declared final 263
Parameter 'graph' is not assigned and could be declared final 263
Local variable 'object' could be declared final 264
Parameter 'rolesetID' is not assigned and could be declared final 268
Parameter 'rolesetID' is not assigned and could be declared final 272
Parameter 'prefix' is not assigned and could be declared final 272
Local variable 'builder' could be declared final 273
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 274
Parameter 'rolesetID' is not assigned and could be declared final 279
Parameter 'rolesetID' is not assigned and could be declared final 283
Avoid reassigning parameters such as 'prefix' 283
Local variable 'builder' could be declared final 287
Parameter 'tokens' is not assigned and could be declared final 296
Parameter 'uriLemma' is not assigned and could be declared final 296
Parameter 'goodLemma' is not assigned and could be declared final 296
Parameter 'mainPos' is not assigned and could be declared final 297
Parameter 'pos' is not assigned and could be declared final 297
Parameter 'lexiconURI' is not assigned and could be declared final 297
Local variable 'leURI' could be declared final 299
Local variable 'token' could be declared final 302
Local variable 'thisURI' could be declared final 309
Parameter 'pos' is not assigned and could be declared final 318
Parameter 'goodLemma' is not assigned and could be declared final 318
Parameter 'lexiconURI' is not assigned and could be declared final 318
Parameter 'uriLemma' is not assigned and could be declared final 318
Local variable 'posURI' could be declared final 319
Local variable 'leURI' could be declared final 323
Local variable 'formURI' could be declared final 324
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 339
Local variable 'wnPos' could be declared final 340
Local variable 'wnLemma' could be declared final 342
Local variable 'wnURI' could be declared final 343
Parameter 'type' is not assigned and could be declared final 385
Parameter 'lemma' is not assigned and could be declared final 385
Local variable 'builder' could be declared final 386
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 387
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 388
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 389
Parameter 'type' is not assigned and could be declared final 394
Parameter 'lemma' is not assigned and could be declared final 394
Local variable 'builder' could be declared final 395
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 396
Parameter 'type' is not assigned and could be declared final 401
Parameter 'lemma' is not assigned and could be declared final 401
Local variable 'builder' could be declared final 403
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 404
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 405
Position literals first in String comparisons 406
Parameter 'rolesetID' is not assigned and could be declared final 414
Parameter 'lemma' is not assigned and could be declared final 414
Parameter 'type' is not assigned and could be declared final 414
Parameter 'rolesetID' is not assigned and could be declared final 418
Parameter 'type' is not assigned and could be declared final 418
Parameter 'lemma' is not assigned and could be declared final 418
Parameter 'prefix' is not assigned and could be declared final 418
Parameter 'type' is not assigned and could be declared final 427
Parameter 'lemma' is not assigned and could be declared final 427
Parameter 'rolesetID' is not assigned and could be declared final 427
Local variable 'posURI' could be declared final 429
Local variable 'builder' could be declared final 431
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 432
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 433
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 434
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 435
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 436
Parameter 'prefix' is not assigned and could be declared final 441
Parameter 'rolesetID' is not assigned and could be declared final 441
Parameter 'argName' is not assigned and could be declared final 441
Local variable 'builder' could be declared final 442
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 443
Parameter 'argName' is not assigned and could be declared final 448
Parameter 'rolesetID' is not assigned and could be declared final 448
Parameter 'argName' is not assigned and could be declared final 456
Parameter 'rolesetID' is not assigned and could be declared final 456
Local variable 'builder' could be declared final 457
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 458
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 459
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 460
Parameter 'prefix' is not assigned and could be declared final 465
Parameter 'argName' is not assigned and could be declared final 465
Parameter 'rolesetID' is not assigned and could be declared final 465
Local variable 'builder' could be declared final 466
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 467
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 468
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 469
Parameter 'class1' is not assigned and could be declared final 474
Parameter 'class2' is not assigned and could be declared final 474
Parameter 'conceptualization1' is not assigned and could be declared final 474
Parameter 'conceptualization2' is not assigned and could be declared final 475
Local variable 'classes' could be declared final 480
Local variable 'conceptualizations' could be declared final 483
Parameter 'conceptualization1' is not assigned and could be declared final 499
Parameter 'class1' is not assigned and could be declared final 499
Parameter 'class2' is not assigned and could be declared final 499
Parameter 'argument2' is not assigned and could be declared final 500
Parameter 'argument1' is not assigned and could be declared final 500
Parameter 'conceptualization2' is not assigned and could be declared final 500
Local variable 'classes' could be declared final 507
Local variable 'conceptualizations' could be declared final 510
Local variable 'arguments' could be declared final 515
Parameter 'arguments' is not assigned and could be declared final 532
Parameter 'conceptualizations' is not assigned and could be declared final 532
Parameter 'classes' is not assigned and could be declared final 532
The method addMappings() has an NPath complexity of 1000 532556
Local variable 'nClasses' could be declared final 533
Local variable 'nConceptualization' could be declared final 534
Local variable 'nArguments' could be declared final 535
Parameter 'uris' is not assigned and could be declared final 558
Parameter 'prefix' is not assigned and could be declared final 567
Parameter 'uris' is not assigned and could be declared final 567
Parameter 'parentMapping' is not assigned and could be declared final 567
Parameter 'suffix' is not assigned and could be declared final 567
Local variable 'cluster' could be declared final 568
Local variable 'uri' could be declared final 569
Parameter 'parentMapping' is not assigned and could be declared final 576
Parameter 'prefix' is not assigned and could be declared final 576
Parameter 'uris' is not assigned and could be declared final 576
Parameter 'suffix' is not assigned and could be declared final 576
Local variable 'cluster' could be declared final 577
Local variable 'uri' could be declared final 578
Parameter 'mapping' is not assigned and could be declared final 585
Parameter 'parentMapping' is not assigned and could be declared final 585
Parameter 'suffix' is not assigned and could be declared final 585
Parameter 'prefix' is not assigned and could be declared final 585
Local variable 'mappingURI' could be declared final 592
Local variable 'uri' could be declared final 610
Parameter 'mapping' is not assigned and could be declared final 617
Parameter 'prefix' is not assigned and could be declared final 617
Parameter 'suffix' is not assigned and could be declared final 617
Local variable 'strings' could be declared final 618
Local variable 'uri' could be declared final 619
Local variable 'hash' could be declared final 623
Local variable 'builder' could be declared final 625
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 626
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 627
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 628
Parameter 'arg' is not assigned and could be declared final 633
Avoid reassigning parameters such as 'text' 641
Avoid reassigning parameters such as 'text' 646
Avoid reassigning parameters such as 'namespace' 646
Parameter 'base' is not assigned and could be declared final 652
Parameter 'end' is not assigned and could be declared final 652
Parameter 'start' is not assigned and could be declared final 652
Local variable 'markableURI' could be declared final 653
Consider simply returning the value vs storing it in local variable 'markableURI' 654
Parameter 'exampleURI' is not assigned and could be declared final 659
Parameter 'addendum' is not assigned and could be declared final 659
Local variable 'builder' could be declared final 660
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 661

premon - eu/fbk/dkm/premon/premonitor/EsoConverter.java

Violation Line
Avoid unused imports such as 'eu.fbk.dkm.premon.vocab.FBMETA' 4
Avoid unused imports such as 'eu.fbk.rdfpro.util.Statements' 11
Avoid unused imports such as 'org.openrdf.model.vocabulary.RDFS' 14
Avoid unused imports such as 'org.openrdf.query.TupleQuery' 17
Private field 'role_FE_query' could be made final; it is only initialized in the declaration or constructor. 35
Private field 'fnLinks' could be made final; it is only initialized in the declaration or constructor. 45
Parameter 'wnInfo' is not assigned and could be declared final 48
Parameter 'premon_prop' is not assigned and could be declared final 87
Parameter 'model' is not assigned and could be declared final 87
Local variable 'fnLink' could be declared final 89
Local variable 'query' could be declared final 91
Local variable 'iterator' could be declared final 92
Local variable 'resultRow' could be declared final 96
Local variable 'fnFE' could be declared final 97
Local variable 'esoRole' could be declared final 98
Parameter 'model' is not assigned and could be declared final 107
Parameter 'frameBroad_prop' is not assigned and could be declared final 107
Parameter 'premon_prop' is not assigned and could be declared final 107
Local variable 'fnLink' could be declared final 109
Local variable 'iter_fb' could be declared final 111
Local variable 'stmt' could be declared final 115
Ensure that resources like this Statement object are closed after use 115
Local variable 'frame' could be declared final 122
Local variable 'fnFrameURI' could be declared final 123
Avoid instantiating new objects inside loops 144
Avoid instantiating new objects inside loops 148159
Avoid unused local variables such as 'p'. 152
Avoid unused local variables such as 'o'. 153
Parameter 'textualPOS' is not assigned and could be declared final 170

premon - eu/fbk/dkm/premon/premonitor/FramebaseCleanerConverter.java

Violation Line
Possible God class (WMC=89, ATFD=83, TCC=0.0) 1443
The class 'FramebaseCleanerConverter' has a Cyclomatic Complexity of 5 (Highest = 11). 22443
The method 'compare' has a Cyclomatic Complexity of 11. 3461
Parameter 'wnInfo' is not assigned and could be declared final 66
Use block level rather than method level synchronization 126145
The class 'URIInfo' has a Standard Cyclomatic Complexity of 5 (Highest = 18). 177438
The class 'URIInfo' has a Cyclomatic Complexity of 5 (Highest = 18). 177438
The class 'URIInfo' has a Modified Cyclomatic Complexity of 5 (Highest = 18). 177438
The method 'update' has a Modified Cyclomatic Complexity of 15. 242289
The method 'update' has a Cyclomatic Complexity of 16. 242289
The method 'update' has a Standard Cyclomatic Complexity of 15. 242289
Avoid empty if statements 256265
The method 'emit' has a Standard Cyclomatic Complexity of 18. 291364
The method 'emit' has a Modified Cyclomatic Complexity of 18. 291364
The method emit() has an NPath complexity of 18432 291364
The method 'emit' has a Cyclomatic Complexity of 18. 291364
Avoid reassigning parameters such as 'set' 406
Avoid reassigning parameters such as 'set' 414
Parameter 'textualPOS' is not assigned and could be declared final 440

premon - eu/fbk/dkm/premon/premonitor/FramebaseConverter.java

Violation Line
Possible God class (WMC=57, ATFD=55, TCC=0.0) 1353
The class 'FramebaseConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 15). 35353
The class 'FramebaseConverter' has a Cyclomatic Complexity of 6 (Highest = 19). 35353
The class 'FramebaseConverter' has a Standard Cyclomatic Complexity of 5 (Highest = 15). 35353
The method 'getPosURI' has a Cyclomatic Complexity of 11. 5785
The method 'getPosURI' has a Standard Cyclomatic Complexity of 11. 5785
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 62
Position literals first in String comparisons for EqualsIgnoreCase 88
Position literals first in String comparisons for EqualsIgnoreCase 88
Position literals first in String comparisons for EqualsIgnoreCase 89
Position literals first in String comparisons for EqualsIgnoreCase 89
Position literals first in String comparisons for EqualsIgnoreCase 90
Position literals first in String comparisons for EqualsIgnoreCase 90
Position literals first in String comparisons for EqualsIgnoreCase 91
The method 'readFramebaseTriples' has a Cyclomatic Complexity of 10. 112140
Avoid instantiating new objects inside loops 116
Avoid instantiating new objects inside loops 119133
The method emitPBNBAlignments() has an NPath complexity of 41492 192298
Avoid really long methods. 192298
The method 'emitPBNBAlignments' has a Cyclomatic Complexity of 19. 192298
The method 'emitPBNBAlignments' has a Standard Cyclomatic Complexity of 15. 192298
The method 'emitPBNBAlignments' has a Modified Cyclomatic Complexity of 15. 192298
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 217
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 270
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 302
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 303
Avoid appending characters as strings in StringBuffer.append. 304
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 304
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 312
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 313
Avoid appending characters as strings in StringBuffer.append. 314
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 314
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 315
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 316
Avoid appending characters as strings in StringBuffer.append. 316
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 317
Position literals first in String comparisons 317
Avoid appending characters as strings in StringBuffer.append. 318
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 318
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 319
Avoid appending characters as strings in StringBuffer.append. 320
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 320
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 328
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 329
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 329
Avoid appending characters as strings in StringBuffer.append. 330
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 330
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 333
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 335
Position literals first in String comparisons 344
Position literals first in String comparisons 346

premon - eu/fbk/dkm/premon/premonitor/FramenetConverter.java

Violation Line
Possible God class (WMC=149, ATFD=219, TCC=0.0) 11063
The class 'FramenetConverter' has a Modified Cyclomatic Complexity of 6 (Highest = 84). 321063
The class 'FramenetConverter' has a Standard Cyclomatic Complexity of 8 (Highest = 103). 321063
The class 'FramenetConverter' has a Cyclomatic Complexity of 9 (Highest = 114). 321063
Private field 'retroMappings' could be made final; it is only initialized in the declaration or constructor. 36
Avoid using redundant field initializer for 'retroMappings' 36
Private field 'thisVersion' could be made final; it is only initialized in the declaration or constructor. 47
Avoid using redundant field initializer for 'thisVersion' 47
Parameter 'path' is not assigned and could be declared final 49
Parameter 'properties' is not assigned and could be declared final 49
Parameter 'sink' is not assigned and could be declared final 49
Parameter 'wnInfo' is not assigned and could be declared final 49
The method 'convert' has a Standard Cyclomatic Complexity of 103. 71882
The method convert() has an NPath complexity of 2036440835 71882
The method convert() has an NCSS line count of 510 71882
The method 'convert' has a Cyclomatic Complexity of 114. 71882
The method 'convert' has a Modified Cyclomatic Complexity of 84. 71882
Avoid really long methods. 71882
Local variable 'key' could be declared final 77
Local variable 'value' could be declared final 78
Position literals first in String comparisons 90
Local variable 'added' could be declared final 97
Local variable 'changed' could be declared final 98
Local variable 'diff' could be declared final 108
Local variable 'diff' could be declared final 112
Local variable 'r15' could be declared final 113
Local variable 'r16' could be declared final 114
Local variable 'diff' could be declared final 119
Local variable 'diff' could be declared final 124
Local variable 'f16' could be declared final 125
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 130
Local variable 'r15' could be declared final 131
Local variable 'r16' could be declared final 132
Local variable 'statusTypes' could be declared final 141
Local variable 'statusType' could be declared final 142
Local variable 'semTypes' could be declared final 149
Local variable 'semType' could be declared final 150
Local variable 'typesFreqs' could be declared final 155
Local variable 'typesFreqsFER' could be declared final 156
Local variable 'frRelationTypes' could be declared final 160
Local variable 'frRelationType' could be declared final 161
Local variable 'name' could be declared final 162
Switch statements should have a default label 167202
Local variable 'frameRelations' could be declared final 209
Local variable 'frameRelation' could be declared final 210
Local variable 'subFrameName' could be declared final 211
Local variable 'superFrameName' could be declared final 212
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 215
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 216
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 218
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 219
Local variable 'feRelations' could be declared final 223
Local variable 'feRelation' could be declared final 224
Local variable 'relURI' could be declared final 227
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 229
Local variable 'arg1' could be declared final 229
Local variable 'arg2' could be declared final 230
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 230
Local variable 'semTypesFreq' could be declared final 249
Local variable 'semTypesForFrame' could be declared final 250
Local variable 'lus' could be declared final 251
These nested if statements could be combined 257259
Local variable 'element' could be declared final 266
Local variable 'cBy' could be declared final 267
Local variable 'cDate' could be declared final 268
Local variable 'identifier' could be declared final 269
Local variable 'frameName' could be declared final 270
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 272
Local variable 'lcFrameName' could be declared final 272
Local variable 'frameURI' could be declared final 274
Avoid unused local variables such as 'classMappingURI'. 276
Local variable 'classMappingURI' could be declared final 276
These nested if statements could be combined 278290
Local variable 'toMap' could be declared final 279
Local variable 'oldFrameURI' could be declared final 281
Static DateFormatter objects should be accessed in a synchronized manner 293
Local variable 'date' could be declared final 293
Local variable 'definition' could be declared final 295
Local variable 'defText' could be declared final 296
Local variable 'elements' could be declared final 298
Local variable 'cbyURI' could be declared final 301
Local variable 'FEs' could be declared final 310
Avoid instantiating new objects inside loops 310
Local variable 'fe' could be declared final 312
Local variable 'feName' could be declared final 314
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 318
Local variable 'lcFeName' could be declared final 318
Local variable 'completeRole' could be declared final 320
Local variable 'toMap' could be declared final 322
Local variable 'argumentURI' could be declared final 324
Local variable 'parts' could be declared final 326
Local variable 'oldFrameURI' could be declared final 327
Local variable 'oldArgumentURI' could be declared final 328
Local variable 'oldFrameURI' could be declared final 338
Local variable 'oldArgumentURI' could be declared final 339
Local variable 'coreType' could be declared final 349
Local variable 'feCBy' could be declared final 350
Local variable 'feCDate' could be declared final 351
Local variable 'feIdentifier' could be declared final 352
Local variable 'abbrev' could be declared final 353
Local variable 'feDate' could be declared final 355
Static DateFormatter objects should be accessed in a synchronized manner 355
Local variable 'feDefinition' could be declared final 357
Local variable 'feDefText' could be declared final 358
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 360
Local variable 'argumentURI' could be declared final 360
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 360
Switch statements should have a default label 362376
Local variable 'feCByURI' could be declared final 378
Local variable 'subElem' could be declared final 394
Local variable 'seName' could be declared final 395
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 396
Local variable 'subElURI' could be declared final 396
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 396
Local variable 'subElem' could be declared final 401
Local variable 'seName' could be declared final 402
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 403
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 403
Local variable 'subElURI' could be declared final 403
Local variable 'members' could be declared final 410
Local variable 'coresetURI' could be declared final 413
Local variable 'memberFEs' could be declared final 417
Local variable 'memberFE' could be declared final 418
Local variable 'mName' could be declared final 419
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 422
Local variable 'itemURI' could be declared final 422
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 422
Local variable 'frameRelation' could be declared final 429
Local variable 'relatedFrames' could be declared final 430
Local variable 'type' could be declared final 431
Switch statements should have a default label 434459
Local variable 'relatedFrame' could be declared final 465
Local variable 'relatedFrameName' could be declared final 466
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 472
Local variable 'lexUnit' could be declared final 479
Local variable 'leCBy' could be declared final 481
Local variable 'leCDate' could be declared final 482
Local variable 'leIdentifier' could be declared final 483
Local variable 'name' could be declared final 487
Local variable 'status' could be declared final 489
Local variable 'pos' could be declared final 490
Local variable 'leDefinition' could be declared final 491
Avoid instantiating new objects inside loops 494
Avoid unused local variables such as 'builder'. 494
Local variable 'builder' could be declared final 494
Local variable 'lexemes' could be declared final 495
Avoid instantiating new objects inside loops 497
Local variable 'lexemeList' could be declared final 497
Local variable 'posList' could be declared final 498
Avoid instantiating new objects inside loops 498
Local variable 'lexeme' could be declared final 499
Local variable 'lemmaName' could be declared final 500
Local variable 'lemmaPos' could be declared final 501
Local variable 'goodLemma' could be declared final 513
Local variable 'uriLemma' could be declared final 514
Local variable 'lexicalEntryURI' could be declared final 515
Local variable 'luURI' could be declared final 516
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 516
Local variable 'stElements' could be declared final 537
Local variable 'statusURI' could be declared final 548
Local variable 'leCByURI' could be declared final 551
Static DateFormatter objects should be accessed in a synchronized manner 552
Local variable 'leDate' could be declared final 552
String.trim().length()==0 is an inefficient way to validate an empty String. 568
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 571
Local variable 'argumentURI' could be declared final 571572
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 572
Local variable 'uri' could be declared final 591
Local variable 'frameName' could be declared final 628
Local variable 'luID' could be declared final 629
Local variable 'frameURI' could be declared final 637
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 637
Local variable 'luURI' could be declared final 638
Avoid unused local variables such as 'luURI'. 638
Local variable 'example' could be declared final 642
Local variable 'tempStatements' could be declared final 649
Avoid instantiating new objects inside loops 649
Local variable 'tempSink' could be declared final 650
Local variable 'id' could be declared final 660
Local variable 'exampleURI' could be declared final 661
Local variable 'text' could be declared final 663
Local variable 'layers' could be declared final 665
Local variable 'starts' could be declared final 667
Avoid instantiating new objects inside loops 667
Avoid instantiating new objects inside loops 668
Local variable 'ends' could be declared final 668
Local variable 'matcher' could be declared final 670
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 677
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 677
Local variable 'layer' could be declared final 683
Local variable 'layerName' could be declared final 684
Position literals first in String comparisons 690
Local variable 'labels' could be declared final 691
Local variable 'label' could be declared final 696
Avoid empty catch blocks 703705
These nested if statements could be combined 718720
These nested if statements could be combined 723725
Local variable 'markableURI' could be declared final 742
Local variable 'anchor' could be declared final 743
Local variable 'aURI' could be declared final 745
Local variable 'layer' could be declared final 779
Local variable 'layerName' could be declared final 780
Position literals first in String comparisons 786
Local variable 'labels' could be declared final 787
Local variable 'label' could be declared final 788
Local variable 'roleName' could be declared final 789
Local variable 'argumentURI' could be declared final 790791
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 790
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 791
Avoid empty catch blocks 804806
Local variable 'aURI' could be declared final 823
Avoid empty if statements 830832
Local variable 'markableURI' could be declared final 834
Local variable 'statement' could be declared final 861
Ensure that resources like this Statement object are closed after use 861
Parameter 'semTypesFreq' is not assigned and could be declared final 884
Parameter 'stElements' is not assigned and could be declared final 884
Parameter 'semTypesForFrame' is not assigned and could be declared final 885
Parameter 'frameURI' is not assigned and could be declared final 885
Parameter 'baseURI' is not assigned and could be declared final 885
Local variable 'stElement' could be declared final 886
Local variable 'LUSemType' could be declared final 887
Parameter 'cBy' is not assigned and could be declared final 900
Local variable 'cbyURI' could be declared final 901
Parameter 'cBy' is not assigned and could be declared final 906
Local variable 'builder' could be declared final 907
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 908
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 909
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 909
Parameter 'exampleID' is not assigned and could be declared final 914
Local variable 'builder' could be declared final 915
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 916
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 917
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 918
Parameter 'semType' is not assigned and could be declared final 923
Local variable 'name' could be declared final 924
Local variable 'abbrev' could be declared final 925
Local variable 'id' could be declared final 926
Local variable 'definition' could be declared final 928
Local variable 'supID' could be declared final 929
Local variable 'superTypeName' could be declared final 930
Local variable 'semTypeURI' could be declared final 932
Local variable 'superSemTypeURI' could be declared final 941
Parameter 'name' is not assigned and could be declared final 946
Avoid reassigning parameters such as 'name' 950
Avoid reassigning parameters such as 'name' 950
Avoid reassigning parameters such as 'name' 950
Avoid reassigning parameters such as 'prefix' 950
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 955
Local variable 'builder' could be declared final 959
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 960
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 961
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 962
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 963
Parameter 'statusType' is not assigned and could be declared final 968
Local variable 'name' could be declared final 969
Local variable 'description' could be declared final 970
Local variable 'statusURI' could be declared final 972
Parameter 'frameName' is not assigned and could be declared final 978
Parameter 'luName' is not assigned and could be declared final 978
Parameter 'pos' is not assigned and could be declared final 978
Parameter 'frameName' is not assigned and could be declared final 982
Avoid reassigning parameters such as 'prefix' 982
Parameter 'luName' is not assigned and could be declared final 982
Parameter 'pos' is not assigned and could be declared final 982
Local variable 'builder' could be declared final 986
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 987
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 988
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 989
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 994
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 996
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 997
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 998
Parameter 'name' is not assigned and could be declared final 1003
Avoid reassigning parameters such as 'prefix' 1007
Parameter 'name' is not assigned and could be declared final 1007
Local variable 'builder' could be declared final 1011
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 1012
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 1013
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 1014
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 1015
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 1015
The method 'getPosURI' has a Standard Cyclomatic Complexity of 14. 10201054
Avoid reassigning parameters such as 'pos' 1020
The method 'getPosURI' has a Cyclomatic Complexity of 14. 10201054
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 1022
Switch statements should have a default label 10241051
Parameter 'arg' is not assigned and could be declared final 1060

premon - eu/fbk/dkm/premon/premonitor/NombankConverter.java

Violation Line
Parameter 'properties' is not assigned and could be declared final 24
Parameter 'path' is not assigned and could be declared final 24
Parameter 'sink' is not assigned and could be declared final 24
Parameter 'wnInfo' is not assigned and could be declared final 24
Parameter 'markableURI' is not assigned and could be declared final 33
Parameter 'argName' is not assigned and could be declared final 33
Parameter 'argType' is not assigned and could be declared final 33
Parameter 'rolesetID' is not assigned and could be declared final 34
Parameter 'asURI' is not assigned and could be declared final 34
Parameter 'f' is not assigned and could be declared final 34
Local variable 'argumentURI' could be declared final 35
Avoid declaring a variable if it is unreferenced before a possible exit point. 35
Local variable 'fType' could be declared final 40
Position literals first in String comparisons 43
Position literals first in String comparisons 47
Parameter 'argType' is not assigned and could be declared final 73
Parameter 'argName' is not assigned and could be declared final 73
Parameter 'markableURI' is not assigned and could be declared final 73
Parameter 'inflection' is not assigned and could be declared final 97
Parameter 'exampleURI' is not assigned and could be declared final 97
Document empty method body 9799
Parameter 'lemma' is not assigned and could be declared final 101
Parameter 'type' is not assigned and could be declared final 101
Avoid long parameter lists. 105106
Parameter 'f' is not assigned and could be declared final 105
Parameter 'argumentURI' is not assigned and could be declared final 105
Parameter 'argName' is not assigned and could be declared final 105
Parameter 'argType' is not assigned and could be declared final 105
Parameter 'type' is not assigned and could be declared final 106
Parameter 'lexicalEntryURI' is not assigned and could be declared final 106
Parameter 'lemma' is not assigned and could be declared final 106
Parameter 'roleset' is not assigned and could be declared final 106
Parameter 'rolesetID' is not assigned and could be declared final 106
Parameter 'role' is not assigned and could be declared final 106
Local variable 'vnLemmas' could be declared final 125
Local variable 'matcher' could be declared final 126
Local variable 'matchers' could be declared final 134
Local variable 'rolesetURI' could be declared final 136
Local variable 'conceptualizationURI' could be declared final 137
Local variable 'matcher' could be declared final 139
Local variable 'pbPredicate' could be declared final 141
Local variable 'source' could be declared final 144
Local variable 'pbLink' could be declared final 149
Local variable 'pbRolesetURI' could be declared final 154
Local variable 'pbConceptualizationURI' could be declared final 155
Local variable 'pbArgumentURI' could be declared final 156
Parameter 'code' is not assigned and could be declared final 163
Local variable 'matcher' could be declared final 172
Position literals first in String comparisons 177
Position literals first in String comparisons 182
Parameter 'uriLemma' is not assigned and could be declared final 191
Parameter 'type' is not assigned and could be declared final 191
Parameter 'complexLemmaWithMappings' is not assigned and could be declared final 191
Parameter 'conceptualizationURI' is not assigned and could be declared final 191
Local variable 'lemmas' could be declared final 193
Local variable 'rolesetID' could be declared final 195
Local variable 'rolesetURI' could be declared final 196
Local variable 'matchers' could be declared final 199
Local variable 'matcher' could be declared final 200
Local variable 'pbLemma' could be declared final 201
Local variable 'lemma' could be declared final 202
Local variable 'pbPredicate' could be declared final 204
Local variable 'pbLink' could be declared final 206
Local variable 'pbRolesetURI' could be declared final 207
Local variable 'pbConceptualizationURI' could be declared final 208
Local variable 'vnClasses' could be declared final 214
Local variable 'vnClass' could be declared final 215
Local variable 'vnLink' could be declared final 216
Local variable 'lemma' could be declared final 217
Local variable 'vnClassURI' could be declared final 218
Local variable 'vnConceptualizationURI' could be declared final 219

premon - eu/fbk/dkm/premon/premonitor/PredMatConverter.java

Violation Line
Possible God class (WMC=64, ATFD=34, TCC=0.0) 1326
The class 'PredMatConverter' has a Standard Cyclomatic Complexity of 8 (Highest = 32). 26326
Too many fields 26326
The class 'PredMatConverter' has a Cyclomatic Complexity of 9 (Highest = 32). 26326
The class 'PredMatConverter' has a Modified Cyclomatic Complexity of 8 (Highest = 32). 26326
This final field could be made static 40
Private field 'vnMap' could be made final; it is only initialized in the declaration or constructor. 47
Private field 'vnLinks' could be made final; it is only initialized in the declaration or constructor. 49
Private field 'fnLinks' could be made final; it is only initialized in the declaration or constructor. 50
Private field 'pbLinks' could be made final; it is only initialized in the declaration or constructor. 51
Parameter 'properties' is not assigned and could be declared final 55
Parameter 'wnInfo' is not assigned and could be declared final 55
Parameter 'sink' is not assigned and could be declared final 55
The constructor 'PredMatConverter' has a Cyclomatic Complexity of 10. 55102
Parameter 'path' is not assigned and could be declared final 55
Local variable 'vnPath' could be declared final 63
Local variable 'vnFile' could be declared final 66
Local variable 'thisClass' could be declared final 79
Local variable 'id' could be declared final 80
Local variable 'mID' could be declared final 81
Deeply nested if..then statements are hard to read 8286
The method 'convert' has a Modified Cyclomatic Complexity of 32. 104226
The method 'convert' has a Standard Cyclomatic Complexity of 32. 104226
The method 'convert' has a Cyclomatic Complexity of 32. 104226
Avoid really long methods. 104226
Local variable 'PredMat' could be declared final 108
Local variable 'classes' could be declared final 112
Local variable 'conceptualizations' could be declared final 113
Local variable 'arguments' could be declared final 114
Local variable 'parser' could be declared final 116
Local variable 'pme' could be declared final 118
Local variable 'lang' could be declared final 120
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 129
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 132
Prefer StringBuffer over += for concatenating strings 134
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 136
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 140
Prefer StringBuffer over += for concatenating strings 149
Prefer StringBuffer over += for concatenating strings 149
Local variable 'hash' could be declared final 151
Local variable 'vnLink' could be declared final 160
Local variable 'fnLink' could be declared final 161
Local variable 'pbLink' could be declared final 162
Local variable 'vnID' could be declared final 163
Local variable 'vnClassURI' could be declared final 164
Local variable 'vnConceptualizationURI' could be declared final 165166
Local variable 'vnArgURI' could be declared final 167
Local variable 'fnFrameURI' could be declared final 169
Local variable 'fnConceptualizationURI' could be declared final 170171
Local variable 'fnArgURI' could be declared final 172
Local variable 'pbRolesetURI' could be declared final 174
Local variable 'pbConceptualizationURI' could be declared final 175177
String.indexOf(char) is faster than String.indexOf(String). 176
Local variable 'pbArgURI' could be declared final 178
Local variable 'wnSenseURI' could be declared final 180181
String.indexOf(char) is faster than String.indexOf(String). 181
Parameter 'textualPOS' is not assigned and could be declared final 228
Parameter 'str' is not assigned and could be declared final 236
Local variable 'strNoNS' could be declared final 237
Parameter 'vnSc' is not assigned and could be declared final 245
Prefer StringBuffer over += for concatenating strings 252
Parameter 'uriLemma' is not assigned and could be declared final 258
Parameter 'wnSense' is not assigned and could be declared final 258
The method 'uriForWnSense' has a Cyclomatic Complexity of 12. 258324
The method 'uriForWnSense' has a Standard Cyclomatic Complexity of 11. 258324
The method 'uriForWnSense' has a Modified Cyclomatic Complexity of 11. 258324
Local variable 'lemma' could be declared final 287
Local variable 'wnuri' could be declared final 300
Local variable 'lexicalEntryURI' could be declared final 312

premon - eu/fbk/dkm/premon/premonitor/Premonitor.java

Violation Line
Possible God class (WMC=199, ATFD=276, TCC=0.0) 11141
The class 'Premonitor' has a Modified Cyclomatic Complexity of 27 (Highest = 39). 871141
The class 'Premonitor' has a Standard Cyclomatic Complexity of 27 (Highest = 39). 871141
The class 'Premonitor' has a Cyclomatic Complexity of 31 (Highest = 49). 871141
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. 871141
Avoid unused private fields such as 'WN_OLD_SENSE'. 104
The method 'main' has a Cyclomatic Complexity of 43. 109399
Avoid really long methods. 109399
The method main() has an NPath complexity of 116760321 109399
The method 'main' has a Modified Cyclomatic Complexity of 39. 109399
The method main() has an NCSS line count of 132 109399
Consider using varargs for methods or constructors which take an array the last parameter. 109
The method 'main' has a Standard Cyclomatic Complexity of 39. 109399
Deeply nested if..then statements are hard to read 192201
These nested if statements could be combined 194200
Deeply nested if..then statements are hard to read 204222
Deeply nested if..then statements are hard to read 244246
Avoid instantiating new objects inside loops 245
Prefer StringBuffer over += for concatenating strings 286
Avoid instantiating new objects inside loops 288
Avoid instantiating new objects inside loops 300
Avoid instantiating new objects inside loops 302
Avoid instantiating new objects inside loops 304330
Use block level rather than method level synchronization 312328
Prefer StringBuffer over += for concatenating strings 376
The method 'emit' has a Modified Cyclomatic Complexity of 38. 401628
The method 'emit' has a Cyclomatic Complexity of 49. 401628
The method emit() has an NPath complexity of 6528600 401628
Avoid really long methods. 401628
The method emit() has an NCSS line count of 153 401628
The method 'emit' has a Standard Cyclomatic Complexity of 38. 401628
Ensure that resources like this Statement object are closed after use 414
Ensure that resources like this Statement object are closed after use 444
Ensure that resources like this Statement object are closed after use 465
Deeply nested if..then statements are hard to read 476478
Deeply nested if..then statements are hard to read 480484
These nested if statements could be combined 481483
Ensure that resources like this Statement object are closed after use 498
Avoid instantiating new objects inside loops 516517
Avoid instantiating new objects inside loops 533534
Avoid instantiating new objects inside loops 544545
Avoid instantiating new objects inside loops 563
The method 'emit' has a Modified Cyclomatic Complexity of 17. 630717
The method 'emit' has a Cyclomatic Complexity of 19. 630717
The method 'emit' has a Standard Cyclomatic Complexity of 17. 630717
The method emit() has an NPath complexity of 69840 630717
This call to Collection.toArray() may be optimizable 656
Ensure that resources like this Statement object are closed after use 704
Avoid instantiating new objects inside loops 705706
The method filterMappings() has an NPath complexity of 4770480 719900
The method 'filterMappings' has a Modified Cyclomatic Complexity of 39. 719900
The method 'filterMappings' has a Cyclomatic Complexity of 43. 719900
The method filterMappings() has an NCSS line count of 114 719900
The method 'filterMappings' has a Standard Cyclomatic Complexity of 39. 719900
Avoid really long methods. 719900
Ensure that resources like this Statement object are closed after use 726
Ensure that resources like this Statement object are closed after use 730
Avoid instantiating new objects inside loops 740741
Ensure that resources like this Statement object are closed after use 755
Ensure that resources like this Statement object are closed after use 785
Ensure that resources like this Statement object are closed after use 790
Ensure that resources like this Statement object are closed after use 812
Ensure that resources like this Statement object are closed after use 837
Local variable 'stmt' could be declared final 837
Ensure that resources like this Statement object are closed after use 862
Local variable 'rel_stmt' could be declared final 877
Ensure that resources like this Statement object are closed after use 877
The class 'InstanceStatistics' has a Standard Cyclomatic Complexity of 19 (Highest = 18). 9061015
The class 'InstanceStatistics' has a Modified Cyclomatic Complexity of 19 (Highest = 18). 9061015
The class 'InstanceStatistics' has a Cyclomatic Complexity of 37 (Highest = 36). 9061015
The constructor 'InstanceStatistics' has a Modified Cyclomatic Complexity of 18. 9261013
The constructor 'InstanceStatistics' has a Cyclomatic Complexity of 36. 9261013
The constructor 'InstanceStatistics' has a Standard Cyclomatic Complexity of 18. 9261013
Ensure that resources like this Statement object are closed after use 970
Ensure that resources like this Statement object are closed after use 982
Ensure that resources like this Statement object are closed after use 994
The class 'MappingStatistics' has a Standard Cyclomatic Complexity of 8 (Highest = 19). 10171139
The class 'MappingStatistics' has a Modified Cyclomatic Complexity of 8 (Highest = 19). 10171139
The class 'MappingStatistics' has a Cyclomatic Complexity of 8 (Highest = 19). 10171139
The constructor 'MappingStatistics' has a Standard Cyclomatic Complexity of 19. 10271118
The constructor 'MappingStatistics' has a Cyclomatic Complexity of 19. 10271118
The constructor 'MappingStatistics' has a Modified Cyclomatic Complexity of 19. 10271118
Local variable 'mappingsCount' could be declared final 1077
Avoid using if statements without curly braces 10781080
Ensure that resources like this Statement object are closed after use 1082
Avoid using if...else statements without curly braces 1096

premon - eu/fbk/dkm/premon/premonitor/PropbankConverter.java

Violation Line
Possible God class (WMC=56, ATFD=58, TCC=0.0) 1301
The class 'PropbankConverter' has a Cyclomatic Complexity of 4 (Highest = 12). 32301
The class 'PropbankConverter' has a Modified Cyclomatic Complexity of 3 (Highest = 12). 32301
The class 'PropbankConverter' has a Standard Cyclomatic Complexity of 4 (Highest = 12). 32301
Parameter 'wnInfo' is not assigned and could be declared final 39
Parameter 'properties' is not assigned and could be declared final 39
Parameter 'path' is not assigned and could be declared final 39
Parameter 'sink' is not assigned and could be declared final 39
Parameter 'part' is not assigned and could be declared final 50
Parameter 'code' is not assigned and could be declared final 54
Local variable 'matcher' could be declared final 66
Position literals first in String comparisons 71
Parameter 'conceptualizationURI' is not assigned and could be declared final 80
Parameter 'complexLemmaWithMappings' is not assigned and could be declared final 80
Parameter 'uriLemma' is not assigned and could be declared final 80
The method 'addExternalLinks' has a Modified Cyclomatic Complexity of 10. 80135
Parameter 'type' is not assigned and could be declared final 80
The method 'addExternalLinks' has a Cyclomatic Complexity of 10. 80135
The method 'addExternalLinks' has a Standard Cyclomatic Complexity of 10. 80135
Avoid using if statements without curly braces 8587
Position literals first in String comparisons 86
Avoid using if statements without curly braces 8687
These nested if statements could be combined 8687
Prefer StringBuffer over += for concatenating strings 87
Local variable 'rolesetURI' could be declared final 89
Local variable 'fnPredicates' could be declared final 92
Local variable 'tmpFnPreds' could be declared final 9495
Local variable 'fnPredicate' could be declared final 104
Local variable 'fnLink' could be declared final 105
Local variable 'fnFrameURI' could be declared final 106
Local variable 'fnConceptualizationURI' could be declared final 107
Local variable 'vnClasses' could be declared final 113
Local variable 'vnClass' could be declared final 114
Local variable 'vnLink' could be declared final 115
Local variable 'vnClassURI' could be declared final 116
Local variable 'vnConceptualizationURI' could be declared final 117
The method 'addInflectionToSink' has a Modified Cyclomatic Complexity of 12. 137193
Parameter 'inflection' is not assigned and could be declared final 137
Parameter 'exampleURI' is not assigned and could be declared final 137
The method 'addInflectionToSink' has a Standard Cyclomatic Complexity of 12. 137193
The method addInflectionToSink() has an NPath complexity of 672 137193
The method 'addInflectionToSink' has a Cyclomatic Complexity of 12. 137193
Local variable 'inflectionParts' could be declared final 143
Local variable 'inflections' could be declared final 144
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 172
Local variable 'builder' could be declared final 175
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 176
Local variable 'part' could be declared final 178
Local variable 'inflectionURI' could be declared final 182
Local variable 'key' could be declared final 184
Local variable 'uri' could be declared final 185
Parameter 'argName' is not assigned and could be declared final 215
Parameter 'argumentURI' is not assigned and could be declared final 215
Avoid long parameter lists. 215216
Parameter 'argType' is not assigned and could be declared final 215
Parameter 'f' is not assigned and could be declared final 215
Parameter 'lemma' is not assigned and could be declared final 216
Parameter 'roleset' is not assigned and could be declared final 216
Parameter 'role' is not assigned and could be declared final 216
Parameter 'type' is not assigned and could be declared final 216
Parameter 'lexicalEntryURI' is not assigned and could be declared final 216
Parameter 'rolesetID' is not assigned and could be declared final 216
Local variable 'vnLemmas' could be declared final 218
Parameter 'argumentURI' is not assigned and could be declared final 239
Parameter 'f' is not assigned and could be declared final 239
Local variable 'lexicalEntry' could be declared final 255
Parameter 'lemma' is not assigned and could be declared final 263
Parameter 'type' is not assigned and could be declared final 263
Parameter 'argType' is not assigned and could be declared final 267
Parameter 'markableURI' is not assigned and could be declared final 267
Parameter 'argName' is not assigned and could be declared final 267
A switch with less than three branches is inefficient, use a if statement instead. 268274
Parameter 'markableURI' is not assigned and could be declared final 277
Parameter 'argType' is not assigned and could be declared final 277
Parameter 'argName' is not assigned and could be declared final 277
Parameter 'rolesetID' is not assigned and could be declared final 278
Parameter 'f' is not assigned and could be declared final 278
Parameter 'asURI' is not assigned and could be declared final 278
Local variable 'argumentURI' could be declared final 279

premon - eu/fbk/dkm/premon/premonitor/SemlinkConverter.java

Violation Line
The class 'SemlinkConverter' has a Modified Cyclomatic Complexity of 7 (Highest = 19). 36320
The class 'SemlinkConverter' has a Cyclomatic Complexity of 7 (Highest = 19). 36320
The class 'SemlinkConverter' has a Standard Cyclomatic Complexity of 7 (Highest = 19). 36320
The constructor 'SemlinkConverter' has a Cyclomatic Complexity of 10. 58104
Parameter 'path' is not assigned and could be declared final 58
Parameter 'properties' is not assigned and could be declared final 58
Parameter 'wnInfo' is not assigned and could be declared final 58
Parameter 'sink' is not assigned and could be declared final 58
Local variable 'vnPath' could be declared final 65
Local variable 'vnFile' could be declared final 68
Local variable 'thisClass' could be declared final 81
Local variable 'id' could be declared final 82
Local variable 'mID' could be declared final 83
Deeply nested if..then statements are hard to read 8488
The method convert() has an NCSS line count of 118 106297
The method convert() has an NPath complexity of 2051 106297
The method 'convert' has a Modified Cyclomatic Complexity of 19. 106297
The method 'convert' has a Cyclomatic Complexity of 19. 106297
The method 'convert' has a Standard Cyclomatic Complexity of 19. 106297
Avoid really long methods. 106297
Local variable 'vnPbMappings' could be declared final 112
Local variable 'vnFnMappings' could be declared final 113
Local variable 'vnFnMappingsRole' could be declared final 114115
Local variable 'predicate' could be declared final 125
Local variable 'lemma' could be declared final 126
Local variable 'uriLemma' could be declared final 127
Local variable 'argmap' could be declared final 130
Local variable 'pbRoleset' could be declared final 131
Local variable 'vnClass' could be declared final 132
Prefer StringBuffer over += for concatenating strings 140
Local variable 'role' could be declared final 145
Local variable 'pbArg' could be declared final 146
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 149
Local variable 'pbLink' could be declared final 151
Local variable 'vnLink' could be declared final 152
Local variable 'pbRolesetURI' could be declared final 154
Local variable 'pbConceptualizationURI' could be declared final 155
Local variable 'pbArgURI' could be declared final 156
Local variable 'vnClassURI' could be declared final 158
Local variable 'vnConceptualizationURI' could be declared final 159
Local variable 'vnArgURI' could be declared final 160
Local variable 'vnfnMap' could be declared final 179
Local variable 'vnfnLemmaMap' could be declared final 180
Local variable 'vnClass' could be declared final 184
Local variable 'vnCls' could be declared final 185
Local variable 'lemma' could be declared final 186
Local variable 'uriLemma' could be declared final 187
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 190
Prefer StringBuffer over += for concatenating strings 200
Local variable 'newVnCls' could be declared final 207
Local variable 'vnClass' could be declared final 222
Local variable 'vnCls' could be declared final 223
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 226
Prefer StringBuffer over += for concatenating strings 233
Local variable 'frames' could be declared final 236
Local variable 'lemmas' could be declared final 243
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 244
Local variable 'role' could be declared final 249
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 253
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 254
Local variable 'fnLink' could be declared final 256
Local variable 'vnLink' could be declared final 257
Local variable 'l' could be declared final 259
Local variable 'index' could be declared final 261
Local variable 'lemma' could be declared final 262
Local variable 'vnSubClass' could be declared final 263
Local variable 'fnFrameURI' could be declared final 265
Local variable 'fnConceptualizationURI' could be declared final 266
Local variable 'fnArgURI' could be declared final 267
Local variable 'vnClassURI' could be declared final 269
Local variable 'vnConceptualizationURI' could be declared final 270
Local variable 'vnArgURI' could be declared final 271
Parameter 'links2' is not assigned and could be declared final 299
Parameter 'p1' is not assigned and could be declared final 299
Parameter 'p2' is not assigned and could be declared final 299
Parameter 'links1' is not assigned and could be declared final 299
Parameter 'uriLemma' is not assigned and could be declared final 299
Local variable 'link1' could be declared final 300
Local variable 'link2' could be declared final 301
Local variable 'firstRolesetURI' could be declared final 302
Local variable 'secondRolesetURI' could be declared final 303
Local variable 'firstConceptualizationURI' could be declared final 304
Local variable 'secondConceptualizationURI' could be declared final 305
Parameter 'textualPOS' is not assigned and could be declared final 312

premon - eu/fbk/dkm/premon/premonitor/VerbnetConverter.java

Violation Line
Possible God class (WMC=121, ATFD=163, TCC=0.0) 1855
The class 'VerbnetConverter' has a Cyclomatic Complexity of 4 (Highest = 24). 32855
The class 'VerbnetConverter' has a Standard Cyclomatic Complexity of 4 (Highest = 22). 32855
The class 'VerbnetConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 22). 32855
The method 'addClassToSink' has a Modified Cyclomatic Complexity of 22. 93283
The method 'addClassToSink' has a Standard Cyclomatic Complexity of 22. 93283
The method 'addClassToSink' has a Cyclomatic Complexity of 24. 93283
The method addClassToSink() has an NPath complexity of 80832 93283
The method addClassToSink() has an NCSS line count of 107 93283
Avoid really long methods. 93283
Avoid instantiating new objects inside loops 139
String.trim().length()==0 is an inefficient way to validate an empty String. 142
Avoid unused local variables such as 'pbRolesetURI'. 150
Avoid unused local variables such as 'questionMark'. 176
This call to String.startsWith can be rewritten using String.charAt(0) 177
Local variable 'lemma' could be declared final 190
Avoid long parameter lists. 317320
The method 'addFrameToSink' has a Modified Cyclomatic Complexity of 10. 374462
The method 'addFrameToSink' has a Standard Cyclomatic Complexity of 10. 374462
The method 'addFrameToSink' has a Cyclomatic Complexity of 10. 374462
Avoid unused local variables such as 'questionMark'. 490
This call to String.startsWith can be rewritten using String.charAt(0) 492
Avoid using if...else statements without curly braces 496
Switch statements should have a default label 500530
Local variable 'thisA' could be declared final 570
The class 'SyntaxArrayLogic' has a Cyclomatic Complexity of 3 (Highest = 10). 601690
The method 'addToSink' has a Cyclomatic Complexity of 10. 629689
Switch statements should have a default label 642688
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 737
Avoid appending characters as strings in StringBuffer.append. 738
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 738
Avoid appending characters as strings in StringBuffer.append. 741
Avoid unused method parameters such as 'frameURI'. 751
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 770
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 771
Avoid appending characters as strings in StringBuffer.append. 771
Avoid appending characters as strings in StringBuffer.append. 774
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 785
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 786
Avoid appending characters as strings in StringBuffer.append. 787
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 787
Avoid appending characters as strings in StringBuffer.append. 790
Position literals first in String comparisons 806
Position literals first in String comparisons 809
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 832
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 833
Avoid appending characters as strings in StringBuffer.append. 833
Avoid appending characters as strings in StringBuffer.append. 836
Switch statements should have a default label 848851
A switch with less than three branches is inefficient, use a if statement instead. 848851

premon - eu/fbk/dkm/premon/premonitor/propbank/Alias.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 21
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 22
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 23
Parameter 'value' is not assigned and could be declared final 41

premon - eu/fbk/dkm/premon/premonitor/propbank/Arg.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 20
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 21
Parameter 'value' is not assigned and could be declared final 40
Parameter 'value' is not assigned and could be declared final 60
Parameter 'value' is not assigned and could be declared final 80

premon - eu/fbk/dkm/premon/premonitor/propbank/Example.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 22
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 23
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 24
Parameter 'value' is not assigned and could be declared final 47
Parameter 'value' is not assigned and could be declared final 67
Parameter 'value' is not assigned and could be declared final 87

premon - eu/fbk/dkm/premon/premonitor/propbank/Inflection.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 19
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 20
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 21
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 22
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 23
Parameter 'value' is not assigned and could be declared final 45
Parameter 'value' is not assigned and could be declared final 69
Parameter 'value' is not assigned and could be declared final 93
Parameter 'value' is not assigned and could be declared final 117
Parameter 'value' is not assigned and could be declared final 141

premon - eu/fbk/dkm/premon/premonitor/propbank/Note.java

Violation Line
Parameter 'value' is not assigned and could be declared final 36

premon - eu/fbk/dkm/premon/premonitor/propbank/ObjectFactory.java

Violation Line
Document empty constructor 3031

premon - eu/fbk/dkm/premon/premonitor/propbank/Predicate.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 22
Parameter 'value' is not assigned and could be declared final 42

premon - eu/fbk/dkm/premon/premonitor/propbank/Rel.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 20
Parameter 'value' is not assigned and could be declared final 39
Parameter 'value' is not assigned and could be declared final 59

premon - eu/fbk/dkm/premon/premonitor/propbank/Role.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 22
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 23
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 24
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 25
Parameter 'value' is not assigned and could be declared final 44
Parameter 'value' is not assigned and could be declared final 64
Parameter 'value' is not assigned and could be declared final 84
Parameter 'value' is not assigned and could be declared final 104

premon - eu/fbk/dkm/premon/premonitor/propbank/Roleset.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 23
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 24
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 25
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 26
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 27
Parameter 'value' is not assigned and could be declared final 49
Parameter 'value' is not assigned and could be declared final 69
Parameter 'value' is not assigned and could be declared final 89
Parameter 'value' is not assigned and could be declared final 109
Parameter 'value' is not assigned and could be declared final 129

premon - eu/fbk/dkm/premon/premonitor/propbank/Text.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 20
Parameter 'value' is not assigned and could be declared final 39
Parameter 'value' is not assigned and could be declared final 59

premon - eu/fbk/dkm/premon/premonitor/propbank/Vnrole.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 20
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 21
Parameter 'value' is not assigned and could be declared final 39
Parameter 'value' is not assigned and could be declared final 59

premon - eu/fbk/dkm/premon/premonitor/verbnet/ARG.java

Violation Line
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. 34
Parameter 'value' is not assigned and could be declared final 56
Parameter 'value' is not assigned and could be declared final 80

premon - eu/fbk/dkm/premon/premonitor/verbnet/DESCRIPTION.java

Violation Line
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. 33
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
Parameter 'value' is not assigned and could be declared final 61
Parameter 'value' is not assigned and could be declared final 85
Parameter 'value' is not assigned and could be declared final 109
Parameter 'value' is not assigned and could be declared final 133

premon - eu/fbk/dkm/premon/premonitor/verbnet/EXAMPLE.java

Violation Line
Parameter 'value' is not assigned and could be declared final 51

premon - eu/fbk/dkm/premon/premonitor/verbnet/FRAME.java

Violation Line
Parameter 'value' is not assigned and could be declared final 60
Parameter 'value' is not assigned and could be declared final 84
Parameter 'value' is not assigned and could be declared final 108
Parameter 'value' is not assigned and could be declared final 132

premon - eu/fbk/dkm/premon/premonitor/verbnet/LEX.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 30
Parameter 'value' is not assigned and could be declared final 52

premon - eu/fbk/dkm/premon/premonitor/verbnet/MEMBER.java

Violation Line
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. 33
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
Parameter 'value' is not assigned and could be declared final 58
Parameter 'value' is not assigned and could be declared final 82
Parameter 'value' is not assigned and could be declared final 106

premon - eu/fbk/dkm/premon/premonitor/verbnet/NP.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
Parameter 'value' is not assigned and could be declared final 63

premon - eu/fbk/dkm/premon/premonitor/verbnet/ObjectFactory.java

Violation Line
Document empty constructor 3637

premon - eu/fbk/dkm/premon/premonitor/verbnet/PRED.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 33
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
Parameter 'value' is not assigned and could be declared final 60
Parameter 'value' is not assigned and could be declared final 84
Parameter 'value' is not assigned and could be declared final 108

premon - eu/fbk/dkm/premon/premonitor/verbnet/PREP.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 33
Parameter 'value' is not assigned and could be declared final 57
Parameter 'value' is not assigned and could be declared final 81

premon - eu/fbk/dkm/premon/premonitor/verbnet/SELRESTR.java

Violation Line
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. 33
Parameter 'value' is not assigned and could be declared final 55
Parameter 'value' is not assigned and could be declared final 79

premon - eu/fbk/dkm/premon/premonitor/verbnet/SELRESTRS.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 36
Parameter 'value' is not assigned and could be declared final 63

premon - eu/fbk/dkm/premon/premonitor/verbnet/SYNRESTR.java

Violation Line
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. 33
Parameter 'value' is not assigned and could be declared final 55
Parameter 'value' is not assigned and could be declared final 79

premon - eu/fbk/dkm/premon/premonitor/verbnet/SYNRESTRS.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 35
Parameter 'value' is not assigned and could be declared final 59

premon - eu/fbk/dkm/premon/premonitor/verbnet/THEMROLE.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 33
Parameter 'value' is not assigned and could be declared final 57
Parameter 'value' is not assigned and could be declared final 81

premon - eu/fbk/dkm/premon/premonitor/verbnet/VNCLASS.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 39
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 42
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 45
Parameter 'value' is not assigned and could be declared final 75
Parameter 'value' is not assigned and could be declared final 99
Parameter 'value' is not assigned and could be declared final 123
Parameter 'value' is not assigned and could be declared final 147
Parameter 'value' is not assigned and could be declared final 171
Parameter 'value' is not assigned and could be declared final 195
Parameter 'value' is not assigned and could be declared final 219

premon - eu/fbk/dkm/premon/premonitor/verbnet/VNSUBCLASS.java

Violation Line
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 38
Parameter 'value' is not assigned and could be declared final 68
Parameter 'value' is not assigned and could be declared final 92
Parameter 'value' is not assigned and could be declared final 116
Parameter 'value' is not assigned and could be declared final 140
Parameter 'value' is not assigned and could be declared final 164

premon - eu/fbk/dkm/premon/util/NF.java

Violation Line
The class 'NF' has a Cyclomatic Complexity of 4 (Highest = 10). 9102
Private field 'n' could be made final; it is only initialized in the declaration or constructor. 11
Private field 'f' could be made final; it is only initialized in the declaration or constructor. 12
Avoid reassigning parameters such as 'f' 17
Avoid reassigning parameters such as 'n' 17
Avoid reassigning parameters such as 'f' 17
Avoid reassigning parameters such as 'f' 17
Avoid reassigning parameters such as 'f' 17
Avoid reassigning parameters such as 'n' 17
Avoid reassigning parameters such as 'n' 17
Avoid reassigning parameters such as 'f' 17
Avoid reassigning parameters such as 'n' 17
The constructor 'NF' has a Cyclomatic Complexity of 10. 1766
Using equalsIgnoreCase() is cleaner than using toUpperCase/toLowerCase().equals(). 33
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 33
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 39
Using equalsIgnoreCase() is cleaner than using toUpperCase/toLowerCase().equals(). 39
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 48
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 58
Position literals first in String comparisons 59
Position literals first in String comparisons 70
Position literals first in String comparisons 72

premon - eu/fbk/dkm/premon/util/PremonResource.java

Violation Line
This abstract class does not have any abstract methods 735

premon - eu/fbk/dkm/premon/util/ProcessorUndoRDFS.java

Violation Line
Returning 'BYPASS' may expose an internal array. 148
The class 'UndoRDFSReducer' has a Standard Cyclomatic Complexity of 7 (Highest = 12). 154217
The class 'UndoRDFSReducer' has a Cyclomatic Complexity of 7 (Highest = 12). 154217
The class 'UndoRDFSReducer' has a Modified Cyclomatic Complexity of 7 (Highest = 12). 154217
The method 'reduce' has a Standard Cyclomatic Complexity of 12. 167215
The method 'reduce' has a Cyclomatic Complexity of 12. 167215
The method 'reduce' has a Modified Cyclomatic Complexity of 12. 167215
Ensure that resources like this Statement object are closed after use 175
Ensure that resources like this Statement object are closed after use 193
Ensure that resources like this Statement object are closed after use 211

premon - eu/fbk/dkm/premon/util/PropBankResource.java

Violation Line
Parameter 'fileName' is not assigned and could be declared final 16
Parameter 'isOntoNotes' is not assigned and could be declared final 16
Parameter 'defaultType' is not assigned and could be declared final 16
Local variable 'matcher' could be declared final 23
Parameter 'main' is not assigned and could be declared final 38

premon - eu/fbk/dkm/premon/util/Replacer.java

Violation Line
Possible God class (WMC=49, ATFD=10, TCC=0.044444444444444446) 1250
Avoid instantiating new objects inside loops 52
Deeply nested if..then statements are hard to read 7072
Deeply nested if..then statements are hard to read 9294
The class 'Rule' has a Cyclomatic Complexity of 3 (Highest = 11). 114248
Deeply nested if..then statements are hard to read 172174
The method 'parse' has a Cyclomatic Complexity of 11. 210246
The method parse() has an NPath complexity of 395 210246
This call to String.startsWith can be rewritten using String.charAt(0) 221
Avoid instantiating new objects inside loops 239
New exception is thrown in catch block, original stack trace may be lost 241242

premon - eu/fbk/dkm/premon/util/WordNet.java

Violation Line
Possible God class (WMC=64, ATFD=51, TCC=0.021052631578947368) 1412
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. 19412
Parameter 'pos' is not assigned and could be declared final 156
Parameter 'lemma' is not assigned and could be declared final 156
Local variable 'indexWord' could be declared final 159
Local variable 'synsets' could be declared final 163
Local variable 'ret' could be declared final 164
Local variable 'synset' could be declared final 166
Do not add empty strings 274
Avoid reassigning parameters such as 'synsetID' 279
Avoid instantiating new objects inside loops 376377

premon - eu/fbk/dkm/premon/vocab/FBMETA.java

Violation Line
A class which only has private constructors should be final 11112

premon - eu/fbk/dkm/premon/vocab/PMOVN.java

Violation Line
Avoid instantiating new objects inside loops 729

Back to top

Last Published: 2018/02/15.

Reflow Maven skin by Andrius Velykis.

Data and Knowledge Management tools