PMD Results
The following document contains the results of PMD 7.23.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]*' |
473 |
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]*' |
146 |
| MethodNamingConventions |
The static method name 'newStringIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
332 |
Priority 3
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/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/AbstractBaseNCodecStreamBuilder.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
43 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setBaseNCodec' should not return any type except void linguistically |
78 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncode' should not return any type except void linguistically |
89 |
org/apache/commons/codec/binary/Base16.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
103 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
103 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
105 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
105 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
107 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
107 |
| LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
114 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
114 |
| LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
128 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
128 |
| LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_TABLE |
135 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
135 |
| LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
150 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
150 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
153 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
200 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
206 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
209 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
254 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
256 |
org/apache/commons/codec/binary/Base16InputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
45 |
org/apache/commons/codec/binary/Base16OutputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
45 |
org/apache/commons/codec/binary/Base32.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
144 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
144 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
146 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
146 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
147 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
147 |
| 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. |
175 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
187 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
209 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
217 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
220 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
223 |
| 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. |
229 |
| 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. |
288 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
523 |
| ShortVariable |
Avoid variables with short names like b |
533 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
629 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
632 |
| ShortVariable |
Avoid variables with short names like b |
699 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
729 |
org/apache/commons/codec/binary/Base32InputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
66 |
org/apache/commons/codec/binary/Base32OutputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
70 |
org/apache/commons/codec/binary/Base58.java
| Rule |
Violation |
Line |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
88 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
96 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
108 |
| ShortVariable |
Avoid variables with short names like b |
179 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
233 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
235 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
266 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
268 |
org/apache/commons/codec/binary/Base58InputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
61 |
org/apache/commons/codec/binary/Base58OutputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
65 |
org/apache/commons/codec/binary/Base64.java
| Rule |
Violation |
Line |
| 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 |
136 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
138 |
| LongVariable |
Avoid excessively long variable names like isStandardEncodeTable |
147 |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
208 |
| 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 BYTES_PER_UNENCODED_BLOCK |
209 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
209 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
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 DECODING_TABLE_LENGTH |
211 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
211 |
| LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
221 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
221 |
| LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
234 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
234 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
256 |
| LongVariable |
Avoid excessively long variable names like STANDARD_DECODE_TABLE |
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 URL_SAFE_DECODE_TABLE |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
310 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
317 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
320 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
517 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
634 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
685 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
738 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toIntegerBytes(BigInteger)' |
767 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
773 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toUrlSafeEncodeTable(boolean)' |
789 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
796 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
801 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
802 |
| LongVariable |
Avoid excessively long variable names like isStandardEncodeTable |
803 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
803 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
986 |
| ShortVariable |
Avoid variables with short names like b |
996 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
1065 |
| ShortVariable |
Avoid variables with short names like in |
1065 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
1067 |
| ShortVariable |
Avoid variables with short names like b |
1115 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
1141 |
org/apache/commons/codec/binary/Base64InputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
68 |
org/apache/commons/codec/binary/Base64OutputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
74 |
org/apache/commons/codec/binary/BaseNCodec.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
61 |
| LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
66 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'AbstractBuilder(byte[])' |
73 |
| LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
73 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
91 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodeTable()' |
95 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodingPolicy()' |
99 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodedBlockSize()' |
103 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodeTable()' |
107 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineLength()' |
111 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
115 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getPadding()' |
119 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getUnencodedBlockSize()' |
123 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodeTable' should not return any type except void linguistically |
134 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodeTableRaw' should not return any type except void linguistically |
145 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setDecodeTableRaw(byte[])' |
145 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodingPolicy' should not return any type except void linguistically |
156 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodedBlockSize' should not return any type except void linguistically |
167 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setEncodedBlockSize(int)' |
167 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTable' should not return any type except void linguistically |
178 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTableRaw' should not return any type except void linguistically |
189 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setEncodeTableRaw(byte[])' |
189 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setLineLength' should not return any type except void linguistically |
200 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setLineSeparator' should not return any type except void linguistically |
211 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setPadding' should not return any type except void linguistically |
222 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setUnencodedBlockSize' should not return any type except void linguistically |
233 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setUnencodedBlockSize(int)' |
233 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
233 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'Context' |
244 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'ibitWorkArea' |
249 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'lbitWorkArea' |
254 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'buffer' |
259 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pos' |
264 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'readPos' |
269 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'eof' |
274 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'currentLinePos' |
280 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'modulus' |
285 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
304 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EOF' |
304 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
315 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
326 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
327 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
327 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
332 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
332 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
343 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
346 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
351 |
| LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
358 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
358 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
365 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'CHUNK_SEPARATOR' |
365 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
370 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EMPTY_BYTE_ARRAY' |
370 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLength(byte[])' |
413 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isInAlphabet(byte, byte[])' |
424 |
| ShortVariable |
Avoid variables with short names like b |
459 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
473 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
476 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
479 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
479 |
| 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. |
488 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
493 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
493 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
508 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
513 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'decodeTable' |
513 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
518 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'encodeTable' |
518 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
552 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
552 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
571 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
571 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
592 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
592 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'available(Context)' |
611 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
627 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
643 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
654 |
| ShortVariable |
Avoid variables with short names like i |
654 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
667 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
694 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
710 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
721 |
| ShortVariable |
Avoid variables with short names like i |
721 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
778 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'hasData(Context)' |
830 |
| LongVariable |
Avoid excessively long variable names like allowWhitespacePad |
849 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
852 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'readResults(byte[], int, int, Context)' |
896 |
| ShortVariable |
Avoid variables with short names like b |
896 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
908 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
60 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setByteArray' should not return any type except void linguistically |
80 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setInputStream' should not return any type except void linguistically |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
96 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
97 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
98 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
99 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
100 |
| ShortVariable |
Avoid variables with short names like r |
192 |
| ShortVariable |
Avoid variables with short names like b |
197 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
198 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
221 |
| ShortVariable |
Avoid variables with short names like c |
237 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
247 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
247 |
| ShortVariable |
Avoid variables with short names like n |
275 |
| ShortVariable |
Avoid variables with short names like b |
280 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
65 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setOutputStream' should not return any type except void linguistically |
84 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
91 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
92 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
93 |
| ShortVariable |
Avoid variables with short names like c |
175 |
| ShortVariable |
Avoid variables with short names like i |
232 |
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 |
82 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
108 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty(byte[])' |
132 |
| 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 |
174 |
| 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 |
238 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
241 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
286 |
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)' |
48 |
| ShortVariable |
Avoid variables with short names like cs |
48 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
51 |
| ShortVariable |
Avoid variables with short names like c1 |
57 |
| ShortVariable |
Avoid variables with short names like c2 |
58 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
63 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
68 |
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 |
96 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
171 |
| ShortVariable |
Avoid variables with short names like e |
270 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
311 |
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 |
516 |
| ShortVariable |
Avoid variables with short names like in |
531 |
org/apache/commons/codec/digest/Crc16.java
| Rule |
Violation |
Line |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
114 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
148 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
149 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
183 |
| 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. |
218 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
219 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
220 |
| 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. |
222 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
223 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
224 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
225 |
| 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. |
227 |
| 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. |
229 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
230 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
265 |
| 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. |
267 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
268 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
535 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
536 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
537 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
538 |
| ShortVariable |
Avoid variables with short names like b |
566 |
| ShortVariable |
Avoid variables with short names like b |
574 |
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 |
org/apache/commons/codec/digest/DigestUtils.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'BUFFER_SIZE' |
69 |
| LongVariable |
Avoid excessively long variable names like defaultMessageDigest |
179 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
181 |
| LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
390 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
1545 |
org/apache/commons/codec/digest/GitIdentifiers.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'DirectoryEntry' |
66 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'DirectoryEntry(String, FileMode, byte[])' |
97 |
| ShortVariable |
Avoid variables with short names like o |
108 |
| 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 |
181 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
184 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
192 |
| 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. |
222 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
223 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
399 |
org/apache/commons/codec/digest/HmacUtils.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
61 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
678 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
735 |
org/apache/commons/codec/digest/MurmurHash3.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
86 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
103 |
| ShortVariable |
Avoid variables with short names like k1 |
106 |
| ShortVariable |
Avoid variables with short names like b1 |
155 |
| ShortVariable |
Avoid variables with short names like b2 |
155 |
| ShortVariable |
Avoid variables with short names like b3 |
155 |
| ShortVariable |
Avoid variables with short names like b4 |
155 |
| 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. |
166 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
172 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
191 |
| ShortVariable |
Avoid variables with short names like k |
216 |
| ShortVariable |
Avoid variables with short names like k |
244 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
276 |
| ShortVariable |
Avoid variables with short names like k1 |
278 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
317 |
| UseUnderscoresInNumericLiterals |
Number 2862933555777941757L 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. |
323 |
| UseUnderscoresInNumericLiterals |
Number 104729 should separate every third digit with an underscore |
323 |
| 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. |
326 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
327 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
328 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
330 |
| 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 C1 |
333 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
333 |
| ShortVariable |
Avoid variables with short names like C2 |
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 R1 |
335 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
335 |
| ShortVariable |
Avoid variables with short names like R2 |
336 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
336 |
| ShortVariable |
Avoid variables with short names like R3 |
337 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
337 |
| ShortVariable |
Avoid variables with short names like M |
338 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
338 |
| ShortVariable |
Avoid variables with short names like N1 |
340 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
340 |
| ShortVariable |
Avoid variables with short names like N2 |
342 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
342 |
| ShortVariable |
Avoid variables with short names like h1 |
512 |
| ShortVariable |
Avoid variables with short names like h2 |
513 |
| ShortVariable |
Avoid variables with short names like k1 |
518 |
| ShortVariable |
Avoid variables with short names like k2 |
519 |
| ShortVariable |
Avoid variables with short names like k1 |
538 |
| ShortVariable |
Avoid variables with short names like k2 |
539 |
| UnnecessaryCast |
Unnecessary cast (long) |
543 |
| UnnecessaryCast |
Unnecessary cast (long) |
546 |
| UnnecessaryCast |
Unnecessary cast (long) |
549 |
| UnnecessaryCast |
Unnecessary cast (long) |
552 |
| UnnecessaryCast |
Unnecessary cast (long) |
555 |
| UnnecessaryCast |
Unnecessary cast (long) |
558 |
| UnnecessaryCast |
Unnecessary cast (long) |
568 |
| UnnecessaryCast |
Unnecessary cast (long) |
571 |
| UnnecessaryCast |
Unnecessary cast (long) |
574 |
| UnnecessaryCast |
Unnecessary cast (long) |
577 |
| UnnecessaryCast |
Unnecessary cast (long) |
580 |
| UnnecessaryCast |
Unnecessary cast (long) |
583 |
| UnnecessaryCast |
Unnecessary cast (long) |
586 |
| ShortVariable |
Avoid variables with short names like k |
718 |
| ShortVariable |
Avoid variables with short names like k1 |
724 |
| ShortVariable |
Avoid variables with short names like r0 |
782 |
| ShortVariable |
Avoid variables with short names like r0 |
833 |
| ShortVariable |
Avoid variables with short names like r1 |
834 |
| ShortVariable |
Avoid variables with short names like k |
918 |
| ShortVariable |
Avoid variables with short names like k1 |
923 |
| ShortVariable |
Avoid variables with short names like k |
1059 |
| ShortVariable |
Avoid variables with short names like k1 |
1068 |
| UnnecessaryCast |
Unnecessary cast (long) |
1072 |
| UnnecessaryCast |
Unnecessary cast (long) |
1075 |
| UnnecessaryCast |
Unnecessary cast (long) |
1078 |
| UnnecessaryCast |
Unnecessary cast (long) |
1081 |
| UnnecessaryCast |
Unnecessary cast (long) |
1084 |
| UnnecessaryCast |
Unnecessary cast (long) |
1087 |
| UnnecessaryCast |
Unnecessary cast (long) |
1090 |
| ShortVariable |
Avoid variables with short names like k1 |
1135 |
| ShortVariable |
Avoid variables with short names like k |
1181 |
| ShortVariable |
Avoid variables with short names like k1 |
1229 |
| UnnecessaryCast |
Unnecessary cast (long) |
1230 |
| UnnecessaryCast |
Unnecessary cast (long) |
1231 |
| ShortVariable |
Avoid variables with short names like k |
1248 |
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 |
131 |
| ShortVariable |
Avoid variables with short names like s0 |
133 |
| ShortVariable |
Avoid variables with short names like s1 |
134 |
| ShortVariable |
Avoid variables with short names like s2 |
135 |
| ShortVariable |
Avoid variables with short names like s3 |
136 |
| ShortVariable |
Avoid variables with short names like b |
157 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
159 |
| ShortVariable |
Avoid variables with short names like b |
193 |
org/apache/commons/codec/language/Caverphone1.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
41 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
56 |
org/apache/commons/codec/language/Caverphone2.java
| Rule |
Violation |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
41 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
56 |
org/apache/commons/codec/language/ColognePhonetic.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty()' |
207 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'length()' |
211 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneInputBuffer(char[])' |
223 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getNextChar()' |
234 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeNext()' |
242 |
| ShortVariable |
Avoid variables with short names like ch |
243 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneOutputBuffer(int)' |
253 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'put(char)' |
269 |
| 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 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
287 |
| ShortVariable |
Avoid variables with short names like SZ |
289 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
289 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
291 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
295 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
303 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
329 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
338–342 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like replacementAtStart |
158 |
| LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
159 |
| LongVariable |
Avoid excessively long variable names like replacementDefault |
160 |
| LongVariable |
Avoid excessively long variable names like replacementAtStart |
162 |
| LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
162 |
| LongVariable |
Avoid excessively long variable names like replacementDefault |
163 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
176 |
| 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 ch |
188 |
| 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 |
| 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_END |
212 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
212 |
| LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
214 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
214 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
217 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
220 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
223 |
| 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. |
230 |
| LongVariable |
Avoid excessively long variable names like inMultilineComment |
243 |
| ShortVariable |
Avoid variables with short names like r |
291 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
314 |
| ShortVariable |
Avoid variables with short names like sb |
348 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
401 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
445 |
| ShortVariable |
Avoid variables with short names like ch |
452 |
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. |
183 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
188 |
| LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
190 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
190 |
| LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
191 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
191 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
192 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
220 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
238 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
248 |
| 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 |
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 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
271 |
| ShortVariable |
Avoid variables with short names like c |
273 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
283 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
287 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
309 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
321 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
347 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
521 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
579 |
| 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 |
589 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
636–641 |
| ShortVariable |
Avoid variables with short names like ch |
1043 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'cleanName(String)' |
86 |
| 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/Metaphone.java
| Rule |
Violation |
Line |
| ShortVariable |
Avoid variables with short names like n |
122 |
| ShortVariable |
Avoid variables with short names like c |
137 |
| ShortVariable |
Avoid variables with short names like c |
145 |
| PrematureDeclaration |
Declaration of 'hard' can be moved closer to its usages |
168 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
171 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
175 |
| ShortVariable |
Avoid variables with short names like n |
221 |
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 |
245 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
252 |
org/apache/commons/codec/language/RefinedSoundex.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
52 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
59 |
| ShortVariable |
Avoid variables with short names like s1 |
129 |
| ShortVariable |
Avoid variables with short names like s2 |
129 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMappingCode(char)' |
175 |
| ShortVariable |
Avoid variables with short names like c |
175 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
177 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
181 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
195 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
199 |
org/apache/commons/codec/language/Soundex.java
| Rule |
Violation |
Line |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
58 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
65 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
84 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
98 |
| ShortVariable |
Avoid variables with short names like s1 |
191 |
| ShortVariable |
Avoid variables with short names like s2 |
191 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
238 |
| ShortVariable |
Avoid variables with short names like ch |
251 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
279 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
283 |
| ShortVariable |
Avoid variables with short names like ch |
291 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.java
| Rule |
Violation |
Line |
| 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 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
106 |
| LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
138 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
186 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
188 |
| ShortVariable |
Avoid variables with short names like ls |
203 |
| ShortVariable |
Avoid variables with short names like ls |
226 |
org/apache/commons/codec/language/bm/Languages.java
| Rule |
Violation |
Line |
| 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' |
65 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
79 |
| ShortVariable |
Avoid variables with short names like i |
169 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RulesApplication(Map<String, List<Rule>>, CharSequence, PhonemeBuilder, int, int)' |
173 |
| ShortVariable |
Avoid variables with short names like i |
173 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
227 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
227 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
229 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
231 |
| 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. |
260 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
262 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
264 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
266 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
318 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
394 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
402 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
441 |
org/apache/commons/codec/language/bm/Rule.java
| Rule |
Violation |
Line |
| ShortVariable |
Avoid variables with short names like c |
98 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
277 |
| LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
282 |
| 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. |
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. |
293 |
| LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
294 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
294 |
| 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. |
296 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
297 |
| ShortVariable |
Avoid variables with short names like rt |
330 |
| ShortVariable |
Avoid variables with short names like rt |
335 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
350 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
354 |
| ShortVariable |
Avoid variables with short names like rt |
368 |
| LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `allRules::addAll` |
371 |
| ShortVariable |
Avoid variables with short names like rt |
383 |
| ShortVariable |
Avoid variables with short names like rt |
396 |
| ShortVariable |
Avoid variables with short names like rt |
409 |
| ShortVariable |
Avoid variables with short names like ph |
417 |
| ShortVariable |
Avoid variables with short names like in |
424 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
426 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'parsePhonemeExpr(String)' |
434 |
| ShortVariable |
Avoid variables with short names like ph |
434 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
449 |
| LongVariable |
Avoid excessively long variable names like inMultilineComment |
457 |
| LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
485 |
| ShortVariable |
Avoid variables with short names like ph |
498 |
| ShortVariable |
Avoid variables with short names like r |
500 |
| ShortVariable |
Avoid variables with short names like sb |
507 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
541–589 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
546 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
548 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
552 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
556 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
560 |
| 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 |
581 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
585 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pattern' |
592 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
604 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
608 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
624 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
625 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
626 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
627 |
| ShortVariable |
Avoid variables with short names like i |
688 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
696 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
701 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
704 |
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 |
164 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
181 |
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 |
158 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
188 |
| ShortVariable |
Avoid variables with short names like b |
200 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
201–205 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
207 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
215 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
240 |
| ShortVariable |
Avoid variables with short names like b |
317 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
| Rule |
Violation |
Line |
| ShortVariable |
Avoid variables with short names like CR |
76 |
| ShortVariable |
Avoid variables with short names like LF |
77 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
114 |
| ShortVariable |
Avoid variables with short names like b |
118 |
| ShortVariable |
Avoid variables with short names like u |
125 |
| ShortVariable |
Avoid variables with short names like l |
126 |
| ShortVariable |
Avoid variables with short names like b |
147 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
149 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
185 |
| 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 |
200 |
| ShortVariable |
Avoid variables with short names like b |
216 |
| ShortVariable |
Avoid variables with short names like b |
235 |
| ShortVariable |
Avoid variables with short names like b |
256 |
| ShortVariable |
Avoid variables with short names like b |
273 |
| ShortVariable |
Avoid variables with short names like b |
286 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
298 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
379 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
382 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
416 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
433 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
463 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
466 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
506 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
525 |
org/apache/commons/codec/net/RFC1522Codec.java
| Rule |
Violation |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RFC1522Codec(Charset)' |
58 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
76 |
| ShortVariable |
Avoid variables with short names like to |
83 |
| 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 |
170 |
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 |
105 |
| ShortVariable |
Avoid variables with short names like b |
109 |
| ShortVariable |
Avoid variables with short names like u |
114 |
| ShortVariable |
Avoid variables with short names like l |
115 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
138 |
| ShortVariable |
Avoid variables with short names like b |
146 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
172 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
219 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
222 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
244 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
270 |
| 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 |
302 |
| 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/AbstractBaseNCodecStreamBuilder.java
| Rule |
Violation |
Line |
| BooleanGetMethodName |
A getX() method which returns a boolean or Boolean should be named isX() |
61 |
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/language/DaitchMokotoffSoundex.java
| Rule |
Violation |
Line |
| ShortClassName |
Avoid short class names like Rule |
155 |
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 |
81 |
Files
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/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/AbstractBaseNCodecStreamBuilder.java
| Rule |
Violation |
Priority |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
3 |
43 |
| BooleanGetMethodName |
A getX() method which returns a boolean or Boolean should be named isX() |
4 |
61 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setBaseNCodec' should not return any type except void linguistically |
3 |
78 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncode' should not return any type except void linguistically |
3 |
89 |
org/apache/commons/codec/binary/Base16.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
103 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
103 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
105 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
105 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
107 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
107 |
| LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
3 |
114 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
114 |
| LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
3 |
128 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
128 |
| LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_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 |
| LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
3 |
150 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
150 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
153 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
200 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
206 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
209 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
254 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
256 |
org/apache/commons/codec/binary/Base16InputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
45 |
org/apache/commons/codec/binary/Base16OutputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
45 |
org/apache/commons/codec/binary/Base32.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
144 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
144 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
146 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
146 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
147 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
147 |
| 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 |
175 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
187 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
209 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
217 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
220 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
223 |
| 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 |
229 |
| 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 |
288 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
523 |
| ShortVariable |
Avoid variables with short names like b |
3 |
533 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
629 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
632 |
| ShortVariable |
Avoid variables with short names like b |
3 |
699 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
729 |
org/apache/commons/codec/binary/Base32InputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
66 |
org/apache/commons/codec/binary/Base32OutputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
70 |
org/apache/commons/codec/binary/Base58.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 |
88 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
96 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
108 |
| ShortVariable |
Avoid variables with short names like b |
3 |
179 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
233 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
235 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
266 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
268 |
org/apache/commons/codec/binary/Base58InputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
61 |
org/apache/commons/codec/binary/Base58OutputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
65 |
org/apache/commons/codec/binary/Base64.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 |
132 |
| 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 |
| LongVariable |
Avoid excessively long variable names like isStandardEncodeTable |
3 |
147 |
| LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
208 |
| 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 BYTES_PER_UNENCODED_BLOCK |
3 |
209 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
209 |
| LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
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 DECODING_TABLE_LENGTH |
3 |
211 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
211 |
| LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
3 |
221 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
221 |
| LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
3 |
234 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
234 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
256 |
| LongVariable |
Avoid excessively long variable names like STANDARD_DECODE_TABLE |
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 URL_SAFE_DECODE_TABLE |
3 |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
310 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
317 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
320 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
517 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
634 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
685 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
738 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toIntegerBytes(BigInteger)' |
3 |
767 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
773 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'toUrlSafeEncodeTable(boolean)' |
3 |
789 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
796 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
801 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
802 |
| LongVariable |
Avoid excessively long variable names like isStandardEncodeTable |
3 |
803 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
803 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
986 |
| ShortVariable |
Avoid variables with short names like b |
3 |
996 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
1065 |
| ShortVariable |
Avoid variables with short names like in |
3 |
1065 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
1067 |
| ShortVariable |
Avoid variables with short names like b |
3 |
1115 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
1141 |
org/apache/commons/codec/binary/Base64InputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
68 |
org/apache/commons/codec/binary/Base64OutputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
74 |
org/apache/commons/codec/binary/BaseNCodec.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
61 |
| LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
3 |
66 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'AbstractBuilder(byte[])' |
3 |
73 |
| LongVariable |
Avoid excessively long variable names like defaultEncodeTable |
3 |
73 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'asThis()' |
3 |
91 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodeTable()' |
3 |
95 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getDecodingPolicy()' |
3 |
99 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodedBlockSize()' |
3 |
103 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getEncodeTable()' |
3 |
107 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineLength()' |
3 |
111 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLineSeparator()' |
3 |
115 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getPadding()' |
3 |
119 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getUnencodedBlockSize()' |
3 |
123 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodeTable' should not return any type except void linguistically |
3 |
134 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodeTableRaw' should not return any type except void linguistically |
3 |
145 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setDecodeTableRaw(byte[])' |
3 |
145 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setDecodingPolicy' should not return any type except void linguistically |
3 |
156 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodedBlockSize' should not return any type except void linguistically |
3 |
167 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setEncodedBlockSize(int)' |
3 |
167 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTable' should not return any type except void linguistically |
3 |
178 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setEncodeTableRaw' should not return any type except void linguistically |
3 |
189 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setEncodeTableRaw(byte[])' |
3 |
189 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setLineLength' should not return any type except void linguistically |
3 |
200 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setLineSeparator' should not return any type except void linguistically |
3 |
211 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setPadding' should not return any type except void linguistically |
3 |
222 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setUnencodedBlockSize' should not return any type except void linguistically |
3 |
233 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'setUnencodedBlockSize(int)' |
3 |
233 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
233 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'Context' |
3 |
244 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'ibitWorkArea' |
3 |
249 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'lbitWorkArea' |
3 |
254 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'buffer' |
3 |
259 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pos' |
3 |
264 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'readPos' |
3 |
269 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'eof' |
3 |
274 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'currentLinePos' |
3 |
280 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'modulus' |
3 |
285 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
304 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EOF' |
3 |
304 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
315 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
326 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
3 |
327 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
327 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
3 |
332 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
332 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
343 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
346 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
351 |
| LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
3 |
358 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
358 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
365 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'CHUNK_SEPARATOR' |
3 |
365 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
370 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'EMPTY_BYTE_ARRAY' |
3 |
370 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getLength(byte[])' |
3 |
413 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isInAlphabet(byte, byte[])' |
3 |
424 |
| ShortVariable |
Avoid variables with short names like b |
3 |
459 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
473 |
| FieldNamingConventions |
The final field name 'PAD' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
473 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
476 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
479 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
479 |
| 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 |
488 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
493 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
493 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
508 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
513 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'decodeTable' |
3 |
513 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
518 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'encodeTable' |
3 |
518 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
552 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
552 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
571 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
571 |
| LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
592 |
| LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
592 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'available(Context)' |
3 |
611 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
627 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
643 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'decode(byte[], int, int, Context)' |
3 |
654 |
| ShortVariable |
Avoid variables with short names like i |
3 |
654 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
667 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
694 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
710 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'encode(byte[], int, int, Context)' |
3 |
721 |
| ShortVariable |
Avoid variables with short names like i |
3 |
721 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
778 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'hasData(Context)' |
3 |
830 |
| LongVariable |
Avoid excessively long variable names like allowWhitespacePad |
3 |
849 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
852 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'readResults(byte[], int, int, Context)' |
3 |
896 |
| ShortVariable |
Avoid variables with short names like b |
3 |
896 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
908 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
60 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setByteArray' should not return any type except void linguistically |
3 |
80 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setInputStream' should not return any type except void linguistically |
3 |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
96 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
97 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
98 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
99 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
100 |
| ShortVariable |
Avoid variables with short names like r |
3 |
192 |
| ShortVariable |
Avoid variables with short names like b |
3 |
197 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
198 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
221 |
| ShortVariable |
Avoid variables with short names like c |
3 |
237 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
247 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
247 |
| ShortVariable |
Avoid variables with short names like n |
3 |
275 |
| ShortVariable |
Avoid variables with short names like b |
3 |
280 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
65 |
| LinguisticNaming |
Linguistics Antipattern - The setter 'setOutputStream' should not return any type except void linguistically |
3 |
84 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
90 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
91 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
92 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
93 |
| ShortVariable |
Avoid variables with short names like c |
3 |
175 |
| ShortVariable |
Avoid variables with short names like i |
3 |
232 |
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 |
82 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
108 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty(byte[])' |
3 |
132 |
| 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 |
174 |
| 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 |
238 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
241 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
286 |
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 |
48 |
| ShortVariable |
Avoid variables with short names like cs |
3 |
48 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
51 |
| ShortVariable |
Avoid variables with short names like c1 |
3 |
57 |
| ShortVariable |
Avoid variables with short names like c2 |
3 |
58 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
63 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
68 |
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 |
96 |
| MethodNamingConventions |
The static method name 'getBytesIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
146 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
171 |
| ShortVariable |
Avoid variables with short names like e |
3 |
270 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
311 |
| MethodNamingConventions |
The static method name 'newStringIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
332 |
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 |
| 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 |
516 |
| ShortVariable |
Avoid variables with short names like in |
3 |
531 |
org/apache/commons/codec/digest/Crc16.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 |
114 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
148 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
149 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
183 |
| 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 |
218 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
219 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
220 |
| 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 |
222 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
223 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
224 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
225 |
| 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 |
227 |
| 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 |
229 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
230 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
265 |
| 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 |
267 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
268 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
535 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
536 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
537 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
538 |
| ShortVariable |
Avoid variables with short names like b |
3 |
566 |
| ShortVariable |
Avoid variables with short names like b |
3 |
574 |
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 |
org/apache/commons/codec/digest/GitIdentifiers.java
| Rule |
Violation |
Priority |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on nested class 'DirectoryEntry' |
3 |
66 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'DirectoryEntry(String, FileMode, byte[])' |
3 |
97 |
| ShortVariable |
Avoid variables with short names like o |
3 |
108 |
| 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 |
181 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
184 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
192 |
| 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 |
222 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
223 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
399 |
org/apache/commons/codec/digest/HmacUtils.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
3 |
61 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
678 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
735 |
org/apache/commons/codec/digest/MurmurHash3.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
86 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
3 |
103 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
106 |
| ShortVariable |
Avoid variables with short names like b1 |
3 |
155 |
| ShortVariable |
Avoid variables with short names like b2 |
3 |
155 |
| ShortVariable |
Avoid variables with short names like b3 |
3 |
155 |
| ShortVariable |
Avoid variables with short names like b4 |
3 |
155 |
| 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 |
166 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
172 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
191 |
| ShortVariable |
Avoid variables with short names like k |
3 |
216 |
| ShortVariable |
Avoid variables with short names like k |
3 |
244 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'finalise(int, int, byte[], int)' |
3 |
276 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
278 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
317 |
| UseUnderscoresInNumericLiterals |
Number 2862933555777941757L 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 |
323 |
| UseUnderscoresInNumericLiterals |
Number 104729 should separate every third digit with an underscore |
3 |
323 |
| 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 |
326 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
327 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
328 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
330 |
| 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 C1 |
3 |
333 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
333 |
| ShortVariable |
Avoid variables with short names like C2 |
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 R1 |
3 |
335 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
335 |
| ShortVariable |
Avoid variables with short names like R2 |
3 |
336 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
336 |
| ShortVariable |
Avoid variables with short names like R3 |
3 |
337 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
337 |
| ShortVariable |
Avoid variables with short names like M |
3 |
338 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
338 |
| ShortVariable |
Avoid variables with short names like N1 |
3 |
340 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
340 |
| ShortVariable |
Avoid variables with short names like N2 |
3 |
342 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
342 |
| ShortVariable |
Avoid variables with short names like h1 |
3 |
512 |
| ShortVariable |
Avoid variables with short names like h2 |
3 |
513 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
518 |
| ShortVariable |
Avoid variables with short names like k2 |
3 |
519 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
538 |
| ShortVariable |
Avoid variables with short names like k2 |
3 |
539 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
543 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
546 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
549 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
552 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
555 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
558 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
568 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
571 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
574 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
577 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
580 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
583 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
586 |
| ShortVariable |
Avoid variables with short names like k |
3 |
718 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
724 |
| ShortVariable |
Avoid variables with short names like r0 |
3 |
782 |
| ShortVariable |
Avoid variables with short names like r0 |
3 |
833 |
| ShortVariable |
Avoid variables with short names like r1 |
3 |
834 |
| ShortVariable |
Avoid variables with short names like k |
3 |
918 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
923 |
| ShortVariable |
Avoid variables with short names like k |
3 |
1059 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
1068 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1072 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1075 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1078 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1081 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1084 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1087 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1090 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
1135 |
| ShortVariable |
Avoid variables with short names like k |
3 |
1181 |
| ShortVariable |
Avoid variables with short names like k1 |
3 |
1229 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1230 |
| UnnecessaryCast |
Unnecessary cast (long) |
3 |
1231 |
| ShortVariable |
Avoid variables with short names like k |
3 |
1248 |
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 |
131 |
| ShortVariable |
Avoid variables with short names like s0 |
3 |
133 |
| ShortVariable |
Avoid variables with short names like s1 |
3 |
134 |
| ShortVariable |
Avoid variables with short names like s2 |
3 |
135 |
| ShortVariable |
Avoid variables with short names like s3 |
3 |
136 |
| ShortVariable |
Avoid variables with short names like b |
3 |
157 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
159 |
| ShortVariable |
Avoid variables with short names like b |
3 |
193 |
org/apache/commons/codec/language/Caverphone1.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
41 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
56 |
org/apache/commons/codec/language/Caverphone2.java
| Rule |
Violation |
Priority |
Line |
| CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
41 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
56 |
org/apache/commons/codec/language/ColognePhonetic.java
| Rule |
Violation |
Priority |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'isEmpty()' |
3 |
207 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'length()' |
3 |
211 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneInputBuffer(char[])' |
3 |
223 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getNextChar()' |
3 |
234 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'removeNext()' |
3 |
242 |
| ShortVariable |
Avoid variables with short names like ch |
3 |
243 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'CologneOutputBuffer(int)' |
3 |
253 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'put(char)' |
3 |
269 |
| 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 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
287 |
| ShortVariable |
Avoid variables with short names like SZ |
3 |
289 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
289 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
291 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
295 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
303 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
329 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
338–342 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
| Rule |
Violation |
Priority |
Line |
| ShortClassName |
Avoid short class names like Rule |
4 |
155 |
| LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
158 |
| LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
159 |
| LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
160 |
| LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
162 |
| LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
162 |
| LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
163 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
176 |
| 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 ch |
3 |
188 |
| 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 |
| 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_END |
3 |
212 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
212 |
| LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
3 |
214 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
214 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
217 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
220 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
223 |
| 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 |
230 |
| LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
243 |
| ShortVariable |
Avoid variables with short names like r |
3 |
291 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
314 |
| ShortVariable |
Avoid variables with short names like sb |
3 |
348 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
401 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
445 |
| ShortVariable |
Avoid variables with short names like ch |
3 |
452 |
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 |
183 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
188 |
| LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
3 |
190 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
190 |
| LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
3 |
191 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
191 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
192 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
220 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
238 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
248 |
| 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 |
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 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
271 |
| ShortVariable |
Avoid variables with short names like c |
3 |
273 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
283 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
287 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
309 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
321 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
347 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
521 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
579 |
| 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 |
589 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
636–641 |
| ShortVariable |
Avoid variables with short names like ch |
3 |
1043 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
| Rule |
Violation |
Priority |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'cleanName(String)' |
3 |
86 |
| 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/Metaphone.java
| Rule |
Violation |
Priority |
Line |
| ShortVariable |
Avoid variables with short names like n |
3 |
122 |
| ShortVariable |
Avoid variables with short names like c |
3 |
137 |
| ShortVariable |
Avoid variables with short names like c |
3 |
145 |
| PrematureDeclaration |
Declaration of 'hard' can be moved closer to its usages |
3 |
168 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
171 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
175 |
| ShortVariable |
Avoid variables with short names like n |
3 |
221 |
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 |
245 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
252 |
org/apache/commons/codec/language/RefinedSoundex.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
52 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
59 |
| ShortVariable |
Avoid variables with short names like s1 |
3 |
129 |
| ShortVariable |
Avoid variables with short names like s2 |
3 |
129 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'getMappingCode(char)' |
3 |
175 |
| ShortVariable |
Avoid variables with short names like c |
3 |
175 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
177 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
181 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
195 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
199 |
org/apache/commons/codec/language/Soundex.java
| Rule |
Violation |
Priority |
Line |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
58 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
65 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
3 |
84 |
| LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
3 |
98 |
| ShortVariable |
Avoid variables with short names like s1 |
3 |
191 |
| ShortVariable |
Avoid variables with short names like s2 |
3 |
191 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
238 |
| ShortVariable |
Avoid variables with short names like ch |
3 |
251 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
279 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
283 |
| ShortVariable |
Avoid variables with short names like ch |
3 |
291 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.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 |
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 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
106 |
| LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
3 |
138 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
186 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
188 |
| ShortVariable |
Avoid variables with short names like ls |
3 |
203 |
| ShortVariable |
Avoid variables with short names like ls |
3 |
226 |
org/apache/commons/codec/language/bm/Languages.java
| Rule |
Violation |
Priority |
Line |
| 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 |
65 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
79 |
| ShortVariable |
Avoid variables with short names like i |
3 |
169 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RulesApplication(Map<String, List<Rule>>, CharSequence, PhonemeBuilder, int, int)' |
3 |
173 |
| ShortVariable |
Avoid variables with short names like i |
3 |
173 |
| LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
3 |
227 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
227 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
229 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
231 |
| 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 |
260 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
262 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
264 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
266 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
318 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
394 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
402 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
441 |
org/apache/commons/codec/language/bm/Rule.java
| Rule |
Violation |
Priority |
Line |
| ShortClassName |
Avoid short class names like Rule |
4 |
81 |
| ShortVariable |
Avoid variables with short names like c |
3 |
98 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
277 |
| LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
3 |
282 |
| 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 |
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 |
293 |
| LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
3 |
294 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
294 |
| 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 |
296 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
297 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
330 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
335 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
350 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
354 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
368 |
| LambdaCanBeMethodReference |
Lambda expression could be written as a method reference: `allRules::addAll` |
3 |
371 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
383 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
396 |
| ShortVariable |
Avoid variables with short names like rt |
3 |
409 |
| ShortVariable |
Avoid variables with short names like ph |
3 |
417 |
| ShortVariable |
Avoid variables with short names like in |
3 |
424 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
426 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on method 'parsePhonemeExpr(String)' |
3 |
434 |
| ShortVariable |
Avoid variables with short names like ph |
3 |
434 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
449 |
| LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
457 |
| LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
3 |
485 |
| ShortVariable |
Avoid variables with short names like ph |
3 |
498 |
| ShortVariable |
Avoid variables with short names like r |
3 |
500 |
| ShortVariable |
Avoid variables with short names like sb |
3 |
507 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
541–589 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
546 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
548 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
552 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
556 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
560 |
| 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 |
581 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
585 |
| CommentDefaultAccessModifier |
Missing commented default access modifier on field 'pattern' |
3 |
592 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
604 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
608 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
624 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
625 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
626 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
627 |
| ShortVariable |
Avoid variables with short names like i |
3 |
688 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
696 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
701 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
704 |
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 |
164 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
181 |
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 |
158 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
188 |
| ShortVariable |
Avoid variables with short names like b |
3 |
200 |
| ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
201–205 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
207 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
215 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
240 |
| ShortVariable |
Avoid variables with short names like b |
3 |
317 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
| Rule |
Violation |
Priority |
Line |
| ShortVariable |
Avoid variables with short names like CR |
3 |
76 |
| ShortVariable |
Avoid variables with short names like LF |
3 |
77 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
114 |
| ShortVariable |
Avoid variables with short names like b |
3 |
118 |
| ShortVariable |
Avoid variables with short names like u |
3 |
125 |
| ShortVariable |
Avoid variables with short names like l |
3 |
126 |
| ShortVariable |
Avoid variables with short names like b |
3 |
147 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
149 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
185 |
| 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 |
200 |
| ShortVariable |
Avoid variables with short names like b |
3 |
216 |
| ShortVariable |
Avoid variables with short names like b |
3 |
235 |
| ShortVariable |
Avoid variables with short names like b |
3 |
256 |
| ShortVariable |
Avoid variables with short names like b |
3 |
273 |
| ShortVariable |
Avoid variables with short names like b |
3 |
286 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
293 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
298 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
379 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
382 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
416 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
433 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
463 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
466 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
506 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
525 |
org/apache/commons/codec/net/RFC1522Codec.java
| Rule |
Violation |
Priority |
Line |
| CommentDefaultAccessModifier |
Missing commented default access modifier on constructor 'RFC1522Codec(Charset)' |
3 |
58 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
76 |
| ShortVariable |
Avoid variables with short names like to |
3 |
83 |
| 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 |
170 |
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 |
105 |
| ShortVariable |
Avoid variables with short names like b |
3 |
109 |
| ShortVariable |
Avoid variables with short names like u |
3 |
114 |
| ShortVariable |
Avoid variables with short names like l |
3 |
115 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
138 |
| ShortVariable |
Avoid variables with short names like b |
3 |
146 |
| FieldDeclarationsShouldBeAtStartOfClass |
Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
3 |
172 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
219 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
222 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
244 |
| OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
270 |
| 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 |
302 |
| 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 |
Suppressed Violations
| Filename |
Rule message |
Suppression type |
Reason |
| org/apache/commons/codec/binary/Base16InputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecInputStream.AbstracBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
| org/apache/commons/codec/binary/Base16OutputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecOutputStream.AbstractBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
| org/apache/commons/codec/binary/Base32InputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecInputStream.AbstracBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
| org/apache/commons/codec/binary/Base32OutputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecOutputStream.AbstractBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
| org/apache/commons/codec/binary/Base64InputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecInputStream.AbstracBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
| org/apache/commons/codec/binary/Base64OutputStream.java |
Unused import 'org.apache.commons.codec.binary.BaseNCodecOutputStream.AbstractBuilder' |
//nopmd |
: Required by ECJ (Eclipse) |
|
