The following document contains the results of Checkstyle. 
| Files | Infos | Warnings | Errors |
|---|---|---|---|
| 26 | 0 | 0 | 1289 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 7 | |
| NewlineAtEndOfFile | 0 | |
| Translation | 0 | |
| JavadocMethod | 72 | |
| JavadocType | 0 | |
| JavadocVariable | 11 | |
| JavadocStyle | 13 | |
| ConstantName | 0 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 8 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 31 | |
| IllegalImport | 0 | |
| RedundantImport | 8 | |
| UnusedImports | 8 | |
| FileLength | 0 | |
| LineLength | 25 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 1 | |
| NoWhitespaceBefore | 21 | |
| OperatorWrap | 13 | |
| ParenPad | 413 | |
| TypecastParenPad | 0 | |
| TabCharacter | 204 | |
| WhitespaceAfter | 14 | |
| WhitespaceAround | 131 | |
| ModifierOrder | 3 | |
| RedundantModifier | 13 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 2 | |
| LeftCurly | 0 | |
| NeedBraces | 29 | |
| RightCurly | 14 | |
| AvoidInlineConditionals | 0 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 9 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 8 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 8 | |
| SimplifyBooleanExpression | 4 | |
| SimplifyBooleanReturn | 1 | |
| DesignForExtension | 17 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 5 | |
| InterfaceIsType | 2 | |
| VisibilityModifier | 10 | |
| ArrayTypeStyle | 1 | |
| FinalParameters | 98 | |
GenericIllegalRegexp
| 83 | |
| TodoComment | 2 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 71 | |
| '6f' is a magic number. | 78 | |
| '0.5f' is a magic number. | 87 | |
| Parameter length should be final. | 118 | |
| 'length' hides a field. | 118 | |
| Parameter length should be final. | 129 | |
| 'length' hides a field. | 129 | |
| Parameter showLabels should be final. | 129 | |
| Parameter lineType should be final. | 129 | |
| 'lineType' hides a field. | 129 | |
| Expected an @return tag. | 162 | |
| Parameter startPoint should be final. | 162 | |
| Expected @param tag for 'startPoint'. | 162 | |
| Parameter endPoint should be final. | 162 | |
| Expected @param tag for 'endPoint'. | 162 | |
| Parameter refColor should be final. | 163 | |
| Expected @param tag for 'refColor'. | 163 | |
| Expected an @return tag. | 218 | |
| Parameter label should be final. | 218 | |
| Expected @param tag for 'label'. | 218 | |
| Parameter position should be final. | 218 | |
| Expected @param tag for 'position'. | 218 | |
| Name 'Textlabel' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 221 | |
| Parameter length should be final. | 258 | |
| 'length' hides a field. | 258 | |
| Parameter lineType should be final. | 267 | |
| 'lineType' hides a field. | 267 | |
| Expected @param tag for 'lineType'. | 267 | |
| Comment matches to-do format 'TODO:'. | 268 | |
| Comment matches to-do format 'TODO:'. | 270 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Unused import - java.util.ArrayList. | 31 | |
| Unused import - javax.vecmath.Color3f. | 35 | |
| Unused import - org.psychomad.psycholib.log.Log. | 38 | |
| Utility classes should not have a public or default constructor. | 46 | |
| Line contains a tab character. | 48 | |
| Line is longer than 80 characters. | 49 | |
| Line contains a tab character. | 49 | |
| Line contains a tab character. | 50 | |
| Line contains a tab character. | 51 | |
| Line is longer than 80 characters. | 52 | |
| Line contains a tab character. | 52 | |
| Line contains a tab character. | 53 | |
| Line is longer than 80 characters. | 54 | |
| Line contains a tab character. | 54 | |
| Line contains a tab character. | 55 | |
| Line contains a tab character. | 56 | |
| '(' is followed by whitespace. | 56 | |
| Parameter tr should be final. | 56 | |
| Parameter index should be final. | 56 | |
| ')' is preceded with whitespace. | 56 | |
| '(' is followed by whitespace. | 60 | |
| ')' is preceded with whitespace. | 60 | |
| '(' is followed by whitespace. | 61 | |
| '+' is not preceded with whitespace. | 61 | |
| '+' is not followed by whitespace. | 61 | |
| ')' is preceded with whitespace. | 61 | |
| '(' is followed by whitespace. | 62 | |
| '+' is not preceded with whitespace. | 62 | |
| '+' is not followed by whitespace. | 62 | |
| ')' is preceded with whitespace. | 62 | |
| Line contains a tab character. | 64 | |
| Line contains a tab character. | 65 | |
| Line contains a tab character. | 66 | |
| Line contains a tab character. | 68 | |
| Line contains a tab character. | 69 | |
| Line contains a tab character. | 71 | |
| Line contains a tab character. | 72 | |
| 'for' construct must use '{}'s. | 74 | |
| Line contains a tab character. | 74 | |
| 'for' is not followed by whitespace. | 74 | |
| '(' is followed by whitespace. | 74 | |
| '+' is not preceded with whitespace. | 74 | |
| '+' is not followed by whitespace. | 74 | |
| '3' is a magic number. | 74 | |
| ')' is preceded with whitespace. | 74 | |
| Line contains a tab character. | 75 | |
| '(' is followed by whitespace. | 75 | |
| ')' is preceded with whitespace. | 75 | |
| Line contains a tab character. | 76 | |
| Line contains a tab character. | 79 | |
| Line contains a tab character. | 80 | |
| Line contains a tab character. | 81 | |
| Line contains a tab character. | 82 | |
| Line is longer than 80 characters. | 83 | |
| Line contains a tab character. | 83 | |
| Line contains a tab character. | 84 | |
| Line is longer than 80 characters. | 85 | |
| Line contains a tab character. | 85 | |
| Line contains a tab character. | 86 | |
| Line contains a tab character. | 87 | |
| Line contains a tab character. | 88 | |
| Line contains a tab character. | 89 | |
| '(' is followed by whitespace. | 89 | |
| Parameter tr should be final. | 89 | |
| Parameter index should be final. | 89 | |
| ')' is preceded with whitespace. | 89 | |
| Line contains a tab character. | 90 | |
| Line contains a tab character. | 91 | |
| Line contains a tab character. | 92 | |
| Line contains a tab character. | 94 | |
| '(' is followed by whitespace. | 94 | |
| ')' is preceded with whitespace. | 94 | |
| Line contains a tab character. | 95 | |
| '(' is followed by whitespace. | 95 | |
| '+' is not preceded with whitespace. | 95 | |
| '+' is not followed by whitespace. | 95 | |
| ')' is preceded with whitespace. | 95 | |
| Line contains a tab character. | 96 | |
| '(' is followed by whitespace. | 96 | |
| '+' is not preceded with whitespace. | 96 | |
| '+' is not followed by whitespace. | 96 | |
| ')' is preceded with whitespace. | 96 | |
| Line contains a tab character. | 98 | |
| Line has trailing spaces. | 103 | |
| Line contains a tab character. | 103 | |
| Line contains a tab character. | 104 | |
| Line contains a tab character. | 105 | |
| '(' is followed by whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| Line contains a tab character. | 106 | |
| '/' is not preceded with whitespace. | 106 | |
| '/' is not followed by whitespace. | 106 | |
| '3f' is a magic number. | 106 | |
| Line contains a tab character. | 108 | |
| Line contains a tab character. | 109 | |
| Line contains a tab character. | 110 | |
| Line contains a tab character. | 111 | |
| Line contains a tab character. | 113 | |
| Line contains a tab character. | 114 | |
| Line contains a tab character. | 117 | |
| Line contains a tab character. | 118 | |
| Line contains a tab character. | 119 | |
| Line contains a tab character. | 120 | |
| Line contains a tab character. | 121 | |
| Line is longer than 80 characters. | 122 | |
| Line contains a tab character. | 122 | |
| Line contains a tab character. | 123 | |
| Line is longer than 80 characters. | 124 | |
| Line contains a tab character. | 124 | |
| Line contains a tab character. | 125 | |
| Expected an @return tag. | 126 | |
| Line contains a tab character. | 126 | |
| '(' is followed by whitespace. | 126 | |
| Parameter tr should be final. | 126 | |
| Parameter index should be final. | 126 | |
| ')' is preceded with whitespace. | 126 | |
| Line contains a tab character. | 127 | |
| Line contains a tab character. | 128 | |
| Line contains a tab character. | 129 | |
| Line contains a tab character. | 131 | |
| '(' is followed by whitespace. | 131 | |
| ')' is preceded with whitespace. | 131 | |
| Line contains a tab character. | 132 | |
| '(' is followed by whitespace. | 132 | |
| '+' is not preceded with whitespace. | 132 | |
| '+' is not followed by whitespace. | 132 | |
| ')' is preceded with whitespace. | 132 | |
| Line contains a tab character. | 133 | |
| '(' is followed by whitespace. | 133 | |
| '+' is not preceded with whitespace. | 133 | |
| '+' is not followed by whitespace. | 133 | |
| ')' is preceded with whitespace. | 133 | |
| Line has trailing spaces. | 134 | |
| Line contains a tab character. | 134 | |
| Line contains a tab character. | 135 | |
| Line contains a tab character. | 136 | |
| Line contains a tab character. | 137 | |
| Line contains a tab character. | 139 | |
| '+' is not preceded with whitespace. | 139 | |
| '+' is not followed by whitespace. | 139 | |
| '+' is not preceded with whitespace. | 139 | |
| '+' is not followed by whitespace. | 139 | |
| Line contains a tab character. | 141 | |
| 'cast' is not followed by whitespace. | 141 | |
| '*' is not preceded with whitespace. | 141 | |
| '*' is not followed by whitespace. | 141 | |
| '-' is not preceded with whitespace. | 141 | |
| '-' is not followed by whitespace. | 141 | |
| '*' is not preceded with whitespace. | 141 | |
| '*' is not followed by whitespace. | 141 | |
| '-' is not preceded with whitespace. | 141 | |
| '-' is not followed by whitespace. | 141 | |
| '*' is not preceded with whitespace. | 141 | |
| '*' is not followed by whitespace. | 141 | |
| '-' is not preceded with whitespace. | 141 | |
| '-' is not followed by whitespace. | 141 | |
| Line contains a tab character. | 143 | |
| Line contains a tab character. | 144 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| interfaces should describe a type and hence have methods. | 34 | |
| Redundant 'public' modifier. | 40 | |
| Line has trailing spaces. | 41 | |
| Redundant 'public' modifier. | 46 | |
| Line has trailing spaces. | 47 | |
| Redundant 'public' modifier. | 52 | |
| Line has trailing spaces. | 53 | |
| First sentence should end with a period. | 54 | |
| Redundant 'public' modifier. | 58 | |
| Line has trailing spaces. | 59 | |
| First sentence should end with a period. | 60 | |
| Redundant 'public' modifier. | 64 | |
| Line has trailing spaces. | 65 | |
| First sentence should end with a period. | 66 | |
| Redundant 'public' modifier. | 70 | |
| Redundant 'public' modifier. | 76 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - javax.swing.*. | 32 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyDialog. | 35 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyActions. | 36 | |
| Unused import - org.psychomad.psycholib.std.ExitCodes. | 39 | |
| '(' is followed by whitespace. | 55 | |
| Parameter myActions should be final. | 55 | |
| Expected @param tag for 'myActions'. | 55 | |
| Parameter config should be final. | 55 | |
| Expected @param tag for 'config'. | 55 | |
| Parameter resources should be final. | 56 | |
| Expected @param tag for 'resources'. | 56 | |
| ')' is preceded with whitespace. | 56 | |
| Line has trailing spaces. | 60 | |
| '(' is followed by whitespace. | 61 | |
| ')' is preceded with whitespace. | 61 | |
| Method 'fillToolBar' is not designed for extension - needs to be abstract, final or empty. | 68 | |
| '(' is followed by whitespace. | 68 | |
| Parameter myActions should be final. | 68 | |
| Expected @param tag for 'myActions'. | 68 | |
| Parameter config should be final. | 68 | |
| Expected @param tag for 'config'. | 68 | |
| Parameter resources should be final. | 69 | |
| Expected @param tag for 'resources'. | 69 | |
| ')' is preceded with whitespace. | 69 | |
| '(' is followed by whitespace. | 71 | |
| ')' is preceded with whitespace. | 71 | |
| 'for' is not followed by whitespace. | 74 | |
| '(' is followed by whitespace. | 74 | |
| ')' is preceded with whitespace. | 74 | |
| '}' should be on the same line. | 77 | |
| 'if' is not followed by whitespace. | 80 | |
| '(' is followed by whitespace. | 80 | |
| ')' is preceded with whitespace. | 80 | |
| '+' should be on a new line. | 81 | |
| '}' should be on the same line. | 83 | |
| 'cast' is not followed by whitespace. | 86 | |
| 'if' is not followed by whitespace. | 87 | |
| '(' is followed by whitespace. | 87 | |
| ')' is preceded with whitespace. | 87 | |
| '(' is followed by whitespace. | 89 | |
| ')' is preceded with whitespace. | 89 | |
| '}' should be on the same line. | 90 | |
| 'if' is not followed by whitespace. | 94 | |
| '(' is followed by whitespace. | 94 | |
| '(' is followed by whitespace. | 94 | |
| ')' is preceded with whitespace. | 94 | |
| Expression can be simplified. | 94 | |
| ')' is preceded with whitespace. | 94 | |
| '+' is not preceded with whitespace. | 97 | |
| '+' is not followed by whitespace. | 97 | |
| 'cast' is not followed by whitespace. | 101 | |
| '(' is followed by whitespace. | 104 | |
| ')' is preceded with whitespace. | 104 | |
| '(' is followed by whitespace. | 109 | |
| ')' is preceded with whitespace. | 109 | |
| '(' is followed by whitespace. | 113 | |
| ')' is preceded with whitespace. | 113 | |
| Line is longer than 80 characters. | 121 | |
| Line is longer than 80 characters. | 122 | |
| Line has trailing spaces. | 128 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.awt.*. | 31 | |
| Using the '.*' form of import should be avoided - java.awt.event.*. | 32 | |
| Using the '.*' form of import should be avoided - java.beans.*. | 33 | |
| Using the '.*' form of import should be avoided - java.util.*. | 34 | |
| Using the '.*' form of import should be avoided - java.io.*. | 35 | |
| Using the '.*' form of import should be avoided - java.applet.*. | 36 | |
| Using the '.*' form of import should be avoided - java.net.*. | 37 | |
| Using the '.*' form of import should be avoided - javax.swing.*. | 39 | |
| Using the '.*' form of import should be avoided - javax.swing.event.*. | 40 | |
| Using the '.*' form of import should be avoided - javax.swing.text.*. | 41 | |
| Using the '.*' form of import should be avoided - javax.swing.text.html.*. | 42 | |
| Using the '.*' form of import should be avoided - javax.swing.border.*. | 43 | |
| Using the '.*' form of import should be avoided - javax.swing.colorchooser.*. | 44 | |
| Using the '.*' form of import should be avoided - javax.swing.filechooser.*. | 45 | |
| Using the '.*' form of import should be avoided - javax.accessibility.*. | 46 | |
| Missing a Javadoc comment. | 54 | |
| Variable 'html' must be private and have accessor methods. | 54 | |
| Line has trailing spaces. | 55 | |
| Missing a Javadoc comment. | 55 | |
| Variable 'panel' must be private and have accessor methods. | 55 | |
| Missing a Javadoc comment. | 56 | |
| Variable 'filename' must be private and have accessor methods. | 56 | |
| Missing a Javadoc comment. | 57 | |
| Variable 'frame' must be private and have accessor methods. | 57 | |
| Line contains a tab character. | 60 | |
| Line contains a tab character. | 61 | |
| Line contains a tab character. | 62 | |
| Method 'mainImpl' is not designed for extension - needs to be abstract, final or empty. | 63 | |
| '(' is followed by whitespace. | 63 | |
| Parameter width should be final. | 63 | |
| Expected @param tag for 'width'. | 63 | |
| Parameter height should be final. | 63 | |
| Expected @param tag for 'height'. | 63 | |
| ')' is preceded with whitespace. | 63 | |
| '(' is followed by whitespace. | 64 | |
| ')' is preceded with whitespace. | 64 | |
| '(' is followed by whitespace. | 65 | |
| '(' is followed by whitespace. | 66 | |
| Parameter e should be final. | 66 | |
| ')' is preceded with whitespace. | 66 | |
| '}' is not followed by whitespace. | 68 | |
| '}' is not preceded with whitespace. | 68 | |
| Line contains a tab character. | 78 | |
| Line contains a tab character. | 79 | |
| Line contains a tab character. | 80 | |
| Method 'display' is not designed for extension - needs to be abstract, final or empty. | 81 | |
| Line has trailing spaces. | 86 | |
| Line contains a tab character. | 88 | |
| Line contains a tab character. | 89 | |
| Line contains a tab character. | 90 | |
| Expected an @return tag. | 91 | |
| Method 'getDemoPanel' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| Line has trailing spaces. | 94 | |
| First sentence should end with a period. | 96 | |
| '(' is followed by whitespace. | 103 | |
| Parameter htmlFile should be final. | 103 | |
| Parameter width should be final. | 103 | |
| Parameter height should be final. | 103 | |
| ')' is preceded with whitespace. | 103 | |
| Line has trailing spaces. | 105 | |
| '+' is not preceded with whitespace. | 116 | |
| '+' is not followed by whitespace. | 116 | |
| '}' should be on the same line. | 117 | |
| Line is longer than 80 characters. | 119 | |
| Line has trailing spaces. | 122 | |
| 'if' is not followed by whitespace. | 123 | |
| Line has trailing spaces. | 127 | |
| '(' is followed by whitespace. | 138 | |
| ')' is preceded with whitespace. | 138 | |
| Expected an @return tag. | 145 | |
| Method 'createHyperLinkListener' is not designed for extension - needs to be abstract, final or empty. | 145 | |
| Line contains a tab character. | 147 | |
| Parameter e should be final. | 147 | |
| Line is longer than 80 characters. | 150 | |
| 'cast' is not followed by whitespace. | 150 | |
| 'cast' is not followed by whitespace. | 150 | |
| Line contains a tab character. | 159 | |
| Line has trailing spaces. | 162 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| interfaces should describe a type and hence have methods. | 36 | |
| First sentence should end with a period. | 38 | |
| Redundant 'public' modifier. | 39 | |
| First sentence should end with a period. | 40 | |
| Redundant 'public' modifier. | 41 | |
| First sentence should end with a period. | 42 | |
| Redundant 'public' modifier. | 43 | |
| First sentence should end with a period. | 44 | |
| Redundant 'public' modifier. | 45 | |
| First sentence should end with a period. | 46 | |
| Redundant 'public' modifier. | 47 | |
| First sentence should end with a period. | 48 | |
| Redundant 'public' modifier. | 49 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 31 | |
| Line contains a tab character. | 42 | |
| '(' is followed by whitespace. | 45 | |
| Parameter msg should be final. | 45 | |
| ')' is preceded with whitespace. | 45 | |
| '(' is followed by whitespace. | 46 | |
| Line contains a tab character. | 47 | |
| ')' is preceded with whitespace. | 47 |
| Violation | Message | Line |
|---|---|---|
| Unused import - org.psychomad.psycholib.resources.Config. | 35 | |
| Line has trailing spaces. | 40 | |
| Line is longer than 80 characters. | 44 | |
| First sentence should end with a period. | 50 | |
| Line contains a tab character. | 59 | |
| '(' is followed by whitespace. | 63 | |
| Parameter actionName should be final. | 63 | |
| Parameter currentClass should be final. | 63 | |
| Expected @param tag for 'currentClass'. | 63 | |
| Parameter resources should be final. | 64 | |
| Expected @param tag for 'resources'. | 64 | |
| ')' is preceded with whitespace. | 64 | |
| '(' is followed by whitespace. | 65 | |
| ')' is preceded with whitespace. | 65 | |
| '(' is followed by whitespace. | 66 | |
| ')' is preceded with whitespace. | 66 | |
| '(' is followed by whitespace. | 79 | |
| Parameter actionDefaultName should be final. | 79 | |
| Parameter currentClass should be final. | 79 | |
| Expected @param tag for 'currentClass'. | 79 | |
| Parameter resources should be final. | 80 | |
| Expected @param tag for 'resources'. | 80 | |
| ')' is preceded with whitespace. | 80 | |
| 'if' is not followed by whitespace. | 81 | |
| '(' is followed by whitespace. | 81 | |
| ')' is preceded with whitespace. | 81 | |
| Line has trailing spaces. | 85 | |
| '(' is followed by whitespace. | 90 | |
| ')' is preceded with whitespace. | 90 | |
| Line has trailing spaces. | 102 | |
| 'if' construct must use '{}'s. | 104 | |
| 'if' is not followed by whitespace. | 104 | |
| '(' is followed by whitespace. | 104 | |
| ')' is preceded with whitespace. | 104 | |
| '(' is followed by whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| '(' is followed by whitespace. | 108 | |
| ')' is preceded with whitespace. | 108 | |
| 'if' is not followed by whitespace. | 109 | |
| '(' is followed by whitespace. | 109 | |
| ')' is preceded with whitespace. | 109 | |
| '(' is followed by whitespace. | 110 | |
| ')' is preceded with whitespace. | 110 | |
| 'if' construct must use '{}'s. | 111 | |
| 'if' is not followed by whitespace. | 111 | |
| '(' is followed by whitespace. | 111 | |
| ')' is preceded with whitespace. | 111 | |
| '(' is followed by whitespace. | 112 | |
| ')' is preceded with whitespace. | 112 | |
| 'else' construct must use '{}'s. | 113 | |
| Line is longer than 80 characters. | 114 | |
| '+' is not preceded with whitespace. | 114 | |
| '+' is not followed by whitespace. | 114 | |
| '+' is not preceded with whitespace. | 114 | |
| '+' is not followed by whitespace. | 114 | |
| '(' is followed by whitespace. | 118 | |
| '+' should be on a new line. | 118 | |
| ')' is preceded with whitespace. | 119 | |
| 'if' construct must use '{}'s. | 120 | |
| 'if' is not followed by whitespace. | 120 | |
| '(' is followed by whitespace. | 120 | |
| ')' is preceded with whitespace. | 120 | |
| '(' is followed by whitespace. | 121 | |
| ')' is preceded with whitespace. | 121 | |
| '+' is not preceded with whitespace. | 126 | |
| '+' is not followed by whitespace. | 126 | |
| ')' is preceded with whitespace. | 126 | |
| 'if' is not followed by whitespace. | 127 | |
| '(' is followed by whitespace. | 127 | |
| ')' is preceded with whitespace. | 127 | |
| '(' is followed by whitespace. | 128 | |
| ')' is preceded with whitespace. | 128 | |
| '(' is followed by whitespace. | 134 | |
| '+' should be on a new line. | 134 | |
| ')' is preceded with whitespace. | 135 | |
| 'if' construct must use '{}'s. | 136 | |
| 'cast' is not followed by whitespace. | 137 | |
| Line has trailing spaces. | 138 | |
| '(' is followed by whitespace. | 138 | |
| ')' is preceded with whitespace. | 138 | |
| '(' is followed by whitespace. | 141 | |
| ')' is preceded with whitespace. | 141 | |
| 'if' construct must use '{}'s. | 142 | |
| '(' is followed by whitespace. | 143 | |
| ')' is preceded with whitespace. | 143 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 27 | |
| Using the '.*' form of import should be avoided - javax.swing.*. | 28 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyDialog. | 30 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyAbstractActionValues. | 31 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyActions. | 32 | |
| Unused import - org.psychomad.psycholib.resources.Config. | 34 | |
| Unused import - org.psychomad.psycholib.std.ExitCodes. | 36 | |
| Parameter myActions should be final. | 59 | |
| Expected @param tag for 'myActions'. | 59 | |
| Parameter currentClass should be final. | 59 | |
| Expected @param tag for 'currentClass'. | 59 | |
| Parameter resources should be final. | 60 | |
| Expected @param tag for 'resources'. | 60 | |
| ')' is preceded with whitespace. | 60 | |
| Line has trailing spaces. | 62 | |
| '(' is followed by whitespace. | 63 | |
| ')' is preceded with whitespace. | 63 | |
| Method 'fillMenuBar' is not designed for extension - needs to be abstract, final or empty. | 70 | |
| '(' is followed by whitespace. | 70 | |
| Parameter myActions should be final. | 70 | |
| Expected @param tag for 'myActions'. | 70 | |
| Parameter currentClass should be final. | 70 | |
| Expected @param tag for 'currentClass'. | 70 | |
| Parameter resources should be final. | 71 | |
| Expected @param tag for 'resources'. | 71 | |
| ')' is preceded with whitespace. | 71 | |
| Line has trailing spaces. | 73 | |
| '(' is followed by whitespace. | 74 | |
| ')' is preceded with whitespace. | 74 | |
| 'for' is not followed by whitespace. | 75 | |
| '(' is followed by whitespace. | 75 | |
| ')' is preceded with whitespace. | 75 | |
| '(' is followed by whitespace. | 76 | |
| ')' is preceded with whitespace. | 76 | |
| Expected an @return tag. | 89 | |
| Method 'createMenu' is not designed for extension - needs to be abstract, final or empty. | 89 | |
| '(' is followed by whitespace. | 89 | |
| Parameter myActions should be final. | 89 | |
| Expected @param tag for 'myActions'. | 89 | |
| Parameter key should be final. | 89 | |
| Expected @param tag for 'key'. | 89 | |
| Parameter currentClass should be final. | 90 | |
| Expected @param tag for 'currentClass'. | 90 | |
| Parameter resources should be final. | 90 | |
| Expected @param tag for 'resources'. | 90 | |
| ')' is preceded with whitespace. | 90 | |
| ',' is not followed by whitespace. | 94 | |
| '+' should be on a new line. | 97 | |
| '(' is followed by whitespace. | 100 | |
| '+' should be on a new line. | 100 | |
| ')' is preceded with whitespace. | 101 | |
| 'if' construct must use '{}'s. | 102 | |
| 'if' is not followed by whitespace. | 102 | |
| '(' is followed by whitespace. | 102 | |
| ')' is preceded with whitespace. | 102 | |
| '(' is followed by whitespace. | 103 | |
| ')' is preceded with whitespace. | 103 | |
| 'for' is not followed by whitespace. | 106 | |
| '(' is followed by whitespace. | 106 | |
| ')' is preceded with whitespace. | 106 | |
| '}' should be on the same line. | 109 | |
| 'if' is not followed by whitespace. | 110 | |
| '(' is followed by whitespace. | 110 | |
| ')' is preceded with whitespace. | 110 | |
| '(' is followed by whitespace. | 116 | |
| ')' is preceded with whitespace. | 116 | |
| '}' should be on the same line. | 117 | |
| 'if' is not followed by whitespace. | 121 | |
| '(' is followed by whitespace. | 121 | |
| ')' is preceded with whitespace. | 121 | |
| '+' should be on a new line. | 122 | |
| '}' should be on the same line. | 124 | |
| 'else' construct must use '{}'s. | 125 | |
| '(' is followed by whitespace. | 126 | |
| ')' is preceded with whitespace. | 126 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 36 | |
| Utility classes should not have a public or default constructor. | 41 | |
| Line has trailing spaces. | 43 | |
| Line contains a tab character. | 53 | |
| '(' is followed by whitespace. | 56 | |
| Parameter msg should be final. | 56 | |
| ')' is preceded with whitespace. | 56 | |
| '(' is followed by whitespace. | 57 | |
| ')' is preceded with whitespace. | 57 | |
| Line has trailing spaces. | 59 | |
| Line contains a tab character. | 59 | |
| Line contains a tab character. | 64 | |
| '(' is followed by whitespace. | 67 | |
| Parameter msg should be final. | 67 | |
| ')' is preceded with whitespace. | 67 | |
| '(' is followed by whitespace. | 68 | |
| ')' is preceded with whitespace. | 68 | |
| Line has trailing spaces. | 71 | |
| Line contains a tab character. | 71 | |
| Line contains a tab character. | 74 | |
| '(' is followed by whitespace. | 78 | |
| Parameter newDebugMode should be final. | 78 | |
| ')' is preceded with whitespace. | 78 | |
| Line has trailing spaces. | 81 | |
| Line contains a tab character. | 84 | |
| Line contains a tab character. | 97 | |
| Line contains a tab character. | 99 | |
| '(' is followed by whitespace. | 103 | |
| Parameter msg should be final. | 103 | |
| ')' is preceded with whitespace. | 103 | |
| 'if' is not followed by whitespace. | 105 | |
| '(' is followed by whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| 'if' is not followed by whitespace. | 107 | |
| '(' is followed by whitespace. | 107 | |
| ')' is preceded with whitespace. | 107 | |
| ')' is preceded with whitespace. | 111 | |
| 'catch' is not followed by whitespace. | 115 | |
| '(' is followed by whitespace. | 115 | |
| ')' is preceded with whitespace. | 115 | |
| Line is longer than 80 characters. | 117 | |
| '(' is followed by whitespace. | 117 | |
| '(' is followed by whitespace. | 118 | |
| '(' is followed by whitespace. | 120 | |
| ')' is preceded with whitespace. | 120 | |
| '(' is followed by whitespace. | 122 | |
| ')' is preceded with whitespace. | 122 | |
| '}' should be on the same line. | 124 | |
| '(' is followed by whitespace. | 126 | |
| ')' is preceded with whitespace. | 126 | |
| Line contains a tab character. | 133 | |
| '(' is followed by whitespace. | 136 | |
| Parameter filename should be final. | 136 | |
| ')' is preceded with whitespace. | 136 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.awt.*. | 31 | |
| Using the '.*' form of import should be avoided - java.util.*. | 32 | |
| Using the '.*' form of import should be avoided - javax.swing.*. | 34 | |
| Utility classes should not have a public or default constructor. | 49 | |
| Line contains a tab character. | 53 | |
| First sentence should end with a period. | 58 | |
| Line contains a tab character. | 65 | |
| Line has trailing spaces. | 72 | |
| Line has trailing spaces. | 75 | |
| Line has trailing spaces. | 76 | |
| Line contains a tab character. | 79 | |
| '(' is followed by whitespace. | 82 | |
| Parameter parentComponent should be final. | 82 | |
| 'parentComponent' hides a field. | 82 | |
| ')' is preceded with whitespace. | 82 | |
| Line has trailing spaces. | 89 | |
| Line contains a tab character. | 89 | |
| '(' is followed by whitespace. | 92 | |
| Parameter defaultDirectory should be final. | 92 | |
| ')' is preceded with whitespace. | 92 | |
| Line has trailing spaces. | 99 | |
| Line contains a tab character. | 99 | |
| '(' is followed by whitespace. | 102 | |
| Parameter msg should be final. | 102 | |
| ')' is preceded with whitespace. | 102 | |
| Line has trailing spaces. | 111 | |
| Line contains a tab character. | 111 | |
| '(' is followed by whitespace. | 114 | |
| Parameter msg should be final. | 114 | |
| ')' is preceded with whitespace. | 114 | |
| Line has trailing spaces. | 123 | |
| Line contains a tab character. | 123 | |
| Line has trailing spaces. | 125 | |
| Line contains a tab character. | 125 | |
| '(' is followed by whitespace. | 128 | |
| Parameter msg should be final. | 128 | |
| ')' is preceded with whitespace. | 128 | |
| Line contains a tab character. | 138 | |
| Line has trailing spaces. | 142 | |
| Line contains a tab character. | 142 | |
| '(' is followed by whitespace. | 145 | |
| Parameter typeOfDialog should be final. | 145 | |
| Parameter tmpDir should be final. | 145 | |
| ')' is preceded with whitespace. | 145 | |
| 'if' is not followed by whitespace. | 147 | |
| '(' is followed by whitespace. | 147 | |
| ')' is preceded with whitespace. | 147 | |
| '(' is followed by whitespace. | 149 | |
| ')' is preceded with whitespace. | 149 | |
| '}' should be on the same line. | 150 | |
| Line has trailing spaces. | 152 | |
| '(' is followed by whitespace. | 153 | |
| ')' is preceded with whitespace. | 153 | |
| '(' is followed by whitespace. | 154 | |
| ')' is preceded with whitespace. | 154 | |
| '(' is followed by whitespace. | 155 | |
| ')' is preceded with whitespace. | 155 | |
| Line has trailing spaces. | 157 | |
| 'if' construct must use '{}'s. | 161 | |
| 'if' is not followed by whitespace. | 161 | |
| '(' is followed by whitespace. | 161 | |
| ')' is preceded with whitespace. | 161 | |
| '(' is followed by whitespace. | 162 | |
| ')' is preceded with whitespace. | 162 | |
| '(' is followed by whitespace. | 165 | |
| ')' is preceded with whitespace. | 165 | |
| 'if' construct must use '{}'s. | 168 | |
| 'if' is not followed by whitespace. | 168 | |
| '(' is followed by whitespace. | 168 | |
| ')' is preceded with whitespace. | 168 | |
| '(' is followed by whitespace. | 169 | |
| ')' is preceded with whitespace. | 169 | |
| 'if' construct must use '{}'s. | 170 | |
| 'if' is not followed by whitespace. | 170 | |
| '(' is followed by whitespace. | 170 | |
| ')' is preceded with whitespace. | 170 | |
| '(' is followed by whitespace. | 171 | |
| ')' is preceded with whitespace. | 171 | |
| '+' should be on a new line. | 174 | |
| '(' is followed by whitespace. | 176 | |
| ')' is preceded with whitespace. | 176 | |
| Line has trailing spaces. | 183 | |
| Line contains a tab character. | 183 | |
| Line has trailing spaces. | 185 | |
| Line contains a tab character. | 185 | |
| '(' is followed by whitespace. | 188 | |
| Parameter typeOfDialog should be final. | 188 | |
| ')' is preceded with whitespace. | 188 | |
| '(' is followed by whitespace. | 189 | |
| ')' is preceded with whitespace. | 189 | |
| Line has trailing spaces. | 196 | |
| Line contains a tab character. | 196 | |
| 'if' is not followed by whitespace. | 200 | |
| '(' is followed by whitespace. | 200 | |
| ')' is preceded with whitespace. | 200 | |
| Line has trailing spaces. | 211 | |
| Line contains a tab character. | 211 | |
| Line has trailing spaces. | 213 | |
| Line contains a tab character. | 213 | |
| '(' is followed by whitespace. | 217 | |
| Parameter pC should be final. | 217 | |
| Expected @param tag for 'pC'. | 217 | |
| Parameter msg should be final. | 217 | |
| ')' is preceded with whitespace. | 217 | |
| Line contains a tab character. | 225 | |
| Expected an @return tag. | 231 | |
| '(' is followed by whitespace. | 231 | |
| Parameter input should be final. | 231 | |
| Expected @param tag for 'input'. | 231 | |
| ')' is preceded with whitespace. | 231 | |
| Array brackets at illegal position. | 234 | |
| Line has trailing spaces. | 235 | |
| 'while' construct must use '{}'s. | 236 | |
| 'while' is not followed by whitespace. | 236 | |
| '(' is followed by whitespace. | 236 | |
| ')' is preceded with whitespace. | 236 | |
| 'for' construct must use '{}'s. | 239 | |
| Line has trailing spaces. | 241 | |
| Line has trailing spaces. | 244 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - org.psychomad.psycholib.resources.*. | 40 | |
| Utility classes should not have a public or default constructor. | 47 | |
| Line contains a tab character. | 51 | |
| 'static' modifier out of order with the JLS suggestions. | 54 | |
| 'static' modifier out of order with the JLS suggestions. | 60 | |
| Expected an @return tag. | 67 | |
| '(' is followed by whitespace. | 68 | |
| ')' is preceded with whitespace. | 68 | |
| Line has trailing spaces. | 71 | |
| Line has trailing spaces. | 78 | |
| Line contains a tab character. | 80 | |
| '(' is followed by whitespace. | 83 | |
| Parameter licenseAgreementFilename should be final. | 83 | |
| ')' is preceded with whitespace. | 83 | |
| '(' is followed by whitespace. | 85 | |
| ')' is preceded with whitespace. | 85 | |
| Line has trailing spaces. | 86 | |
| Line has trailing spaces. | 88 | |
| '(' is followed by whitespace. | 91 | |
| ')' is preceded with whitespace. | 91 | |
| 'catch' is not followed by whitespace. | 92 | |
| '(' is followed by whitespace. | 92 | |
| ')' is preceded with whitespace. | 92 | |
| Line is longer than 80 characters. | 93 | |
| '(' is followed by whitespace. | 93 | |
| '(' is followed by whitespace. | 97 | |
| ')' is preceded with whitespace. | 97 | |
| 'if' is not followed by whitespace. | 100 | |
| '(' is followed by whitespace. | 100 | |
| Expression can be simplified. | 100 | |
| ')' is preceded with whitespace. | 100 | |
| 'if' is not followed by whitespace. | 105 | |
| '(' is followed by whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| '(' is followed by whitespace. | 106 | |
| 'catch' is not followed by whitespace. | 110 | |
| '(' is followed by whitespace. | 110 | |
| ')' is preceded with whitespace. | 110 | |
| '+' should be on a new line. | 111 | |
| '(' is followed by whitespace. | 129 | |
| ')' is preceded with whitespace. | 129 | |
| '(' is followed by whitespace. | 130 | |
| Line has trailing spaces. | 133 | |
| '{' is followed by whitespace. | 133 | |
| Name 'TMP_filename' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 140 | |
| Line has trailing spaces. | 145 | |
| 'while' is not followed by whitespace. | 147 | |
| '(' is followed by whitespace. | 147 | |
| ')' is preceded with whitespace. | 147 | |
| 'catch' is not followed by whitespace. | 152 | |
| '(' is followed by whitespace. | 152 | |
| ')' is preceded with whitespace. | 152 | |
| '(' is followed by whitespace. | 153 | |
| ')' is preceded with whitespace. | 153 | |
| '}' should be on the same line. | 154 | |
| 'catch' is not followed by whitespace. | 155 | |
| '(' is followed by whitespace. | 155 | |
| ')' is preceded with whitespace. | 155 | |
| '(' is followed by whitespace. | 156 | |
| ')' is preceded with whitespace. | 156 | |
| Name 'TEXT_ROWS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 160 | |
| '20' is a magic number. | 160 | |
| Name 'TEXT_COLS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 161 | |
| Line has trailing spaces. | 162 | |
| Line has trailing spaces. | 168 | |
| 'if' construct must use '{}'s. | 173 | |
| Conditional logic can be removed. | 173 | |
| 'if' is not followed by whitespace. | 173 | |
| '(' is followed by whitespace. | 173 | |
| ')' is preceded with whitespace. | 173 | |
| 'else' construct must use '{}'s. | 175 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 31 | |
| ';' is preceded with whitespace. | 31 | |
| Using the '.*' form of import should be avoided - java.io.*. | 32 | |
| ';' is preceded with whitespace. | 32 | |
| ';' is preceded with whitespace. | 43 | |
| Line has trailing spaces. | 51 | |
| Line contains a tab character. | 53 | |
| '(' is followed by whitespace. | 57 | |
| Parameter path should be final. | 57 | |
| Parameter modifAllowed should be final. | 57 | |
| 'modifAllowed' hides a field. | 57 | |
| ')' is preceded with whitespace. | 57 | |
| ';' is preceded with whitespace. | 61 | |
| ';' is preceded with whitespace. | 63 | |
| 'catch' is not followed by whitespace. | 64 | |
| '(' is followed by whitespace. | 64 | |
| ')' is preceded with whitespace. | 64 | |
| Must have at least one statement. | 64 | |
| Line is longer than 80 characters. | 65 | |
| '}' should be on the same line. | 66 | |
| 'catch' is not followed by whitespace. | 67 | |
| '(' is followed by whitespace. | 67 | |
| ')' is preceded with whitespace. | 67 | |
| Must have at least one statement. | 67 | |
| Line is longer than 80 characters. | 68 | |
| Line has trailing spaces. | 71 | |
| Line contains a tab character. | 71 | |
| Method 'getPath' is not designed for extension - needs to be abstract, final or empty. | 76 | |
| Line contains a tab character. | 83 | |
| Line contains a tab character. | 85 | |
| Method 'getString' is not designed for extension - needs to be abstract, final or empty. | 88 | |
| '(' is followed by whitespace. | 88 | |
| Parameter str should be final. | 88 | |
| ')' is preceded with whitespace. | 88 | |
| Expected @throws tag for 'MissingPropertiesException'. | 88 | |
| Redundant throws: 'MissingPropertiesException' is unchecked exception. | 88 | |
| Name 'str_val' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 89 | |
| '(' is followed by whitespace. | 91 | |
| ')' is preceded with whitespace. | 91 | |
| 'if' construct must use '{}'s. | 92 | |
| 'if' is not followed by whitespace. | 92 | |
| '(' is followed by whitespace. | 92 | |
| ')' is preceded with whitespace. | 92 | |
| ';' is preceded with whitespace. | 93 | |
| Line contains a tab character. | 98 | |
| Line contains a tab character. | 99 | |
| Line contains a tab character. | 100 | |
| Expected an @return tag. | 101 | |
| Method 'getStringNoWarning' is not designed for extension - needs to be abstract, final or empty. | 101 | |
| '(' is followed by whitespace. | 101 | |
| Parameter str should be final. | 101 | |
| Expected @param tag for 'str'. | 101 | |
| ')' is preceded with whitespace. | 101 | |
| Name 'str_val' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 102 | |
| '(' is followed by whitespace. | 104 | |
| ')' is preceded with whitespace. | 104 | |
| 'catch' is not followed by whitespace. | 105 | |
| '(' is followed by whitespace. | 105 | |
| ')' is preceded with whitespace. | 105 | |
| Line has trailing spaces. | 110 | |
| Line contains a tab character. | 115 | |
| Line contains a tab character. | 117 | |
| Line has trailing spaces. | 118 | |
| Method 'getInt' is not designed for extension - needs to be abstract, final or empty. | 120 | |
| '(' is followed by whitespace. | 120 | |
| Parameter str should be final. | 120 | |
| ')' is preceded with whitespace. | 120 | |
| Expected @throws tag for 'MissingPropertiesException'. | 120 | |
| Redundant throws: 'MissingPropertiesException' is unchecked exception. | 120 | |
| Expected @throws tag for 'NumberFormatException'. | 121 | |
| Redundant throws: 'NumberFormatException' is unchecked exception. | 121 | |
| Name 'str_val' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 122 | |
| ';' is preceded with whitespace. | 123 | |
| '(' is followed by whitespace. | 125 | |
| ')' is preceded with whitespace. | 125 | |
| 'if' construct must use '{}'s. | 126 | |
| 'if' is not followed by whitespace. | 126 | |
| '(' is followed by whitespace. | 126 | |
| ')' is preceded with whitespace. | 126 | |
| ';' is preceded with whitespace. | 127 | |
| Line has trailing spaces. | 135 | |
| Line contains a tab character. | 135 | |
| Line contains a tab character. | 137 | |
| Line has trailing spaces. | 138 | |
| Method 'getDouble' is not designed for extension - needs to be abstract, final or empty. | 140 | |
| '(' is followed by whitespace. | 140 | |
| Parameter str should be final. | 140 | |
| ')' is preceded with whitespace. | 140 | |
| Expected @throws tag for 'MissingPropertiesException'. | 140 | |
| Redundant throws: 'MissingPropertiesException' is unchecked exception. | 140 | |
| Expected @throws tag for 'NumberFormatException'. | 141 | |
| Redundant throws: 'NumberFormatException' is unchecked exception. | 141 | |
| Name 'str_val' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 142 | |
| ';' is preceded with whitespace. | 143 | |
| '(' is followed by whitespace. | 145 | |
| ')' is preceded with whitespace. | 145 | |
| 'if' construct must use '{}'s. | 146 | |
| 'if' is not followed by whitespace. | 146 | |
| '(' is followed by whitespace. | 146 | |
| ')' is preceded with whitespace. | 146 | |
| ';' is preceded with whitespace. | 147 | |
| Expected an @return tag. | 160 | |
| Method 'setProperty' is not designed for extension - needs to be abstract, final or empty. | 160 | |
| '(' is followed by whitespace. | 160 | |
| Parameter key should be final. | 160 | |
| Expected @param tag for 'key'. | 160 | |
| Parameter value should be final. | 160 | |
| Expected @param tag for 'value'. | 160 | |
| ')' is preceded with whitespace. | 160 | |
| 'if' is not followed by whitespace. | 162 | |
| '(' is followed by whitespace. | 162 | |
| Expression can be simplified. | 162 | |
| ')' is preceded with whitespace. | 162 | |
| '(' is followed by whitespace. | 167 | |
| ')' is preceded with whitespace. | 167 | |
| Method 'store' is not designed for extension - needs to be abstract, final or empty. | 174 | |
| Expected @throws tag for 'IOException'. | 174 | |
| 'if' is not followed by whitespace. | 176 | |
| '(' is followed by whitespace. | 176 | |
| Expression can be simplified. | 176 | |
| ')' is preceded with whitespace. | 176 | |
| '(' is followed by whitespace. | 182 | |
| ')' is preceded with whitespace. | 182 | |
| Line has trailing spaces. | 183 | |
| '(' is followed by whitespace. | 184 | |
| ')' is preceded with whitespace. | 184 | |
| Line has trailing spaces. | 185 | |
| 'if' construct must use '{}'s. | 186 | |
| Line has trailing spaces. | 187 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 49 | |
| '<' is not preceded with whitespace. | 49 | |
| '<' is not followed by whitespace. | 49 | |
| ',' is not followed by whitespace. | 49 | |
| '>' is not preceded with whitespace. | 49 | |
| Variable 'commands' must be private and have accessor methods. | 49 | |
| '<' is not preceded with whitespace. | 57 | |
| '<' is not followed by whitespace. | 57 | |
| ',' is not followed by whitespace. | 57 | |
| '>' is not preceded with whitespace. | 57 | |
| '>' is not followed by whitespace. | 57 | |
| 'cast' is not followed by whitespace. | 66 | |
| Expected an @return tag. | 79 | |
| Expected an @return tag. | 85 | |
| Method 'getAction' is not designed for extension - needs to be abstract, final or empty. | 85 | |
| Parameter cmd should be final. | 85 | |
| Expected @param tag for 'cmd'. | 85 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.io.*. | 31 | |
| ';' is preceded with whitespace. | 31 | |
| Using the '.*' form of import should be avoided - java.util.*. | 32 | |
| ';' is preceded with whitespace. | 32 | |
| Utility classes should not have a public or default constructor. | 41 | |
| Line has trailing spaces. | 42 | |
| Line contains a tab character. | 42 | |
| Missing a Javadoc comment. | 43 | |
| ';' is preceded with whitespace. | 43 | |
| Missing a Javadoc comment. | 44 | |
| 'static' modifier out of order with the JLS suggestions. | 44 | |
| ';' is preceded with whitespace. | 44 | |
| '(' is followed by whitespace. | 48 | |
| ';' is preceded with whitespace. | 49 | |
| Line has trailing spaces. | 52 | |
| Line contains a tab character. | 52 | |
| Line has trailing spaces. | 55 | |
| Line contains a tab character. | 55 | |
| Line contains a tab character. | 57 | |
| '(' is followed by whitespace. | 60 | |
| Parameter key should be final. | 60 | |
| ')' is preceded with whitespace. | 60 | |
| Expected @throws tag for 'MissingResourceException'. | 60 | |
| Redundant throws: 'MissingResourceException' is unchecked exception. | 60 | |
| ';' is preceded with whitespace. | 61 | |
| Line has trailing spaces. | 64 | |
| Line contains a tab character. | 64 | |
| Line has trailing spaces. | 67 | |
| Line contains a tab character. | 67 | |
| Line contains a tab character. | 69 | |
| '(' is followed by whitespace. | 72 | |
| Parameter key should be final. | 72 | |
| ')' is preceded with whitespace. | 72 | |
| ';' is preceded with whitespace. | 74 | |
| 'catch' is not followed by whitespace. | 75 | |
| '(' is followed by whitespace. | 75 | |
| ')' is preceded with whitespace. | 75 | |
| Line has trailing spaces. | 80 | |
| Line contains a tab character. | 80 | |
| Line has trailing spaces. | 83 | |
| Line contains a tab character. | 83 | |
| Line has trailing spaces. | 85 | |
| Line contains a tab character. | 85 | |
| '(' is followed by whitespace. | 88 | |
| Parameter key should be final. | 88 | |
| ')' is preceded with whitespace. | 88 | |
| Expected @throws tag for 'MissingResourceException'. | 88 | |
| Redundant throws: 'MissingResourceException' is unchecked exception. | 88 | |
| Expected @throws tag for 'NumberFormatException'. | 89 | |
| Redundant throws: 'NumberFormatException' is unchecked exception. | 89 | |
| ';' is preceded with whitespace. | 90 | |
| '(' is followed by whitespace. | 91 | |
| ')' is preceded with whitespace. | 91 | |
| ';' is preceded with whitespace. | 91 | |
| '(' is followed by whitespace. | 92 | |
| ';' is preceded with whitespace. | 92 | |
| Line has trailing spaces. | 95 | |
| Line contains a tab character. | 95 | |
| Line has trailing spaces. | 98 | |
| Line contains a tab character. | 98 | |
| '(' is followed by whitespace. | 101 | |
| Parameter language should be final. | 101 | |
| ')' is preceded with whitespace. | 101 | |
| '(' is followed by whitespace. | 102 | |
| ';' is preceded with whitespace. | 102 | |
| Line has trailing spaces. | 104 | |
| Line contains a tab character. | 104 | |
| Line has trailing spaces. | 111 | |
| Line contains a tab character. | 111 | |
| '(' is followed by whitespace. | 115 | |
| Parameter key should be final. | 115 | |
| ')' is preceded with whitespace. | 115 | |
| '(' is followed by whitespace. | 116 | |
| ')' is preceded with whitespace. | 116 | |
| 'if' is not followed by whitespace. | 117 | |
| '(' is followed by whitespace. | 117 | |
| ')' is preceded with whitespace. | 117 | |
| '(' is followed by whitespace. | 124 | |
| ')' is preceded with whitespace. | 124 | |
| Line has trailing spaces. | 125 | |
| '(' is followed by whitespace. | 126 | |
| ')' is preceded with whitespace. | 126 | |
| 'catch' is not followed by whitespace. | 127 | |
| '(' is followed by whitespace. | 127 | |
| ')' is preceded with whitespace. | 127 | |
| '+' is not preceded with whitespace. | 128 | |
| '+' is not followed by whitespace. | 128 | |
| 'catch' is not followed by whitespace. | 129 | |
| '(' is followed by whitespace. | 129 | |
| ')' is preceded with whitespace. | 129 | |
| Line is longer than 80 characters. | 130 | |
| '+' is not preceded with whitespace. | 130 | |
| '+' is not followed by whitespace. | 130 | |
| '+' is not preceded with whitespace. | 130 | |
| '+' is not followed by whitespace. | 130 | |
| 'catch' is not followed by whitespace. | 131 | |
| '(' is followed by whitespace. | 131 | |
| ')' is preceded with whitespace. | 131 | |
| '+' is not preceded with whitespace. | 132 | |
| '+' is not followed by whitespace. | 132 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - javax.media.j3d.*. | 27 | |
| Using the '.*' form of import should be avoided - javax.vecmath.*. | 28 | |
| Line contains a tab character. | 42 | |
| Missing a Javadoc comment. | 42 | |
| Variable 'nFrames' must be private and have accessor methods. | 42 | |
| Line contains a tab character. | 43 | |
| Missing a Javadoc comment. | 43 | |
| Variable 'startTime' must be private and have accessor methods. | 43 | |
| Line contains a tab character. | 44 | |
| Missing a Javadoc comment. | 44 | |
| Variable 'w' must be private and have accessor methods. | 44 | |
| Line contains a tab character. | 45 | |
| Missing a Javadoc comment. | 45 | |
| Variable 'frameRate' must be private and have accessor methods. | 45 | |
| Line contains a tab character. | 46 | |
| Missing a Javadoc comment. | 46 | |
| Variable 'text2D' must be private and have accessor methods. | 46 | |
| Line contains a tab character. | 49 | |
| Line contains a tab character. | 50 | |
| Line contains a tab character. | 51 | |
| Line contains a tab character. | 52 | |
| Line contains a tab character. | 53 | |
| '200' is a magic number. | 53 | |
| Line contains a tab character. | 54 | |
| Line contains a tab character. | 57 | |
| Line contains a tab character. | 58 | |
| Line contains a tab character. | 59 | |
| Line contains a tab character. | 60 | |
| Line contains a tab character. | 61 | |
| Line contains a tab character. | 62 | |
| Parameter nFrames should be final. | 62 | |
| 'nFrames' hides a field. | 62 | |
| Line contains a tab character. | 63 | |
| Line contains a tab character. | 64 | |
| Line contains a tab character. | 67 | |
| Line contains a tab character. | 68 | |
| Line contains a tab character. | 69 | |
| Line contains a tab character. | 70 | |
| Line is longer than 80 characters. | 71 | |
| Line contains a tab character. | 71 | |
| Line contains a tab character. | 72 | |
| Line contains a tab character. | 73 | |
| Line contains a tab character. | 74 | |
| Parameter nFrames should be final. | 74 | |
| 'nFrames' hides a field. | 74 | |
| Parameter texto should be final. | 74 | |
| Line contains a tab character. | 75 | |
| Line contains a tab character. | 76 | |
| 'if' construct must use '{}'s. | 78 | |
| Line contains a tab character. | 78 | |
| Line contains a tab character. | 79 | |
| 'cast' is not followed by whitespace. | 79 | |
| Line contains a tab character. | 80 | |
| Line contains a tab character. | 83 | |
| Line contains a tab character. | 84 | |
| Line contains a tab character. | 85 | |
| Line contains a tab character. | 86 | |
| Line contains a tab character. | 87 | |
| '1000' is a magic number. | 87 | |
| Line contains a tab character. | 88 | |
| Line contains a tab character. | 89 | |
| Line contains a tab character. | 90 | |
| Line has trailing spaces. | 91 | |
| Line contains a tab character. | 91 | |
| Line contains a tab character. | 93 | |
| Line has trailing spaces. | 94 | |
| Line contains a tab character. | 94 | |
| Line contains a tab character. | 95 | |
| Line contains a tab character. | 96 | |
| Parameter criteria should be final. | 96 | |
| Expected @param tag for 'criteria'. | 96 | |
| Line contains a tab character. | 97 | |
| Line contains a tab character. | 98 | |
| Line is longer than 80 characters. | 100 | |
| Line contains a tab character. | 100 | |
| Line is longer than 80 characters. | 101 | |
| Line contains a tab character. | 101 | |
| '1000' is a magic number. | 101 | |
| '*' is not preceded with whitespace. | 101 | |
| '*' is not followed by whitespace. | 101 | |
| 'cast' is not followed by whitespace. | 101 | |
| '/' is not preceded with whitespace. | 101 | |
| '/' is not followed by whitespace. | 101 | |
| 'cast' is not followed by whitespace. | 101 | |
| '-' is not preceded with whitespace. | 101 | |
| '-' is not followed by whitespace. | 101 | |
| Line contains a tab character. | 103 | |
| Line contains a tab character. | 104 | |
| Line contains a tab character. | 106 | |
| Line contains a tab character. | 107 | |
| Line contains a tab character. | 108 | |
| Line contains a tab character. | 109 | |
| Line contains a tab character. | 114 | |
| Line contains a tab character. | 115 | |
| Line contains a tab character. | 117 | |
| Line contains a tab character. | 118 | |
| Line contains a tab character. | 119 | |
| Line contains a tab character. | 122 | |
| Line contains a tab character. | 123 | |
| Line contains a tab character. | 124 | |
| Expected an @return tag. | 125 | |
| Line contains a tab character. | 125 | |
| Line contains a tab character. | 126 | |
| Line contains a tab character. | 127 |
| Violation | Message | Line |
|---|---|---|
| Line has trailing spaces. | 43 | |
| Line is longer than 80 characters. | 47 | |
| First sentence should end with a period. | 49 | |
| Line contains a tab character. | 58 | |
| '(' is followed by whitespace. | 62 | |
| Parameter actionName should be final. | 62 | |
| Parameter config should be final. | 62 | |
| Expected @param tag for 'config'. | 62 | |
| Parameter resources should be final. | 63 | |
| Expected @param tag for 'resources'. | 63 | |
| ')' is preceded with whitespace. | 63 | |
| '(' is followed by whitespace. | 64 | |
| ')' is preceded with whitespace. | 64 | |
| '(' is followed by whitespace. | 65 | |
| ')' is preceded with whitespace. | 65 | |
| '(' is followed by whitespace. | 78 | |
| Parameter actionDefaultName should be final. | 78 | |
| Parameter config should be final. | 78 | |
| Expected @param tag for 'config'. | 78 | |
| Parameter resources should be final. | 79 | |
| Expected @param tag for 'resources'. | 79 | |
| ')' is preceded with whitespace. | 79 | |
| 'if' is not followed by whitespace. | 80 | |
| '(' is followed by whitespace. | 80 | |
| ')' is preceded with whitespace. | 80 | |
| '(' is followed by whitespace. | 86 | |
| ')' is preceded with whitespace. | 86 | |
| 'if' construct must use '{}'s. | 94 | |
| 'if' is not followed by whitespace. | 94 | |
| '(' is followed by whitespace. | 94 | |
| ')' is preceded with whitespace. | 94 | |
| '(' is followed by whitespace. | 95 | |
| ')' is preceded with whitespace. | 95 | |
| ')' is preceded with whitespace. | 95 | |
| '(' is followed by whitespace. | 98 | |
| '+' should be on a new line. | 98 | |
| ')' is preceded with whitespace. | 99 | |
| 'if' is not followed by whitespace. | 100 | |
| '(' is followed by whitespace. | 100 | |
| ')' is preceded with whitespace. | 100 | |
| '(' is followed by whitespace. | 101 | |
| ')' is preceded with whitespace. | 101 | |
| 'if' construct must use '{}'s. | 102 | |
| 'if' is not followed by whitespace. | 102 | |
| '(' is followed by whitespace. | 102 | |
| ')' is preceded with whitespace. | 102 | |
| '(' is followed by whitespace. | 103 | |
| ')' is preceded with whitespace. | 103 | |
| 'else' construct must use '{}'s. | 104 | |
| Line is longer than 80 characters. | 105 | |
| '+' is not preceded with whitespace. | 105 | |
| '+' is not followed by whitespace. | 105 | |
| '+' is not preceded with whitespace. | 105 | |
| '+' is not followed by whitespace. | 105 | |
| '(' is followed by whitespace. | 109 | |
| '+' should be on a new line. | 109 | |
| ')' is preceded with whitespace. | 110 | |
| 'if' construct must use '{}'s. | 111 | |
| 'if' is not followed by whitespace. | 111 | |
| '(' is followed by whitespace. | 111 | |
| ')' is preceded with whitespace. | 111 | |
| '(' is followed by whitespace. | 112 | |
| ')' is preceded with whitespace. | 112 | |
| '+' is not preceded with whitespace. | 117 | |
| '+' is not followed by whitespace. | 117 | |
| ')' is preceded with whitespace. | 117 | |
| 'if' is not followed by whitespace. | 118 | |
| '(' is followed by whitespace. | 118 | |
| ')' is preceded with whitespace. | 118 | |
| '(' is followed by whitespace. | 119 | |
| ')' is preceded with whitespace. | 119 | |
| '(' is followed by whitespace. | 125 | |
| '+' should be on a new line. | 125 | |
| ')' is preceded with whitespace. | 126 | |
| 'if' construct must use '{}'s. | 127 | |
| 'if' is not followed by whitespace. | 127 | |
| '(' is followed by whitespace. | 127 | |
| ')' is preceded with whitespace. | 127 | |
| 'cast' is not followed by whitespace. | 128 | |
| Line has trailing spaces. | 129 | |
| '(' is followed by whitespace. | 129 | |
| ')' is preceded with whitespace. | 129 | |
| '(' is followed by whitespace. | 132 | |
| ')' is preceded with whitespace. | 132 | |
| 'if' construct must use '{}'s. | 133 | |
| 'if' is not followed by whitespace. | 133 | |
| '(' is followed by whitespace. | 133 | |
| ')' is preceded with whitespace. | 133 | |
| '(' is followed by whitespace. | 134 | |
| ')' is preceded with whitespace. | 134 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 31 | |
| Using the '.*' form of import should be avoided - javax.swing.*. | 32 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyDialog. | 34 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyAbstractActionValues. | 35 | |
| Redundant import from the same package - org.psychomad.psycholib.ui.EasyActions. | 36 | |
| Unused import - org.psychomad.psycholib.std.ExitCodes. | 40 | |
| '(' is followed by whitespace. | 59 | |
| Parameter myActions should be final. | 59 | |
| Expected @param tag for 'myActions'. | 59 | |
| Parameter config should be final. | 59 | |
| Expected @param tag for 'config'. | 59 | |
| Parameter resources should be final. | 60 | |
| Expected @param tag for 'resources'. | 60 | |
| ')' is preceded with whitespace. | 60 | |
| Line has trailing spaces. | 62 | |
| '(' is followed by whitespace. | 63 | |
| ')' is preceded with whitespace. | 63 | |
| Method 'fillMenuBar' is not designed for extension - needs to be abstract, final or empty. | 70 | |
| '(' is followed by whitespace. | 70 | |
| Parameter myActions should be final. | 70 | |
| Expected @param tag for 'myActions'. | 70 | |
| Parameter config should be final. | 70 | |
| Expected @param tag for 'config'. | 70 | |
| Parameter resources should be final. | 71 | |
| Expected @param tag for 'resources'. | 71 | |
| ')' is preceded with whitespace. | 71 | |
| Line has trailing spaces. | 72 | |
| '(' is followed by whitespace. | 73 | |
| ')' is preceded with whitespace. | 73 | |
| 'for' is not followed by whitespace. | 74 | |
| '(' is followed by whitespace. | 74 | |
| ')' is preceded with whitespace. | 74 | |
| '(' is followed by whitespace. | 75 | |
| ')' is preceded with whitespace. | 75 | |
| Expected an @return tag. | 88 | |
| Method 'createMenu' is not designed for extension - needs to be abstract, final or empty. | 88 | |
| '(' is followed by whitespace. | 88 | |
| Parameter myActions should be final. | 88 | |
| Expected @param tag for 'myActions'. | 88 | |
| Parameter key should be final. | 88 | |
| Expected @param tag for 'key'. | 88 | |
| Parameter config should be final. | 88 | |
| Expected @param tag for 'config'. | 88 | |
| Parameter resources should be final. | 89 | |
| Expected @param tag for 'resources'. | 89 | |
| ')' is preceded with whitespace. | 89 | |
| Line is longer than 80 characters. | 95 | |
| '(' is followed by whitespace. | 97 | |
| '+' should be on a new line. | 97 | |
| ')' is preceded with whitespace. | 98 | |
| 'if' construct must use '{}'s. | 99 | |
| 'if' is not followed by whitespace. | 99 | |
| '(' is followed by whitespace. | 99 | |
| ')' is preceded with whitespace. | 99 | |
| '(' is followed by whitespace. | 100 | |
| ')' is preceded with whitespace. | 100 | |
| 'for' is not followed by whitespace. | 103 | |
| '(' is followed by whitespace. | 103 | |
| ')' is preceded with whitespace. | 103 | |
| '}' should be on the same line. | 106 | |
| 'if' is not followed by whitespace. | 107 | |
| '(' is followed by whitespace. | 107 | |
| ')' is preceded with whitespace. | 107 | |
| '(' is followed by whitespace. | 112 | |
| ')' is preceded with whitespace. | 112 | |
| '}' should be on the same line. | 113 | |
| 'if' is not followed by whitespace. | 117 | |
| '(' is followed by whitespace. | 117 | |
| ')' is preceded with whitespace. | 117 | |
| '+' should be on a new line. | 118 | |
| '}' should be on the same line. | 120 | |
| 'else' construct must use '{}'s. | 121 | |
| '(' is followed by whitespace. | 122 | |
| ')' is preceded with whitespace. | 122 |