PMD Results
The following document contains the results of PMD 7.6.0.
Violations By Priority
Priority 1
org/apache/commons/codec/binary/BaseNCodec.java
Rule |
Violation |
Line |
FieldNamingConventions |
The final field name 'PAD' doesn't match '[a-z][a-zA-Z0-9]*' |
392 |
org/apache/commons/codec/binary/StringUtils.java
Rule |
Violation |
Line |
MethodNamingConventions |
The static method name 'getBytesIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
144 |
MethodNamingConventions |
The static method name 'newStringIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
331 |
Priority 3
org/apache/commons/codec/CharEncoding.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
128 |
org/apache/commons/codec/Charsets.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
169 |
org/apache/commons/codec/DecoderException.java
Rule |
Violation |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
39 |
org/apache/commons/codec/EncoderException.java
Rule |
Violation |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
40 |
org/apache/commons/codec/Resources.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
52 |
org/apache/commons/codec/StringEncoderComparator.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like o1 |
73 |
ShortVariable |
Avoid variables with short names like o2 |
73 |
ShortVariable |
Avoid variables with short names like s1 |
80 |
ShortVariable |
Avoid variables with short names like s2 |
81 |
org/apache/commons/codec/binary/Base16.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
47 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
48 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
49 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
56 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
70 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_TABLE |
77 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
92 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
148 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
154 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
157 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
202 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
204 |
org/apache/commons/codec/binary/Base32.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
84 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
84 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
86 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
86 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
87 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
87 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
93 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
112 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
124 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
143 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
151 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
154 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
157 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
160 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
163 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
182 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
193 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
198 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
402 |
ShortVariable |
Avoid variables with short names like b |
412 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
507 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
510 |
ShortVariable |
Avoid variables with short names like b |
577 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
607 |
org/apache/commons/codec/binary/Base64.java
Rule |
Violation |
Line |
LinguisticNaming |
Linguistics Antipattern - The setter 'setUrlSafe' should not return any type except void linguistically |
97 |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
108 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
108 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
109 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
109 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
110 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
110 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
111 |
LongVariable |
Avoid excessively long variable names like DECODING_TABLE_LENGTH |
112 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
112 |
LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
122 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
122 |
LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
135 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
135 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
156 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
172 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
179 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
181 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
299 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
421 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toIntegerBytes(BigInteger)' |
448 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
455 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
482 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
487 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
492 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
498 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
500 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
743 |
ShortVariable |
Avoid variables with short names like b |
753 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
825 |
ShortVariable |
Avoid variables with short names like in |
825 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
827 |
ShortVariable |
Avoid variables with short names like b |
875 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
901 |
org/apache/commons/codec/binary/BaseNCodec.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
67 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'AbstractBuilder(byte[])' |
72 |
LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
72 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
89 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodingPolicy()' |
93 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodeTable()' |
97 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineLength()' |
101 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
105 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getPadding()' |
109 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodingPolicy' should not return any type except void linguistically |
119 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTable' should not return any type except void linguistically |
130 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setLineLength' should not return any type except void linguistically |
141 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setLineSeparator' should not return any type except void linguistically |
152 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPadding' should not return any type except void linguistically |
163 |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'Context' |
175 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'ibitWorkArea' |
181 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'lbitWorkArea' |
187 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'buffer' |
192 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pos' |
197 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'readPos' |
202 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'eof' |
208 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'currentLinePos' |
214 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'modulus' |
220 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
240 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EOF' |
240 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
252 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
264 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
266 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
284 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
287 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
292 |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
298 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
298 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
305 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'CHUNK_SEPARATOR' |
305 |
ShortVariable |
Avoid variables with short names like b |
373 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toLength(byte[])' |
384 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
392 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
395 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
398 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
398 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
401 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
408 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
413 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
413 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
434 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
448 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
448 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
465 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
465 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
484 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
484 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'available(Context)' |
501 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
518 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
535 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
546 |
ShortVariable |
Avoid variables with short names like i |
546 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
562 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
591 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
611 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
622 |
ShortVariable |
Avoid variables with short names like i |
622 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
684 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'hasData(Context)' |
738 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
763 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'readResults(byte[], int, int, Context)' |
815 |
ShortVariable |
Avoid variables with short names like b |
815 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
827 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like r |
126 |
ShortVariable |
Avoid variables with short names like b |
131 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
132 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
163 |
ShortVariable |
Avoid variables with short names like c |
187 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
197 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
197 |
ShortVariable |
Avoid variables with short names like n |
225 |
ShortVariable |
Avoid variables with short names like b |
230 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like c |
122 |
ShortVariable |
Avoid variables with short names like i |
191 |
org/apache/commons/codec/binary/BinaryCodec.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
109 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty(byte[])' |
135 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
150 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
181 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
217 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
247 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
250 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
253 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
302 |
org/apache/commons/codec/binary/CharSequenceUtils.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'regionMatches(CharSequence, boolean, int, CharSequence, int, int)' |
54 |
ShortVariable |
Avoid variables with short names like cs |
54 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
57 |
ShortVariable |
Avoid variables with short names like c1 |
63 |
ShortVariable |
Avoid variables with short names like c2 |
64 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
69 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
74 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
86 |
org/apache/commons/codec/binary/Hex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like DEFAULT_CHARSET_NAME |
52 |
ShortVariable |
Avoid variables with short names like f |
102 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
348 |
ShortVariable |
Avoid variables with short names like ch |
365 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
373 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
452 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
455 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
458 |
org/apache/commons/codec/binary/StringUtils.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
75 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
94 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
169 |
ShortVariable |
Avoid variables with short names like e |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
310 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
421 |
org/apache/commons/codec/digest/Blake3.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like in |
166 |
LongVariable |
Avoid excessively long variable names like parentNodesRemaining |
191 |
ShortVariable |
Avoid variables with short names like cv |
203 |
LongVariable |
Avoid excessively long variable names like inputChainingValue |
220 |
LongVariable |
Avoid excessively long variable names like inputChainingValue |
226 |
LongVariable |
Avoid excessively long variable names like outputBlockCounter |
239 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
255 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
256 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
257 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
258 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
259 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
260 |
LongVariable |
Avoid excessively long variable names like CHAINING_VALUE_INTS |
261 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
261 |
ShortVariable |
Avoid variables with short names like IV |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
269 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
270 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
271 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
273 |
LongVariable |
Avoid excessively long variable names like DERIVE_KEY_CONTEXT |
274 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
274 |
LongVariable |
Avoid excessively long variable names like DERIVE_KEY_MATERIAL |
275 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
275 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
281 |
ShortMethodName |
Avoid using short method names |
327 |
ShortVariable |
Avoid variables with short names like a |
327 |
ShortVariable |
Avoid variables with short names like b |
327 |
ShortVariable |
Avoid variables with short names like c |
327 |
ShortVariable |
Avoid variables with short names like d |
327 |
ShortVariable |
Avoid variables with short names like mx |
327 |
ShortVariable |
Avoid variables with short names like my |
327 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
447 |
ShortVariable |
Avoid variables with short names like in |
515 |
ShortVariable |
Avoid variables with short names like in |
530 |
org/apache/commons/codec/digest/Crypt.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
77 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
80 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
83 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
86 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
182 |
org/apache/commons/codec/digest/DigestUtils.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'BUFFER_SIZE' |
63 |
LongVariable |
Avoid excessively long variable names like defaultMessageDigest |
173 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
175 |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
352 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
1372 |
org/apache/commons/codec/digest/HmacUtils.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
788 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
804 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
873 |
org/apache/commons/codec/digest/MurmurHash3.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
79 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
79 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
94 |
ShortVariable |
Avoid variables with short names like k1 |
99 |
ShortVariable |
Avoid variables with short names like b1 |
146 |
ShortVariable |
Avoid variables with short names like b2 |
146 |
ShortVariable |
Avoid variables with short names like b3 |
146 |
ShortVariable |
Avoid variables with short names like b4 |
146 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
151 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
154 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
157 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
163 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
168 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
182 |
ShortVariable |
Avoid variables with short names like k |
209 |
ShortVariable |
Avoid variables with short names like k |
238 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
271 |
ShortVariable |
Avoid variables with short names like k1 |
273 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
312 |
UseUnderscoresInNumericLiterals |
Number 2862933555777941757L should separate every third digit with an underscore |
312 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
317 |
UseUnderscoresInNumericLiterals |
Number 104729 should separate every third digit with an underscore |
317 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
319 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
320 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
321 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
322 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
324 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
325 |
ShortVariable |
Avoid variables with short names like C1 |
327 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
327 |
ShortVariable |
Avoid variables with short names like C2 |
328 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
328 |
ShortVariable |
Avoid variables with short names like R1 |
329 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
329 |
ShortVariable |
Avoid variables with short names like R2 |
330 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
330 |
ShortVariable |
Avoid variables with short names like R3 |
331 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
331 |
ShortVariable |
Avoid variables with short names like M |
332 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
332 |
ShortVariable |
Avoid variables with short names like N1 |
334 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
334 |
ShortVariable |
Avoid variables with short names like N2 |
336 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
336 |
UnnecessaryCast |
Unnecessary cast (long) |
390 |
UnnecessaryCast |
Unnecessary cast (long) |
391 |
UnnecessaryCast |
Unnecessary cast (long) |
392 |
UnnecessaryCast |
Unnecessary cast (long) |
393 |
UnnecessaryCast |
Unnecessary cast (long) |
394 |
UnnecessaryCast |
Unnecessary cast (long) |
395 |
UnnecessaryCast |
Unnecessary cast (long) |
396 |
UnnecessaryCast |
Unnecessary cast (long) |
397 |
ShortVariable |
Avoid variables with short names like h1 |
530 |
ShortVariable |
Avoid variables with short names like h2 |
531 |
ShortVariable |
Avoid variables with short names like k1 |
537 |
ShortVariable |
Avoid variables with short names like k2 |
538 |
ShortVariable |
Avoid variables with short names like k1 |
560 |
ShortVariable |
Avoid variables with short names like k2 |
561 |
UnnecessaryCast |
Unnecessary cast (long) |
565 |
UnnecessaryCast |
Unnecessary cast (long) |
567 |
UnnecessaryCast |
Unnecessary cast (long) |
569 |
UnnecessaryCast |
Unnecessary cast (long) |
571 |
UnnecessaryCast |
Unnecessary cast (long) |
573 |
UnnecessaryCast |
Unnecessary cast (long) |
575 |
UnnecessaryCast |
Unnecessary cast (long) |
584 |
UnnecessaryCast |
Unnecessary cast (long) |
586 |
UnnecessaryCast |
Unnecessary cast (long) |
588 |
UnnecessaryCast |
Unnecessary cast (long) |
590 |
UnnecessaryCast |
Unnecessary cast (long) |
592 |
UnnecessaryCast |
Unnecessary cast (long) |
594 |
UnnecessaryCast |
Unnecessary cast (long) |
596 |
ShortVariable |
Avoid variables with short names like k |
720 |
ShortVariable |
Avoid variables with short names like k1 |
729 |
ShortVariable |
Avoid variables with short names like r0 |
787 |
ShortVariable |
Avoid variables with short names like r0 |
838 |
ShortVariable |
Avoid variables with short names like r1 |
839 |
ShortVariable |
Avoid variables with short names like k |
920 |
ShortVariable |
Avoid variables with short names like k1 |
926 |
ShortVariable |
Avoid variables with short names like k |
1050 |
ShortVariable |
Avoid variables with short names like k1 |
1061 |
UnnecessaryCast |
Unnecessary cast (long) |
1065 |
UnnecessaryCast |
Unnecessary cast (long) |
1067 |
UnnecessaryCast |
Unnecessary cast (long) |
1069 |
UnnecessaryCast |
Unnecessary cast (long) |
1071 |
UnnecessaryCast |
Unnecessary cast (long) |
1073 |
UnnecessaryCast |
Unnecessary cast (long) |
1075 |
UnnecessaryCast |
Unnecessary cast (long) |
1077 |
ShortVariable |
Avoid variables with short names like k1 |
1120 |
ShortVariable |
Avoid variables with short names like k |
1163 |
ShortVariable |
Avoid variables with short names like k1 |
1207 |
UnnecessaryCast |
Unnecessary cast (long) |
1208 |
UnnecessaryCast |
Unnecessary cast (long) |
1209 |
ShortVariable |
Avoid variables with short names like k |
1227 |
org/apache/commons/codec/digest/XXHash32.java
Rule |
Violation |
Line |
UseUnderscoresInNumericLiterals |
Number 2654435761L should separate every third digit with an underscore |
43 |
UseUnderscoresInNumericLiterals |
Number 2246822519L should separate every third digit with an underscore |
44 |
UseUnderscoresInNumericLiterals |
Number 3266489917L should separate every third digit with an underscore |
45 |
UseUnderscoresInNumericLiterals |
Number 668265263 should separate every third digit with an underscore |
46 |
UseUnderscoresInNumericLiterals |
Number 374761393 should separate every third digit with an underscore |
47 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
62 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
63 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
66 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
68 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
69 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
71 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
74 |
ShortVariable |
Avoid variables with short names like b |
132 |
ShortVariable |
Avoid variables with short names like s0 |
134 |
ShortVariable |
Avoid variables with short names like s1 |
135 |
ShortVariable |
Avoid variables with short names like s2 |
136 |
ShortVariable |
Avoid variables with short names like s3 |
137 |
ShortVariable |
Avoid variables with short names like b |
161 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
163 |
ShortVariable |
Avoid variables with short names like b |
202 |
org/apache/commons/codec/language/AbstractCaverphone.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
39 |
org/apache/commons/codec/language/Caverphone.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
45 |
org/apache/commons/codec/language/Caverphone1.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
39 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
39 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
54 |
org/apache/commons/codec/language/Caverphone2.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
39 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
39 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
54 |
org/apache/commons/codec/language/ColognePhonetic.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'CologneBuffer' |
190 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneBuffer(char[])' |
196 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneBuffer(int)' |
201 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneInputBuffer(char[])' |
224 |
ShortVariable |
Avoid variables with short names like ch |
244 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneOutputBuffer(int)' |
254 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
281 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
282 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
283 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
284 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
285 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
286 |
ShortVariable |
Avoid variables with short names like SZ |
288 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
288 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
292 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
294 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
302 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
311 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
328 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
342–346 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
100 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
103 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
161 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
162 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
163 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
165 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
165 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
166 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
179 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
185 |
ShortVariable |
Avoid variables with short names like ch |
191 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
206 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
208 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_END |
210 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
210 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
212 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
212 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
215 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
218 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
221 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
224 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
238 |
ShortVariable |
Avoid variables with short names like r |
297 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
323 |
ShortVariable |
Avoid variables with short names like sb |
357 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
412 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
456 |
ShortVariable |
Avoid variables with short names like ch |
466 |
org/apache/commons/codec/language/DoubleMetaphone.java
Rule |
Violation |
Line |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
180 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
185 |
LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
187 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
187 |
LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
189 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
217 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
222 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
235 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
245 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
249 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
259 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
262 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
265 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
268 |
ShortVariable |
Avoid variables with short names like c |
270 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
280 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
284 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
306 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
320 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
346 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
519 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
577 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
582 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
587 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
635–640 |
ShortVariable |
Avoid variables with short names like ch |
1050 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
68 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'cleanName(String)' |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
128 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
145 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getFirst3Last3(String)' |
166 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
172 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMinRating(int)' |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
221 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
224 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
227 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
230 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
254 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isVowel(String)' |
284 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'leftToRightThenRightToLeftProcessing(String, String)' |
302 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
345 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeAccents(String)' |
358 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
360 |
ShortVariable |
Avoid variables with short names like sb |
363 |
ShortVariable |
Avoid variables with short names like n |
364 |
ShortVariable |
Avoid variables with short names like c |
367 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeDoubleConsonants(String)' |
391 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeVowels(String)' |
414 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
428 |
org/apache/commons/codec/language/Nysiis.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like c |
102 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
123 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
128 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
138 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
140 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
143 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
145 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
152 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
157 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
162 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
167 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
252 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
259 |
org/apache/commons/codec/language/RefinedSoundex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
49 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
56 |
ShortVariable |
Avoid variables with short names like s1 |
126 |
ShortVariable |
Avoid variables with short names like s2 |
126 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMappingCode(char)' |
172 |
ShortVariable |
Avoid variables with short names like c |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
178 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
192 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
196 |
org/apache/commons/codec/language/Soundex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
61 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
69 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
93 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
110 |
ShortVariable |
Avoid variables with short names like s1 |
215 |
ShortVariable |
Avoid variables with short names like s2 |
215 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
268 |
ShortVariable |
Avoid variables with short names like ch |
283 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
314 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
318 |
ShortVariable |
Avoid variables with short names like ch |
326 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
93 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
108 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Line |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
102 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
104 |
LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
136 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
184 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
186 |
ShortVariable |
Avoid variables with short names like ls |
201 |
ShortVariable |
Avoid variables with short names like ls |
224 |
org/apache/commons/codec/language/bm/Languages.java
Rule |
Violation |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
86 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'merge(LanguageSet)' |
119 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
175 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
186 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
189 |
LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `someLanguages.languages::contains` |
192 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
205 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
207 |
LongVariable |
Avoid excessively long variable names like languagesResourceName |
313 |
ShortVariable |
Avoid variables with short names like ls |
315 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
339 |
org/apache/commons/codec/language/bm/PhoneticEngine.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'PhonemeBuilder' |
64 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
78 |
ShortVariable |
Avoid variables with short names like i |
168 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RulesApplication(Map<String, List<Rule>>, CharSequence, PhonemeBuilder, int, int)' |
172 |
ShortVariable |
Avoid variables with short names like i |
172 |
LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
226 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
226 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
228 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
255 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
257 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
259 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
261 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
263 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
315 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
391 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
399 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
438 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like c |
102 |
LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
295 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
297 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
299 |
LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
301 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
301 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
303 |
ShortVariable |
Avoid variables with short names like rs |
312 |
ShortVariable |
Avoid variables with short names like ls |
314 |
ShortVariable |
Avoid variables with short names like rt |
339 |
ShortVariable |
Avoid variables with short names like rt |
345 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
361 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
365 |
ShortVariable |
Avoid variables with short names like rt |
382 |
LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `allRules::addAll` |
386 |
ShortVariable |
Avoid variables with short names like rt |
401 |
ShortVariable |
Avoid variables with short names like rt |
417 |
ShortVariable |
Avoid variables with short names like rt |
435 |
ShortVariable |
Avoid variables with short names like ph |
447 |
ShortVariable |
Avoid variables with short names like in |
454 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
457 |
ShortVariable |
Avoid variables with short names like ph |
462 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
477 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
486 |
LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
519 |
ShortVariable |
Avoid variables with short names like ph |
533 |
ShortVariable |
Avoid variables with short names like r |
535 |
ShortVariable |
Avoid variables with short names like sb |
541 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
579–630 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
584 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
586 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
590 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
594 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
598 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
617 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
621 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
625–626 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pattern' |
633 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
645 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
649 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
667 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
669 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
671 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
673 |
ShortVariable |
Avoid variables with short names like i |
741 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
751 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
757 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
760 |
org/apache/commons/codec/net/BCodec.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
55 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
125 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
155 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
163 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
180 |
org/apache/commons/codec/net/PercentCodec.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMin |
60 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMax |
60 |
ShortVariable |
Avoid variables with short names like c |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
92 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
105 |
ShortVariable |
Avoid variables with short names like b |
109 |
ShortVariable |
Avoid variables with short names like u |
112 |
ShortVariable |
Avoid variables with short names like l |
113 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
137 |
ShortVariable |
Avoid variables with short names like bb |
149 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
174 |
LongVariable |
Avoid excessively long variable names like expectedEncodingBytes |
176 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
179 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
194 |
ShortVariable |
Avoid variables with short names like b |
205 |
ShortVariable |
Avoid variables with short names like c |
220 |
ShortVariable |
Avoid variables with short names like b |
230 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsArray |
248 |
ShortVariable |
Avoid variables with short names like c |
257 |
org/apache/commons/codec/net/QCodec.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
157 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
187 |
ShortVariable |
Avoid variables with short names like b |
199 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
200–204 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
206 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
214 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
239 |
ShortVariable |
Avoid variables with short names like b |
316 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like CR |
83 |
ShortVariable |
Avoid variables with short names like LF |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
126 |
ShortVariable |
Avoid variables with short names like b |
130 |
ShortVariable |
Avoid variables with short names like u |
137 |
ShortVariable |
Avoid variables with short names like l |
138 |
ShortVariable |
Avoid variables with short names like b |
162 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
164 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
206 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
216 |
ShortVariable |
Avoid variables with short names like b |
223 |
ShortVariable |
Avoid variables with short names like b |
241 |
ShortVariable |
Avoid variables with short names like b |
261 |
ShortVariable |
Avoid variables with short names like b |
284 |
ShortVariable |
Avoid variables with short names like b |
304 |
ShortVariable |
Avoid variables with short names like b |
318 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
325 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
330 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
427 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
430 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
469 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
490 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
525 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
528 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
574 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
597 |
org/apache/commons/codec/net/RFC1522Codec.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RFC1522Codec(Charset)' |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
81 |
ShortVariable |
Avoid variables with short names like to |
88 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
152 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
186 |
org/apache/commons/codec/net/URLCodec.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
104 |
ShortVariable |
Avoid variables with short names like b |
108 |
ShortVariable |
Avoid variables with short names like u |
113 |
ShortVariable |
Avoid variables with short names like l |
114 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
137 |
ShortVariable |
Avoid variables with short names like b |
145 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
171 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
218 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
221 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
243 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
298 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
301 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
323 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
345 |
Priority 4
org/apache/commons/codec/binary/Hex.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Hex |
38 |
org/apache/commons/codec/digest/B64.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like B64 |
31 |
org/apache/commons/codec/digest/Md5Crypt.java
Rule |
Violation |
Line |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'Md5Crypt': 'md5Crypt' is already in scope |
123 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Rule |
159 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Lang |
77 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Rule |
85 |
Files
org/apache/commons/codec/CharEncoding.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
128 |
org/apache/commons/codec/Charsets.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
169 |
org/apache/commons/codec/DecoderException.java
Rule |
Violation |
Priority |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
39 |
org/apache/commons/codec/EncoderException.java
Rule |
Violation |
Priority |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
40 |
org/apache/commons/codec/Resources.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
52 |
org/apache/commons/codec/StringEncoderComparator.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like o1 |
3 |
73 |
ShortVariable |
Avoid variables with short names like o2 |
3 |
73 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
80 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
81 |
org/apache/commons/codec/binary/Base16.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
47 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
48 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
49 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
3 |
56 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
3 |
70 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_TABLE |
3 |
77 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
3 |
92 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
148 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
154 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
157 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
202 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
204 |
org/apache/commons/codec/binary/Base32.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
84 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
84 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
86 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
86 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
87 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
87 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
93 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
112 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
124 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
143 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
151 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
154 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
157 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
160 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
163 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
182 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
193 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
198 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
402 |
ShortVariable |
Avoid variables with short names like b |
3 |
412 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
507 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
510 |
ShortVariable |
Avoid variables with short names like b |
3 |
577 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
607 |
org/apache/commons/codec/binary/Base64.java
Rule |
Violation |
Priority |
Line |
LinguisticNaming |
Linguistics Antipattern - The setter 'setUrlSafe' should not return any type except void linguistically |
3 |
97 |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
108 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
108 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
109 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
109 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
110 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
110 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
111 |
LongVariable |
Avoid excessively long variable names like DECODING_TABLE_LENGTH |
3 |
112 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
112 |
LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
3 |
122 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
122 |
LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
3 |
135 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
135 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
156 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
172 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
179 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
181 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
299 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
421 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toIntegerBytes(BigInteger)' |
3 |
448 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
455 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
482 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
487 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
492 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
498 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
500 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
743 |
ShortVariable |
Avoid variables with short names like b |
3 |
753 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
825 |
ShortVariable |
Avoid variables with short names like in |
3 |
825 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
827 |
ShortVariable |
Avoid variables with short names like b |
3 |
875 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
901 |
org/apache/commons/codec/binary/BaseNCodec.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
3 |
67 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'AbstractBuilder(byte[])' |
3 |
72 |
LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
3 |
72 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
3 |
89 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodingPolicy()' |
3 |
93 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodeTable()' |
3 |
97 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineLength()' |
3 |
101 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
105 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getPadding()' |
3 |
109 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodingPolicy' should not return any type except void linguistically |
3 |
119 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTable' should not return any type except void linguistically |
3 |
130 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setLineLength' should not return any type except void linguistically |
3 |
141 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setLineSeparator' should not return any type except void linguistically |
3 |
152 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPadding' should not return any type except void linguistically |
3 |
163 |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'Context' |
3 |
175 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'ibitWorkArea' |
3 |
181 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'lbitWorkArea' |
3 |
187 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'buffer' |
3 |
192 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pos' |
3 |
197 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'readPos' |
3 |
202 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'eof' |
3 |
208 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'currentLinePos' |
3 |
214 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'modulus' |
3 |
220 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
240 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EOF' |
3 |
240 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
252 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
264 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
3 |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
266 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
3 |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
284 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
287 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
292 |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
3 |
298 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
298 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
305 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'CHUNK_SEPARATOR' |
3 |
305 |
ShortVariable |
Avoid variables with short names like b |
3 |
373 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toLength(byte[])' |
3 |
384 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
392 |
FieldNamingConventions |
The final field name 'PAD' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
392 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
395 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
398 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
398 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
401 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
408 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
413 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
413 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
434 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
448 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
448 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
465 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
465 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
484 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
484 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'available(Context)' |
3 |
501 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
518 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
535 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
546 |
ShortVariable |
Avoid variables with short names like i |
3 |
546 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
562 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
591 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
611 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
622 |
ShortVariable |
Avoid variables with short names like i |
3 |
622 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
684 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'hasData(Context)' |
3 |
738 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
763 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'readResults(byte[], int, int, Context)' |
3 |
815 |
ShortVariable |
Avoid variables with short names like b |
3 |
815 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
827 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like r |
3 |
126 |
ShortVariable |
Avoid variables with short names like b |
3 |
131 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
132 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
163 |
ShortVariable |
Avoid variables with short names like c |
3 |
187 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
197 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
197 |
ShortVariable |
Avoid variables with short names like n |
3 |
225 |
ShortVariable |
Avoid variables with short names like b |
3 |
230 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like c |
3 |
122 |
ShortVariable |
Avoid variables with short names like i |
3 |
191 |
org/apache/commons/codec/binary/BinaryCodec.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
109 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty(byte[])' |
3 |
135 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
150 |
LocalVariableNamingConventions |
The final local variable name 'l_ascii' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
154 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
181 |
LocalVariableNamingConventions |
The final local variable name 'l_ascii' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
185 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
217 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
247 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
250 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
253 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
302 |
org/apache/commons/codec/binary/CharSequenceUtils.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'regionMatches(CharSequence, boolean, int, CharSequence, int, int)' |
3 |
54 |
ShortVariable |
Avoid variables with short names like cs |
3 |
54 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
57 |
ShortVariable |
Avoid variables with short names like c1 |
3 |
63 |
ShortVariable |
Avoid variables with short names like c2 |
3 |
64 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
69 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
74 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
86 |
org/apache/commons/codec/binary/Hex.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Hex |
4 |
38 |
LongVariable |
Avoid excessively long variable names like DEFAULT_CHARSET_NAME |
3 |
52 |
ShortVariable |
Avoid variables with short names like f |
3 |
102 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
348 |
ShortVariable |
Avoid variables with short names like ch |
3 |
365 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
373 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
452 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
455 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
458 |
org/apache/commons/codec/binary/StringUtils.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
75 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
94 |
MethodNamingConventions |
The static method name 'getBytesIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
144 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
169 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'StringUtils': 'newIllegalStateException' is already in scope |
4 |
174 |
ShortVariable |
Avoid variables with short names like e |
3 |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
310 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'StringUtils': 'newIllegalStateException' is already in scope |
4 |
315 |
MethodNamingConventions |
The static method name 'newStringIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
331 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
421 |
org/apache/commons/codec/digest/Blake3.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like in |
3 |
166 |
LongVariable |
Avoid excessively long variable names like parentNodesRemaining |
3 |
191 |
ShortVariable |
Avoid variables with short names like cv |
3 |
203 |
LongVariable |
Avoid excessively long variable names like inputChainingValue |
3 |
220 |
LongVariable |
Avoid excessively long variable names like inputChainingValue |
3 |
226 |
LongVariable |
Avoid excessively long variable names like outputBlockCounter |
3 |
239 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
255 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
256 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
257 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
258 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
259 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
260 |
LongVariable |
Avoid excessively long variable names like CHAINING_VALUE_INTS |
3 |
261 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
261 |
ShortVariable |
Avoid variables with short names like IV |
3 |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
266 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
269 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
270 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
271 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
272 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
273 |
LongVariable |
Avoid excessively long variable names like DERIVE_KEY_CONTEXT |
3 |
274 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
274 |
LongVariable |
Avoid excessively long variable names like DERIVE_KEY_MATERIAL |
3 |
275 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
275 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
281 |
ShortMethodName |
Avoid using short method names |
3 |
327 |
ShortVariable |
Avoid variables with short names like a |
3 |
327 |
ShortVariable |
Avoid variables with short names like b |
3 |
327 |
ShortVariable |
Avoid variables with short names like c |
3 |
327 |
ShortVariable |
Avoid variables with short names like d |
3 |
327 |
ShortVariable |
Avoid variables with short names like mx |
3 |
327 |
ShortVariable |
Avoid variables with short names like my |
3 |
327 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'Blake3': 'initHash' is already in scope |
4 |
346 |
UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'Blake3': 'initKeyedHash' is already in scope |
4 |
402 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
447 |
ShortVariable |
Avoid variables with short names like in |
3 |
515 |
ShortVariable |
Avoid variables with short names like in |
3 |
530 |
org/apache/commons/codec/digest/Crypt.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
77 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
80 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
83 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
86 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
182 |
org/apache/commons/codec/digest/HmacUtils.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
3 |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
788 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
804 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
873 |
org/apache/commons/codec/digest/MurmurHash3.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
79 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
79 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
3 |
94 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
99 |
ShortVariable |
Avoid variables with short names like b1 |
3 |
146 |
ShortVariable |
Avoid variables with short names like b2 |
3 |
146 |
ShortVariable |
Avoid variables with short names like b3 |
3 |
146 |
ShortVariable |
Avoid variables with short names like b4 |
3 |
146 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
151 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
154 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
157 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
163 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
168 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
182 |
ShortVariable |
Avoid variables with short names like k |
3 |
209 |
ShortVariable |
Avoid variables with short names like k |
3 |
238 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
3 |
271 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
273 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
312 |
UseUnderscoresInNumericLiterals |
Number 2862933555777941757L should separate every third digit with an underscore |
3 |
312 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
317 |
UseUnderscoresInNumericLiterals |
Number 104729 should separate every third digit with an underscore |
3 |
317 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
319 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
320 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
321 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
322 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
324 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
325 |
ShortVariable |
Avoid variables with short names like C1 |
3 |
327 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
327 |
ShortVariable |
Avoid variables with short names like C2 |
3 |
328 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
328 |
ShortVariable |
Avoid variables with short names like R1 |
3 |
329 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
329 |
ShortVariable |
Avoid variables with short names like R2 |
3 |
330 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
330 |
ShortVariable |
Avoid variables with short names like R3 |
3 |
331 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
331 |
ShortVariable |
Avoid variables with short names like M |
3 |
332 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
332 |
ShortVariable |
Avoid variables with short names like N1 |
3 |
334 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
334 |
ShortVariable |
Avoid variables with short names like N2 |
3 |
336 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
336 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
390 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
391 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
392 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
393 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
394 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
395 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
396 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
397 |
ShortVariable |
Avoid variables with short names like h1 |
3 |
530 |
ShortVariable |
Avoid variables with short names like h2 |
3 |
531 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
537 |
ShortVariable |
Avoid variables with short names like k2 |
3 |
538 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
560 |
ShortVariable |
Avoid variables with short names like k2 |
3 |
561 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
565 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
567 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
569 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
571 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
573 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
575 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
584 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
586 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
588 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
590 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
592 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
594 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
596 |
ShortVariable |
Avoid variables with short names like k |
3 |
720 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
729 |
ShortVariable |
Avoid variables with short names like r0 |
3 |
787 |
ShortVariable |
Avoid variables with short names like r0 |
3 |
838 |
ShortVariable |
Avoid variables with short names like r1 |
3 |
839 |
ShortVariable |
Avoid variables with short names like k |
3 |
920 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
926 |
ShortVariable |
Avoid variables with short names like k |
3 |
1050 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
1061 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1065 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1067 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1069 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1071 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1073 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1075 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1077 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
1120 |
ShortVariable |
Avoid variables with short names like k |
3 |
1163 |
ShortVariable |
Avoid variables with short names like k1 |
3 |
1207 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1208 |
UnnecessaryCast |
Unnecessary cast (long) |
3 |
1209 |
ShortVariable |
Avoid variables with short names like k |
3 |
1227 |
org/apache/commons/codec/digest/XXHash32.java
Rule |
Violation |
Priority |
Line |
UseUnderscoresInNumericLiterals |
Number 2654435761L should separate every third digit with an underscore |
3 |
43 |
UseUnderscoresInNumericLiterals |
Number 2246822519L should separate every third digit with an underscore |
3 |
44 |
UseUnderscoresInNumericLiterals |
Number 3266489917L should separate every third digit with an underscore |
3 |
45 |
UseUnderscoresInNumericLiterals |
Number 668265263 should separate every third digit with an underscore |
3 |
46 |
UseUnderscoresInNumericLiterals |
Number 374761393 should separate every third digit with an underscore |
3 |
47 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
62 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
63 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
66 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
68 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
69 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
71 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
74 |
ShortVariable |
Avoid variables with short names like b |
3 |
132 |
ShortVariable |
Avoid variables with short names like s0 |
3 |
134 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
135 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
136 |
ShortVariable |
Avoid variables with short names like s3 |
3 |
137 |
ShortVariable |
Avoid variables with short names like b |
3 |
161 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
163 |
ShortVariable |
Avoid variables with short names like b |
3 |
202 |
org/apache/commons/codec/language/AbstractCaverphone.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
39 |
org/apache/commons/codec/language/Caverphone.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
45 |
org/apache/commons/codec/language/Caverphone1.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
39 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
39 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
54 |
org/apache/commons/codec/language/Caverphone2.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
39 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
39 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
54 |
org/apache/commons/codec/language/ColognePhonetic.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'CologneBuffer' |
3 |
190 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneBuffer(char[])' |
3 |
196 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneBuffer(int)' |
3 |
201 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneInputBuffer(char[])' |
3 |
224 |
ShortVariable |
Avoid variables with short names like ch |
3 |
244 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneOutputBuffer(int)' |
3 |
254 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
281 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
282 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
283 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
284 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
285 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
286 |
ShortVariable |
Avoid variables with short names like SZ |
3 |
288 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
288 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
292 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
294 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
302 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
311 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
328 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
342–346 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
100 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
103 |
ShortClassName |
Avoid short class names like Rule |
4 |
159 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
161 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
162 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
163 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
165 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
165 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
166 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
179 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
185 |
ShortVariable |
Avoid variables with short names like ch |
3 |
191 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
206 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
208 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_END |
3 |
210 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
210 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
3 |
212 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
212 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
215 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
218 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
221 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
224 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
238 |
ShortVariable |
Avoid variables with short names like r |
3 |
297 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
323 |
ShortVariable |
Avoid variables with short names like sb |
3 |
357 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
412 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
456 |
ShortVariable |
Avoid variables with short names like ch |
3 |
466 |
org/apache/commons/codec/language/DoubleMetaphone.java
Rule |
Violation |
Priority |
Line |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
180 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
185 |
LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
3 |
187 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
187 |
LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
3 |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
188 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
189 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
217 |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
222 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
235 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
245 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
249 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
259 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
262 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
265 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
268 |
ShortVariable |
Avoid variables with short names like c |
3 |
270 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
280 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
284 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
306 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
320 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
346 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
519 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
577 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
582 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
587 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
635–640 |
ShortVariable |
Avoid variables with short names like ch |
3 |
1050 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
68 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'cleanName(String)' |
3 |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
128 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
145 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getFirst3Last3(String)' |
3 |
166 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
172 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMinRating(int)' |
3 |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
221 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
224 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
227 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
230 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
254 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isVowel(String)' |
3 |
284 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'leftToRightThenRightToLeftProcessing(String, String)' |
3 |
302 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
345 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeAccents(String)' |
3 |
358 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
360 |
ShortVariable |
Avoid variables with short names like sb |
3 |
363 |
ShortVariable |
Avoid variables with short names like n |
3 |
364 |
ShortVariable |
Avoid variables with short names like c |
3 |
367 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeDoubleConsonants(String)' |
3 |
391 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeVowels(String)' |
3 |
414 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
428 |
org/apache/commons/codec/language/Nysiis.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like c |
3 |
102 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
123 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
128 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
138 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
140 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
143 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
145 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
152 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
157 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
162 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
167 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
252 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
259 |
org/apache/commons/codec/language/RefinedSoundex.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
49 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
56 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
126 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
126 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMappingCode(char)' |
3 |
172 |
ShortVariable |
Avoid variables with short names like c |
3 |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
178 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
192 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
196 |
org/apache/commons/codec/language/Soundex.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
61 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
69 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
3 |
93 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
3 |
110 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
215 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
215 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
268 |
ShortVariable |
Avoid variables with short names like ch |
3 |
283 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
314 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
318 |
ShortVariable |
Avoid variables with short names like ch |
3 |
326 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
93 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
108 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Lang |
4 |
77 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
102 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
104 |
LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
3 |
136 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
184 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
186 |
ShortVariable |
Avoid variables with short names like ls |
3 |
201 |
ShortVariable |
Avoid variables with short names like ls |
3 |
224 |
org/apache/commons/codec/language/bm/Languages.java
Rule |
Violation |
Priority |
Line |
UnnecessaryConstructor |
Avoid unnecessary constructors - the compiler will generate these for you |
3 |
86 |
CommentDefaultAccessModifier |
Missing commented default access modifier on method 'merge(LanguageSet)' |
3 |
119 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
175 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
186 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
189 |
LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `someLanguages.languages::contains` |
3 |
192 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
205 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
207 |
LongVariable |
Avoid excessively long variable names like languagesResourceName |
3 |
313 |
ShortVariable |
Avoid variables with short names like ls |
3 |
315 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
339 |
org/apache/commons/codec/language/bm/PhoneticEngine.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'PhonemeBuilder' |
3 |
64 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
78 |
ShortVariable |
Avoid variables with short names like i |
3 |
168 |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RulesApplication(Map<String, List<Rule>>, CharSequence, PhonemeBuilder, int, int)' |
3 |
172 |
ShortVariable |
Avoid variables with short names like i |
3 |
172 |
LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
3 |
226 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
226 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
228 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
255 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
257 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
259 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
261 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
263 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
315 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
391 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
399 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
438 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Rule |
4 |
85 |
ShortVariable |
Avoid variables with short names like c |
3 |
102 |
LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
3 |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
290 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
295 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
297 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
299 |
LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
3 |
301 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
301 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
303 |
ShortVariable |
Avoid variables with short names like rs |
3 |
312 |
ShortVariable |
Avoid variables with short names like ls |
3 |
314 |
ShortVariable |
Avoid variables with short names like rt |
3 |
339 |
ShortVariable |
Avoid variables with short names like rt |
3 |
345 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
361 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
365 |
ShortVariable |
Avoid variables with short names like rt |
3 |
382 |
LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `allRules::addAll` |
3 |
386 |
ShortVariable |
Avoid variables with short names like rt |
3 |
401 |
ShortVariable |
Avoid variables with short names like rt |
3 |
417 |
ShortVariable |
Avoid variables with short names like rt |
3 |
435 |
ShortVariable |
Avoid variables with short names like ph |
3 |
447 |
ShortVariable |
Avoid variables with short names like in |
3 |
454 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
457 |
ShortVariable |
Avoid variables with short names like ph |
3 |
462 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
477 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
486 |
LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
3 |
519 |
ShortVariable |
Avoid variables with short names like ph |
3 |
533 |
ShortVariable |
Avoid variables with short names like r |
3 |
535 |
ShortVariable |
Avoid variables with short names like sb |
3 |
541 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
579–630 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
584 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
586 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
590 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
594 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
598 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
617 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
621 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
625–626 |
CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pattern' |
3 |
633 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
645 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
649 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
667 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
669 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
671 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
673 |
ShortVariable |
Avoid variables with short names like i |
3 |
741 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
751 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
757 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
760 |
org/apache/commons/codec/net/BCodec.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
3 |
55 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
125 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
155 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
163 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
180 |
org/apache/commons/codec/net/PercentCodec.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMin |
3 |
60 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMax |
3 |
60 |
ShortVariable |
Avoid variables with short names like c |
3 |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
92 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
105 |
ShortVariable |
Avoid variables with short names like b |
3 |
109 |
ShortVariable |
Avoid variables with short names like u |
3 |
112 |
ShortVariable |
Avoid variables with short names like l |
3 |
113 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
137 |
ShortVariable |
Avoid variables with short names like bb |
3 |
149 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
174 |
LongVariable |
Avoid excessively long variable names like expectedEncodingBytes |
3 |
176 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
179 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
194 |
ShortVariable |
Avoid variables with short names like b |
3 |
205 |
ShortVariable |
Avoid variables with short names like c |
3 |
220 |
ShortVariable |
Avoid variables with short names like b |
3 |
230 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsArray |
3 |
248 |
ShortVariable |
Avoid variables with short names like c |
3 |
257 |
org/apache/commons/codec/net/QCodec.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
157 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
187 |
ShortVariable |
Avoid variables with short names like b |
3 |
199 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
200–204 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
206 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
214 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
239 |
ShortVariable |
Avoid variables with short names like b |
3 |
316 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like CR |
3 |
83 |
ShortVariable |
Avoid variables with short names like LF |
3 |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
126 |
ShortVariable |
Avoid variables with short names like b |
3 |
130 |
ShortVariable |
Avoid variables with short names like u |
3 |
137 |
ShortVariable |
Avoid variables with short names like l |
3 |
138 |
ShortVariable |
Avoid variables with short names like b |
3 |
162 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
164 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
206 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
216 |
ShortVariable |
Avoid variables with short names like b |
3 |
223 |
ShortVariable |
Avoid variables with short names like b |
3 |
241 |
ShortVariable |
Avoid variables with short names like b |
3 |
261 |
ShortVariable |
Avoid variables with short names like b |
3 |
284 |
ShortVariable |
Avoid variables with short names like b |
3 |
304 |
ShortVariable |
Avoid variables with short names like b |
3 |
318 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
325 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
330 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
427 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
430 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
469 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
490 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
525 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
528 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
574 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
597 |
org/apache/commons/codec/net/RFC1522Codec.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RFC1522Codec(Charset)' |
3 |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
81 |
ShortVariable |
Avoid variables with short names like to |
3 |
88 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
152 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
186 |
org/apache/commons/codec/net/URLCodec.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
104 |
ShortVariable |
Avoid variables with short names like b |
3 |
108 |
ShortVariable |
Avoid variables with short names like u |
3 |
113 |
ShortVariable |
Avoid variables with short names like l |
3 |
114 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
137 |
ShortVariable |
Avoid variables with short names like b |
3 |
145 |
FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
171 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
218 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
221 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
243 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
298 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
301 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
323 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
345 |
|