The following document contains the results of Checkstyle.
Rules | Violations | Severity |
---|---|---|
FileLength | 0 | Error |
NewlineAtEndOfFile | 0 | Error |
AbstractClassName | 0 | Error |
AnnotationUseStyle | 0 | Error |
AnonInnerLength | 5 | Error |
ArrayTypeStyle | 1 | Error |
AvoidInlineConditionals | 0 | Error |
AvoidStaticImport | 0 | Error |
ClassDataAbstractionCoupling | 4 | Error |
ClassFanOutComplexity | 2 | Error |
ClassTypeParameterName | 0 | Error |
ConstantName | 9 | Error |
CovariantEquals | 3 | Error |
DeclarationOrder | 15 | Error |
DefaultComesLast | 0 | Error |
DescendantToken | 0 | Error |
DesignForExtension | 3 | Error |
DoubleCheckedLocking | 0 | Error |
EmptyBlock | 0 | Error |
EmptyForInitializerPad | 0 | Error |
EmptyForIteratorPad | 0 | Error |
EmptyStatement | 0 | Error |
EqualsAvoidNull | 0 | Error |
EqualsHashCode | 2 | Error |
ExecutableStatementCount | 13 | Error |
ExplicitInitialization | 5 | Error |
FallThrough | 0 | Error |
FinalClass | 0 | Error |
FinalLocalVariable | 45 | Error |
FinalParameters | 35 | Error |
GenericWhitespace | 0 | Error |
HiddenField
| 2 | Error |
HideUtilityClassConstructor | 0 | Error |
IllegalCatch | 7 | Error |
IllegalImport | 0 | Error |
IllegalInstantiation | 0 | Error |
IllegalThrows | 0 | Error |
IllegalTokenText | 0 | Error |
IllegalType | 3 | Error |
Indentation | 15 | Error |
InnerAssignment | 0 | Error |
InterfaceIsType | 0 | Error |
JUnitTestCase | 0 | Error |
JavaNCSS | 7 | Error |
LeftCurly | 0 | Error |
LocalFinalVariableName | 11 | Error |
LocalVariableName | 9 | Error |
MemberName | 10 | Error |
MethodName | 0 | Error |
MethodParamPad | 0 | Error |
MethodTypeParameterName | 0 | Error |
MissingCtor | 0 | Error |
MissingDeprecated | 0 | Error |
MissingOverride | 0 | Error |
ModifierOrder | 0 | Error |
MultipleStringLiterals | 10 | Error |
MultipleVariableDeclarations | 0 | Error |
MutableException | 0 | Error |
NPathComplexity | 4 | Error |
NeedBraces | 0 | Error |
NoClone | 0 | Error |
NoFinalizer | 0 | Error |
NoWhitespaceBefore | 0 | Error |
OperatorWrap | 2 | Error |
OuterTypeNumber | 0 | Error |
PackageAnnotation | 0 | Error |
PackageDeclaration | 0 | Error |
PackageName | 0 | Error |
ParameterName | 16 | Error |
ParameterNumber | 0 | Error |
ParenPad | 2 | Error |
RedundantImport | 0 | Error |
RedundantModifier | 14 | Error |
RedundantThrows | 0 | Error |
RightCurly | 0 | Error |
StaticVariableName | 0 | Error |
SuperClone | 0 | Error |
SuperFinalize | 0 | Error |
SuppressWarnings | 0 | Error |
ThrowsCount | 1 | Error |
TodoComment | 0 | Error |
TypeName | 0 | Error |
TypecastParenPad | 0 | Error |
UncommentedMain
| 1 | Error |
UpperEll | 0 | Error |
VisibilityModifier | 10 | Error |
WhitespaceAfter | 19 | Error |
WhitespaceAround | 125 | Error |
MagicNumber | 141 | Error |
UnusedImports | 0 | Error |
AvoidStarImport | 0 | Error |
NoWhitespaceAfter | 0 | Error |
BooleanExpressionComplexity | 0 | Error |
CyclomaticComplexity | 10 | Error |
MethodLength | 0 | Error |
MissingSwitchDefault | 1 | Error |
ModifiedControlVariable | 4 | Error |
NestedTryDepth | 0 | Error |
ParameterAssignment | 1 | Error |
SimplifyBooleanExpression | 0 | Error |
SimplifyBooleanReturn | 0 | Error |
StringLiteralEquality | 0 | Error |
LineLength
| 4 | Error |
NestedIfDepth
| 7 | Error |
ReturnCount | 5 | Error |
Violation | Message | Line |
---|---|---|
Class Data Abstraction Coupling is 27 (max allowed is 7) classes [ActionListener, ArrayList, BorderLayout, CSVFileFilter, CSVTypeLog, Dimension, EntireGraphingPanel, File, FileInputStream, FileOutputStream, FreeEMSBin, FreeEMSBinFileFilter, FreeEMSFileFilter, FreeEMSLAFileFilter, JFileChooser, JMenu, JMenuBar, JMenuItem, JPanel, LogFileFilter, MSTypeFileFilter, OptionFrameV2, PlayBarPanel, Properties, PropertiesPane, Runnable, WindowEvent]. | 77 | |
Class Fan-Out Complexity is 42 (max allowed is 20). | 77 | |
Declaring variables, return values or parameters of type 'AbstractDecoder' is not allowed. | 115 | |
Executable statement count is 49 (max allowed is 30). | 119 | |
NCSS for this method is 65 (max allowed is 50). | 119 | |
'600' is a magic number. | 127 | |
'420' is a magic number. | 127 | |
Variable 'wev' should be declared final. | 155 | |
Uncommented main method found. | 229 | |
Anonymous inner class length is 35 lines (max allowed is 20). | 230 | |
Variable 'currentLocale' should be declared final. | 234 | |
Cyclomatic Complexity is 17 (max allowed is 10). | 267 | |
Executable statement count is 46 (max allowed is 30). | 267 | |
NCSS for this method is 54 (max allowed is 50). | 267 | |
NPath Complexity is 312 (max allowed is 200). | 267 | |
Line is longer than 160 characters. | 331 | |
Name 'AppWide' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 348 | |
Name 'AppWideFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 349 | |
Name 'AppWide' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 363 | |
Name 'AppWideFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 364 | |
Name 'AppWide' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 379 | |
Name 'AppWideFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 380 | |
Name 'AppWide' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 393 | |
Name 'AppWideFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 394 | |
Name 'IOE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 414 | |
')' is preceded with whitespace. | 427 | |
method call child at indentation level 16 not at correct indentation, 20 | 434 | |
try child at indentation level 16 not at correct indentation, 20 | 434 | |
method call child at indentation level 16 not at correct indentation, 20 | 435 | |
try child at indentation level 16 not at correct indentation, 20 | 435 | |
method call child at indentation level 16 not at correct indentation, 20 | 436 | |
try child at indentation level 16 not at correct indentation, 20 | 436 | |
method call child at indentation level 16 not at correct indentation, 20 | 437 | |
try child at indentation level 16 not at correct indentation, 20 | 437 | |
method call child at indentation level 16 not at correct indentation, 20 | 438 | |
try child at indentation level 16 not at correct indentation, 20 | 438 | |
method call child at indentation level 16 not at correct indentation, 20 | 439 | |
try child at indentation level 16 not at correct indentation, 20 | 439 | |
method call child at indentation level 16 not at correct indentation, 20 | 440 | |
try child at indentation level 16 not at correct indentation, 20 | 440 | |
try child at indentation level 16 not at correct indentation, 20 | 441 | |
Catching 'Exception' is not allowed. | 442 | |
'catch' is not followed by whitespace. | 442 | |
Variable 'ge' should be declared final. | 454 | |
Variable 'gs' should be declared final. | 455 | |
'{' is not preceded with whitespace. | 468 | |
'if' is not followed by whitespace. | 469 | |
'{' is not preceded with whitespace. | 469 |
Violation | Message | Line |
---|---|---|
Executable statement count is 84 (max allowed is 30). | 71 | |
NCSS for this method is 103 (max allowed is 50). | 71 | |
Variable 'zoom' should be declared final. | 234 | |
Variable 'zoomedOut' should be declared final. | 235 | |
'if' is not followed by whitespace. | 236 | |
'{' is not preceded with whitespace. | 236 | |
'for' is not followed by whitespace. | 237 | |
'{' is not preceded with whitespace. | 237 | |
'for' is not followed by whitespace. | 241 | |
'{' is not preceded with whitespace. | 241 |
Violation | Message | Line |
---|---|---|
'0.25F' is a magic number. | 45 | |
'0.5F' is a magic number. | 46 | |
'0.75F' is a magic number. | 47 | |
'8' is a magic number. | 49 | |
'128' is a magic number. | 49 | |
'4' is a magic number. | 62 | |
'0.25' is a magic number. | 67 | |
Control variable 'found' is modified. | 82 |
Violation | Message | Line |
---|---|---|
covariant equals without overriding equals(java.lang.Object). | 90 | |
Definition of 'equals()' without corresponding definition of 'hashCode()'. | 94 | |
covariant equals without overriding equals(java.lang.Object). | 94 |
Violation | Message | Line |
---|---|---|
Name 'initialLength' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 35 | |
Name 'loadFactor' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 36 | |
Line is longer than 160 characters. | 57 | |
Catching 'Exception' is not allowed. | 58 |
Violation | Message | Line |
---|---|---|
Name 'initialLength' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 42 | |
Name 'loadFactor' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 43 | |
Static variable definition in wrong order. | 67 | |
Static variable definition in wrong order. | 78 | |
Static variable definition in wrong order. | 89 | |
Static variable definition in wrong order. | 109 | |
'100' is a magic number. | 111 | |
'100' is a magic number. | 112 | |
'640' is a magic number. | 113 | |
'32768' is a magic number. | 114 | |
'100' is a magic number. | 115 | |
'100' is a magic number. | 116 | |
'1000' is a magic number. | 117 | |
'100' is a magic number. | 118 | |
'32768' is a magic number. | 119 | |
'100' is a magic number. | 120 | |
'512' is a magic number. | 129 | |
'512' is a magic number. | 130 | |
'32768' is a magic number. | 131 | |
'1250' is a magic number. | 134 | |
'100.0' is a magic number. | 135 | |
'32768.0' is a magic number. | 135 | |
'1250' is a magic number. | 136 | |
'1250' is a magic number. | 137 | |
'1250' is a magic number. | 138 | |
'1250' is a magic number. | 139 | |
'50' is a magic number. | 140 | |
'1250' is a magic number. | 141 | |
'8' is a magic number. | 168 | |
'6000' is a magic number. | 190 | |
'32' is a magic number. | 196 | |
Cyclomatic Complexity is 18 (max allowed is 10). | 223 | |
Executable statement count is 74 (max allowed is 30). | 223 | |
NCSS for this method is 90 (max allowed is 50). | 223 | |
Nested if-else depth is 3 (max allowed is 2). | 265 | |
Nested if-else depth is 4 (max allowed is 2). | 266 | |
Nested if-else depth is 3 (max allowed is 2). | 280 | |
Catching 'Exception' is not allowed. | 312 | |
Cyclomatic Complexity is 19 (max allowed is 10). | 346 | |
Executable statement count is 41 (max allowed is 30). | 346 | |
NCSS for this method is 56 (max allowed is 50). | 346 | |
NPath Complexity is 720 (max allowed is 200). | 346 | |
Return count is 3 (max allowed is 2). | 346 | |
Name 'HEADER_HAS_LENGTH_INDEX' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 347 | |
Name 'HEADER_HAS_SEQUENCE_INDEX' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 349 | |
'256' is a magic number. | 367 | |
'256' is a magic number. | 407 | |
Redundant 'final' modifier. | 433 | |
'8' is a magic number. | 434 | |
'16' is a magic number. | 434 | |
'32' is a magic number. | 434 | |
'256' is a magic number. | 466 | |
Return count is 4 (max allowed is 2). | 479 | |
Redundant 'final' modifier. | 497 | |
'0xFF' is a magic number. | 498 |
Violation | Message | Line |
---|---|---|
Name 'divideByOne' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 26 | |
Name 'addToZero' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 27 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 29 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 41 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 44 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 47 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 51 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 54 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 58 | |
The String "ID is null, needs to be a valid string!" appears 2 times in the file. | 65 | |
Cyclomatic Complexity is 12 (max allowed is 10). | 75 | |
Name 'ID' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 75 | |
'8' is a magic number. | 88 | |
'16' is a magic number. | 90 | |
'32' is a magic number. | 92 |
Violation | Message | Line |
---|---|---|
Return count is 4 (max allowed is 2). | 40 |
Violation | Message | Line |
---|---|---|
Return count is 6 (max allowed is 2). | 40 |
Violation | Message | Line |
---|---|---|
Array brackets at illegal position. | 52 | |
Variable 'realMinAndMaxFound' explicitly initialized to 'false' (default value for its type). | 57 | |
Variable 'displayMinAndMaxSet' explicitly initialized to 'false' (default value for its type). | 63 | |
Name 'PCS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 74 | |
'3' is a magic number. | 111 | |
Name 'PCL' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 160 | |
Name 'PCL' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 164 | |
Return count is 3 (max allowed is 2). | 178 | |
Throws count is 2 (max allowed is 1). | 179 | |
The String "Unsupported" appears 2 times in the file. | 183 | |
Redundant 'final' modifier. | 284 |
Violation | Message | Line |
---|---|---|
Variable access definition in wrong order. | 38 | |
Variable access definition in wrong order. | 39 | |
Variable access definition in wrong order. | 40 | |
Variable access definition in wrong order. | 43 | |
Name 'recordCountKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 43 | |
Variable access definition in wrong order. | 44 | |
Name 'tempResetKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 44 | |
Variable access definition in wrong order. | 45 | |
Name 'elapsedTimeKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 45 | |
Variable access definition in wrong order. | 55 | |
Name 'PCS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 55 | |
Variable 'PCS' must be private and have accessor methods. | 55 | |
Variable 'logStatusMessage' explicitly initialized to 'null' (default value for its type). | 57 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 111 | |
Variable 'GDE' should be declared final. | 111 | |
Executable statement count is 32 (max allowed is 30). | 129 | |
'if' is not followed by whitespace. | 132 | |
'{' is not preceded with whitespace. | 132 | |
Variable 'ourRuntime' should be declared final. | 136 | |
Line is longer than 160 characters. | 138 | |
The String ", Free: " appears 2 times in the file. | 138 | |
The String ", Total: " appears 2 times in the file. | 138 | |
'if' is not followed by whitespace. | 152 | |
Line is longer than 160 characters. | 156 | |
'+' should be on a new line. | 159 | |
'+' should be on a new line. | 160 | |
'+' is not preceded with whitespace. | 161 | |
Variable 'oldLogStatus' should be declared final. | 185 | |
Method 'setLogStatusMessage' is not designed for extension - needs to be abstract, final or empty. | 245 | |
Parameter message should be final. | 245 |
Violation | Message | Line |
---|---|---|
'600' is a magic number. | 71 | |
'400' is a magic number. | 71 | |
'600' is a magic number. | 74 | |
'20' is a magic number. | 74 | |
'10' is a magic number. | 82 | |
'10' is a magic number. | 84 | |
'if' is not followed by whitespace. | 98 | |
'{' is not preceded with whitespace. | 98 | |
Redundant 'final' modifier. | 136 | |
'{' is not preceded with whitespace. | 136 | |
Redundant 'final' modifier. | 143 | |
'{' is not preceded with whitespace. | 143 | |
'if' is not followed by whitespace. | 157 | |
'cast' is not followed by whitespace. | 162 | |
'cast' is not followed by whitespace. | 164 | |
'{' is not preceded with whitespace. | 177 | |
'cast' is not followed by whitespace. | 178 | |
'{' is not preceded with whitespace. | 179 | |
'cast' is not followed by whitespace. | 198 | |
'cast' is not followed by whitespace. | 200 | |
'{' is not preceded with whitespace. | 203 | |
'cast' is not followed by whitespace. | 205 | |
'{' is not preceded with whitespace. | 217 | |
'cast' is not followed by whitespace. | 218 | |
'{' is not preceded with whitespace. | 219 | |
Redundant 'final' modifier. | 237 | |
'{' is not preceded with whitespace. | 237 | |
'8' is a magic number. | 238 | |
'{' is not preceded with whitespace. | 240 | |
'{' is not preceded with whitespace. | 247 | |
'{' is not preceded with whitespace. | 249 | |
'{' is not preceded with whitespace. | 257 | |
'{' is not preceded with whitespace. | 259 | |
Method 'isZoomedOutBeyondOneToOne' is not designed for extension - needs to be abstract, final or empty. | 267 | |
'{' is not preceded with whitespace. | 267 | |
Redundant 'final' modifier. | 323 | |
'{' is not preceded with whitespace. | 323 | |
'if' is not followed by whitespace. | 325 | |
'{' is not preceded with whitespace. | 325 | |
'cast' is not followed by whitespace. | 328 | |
'cast' is not followed by whitespace. | 328 | |
Redundant 'final' modifier. | 337 | |
'if' is not followed by whitespace. | 351 | |
'{' is not preceded with whitespace. | 351 | |
Redundant 'final' modifier. | 361 | |
Redundant 'final' modifier. | 372 | |
'{' is not preceded with whitespace. | 375 | |
'{' is not preceded with whitespace. | 407 | |
'{' is not preceded with whitespace. | 409 | |
'if' is not followed by whitespace. | 422 | |
'{' is not preceded with whitespace. | 422 | |
'50' is a magic number. | 506 | |
Cyclomatic Complexity is 11 (max allowed is 10). | 519 | |
'cast' is not followed by whitespace. | 525 | |
'{' is not preceded with whitespace. | 527 | |
'if' is not followed by whitespace. | 528 | |
'{' is not preceded with whitespace. | 528 | |
'{' is not preceded with whitespace. | 533 | |
'{' is not preceded with whitespace. | 539 | |
'if' is not followed by whitespace. | 540 | |
'{' is not preceded with whitespace. | 540 | |
'{' is not preceded with whitespace. | 545 | |
Cyclomatic Complexity is 31 (max allowed is 10). | 554 | |
Executable statement count is 49 (max allowed is 30). | 554 | |
NCSS for this method is 79 (max allowed is 50). | 554 | |
NPath Complexity is 2,049 (max allowed is 200). | 554 | |
switch without "default" clause. | 556 | |
'0.75' is a magic number. | 599 | |
'if' is not followed by whitespace. | 605 | |
'{' is not preceded with whitespace. | 605 | |
'0.75' is a magic number. | 610 | |
'if' is not followed by whitespace. | 619 | |
'{' is not preceded with whitespace. | 619 | |
'0.75' is a magic number. | 624 | |
'0.75' is a magic number. | 634 | |
'if' is not followed by whitespace. | 640 | |
'{' is not preceded with whitespace. | 640 | |
'0.75' is a magic number. | 645 | |
'if' is not followed by whitespace. | 654 | |
'{' is not preceded with whitespace. | 654 | |
'0.75' is a magic number. | 659 |
Violation | Message | Line |
---|---|---|
'2.5' is a magic number. | 60 | |
'if' is not followed by whitespace. | 83 | |
'{' is not preceded with whitespace. | 83 | |
'if' is not followed by whitespace. | 94 | |
'{' is not preceded with whitespace. | 94 | |
'cast' is not followed by whitespace. | 107 | |
'cast' is not followed by whitespace. | 107 | |
'{' is not preceded with whitespace. | 108 | |
'6' is a magic number. | 109 | |
'if' is not followed by whitespace. | 112 | |
'{' is not preceded with whitespace. | 112 | |
'if' is not followed by whitespace. | 125 | |
'{' is not preceded with whitespace. | 125 | |
Variable 'fm' should be declared final. | 131 | |
'cast' is not followed by whitespace. | 138 | |
'cast' is not followed by whitespace. | 138 | |
'{' is not preceded with whitespace. | 139 | |
Variable 'positionData' should be declared final. | 141 | |
'if' is not followed by whitespace. | 142 | |
'0.5' is a magic number. | 142 | |
'{' is not preceded with whitespace. | 142 | |
Variable 'stringWidth' should be declared final. | 147 | |
'18' is a magic number. | 148 | |
'if' is not followed by whitespace. | 152 | |
'{' is not preceded with whitespace. | 152 | |
Redundant 'final' modifier. | 160 | |
'{' is not preceded with whitespace. | 160 | |
'if' is not followed by whitespace. | 164 | |
'{' is not preceded with whitespace. | 164 | |
'{' is not preceded with whitespace. | 165 | |
'{' is not preceded with whitespace. | 169 | |
'100.0' is a magic number. | 206 | |
Variable 'count' should be declared final. | 207 | |
'cast' is not followed by whitespace. | 207 | |
'cast' is not followed by whitespace. | 207 | |
'{' is not preceded with whitespace. | 209 | |
'{' is not preceded with whitespace. | 210 | |
'3' is a magic number. | 211 | |
'{' is not preceded with whitespace. | 215 | |
'3' is a magic number. | 216 | |
Redundant 'final' modifier. | 254 | |
'if' is not followed by whitespace. | 260 | |
'{' is not preceded with whitespace. | 260 | |
'{' is not preceded with whitespace. | 270 | |
'{' is not preceded with whitespace. | 273 | |
'{' is not preceded with whitespace. | 274 | |
'{' is not preceded with whitespace. | 292 | |
'if' is not followed by whitespace. | 297 | |
'16' is a magic number. | 297 | |
'{' is not preceded with whitespace. | 297 | |
'16' is a magic number. | 298 | |
'15' is a magic number. | 299 | |
'{' is not preceded with whitespace. | 299 |
Violation | Message | Line |
---|---|---|
Name 'FPScounter' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 44 | |
Variable 'FPScounter' explicitly initialized to '0' (default value for its type). | 44 | |
Name 'FPS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 45 | |
Variable 'FPS' explicitly initialized to '0' (default value for its type). | 45 | |
'255' is a magic number. | 49 | |
'255' is a magic number. | 49 | |
'255' is a magic number. | 49 | |
'100' is a magic number. | 49 | |
'170' is a magic number. | 50 | |
'-100' is a magic number. | 56 | |
'-100' is a magic number. | 57 | |
Cyclomatic Complexity is 14 (max allowed is 10). | 62 | |
Executable statement count is 49 (max allowed is 30). | 62 | |
NCSS for this method is 58 (max allowed is 50). | 62 | |
'1000' is a magic number. | 66 | |
'1000' is a magic number. | 71 | |
'0.001' is a magic number. | 71 | |
'12' is a magic number. | 81 | |
'3' is a magic number. | 95 | |
'4' is a magic number. | 96 | |
'if' is not followed by whitespace. | 112 | |
'{' is not preceded with whitespace. | 112 | |
Nested if-else depth is 3 (max allowed is 2). | 125 | |
'if' is not followed by whitespace. | 125 | |
'{' is not preceded with whitespace. | 125 | |
'15' is a magic number. | 128 | |
'8' is a magic number. | 128 | |
'15' is a magic number. | 128 | |
'15' is a magic number. | 130 | |
'15' is a magic number. | 130 |
Violation | Message | Line |
---|---|---|
'400' is a magic number. | 47 | |
'600' is a magic number. | 47 | |
'99' is a magic number. | 48 | |
Control variable 'i' is modified. | 105 |
Violation | Message | Line |
---|---|---|
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 49 | |
Variable 'zoomedOut' should be declared final. | 80 | |
'if' is not followed by whitespace. | 81 | |
'{' is not preceded with whitespace. | 81 | |
'else' is not followed by whitespace. | 83 | |
'{' is not preceded with whitespace. | 83 | |
Cyclomatic Complexity is 16 (max allowed is 10). | 91 | |
Executable statement count is 37 (max allowed is 30). | 91 | |
NPath Complexity is 258 (max allowed is 200). | 91 | |
'if' is not followed by whitespace. | 110 | |
'{' is not preceded with whitespace. | 110 | |
'cast' is not followed by whitespace. | 115 | |
'try' is not followed by whitespace. | 123 | |
'{' is not preceded with whitespace. | 123 | |
'{' is not preceded with whitespace. | 125 | |
'try' is not followed by whitespace. | 129 | |
'{' is not preceded with whitespace. | 129 | |
'{' is not preceded with whitespace. | 131 | |
'if' is not followed by whitespace. | 137 | |
'{' is not preceded with whitespace. | 137 | |
'if' is not followed by whitespace. | 141 | |
'{' is not preceded with whitespace. | 141 | |
'if' is not followed by whitespace. | 150 | |
'5' is a magic number. | 150 | |
'{' is not preceded with whitespace. | 150 | |
'3' is a magic number. | 154 | |
'3' is a magic number. | 154 | |
'{' is not preceded with whitespace. | 155 | |
'3' is a magic number. | 156 | |
'3' is a magic number. | 156 | |
'{' is not preceded with whitespace. | 165 | |
'GDE' hides a field. | 195 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 195 | |
Variable 'zoomedOut' should be declared final. | 213 | |
'if' is not followed by whitespace. | 215 | |
'{' is not preceded with whitespace. | 215 | |
Redundant 'final' modifier. | 229 | |
'{' is not preceded with whitespace. | 229 | |
Redundant 'final' modifier. | 250 | |
Parameter cursorPosition should be final. | 250 | |
'{' is not preceded with whitespace. | 250 | |
'cast' is not followed by whitespace. | 274 | |
Variable 'graphWindowWidth' should be declared final. | 275 | |
'3' is a magic number. | 278 | |
Cyclomatic Complexity is 12 (max allowed is 10). | 297 | |
Executable statement count is 34 (max allowed is 30). | 297 | |
'cast' is not followed by whitespace. | 299 | |
Variable 'graphWindowWidth' should be declared final. | 300 | |
'+=' is not preceded with whitespace. | 324 | |
'+=' is not followed by whitespace. | 324 | |
'{' is not preceded with whitespace. | 333 | |
Nested if-else depth is 3 (max allowed is 2). | 338 | |
'{' is not preceded with whitespace. | 338 | |
Nested if-else depth is 3 (max allowed is 2). | 341 | |
'{' is not preceded with whitespace. | 341 | |
Variable 'averageData' should be declared final. | 346 | |
'{' is not preceded with whitespace. | 347 | |
'{' is not preceded with whitespace. | 350 | |
'if' is not followed by whitespace. | 382 | |
'{' is not preceded with whitespace. | 382 |
Violation | Message | Line |
---|---|---|
Class Data Abstraction Coupling is 11 (max allowed is 7) classes [ActionListener, ActiveHeaderLabel, ArrayList, ContainerListener, Dimension, JButton, JLayeredPane, JPanel, JScrollPane, ModifyGraphPane, MouseMotionAdapter]. | 56 | |
Declaring variables, return values or parameters of type 'ArrayList' is not allowed. | 87 | |
Variable 'scroll' should be declared final. | 100 | |
Variable 'ih' should be declared final. | 112 | |
'1280' is a magic number. | 119 | |
Instance variable definition in wrong order. | 123 | |
Parameter e should be final. | 125 | |
Instance variable definition in wrong order. | 130 | |
Instance variable definition in wrong order. | 137 | |
Anonymous inner class length is 28 lines (max allowed is 20). | 137 | |
Cyclomatic Complexity is 12 (max allowed is 10). | 194 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 201 | |
Control variable 'i' is modified. | 208 | |
Instance variable definition in wrong order. | 244 | |
Anonymous inner class length is 31 lines (max allowed is 20). | 244 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 249 | |
'16' is a magic number. | 253 | |
Nested if-else depth is 3 (max allowed is 2). | 259 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 276 | |
Executable statement count is 35 (max allowed is 30). | 289 | |
Method 'updateFromLog' is not designed for extension - needs to be abstract, final or empty. | 289 | |
Declaring variables, return values or parameters of type 'ArrayList' is not allowed. | 305 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 312 | |
Parameter GCB should be final. | 344 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 344 | |
Parameter GDE should be final. | 344 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 344 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 363 | |
Variable 'GDE' must be private and have accessor methods. | 363 | |
Name 'AHL' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 364 | |
Variable 'AHL' must be private and have accessor methods. | 364 | |
Variable 'c' should be declared final. | 409 | |
Executable statement count is 34 (max allowed is 30). | 419 | |
'10' is a magic number. | 423 | |
'10' is a magic number. | 424 | |
'100' is a magic number. | 438 | |
'20' is a magic number. | 439 | |
'100' is a magic number. | 440 | |
'20' is a magic number. | 440 | |
'40' is a magic number. | 441 | |
'60' is a magic number. | 442 | |
'100' is a magic number. | 443 | |
'60' is a magic number. | 443 | |
'80' is a magic number. | 444 | |
'500' is a magic number. | 456 | |
'180' is a magic number. | 456 | |
'210' is a magic number. | 456 | |
'133' is a magic number. | 456 | |
Parameter gde should be final. | 462 | |
Parameter ahl should be final. | 462 | |
Catching 'Exception' is not allowed. | 474 | |
The String "TODO: do something with Auto field" appears 2 times in the file. | 475 | |
Catching 'Exception' is not allowed. | 480 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 494 | |
Anonymous inner class length is 56 lines (max allowed is 20). | 500 | |
Parameter e should be final. | 503 | |
'16' is a magic number. | 504 | |
'18' is a magic number. | 506 | |
Parameter e should be final. | 515 | |
Parameter e should be final. | 519 | |
Parameter e should be final. | 523 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 524 | |
Variable 'GCB' should be declared final. | 524 | |
Parameter e should be final. | 529 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 530 | |
Variable 'GCB' should be declared final. | 530 | |
Parameter x should be final. | 568 | |
Parameter y should be final. | 568 | |
Parameter widht should be final. | 568 | |
Parameter height should be final. | 568 | |
Parameter GDE should be final. | 575 | |
'GDE' hides a field. | 575 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 575 | |
Parameter e should be final. | 584 | |
The String "<HTML> Data Stream: </b>" appears 2 times in the file. | 590 | |
The String "<br>To modify Min and Max values for scaling purposes Ctrl+LeftClick</HTML>" appears 2 times in the file. | 595 | |
Parameter previousPanel should be final. | 615 | |
Parameter dragging should be final. | 631 | |
Parameter selected should be final. | 635 | |
Parameter o should be final. | 658 | |
Name 'GCB' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 660 | |
Variable 'GCB' should be declared final. | 660 |
Violation | Message | Line |
---|---|---|
Class Data Abstraction Coupling is 19 (max allowed is 7) classes [ActionListener, ArrayList, BorderLayout, BufferedWriter, Color, Dimension, File, FileReader, FileWriter, FlowLayout, JButton, JMenu, JMenuBar, JMenuItem, JPanel, JScrollPane, PropertyPanel, Scanner, SingleProperty]. | 59 | |
Class Fan-Out Complexity is 23 (max allowed is 20). | 59 | |
Name 'OLVProperties' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 62 | |
'350' is a magic number. | 71 | |
'500' is a magic number. | 71 | |
'550' is a magic number. | 72 | |
'500' is a magic number. | 72 | |
'400' is a magic number. | 78 | |
The String ".OpenLogViewer" appears 3 times in the file. | 102 | |
The String "OLVProperties.olv" appears 2 times in the file. | 108 | |
Name 'IOE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 115 | |
Catching 'Exception' is not allowed. | 122 | |
Name 'E' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 122 | |
Parameter evt should be final. | 137 | |
Variable 's' should be declared final. | 138 | |
')' is preceded with whitespace. | 139 | |
Variable 'newprop' should be declared final. | 140 | |
Parameter evt should be final. | 150 | |
'500' is a magic number. | 163 | |
'32' is a magic number. | 163 | |
Variable 'okButton' should be declared final. | 166 | |
Variable 'cancel' should be declared final. | 167 | |
Parameter e should be final. | 172 | |
Parameter e should be final. | 181 | |
Variable 'scan' should be declared final. | 196 | |
Variable 'propLine' should be declared final. | 199 | |
Variable 'sp' should be declared final. | 200 | |
Variable 'prop' should be declared final. | 201 | |
'3' is a magic number. | 207 | |
'4' is a magic number. | 208 | |
'5' is a magic number. | 209 | |
'6' is a magic number. | 210 | |
Name 'FNF' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 215 | |
Catching 'Exception' is not allowed. | 234 | |
Variable 'pp' should be declared final. | 242 | |
Variable 'pp' should be declared final. | 249 | |
Variable 'pp' should be declared final. | 263 | |
Variable 'pp' should be declared final. | 272 | |
'60' is a magic number. | 277 | |
Variable 'pp' should be declared final. | 303 | |
'60' is a magic number. | 311 | |
Control variable 'i' is modified. | 314 | |
Class Data Abstraction Coupling is 8 (max allowed is 7) classes [Dimension, FlowLayout, JCheckBox, JComboBox, JLabel, JPanel, JTextField, MouseListener]. | 320 | |
Variable 'sp' must be private and have accessor methods. | 322 | |
Variable 'check' must be private and have accessor methods. | 323 | |
Variable 'colorBox' must be private and have accessor methods. | 324 | |
Variable 'minBox' must be private and have accessor methods. | 325 | |
Variable 'maxBox' must be private and have accessor methods. | 326 | |
Variable 'splitBox' must be private and have accessor methods. | 327 | |
Variable 'activeBox' must be private and have accessor methods. | 328 | |
Executable statement count is 39 (max allowed is 30). | 330 | |
'10' is a magic number. | 333 | |
'500' is a magic number. | 335 | |
'50' is a magic number. | 335 | |
Variable 'minLabel' should be declared final. | 336 | |
Variable 'maxLabel' should be declared final. | 337 | |
Variable 'colorLabel' should be declared final. | 338 | |
Variable 'splitLabel' should be declared final. | 339 | |
Variable 'activeLabel' should be declared final. | 340 | |
'15' is a magic number. | 342 | |
'20' is a magic number. | 342 | |
'50' is a magic number. | 345 | |
'20' is a magic number. | 345 | |
'50' is a magic number. | 348 | |
'20' is a magic number. | 348 | |
'30' is a magic number. | 352 | |
'20' is a magic number. | 352 | |
Variable 'tf' should be declared final. | 353 | |
'60' is a magic number. | 360 | |
'20' is a magic number. | 360 | |
Anonymous inner class length is 29 lines (max allowed is 20). | 363 | |
Parameter e should be final. | 366 | |
Variable 'newColor' should be declared final. | 367 | |
Parameter e should be final. | 377 | |
Parameter e should be final. | 381 | |
Parameter e should be final. | 385 | |
Parameter e should be final. | 389 | |
Parameter sp should be final. | 414 | |
Variable 'active' should be declared final. | 423 | |
Parameter pp should be final. | 436 |
Violation | Message | Line |
---|---|---|
Parameter GDE should be final. | 45 | |
Name 'GDE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 45 | |
Parameter split should be final. | 95 | |
Assignment of parameter 'split' is not allowed. | 97 | |
The String "," appears 6 times in the file. | 113 | |
Definition of 'equals()' without corresponding definition of 'hashCode()'. | 125 | |
covariant equals without overriding equals(java.lang.Object). | 125 |