PMD Results
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) | 1–783 |
| The class 'BankConverter' has a Cyclomatic Complexity of 5 (Highest = 52). | 40–783 |
| The class 'BankConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 49). | 40–783 |
| The class 'BankConverter' has a Standard Cyclomatic Complexity of 5 (Highest = 49). | 40–783 |
| 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. | 91–137 |
| 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 | 116–120 |
| 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 | 151–154 |
| Avoid really long methods. | 161–571 |
| The method 'convert' has a Modified Cyclomatic Complexity of 49. | 161–571 |
| The method 'convert' has a Standard Cyclomatic Complexity of 49. | 161–571 |
| The method 'convert' has a Cyclomatic Complexity of 52. | 161–571 |
| The method convert() has an NPath complexity of 1377860802 | 161–571 |
| The method convert() has an NCSS line count of 231 | 161–571 |
| 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 | 262–280 |
| 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 | 270–271 |
| Avoid instantiating new objects inside loops | 272–273 |
| Local variable 'aliasComplexLemma' could be declared final | 272–273 |
| 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 | 298–300 |
| Position literals first in String comparisons | 299 |
| Avoid using if statements without curly braces | 299–300 |
| Deeply nested if..then statements are hard to read | 299–300 |
| These nested if statements could be combined | 299–300 |
| 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 | 344–388 |
| 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 | 414–415 |
| 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 | 446–448 |
| 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 | 459–460 |
| Local variable 'start' could be declared final | 463 |
| Deeply nested if..then statements are hard to read | 464–469 |
| 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 | 482–486 |
| Local variable 'conceptualizationURI' could be declared final | 483–484 |
| 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 | 504–505 |
| Local variable 'start' could be declared final | 507 |
| Deeply nested if..then statements are hard to read | 508–512 |
| 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 | 535–538 |
| Deeply nested if..then statements are hard to read | 541–543 |
| Local variable 'argumentURI' could be declared final | 553–554 |
| 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 | 661–662 |
| 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. | 746–747 |
| Parameter 'textualPOS' is not assigned and could be declared final | 756 |
| Switch statements should have a default label | 761–771 |
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) | 1–686 |
| Avoid unused imports such as 'org.openrdf.query.algebra.evaluation.function.hash.MD5' | 23 |
| Too many fields | 39–686 |
| 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 | 532–556 |
| 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 | 148–159 |
| 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) | 1–443 |
| The class 'FramebaseCleanerConverter' has a Cyclomatic Complexity of 5 (Highest = 11). | 22–443 |
| The method 'compare' has a Cyclomatic Complexity of 11. | 34–61 |
| Parameter 'wnInfo' is not assigned and could be declared final | 66 |
| Use block level rather than method level synchronization | 126–145 |
| The class 'URIInfo' has a Standard Cyclomatic Complexity of 5 (Highest = 18). | 177–438 |
| The class 'URIInfo' has a Cyclomatic Complexity of 5 (Highest = 18). | 177–438 |
| The class 'URIInfo' has a Modified Cyclomatic Complexity of 5 (Highest = 18). | 177–438 |
| The method 'update' has a Modified Cyclomatic Complexity of 15. | 242–289 |
| The method 'update' has a Cyclomatic Complexity of 16. | 242–289 |
| The method 'update' has a Standard Cyclomatic Complexity of 15. | 242–289 |
| Avoid empty if statements | 256–265 |
| The method 'emit' has a Standard Cyclomatic Complexity of 18. | 291–364 |
| The method 'emit' has a Modified Cyclomatic Complexity of 18. | 291–364 |
| The method emit() has an NPath complexity of 18432 | 291–364 |
| The method 'emit' has a Cyclomatic Complexity of 18. | 291–364 |
| 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) | 1–353 |
| The class 'FramebaseConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 15). | 35–353 |
| The class 'FramebaseConverter' has a Cyclomatic Complexity of 6 (Highest = 19). | 35–353 |
| The class 'FramebaseConverter' has a Standard Cyclomatic Complexity of 5 (Highest = 15). | 35–353 |
| The method 'getPosURI' has a Cyclomatic Complexity of 11. | 57–85 |
| The method 'getPosURI' has a Standard Cyclomatic Complexity of 11. | 57–85 |
| 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. | 112–140 |
| Avoid instantiating new objects inside loops | 116 |
| Avoid instantiating new objects inside loops | 119–133 |
| The method emitPBNBAlignments() has an NPath complexity of 41492 | 192–298 |
| Avoid really long methods. | 192–298 |
| The method 'emitPBNBAlignments' has a Cyclomatic Complexity of 19. | 192–298 |
| The method 'emitPBNBAlignments' has a Standard Cyclomatic Complexity of 15. | 192–298 |
| The method 'emitPBNBAlignments' has a Modified Cyclomatic Complexity of 15. | 192–298 |
| 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) | 1–1063 |
| The class 'FramenetConverter' has a Modified Cyclomatic Complexity of 6 (Highest = 84). | 32–1063 |
| The class 'FramenetConverter' has a Standard Cyclomatic Complexity of 8 (Highest = 103). | 32–1063 |
| The class 'FramenetConverter' has a Cyclomatic Complexity of 9 (Highest = 114). | 32–1063 |
| 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. | 71–882 |
| The method convert() has an NPath complexity of 2036440835 | 71–882 |
| The method convert() has an NCSS line count of 510 | 71–882 |
| The method 'convert' has a Cyclomatic Complexity of 114. | 71–882 |
| The method 'convert' has a Modified Cyclomatic Complexity of 84. | 71–882 |
| Avoid really long methods. | 71–882 |
| 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 | 167–202 |
| 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 | 257–259 |
| 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 | 278–290 |
| 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 | 362–376 |
| 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 | 434–459 |
| 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 | 571–572 |
| 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 | 703–705 |
| These nested if statements could be combined | 718–720 |
| These nested if statements could be combined | 723–725 |
| 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 | 790–791 |
| 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 | 804–806 |
| Local variable 'aURI' could be declared final | 823 |
| Avoid empty if statements | 830–832 |
| 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. | 1020–1054 |
| Avoid reassigning parameters such as 'pos' | 1020 |
| The method 'getPosURI' has a Cyclomatic Complexity of 14. | 1020–1054 |
| When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 1022 |
| Switch statements should have a default label | 1024–1051 |
| 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 | 97–99 |
| 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. | 105–106 |
| 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) | 1–326 |
| The class 'PredMatConverter' has a Standard Cyclomatic Complexity of 8 (Highest = 32). | 26–326 |
| Too many fields | 26–326 |
| The class 'PredMatConverter' has a Cyclomatic Complexity of 9 (Highest = 32). | 26–326 |
| The class 'PredMatConverter' has a Modified Cyclomatic Complexity of 8 (Highest = 32). | 26–326 |
| 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. | 55–102 |
| 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 | 82–86 |
| The method 'convert' has a Modified Cyclomatic Complexity of 32. | 104–226 |
| The method 'convert' has a Standard Cyclomatic Complexity of 32. | 104–226 |
| The method 'convert' has a Cyclomatic Complexity of 32. | 104–226 |
| Avoid really long methods. | 104–226 |
| 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 | 165–166 |
| Local variable 'vnArgURI' could be declared final | 167 |
| Local variable 'fnFrameURI' could be declared final | 169 |
| Local variable 'fnConceptualizationURI' could be declared final | 170–171 |
| Local variable 'fnArgURI' could be declared final | 172 |
| Local variable 'pbRolesetURI' could be declared final | 174 |
| Local variable 'pbConceptualizationURI' could be declared final | 175–177 |
| 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 | 180–181 |
| 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. | 258–324 |
| The method 'uriForWnSense' has a Standard Cyclomatic Complexity of 11. | 258–324 |
| The method 'uriForWnSense' has a Modified Cyclomatic Complexity of 11. | 258–324 |
| 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) | 1–1141 |
| The class 'Premonitor' has a Modified Cyclomatic Complexity of 27 (Highest = 39). | 87–1141 |
| The class 'Premonitor' has a Standard Cyclomatic Complexity of 27 (Highest = 39). | 87–1141 |
| The class 'Premonitor' has a Cyclomatic Complexity of 31 (Highest = 49). | 87–1141 |
| 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. | 87–1141 |
| Avoid unused private fields such as 'WN_OLD_SENSE'. | 104 |
| The method 'main' has a Cyclomatic Complexity of 43. | 109–399 |
| Avoid really long methods. | 109–399 |
| The method main() has an NPath complexity of 116760321 | 109–399 |
| The method 'main' has a Modified Cyclomatic Complexity of 39. | 109–399 |
| The method main() has an NCSS line count of 132 | 109–399 |
| 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. | 109–399 |
| Deeply nested if..then statements are hard to read | 192–201 |
| These nested if statements could be combined | 194–200 |
| Deeply nested if..then statements are hard to read | 204–222 |
| Deeply nested if..then statements are hard to read | 244–246 |
| 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 | 304–330 |
| Use block level rather than method level synchronization | 312–328 |
| Prefer StringBuffer over += for concatenating strings | 376 |
| The method 'emit' has a Modified Cyclomatic Complexity of 38. | 401–628 |
| The method 'emit' has a Cyclomatic Complexity of 49. | 401–628 |
| The method emit() has an NPath complexity of 6528600 | 401–628 |
| Avoid really long methods. | 401–628 |
| The method emit() has an NCSS line count of 153 | 401–628 |
| The method 'emit' has a Standard Cyclomatic Complexity of 38. | 401–628 |
| 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 | 476–478 |
| Deeply nested if..then statements are hard to read | 480–484 |
| These nested if statements could be combined | 481–483 |
| Ensure that resources like this Statement object are closed after use | 498 |
| Avoid instantiating new objects inside loops | 516–517 |
| Avoid instantiating new objects inside loops | 533–534 |
| Avoid instantiating new objects inside loops | 544–545 |
| Avoid instantiating new objects inside loops | 563 |
| The method 'emit' has a Modified Cyclomatic Complexity of 17. | 630–717 |
| The method 'emit' has a Cyclomatic Complexity of 19. | 630–717 |
| The method 'emit' has a Standard Cyclomatic Complexity of 17. | 630–717 |
| The method emit() has an NPath complexity of 69840 | 630–717 |
| 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 | 705–706 |
| The method filterMappings() has an NPath complexity of 4770480 | 719–900 |
| The method 'filterMappings' has a Modified Cyclomatic Complexity of 39. | 719–900 |
| The method 'filterMappings' has a Cyclomatic Complexity of 43. | 719–900 |
| The method filterMappings() has an NCSS line count of 114 | 719–900 |
| The method 'filterMappings' has a Standard Cyclomatic Complexity of 39. | 719–900 |
| Avoid really long methods. | 719–900 |
| 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 | 740–741 |
| 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). | 906–1015 |
| The class 'InstanceStatistics' has a Modified Cyclomatic Complexity of 19 (Highest = 18). | 906–1015 |
| The class 'InstanceStatistics' has a Cyclomatic Complexity of 37 (Highest = 36). | 906–1015 |
| The constructor 'InstanceStatistics' has a Modified Cyclomatic Complexity of 18. | 926–1013 |
| The constructor 'InstanceStatistics' has a Cyclomatic Complexity of 36. | 926–1013 |
| The constructor 'InstanceStatistics' has a Standard Cyclomatic Complexity of 18. | 926–1013 |
| 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). | 1017–1139 |
| The class 'MappingStatistics' has a Modified Cyclomatic Complexity of 8 (Highest = 19). | 1017–1139 |
| The class 'MappingStatistics' has a Cyclomatic Complexity of 8 (Highest = 19). | 1017–1139 |
| The constructor 'MappingStatistics' has a Standard Cyclomatic Complexity of 19. | 1027–1118 |
| The constructor 'MappingStatistics' has a Cyclomatic Complexity of 19. | 1027–1118 |
| The constructor 'MappingStatistics' has a Modified Cyclomatic Complexity of 19. | 1027–1118 |
| Local variable 'mappingsCount' could be declared final | 1077 |
| Avoid using if statements without curly braces | 1078–1080 |
| 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) | 1–301 |
| The class 'PropbankConverter' has a Cyclomatic Complexity of 4 (Highest = 12). | 32–301 |
| The class 'PropbankConverter' has a Modified Cyclomatic Complexity of 3 (Highest = 12). | 32–301 |
| The class 'PropbankConverter' has a Standard Cyclomatic Complexity of 4 (Highest = 12). | 32–301 |
| 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. | 80–135 |
| Parameter 'type' is not assigned and could be declared final | 80 |
| The method 'addExternalLinks' has a Cyclomatic Complexity of 10. | 80–135 |
| The method 'addExternalLinks' has a Standard Cyclomatic Complexity of 10. | 80–135 |
| Avoid using if statements without curly braces | 85–87 |
| Position literals first in String comparisons | 86 |
| Avoid using if statements without curly braces | 86–87 |
| These nested if statements could be combined | 86–87 |
| 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 | 94–95 |
| 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. | 137–193 |
| 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. | 137–193 |
| The method addInflectionToSink() has an NPath complexity of 672 | 137–193 |
| The method 'addInflectionToSink' has a Cyclomatic Complexity of 12. | 137–193 |
| 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. | 215–216 |
| 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. | 268–274 |
| 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). | 36–320 |
| The class 'SemlinkConverter' has a Cyclomatic Complexity of 7 (Highest = 19). | 36–320 |
| The class 'SemlinkConverter' has a Standard Cyclomatic Complexity of 7 (Highest = 19). | 36–320 |
| The constructor 'SemlinkConverter' has a Cyclomatic Complexity of 10. | 58–104 |
| 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 | 84–88 |
| The method convert() has an NCSS line count of 118 | 106–297 |
| The method convert() has an NPath complexity of 2051 | 106–297 |
| The method 'convert' has a Modified Cyclomatic Complexity of 19. | 106–297 |
| The method 'convert' has a Cyclomatic Complexity of 19. | 106–297 |
| The method 'convert' has a Standard Cyclomatic Complexity of 19. | 106–297 |
| Avoid really long methods. | 106–297 |
| Local variable 'vnPbMappings' could be declared final | 112 |
| Local variable 'vnFnMappings' could be declared final | 113 |
| Local variable 'vnFnMappingsRole' could be declared final | 114–115 |
| 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) | 1–855 |
| The class 'VerbnetConverter' has a Cyclomatic Complexity of 4 (Highest = 24). | 32–855 |
| The class 'VerbnetConverter' has a Standard Cyclomatic Complexity of 4 (Highest = 22). | 32–855 |
| The class 'VerbnetConverter' has a Modified Cyclomatic Complexity of 4 (Highest = 22). | 32–855 |
| The method 'addClassToSink' has a Modified Cyclomatic Complexity of 22. | 93–283 |
| The method 'addClassToSink' has a Standard Cyclomatic Complexity of 22. | 93–283 |
| The method 'addClassToSink' has a Cyclomatic Complexity of 24. | 93–283 |
| The method addClassToSink() has an NPath complexity of 80832 | 93–283 |
| The method addClassToSink() has an NCSS line count of 107 | 93–283 |
| Avoid really long methods. | 93–283 |
| 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. | 317–320 |
| The method 'addFrameToSink' has a Modified Cyclomatic Complexity of 10. | 374–462 |
| The method 'addFrameToSink' has a Standard Cyclomatic Complexity of 10. | 374–462 |
| The method 'addFrameToSink' has a Cyclomatic Complexity of 10. | 374–462 |
| 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 | 500–530 |
| Local variable 'thisA' could be declared final | 570 |
| The class 'SyntaxArrayLogic' has a Cyclomatic Complexity of 3 (Highest = 10). | 601–690 |
| The method 'addToSink' has a Cyclomatic Complexity of 10. | 629–689 |
| Switch statements should have a default label | 642–688 |
| 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 | 848–851 |
| A switch with less than three branches is inefficient, use a if statement instead. | 848–851 |
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 | 30–31 |
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 | 36–37 |
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). | 9–102 |
| 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. | 17–66 |
| 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 | 7–35 |
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). | 154–217 |
| The class 'UndoRDFSReducer' has a Cyclomatic Complexity of 7 (Highest = 12). | 154–217 |
| The class 'UndoRDFSReducer' has a Modified Cyclomatic Complexity of 7 (Highest = 12). | 154–217 |
| The method 'reduce' has a Standard Cyclomatic Complexity of 12. | 167–215 |
| The method 'reduce' has a Cyclomatic Complexity of 12. | 167–215 |
| The method 'reduce' has a Modified Cyclomatic Complexity of 12. | 167–215 |
| 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) | 1–250 |
| Avoid instantiating new objects inside loops | 52 |
| Deeply nested if..then statements are hard to read | 70–72 |
| Deeply nested if..then statements are hard to read | 92–94 |
| The class 'Rule' has a Cyclomatic Complexity of 3 (Highest = 11). | 114–248 |
| Deeply nested if..then statements are hard to read | 172–174 |
| The method 'parse' has a Cyclomatic Complexity of 11. | 210–246 |
| The method parse() has an NPath complexity of 395 | 210–246 |
| 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 | 241–242 |
premon - eu/fbk/dkm/premon/util/WordNet.java
| Violation | Line |
|---|---|
| Possible God class (WMC=64, ATFD=51, TCC=0.021052631578947368) | 1–412 |
| 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. | 19–412 |
| 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 | 376–377 |
premon - eu/fbk/dkm/premon/vocab/FBMETA.java
| Violation | Line |
|---|---|
| A class which only has private constructors should be final | 11–112 |
premon - eu/fbk/dkm/premon/vocab/PMOVN.java
| Violation | Line |
|---|---|
| Avoid instantiating new objects inside loops | 729 |
