PMD Results
The following document contains the results of PMD 6.55.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]*' |
130 |
Priority 3
org/apache/commons/codec/DecoderException.java
Rule |
Violation |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
39–40 |
org/apache/commons/codec/EncoderException.java
Rule |
Violation |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
40–41 |
org/apache/commons/codec/StringEncoderComparator.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like o1 |
73 |
ShortVariable |
Avoid variables with short names like o2 |
73 |
ShortVariable |
Avoid variables with short names like s1 |
80 |
ShortVariable |
Avoid variables with short names like s2 |
81 |
org/apache/commons/codec/binary/Base16.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
51 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
52 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
53 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
60 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
73 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_TABLE |
83 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
98 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
151–200 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
157 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
161 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
216–241 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
218 |
org/apache/commons/codec/binary/Base32.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
52 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
53 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
54 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
365–458 |
ShortVariable |
Avoid variables with short names like b |
375 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
476–570 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
480 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
490 |
ShortVariable |
Avoid variables with short names like b |
547 |
org/apache/commons/codec/binary/Base64.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
63 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
64 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
65 |
LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
75 |
LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
88 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
393 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the toIntegerBytes method if you want a default access modifier |
420–442 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
427 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
643–700 |
ShortVariable |
Avoid variables with short names like b |
652 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
724–793 |
ShortVariable |
Avoid variables with short names like in |
724 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
726 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
736 |
ShortVariable |
Avoid variables with short names like b |
774 |
org/apache/commons/codec/binary/BaseNCodec.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EOF field if you want a default access modifier |
59 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
85 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
91 |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
117 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the CHUNK_SEPARATOR field if you want a default access modifier |
124 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
136 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
151 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the Context nested class if you want a default access modifier |
181–240 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the ibitWorkArea field if you want a default access modifier |
187 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the lbitWorkArea field if you want a default access modifier |
193 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the buffer field if you want a default access modifier |
198 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the pos field if you want a default access modifier |
203 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the readPos field if you want a default access modifier |
208 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the eof field if you want a default access modifier |
214 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the currentLinePos field if you want a default access modifier |
220 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the modulus field if you want a default access modifier |
226 |
ShortVariable |
Avoid variables with short names like b |
309 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
323 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
324 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
338 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
339 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
355 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
356 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the available method if you want a default access modifier |
373–375 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
388 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
392 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
408 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
419 |
ShortVariable |
Avoid variables with short names like i |
419 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
435 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
464 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
484 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
495 |
ShortVariable |
Avoid variables with short names like i |
495 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
556 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the hasData method if you want a default access modifier |
611–613 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
639 |
ShortVariable |
Avoid variables with short names like b |
691 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the readResults method if you want a default access modifier |
691–706 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
703 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like r |
127 |
ShortVariable |
Avoid variables with short names like b |
132 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
133 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
167 |
ShortVariable |
Avoid variables with short names like c |
191 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
201 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
201 |
ShortVariable |
Avoid variables with short names like n |
228 |
ShortVariable |
Avoid variables with short names like b |
234 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like c |
120 |
ShortVariable |
Avoid variables with short names like i |
192 |
org/apache/commons/codec/binary/BinaryCodec.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
35–304 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
114 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isEmpty method if you want a default access modifier |
140–142 |
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 |
186 |
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 |
248 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
251 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
300 |
org/apache/commons/codec/binary/CharSequenceUtils.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the regionMatches method if you want a default access modifier |
54–83 |
ShortVariable |
Avoid variables with short names like cs |
54 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
57 |
ShortVariable |
Avoid variables with short names like c1 |
64 |
ShortVariable |
Avoid variables with short names like c2 |
65 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
78 |
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 |
109 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
351 |
ShortVariable |
Avoid variables with short names like ch |
368 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
453 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
456 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
459 |
org/apache/commons/codec/binary/StringUtils.java
Rule |
Violation |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
75 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
94 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
170 |
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/cli/Digest.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
105 |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
142 |
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 |
79 |
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 |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
88 |
org/apache/commons/codec/digest/DigestUtils.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
61 |
LongVariable |
Avoid excessively long variable names like defaultMessageDigest |
197 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
199 |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
384 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'inputStream': resource specifications are implicitly final |
1422 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'inputStream': resource specifications are implicitly final |
1489 |
org/apache/commons/codec/digest/HmacUtils.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
74 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
90 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'stream': resource specifications are implicitly final |
1060 |
org/apache/commons/codec/language/AbstractCaverphone.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
34–72 |
org/apache/commons/codec/language/Caverphone.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
35–97 |
org/apache/commons/codec/language/Caverphone1.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
32–126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
47 |
org/apache/commons/codec/language/Caverphone2.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
32–130 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
47 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_END |
76 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
124 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
127 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
185 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
186 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
187 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
189 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
189 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
203 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
209 |
ShortVariable |
Avoid variables with short names like ch |
215 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
231 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
242 |
ShortVariable |
Avoid variables with short names like r |
301 |
ShortVariable |
Avoid variables with short names like sb |
358 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
415 |
ShortVariable |
Avoid variables with short names like sb |
445 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
468 |
ShortVariable |
Avoid variables with short names like ch |
478 |
org/apache/commons/codec/language/DoubleMetaphone.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
37–992 |
LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
49 |
LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
51 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
292 |
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 |
355 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
360 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
408–413 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
769 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
772 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
775 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
778 |
ShortVariable |
Avoid variables with short names like c |
780 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
790 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
794 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
816 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
828 |
ShortVariable |
Avoid variables with short names like ch |
848 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
873 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
877 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
889 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
33–420 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the cleanName method if you want a default access modifier |
78–90 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
123 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getFirst3Last3 method if you want a default access modifier |
154–163 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
160 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getMinRating method if you want a default access modifier |
178–194 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
209 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
212 |
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 |
218 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
242 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isVowel method if you want a default access modifier |
272–275 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the leftToRightThenRightToLeftProcessing method if you want a default access modifier |
290–336 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
333 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeAccents method if you want a default access modifier |
346–365 |
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 sb |
351 |
ShortVariable |
Avoid variables with short names like n |
352 |
ShortVariable |
Avoid variables with short names like c |
355 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeDoubleConsonants method if you want a default access modifier |
379–388 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeVowels method if you want a default access modifier |
402–419 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
416 |
org/apache/commons/codec/language/Nysiis.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like c |
105 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
131 |
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 |
142 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
144 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
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 |
156 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
161 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
166 |
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 |
253 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
260 |
org/apache/commons/codec/language/RefinedSoundex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
49 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
56 |
ShortVariable |
Avoid variables with short names like s2 |
126 |
ShortVariable |
Avoid variables with short names like s1 |
126 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getMappingCode method if you want a default access modifier |
172–177 |
ShortVariable |
Avoid variables with short names like c |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
192 |
org/apache/commons/codec/language/Soundex.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
61 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
69 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
93 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
110 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
170 |
ShortVariable |
Avoid variables with short names like s1 |
225 |
ShortVariable |
Avoid variables with short names like s2 |
225 |
ShortVariable |
Avoid variables with short names like ch |
284 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
315 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
319 |
ShortVariable |
Avoid variables with short names like ch |
327 |
org/apache/commons/codec/language/SoundexUtils.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the clean method if you want a default access modifier |
40–56 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
42 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
53 |
ShortVariable |
Avoid variables with short names like s1 |
84 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the difference method if you want a default access modifier |
84–86 |
ShortVariable |
Avoid variables with short names like s2 |
84 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the differenceEncoded method if you want a default access modifier |
108–121 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
111 |
ShortVariable |
Avoid variables with short names like cs |
137 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isEmpty method if you want a default access modifier |
137–139 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
82–187 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
100 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
140 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
142–143 |
ShortVariable |
Avoid variables with short names like ls |
201 |
ShortVariable |
Avoid variables with short names like ls |
226 |
org/apache/commons/codec/language/bm/Languages.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the merge method if you want a default access modifier |
173 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
182–184 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
213 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
216 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
225 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
228 |
LongVariable |
Avoid excessively long variable names like languagesResourceName |
253 |
ShortVariable |
Avoid variables with short names like ls |
255 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'lsScanner': resource specifications are implicitly final |
256–257 |
org/apache/commons/codec/language/bm/PhoneticEngine.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
58 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the PhonemeBuilder nested class if you want a default access modifier |
92–177 |
ShortVariable |
Avoid variables with short names like i |
196 |
ShortVariable |
Avoid variables with short names like i |
201 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
316 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
378 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
380 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
382 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
392 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
400 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
439 |
org/apache/commons/codec/language/bm/ResourceConstants.java
Rule |
Violation |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the CMT field if you want a default access modifier |
31 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the ENCODING field if you want a default access modifier |
32 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EXT_CMT_END field if you want a default access modifier |
33 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EXT_CMT_START field if you want a default access modifier |
34 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
87 |
LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
95 |
ShortVariable |
Avoid variables with short names like c |
120 |
ShortVariable |
Avoid variables with short names like rs |
227 |
ShortVariable |
Avoid variables with short names like ls |
229 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
231 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
238 |
ShortVariable |
Avoid variables with short names like rt |
254 |
ShortVariable |
Avoid variables with short names like rt |
260 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
276 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
280 |
ShortVariable |
Avoid variables with short names like rt |
297 |
ShortVariable |
Avoid variables with short names like rt |
316 |
ShortVariable |
Avoid variables with short names like rt |
332 |
ShortVariable |
Avoid variables with short names like rt |
350 |
ShortVariable |
Avoid variables with short names like ph |
362 |
ShortVariable |
Avoid variables with short names like in |
369 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
372 |
ShortVariable |
Avoid variables with short names like ph |
377 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
392 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
401 |
LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
434 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'hashIncludeScanner': resource specifications are implicitly final |
434 |
ShortVariable |
Avoid variables with short names like ph |
448 |
ShortVariable |
Avoid variables with short names like r |
450 |
ShortVariable |
Avoid variables with short names like sb |
456 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
494–545 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
499 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
501 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
505 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
509 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
513 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
532 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
536 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
540–541 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the pattern field if you want a default access modifier |
548 |
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 |
564 |
ShortVariable |
Avoid variables with short names like i |
648 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
658 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
664 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
667 |
org/apache/commons/codec/net/BCodec.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
54 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
97–100 |
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 |
146 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
165 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
183 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
204 |
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 |
243 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
267 |
org/apache/commons/codec/net/PercentCodec.java
Rule |
Violation |
Line |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMax |
61 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMin |
61 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsArray |
91 |
ShortVariable |
Avoid variables with short names like b |
106 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
123 |
LongVariable |
Avoid excessively long variable names like expectedEncodingBytes |
126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
129 |
ShortVariable |
Avoid variables with short names like bb |
138 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
167 |
ShortVariable |
Avoid variables with short names like c |
173 |
ShortVariable |
Avoid variables with short names like c |
177 |
ShortVariable |
Avoid variables with short names like c |
181 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
192 |
ShortVariable |
Avoid variables with short names like b |
197 |
ShortVariable |
Avoid variables with short names like u |
200 |
ShortVariable |
Avoid variables with short names like l |
201 |
ShortVariable |
Avoid variables with short names like b |
218 |
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 |
253 |
org/apache/commons/codec/net/QCodec.java
Rule |
Violation |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
129–131 |
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 |
171 |
ShortVariable |
Avoid variables with short names like b |
183 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
184–188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
209 |
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 |
248 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
266 |
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 |
311 |
ShortVariable |
Avoid variables with short names like b |
355 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like CR |
93 |
ShortVariable |
Avoid variables with short names like LF |
95 |
UnnecessaryModifier |
Unnecessary modifier 'final' on method 'encodeQuotedPrintable': private methods cannot be overridden |
187–194 |
ShortVariable |
Avoid variables with short names like b |
187 |
ShortVariable |
Avoid variables with short names like b |
207 |
ShortVariable |
Avoid variables with short names like b |
225 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
228 |
ShortVariable |
Avoid variables with short names like b |
241 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
281 |
ShortVariable |
Avoid variables with short names like b |
294 |
ShortVariable |
Avoid variables with short names like b |
312 |
ShortVariable |
Avoid variables with short names like b |
332 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
362 |
ShortVariable |
Avoid variables with short names like b |
366 |
ShortVariable |
Avoid variables with short names like u |
373 |
ShortVariable |
Avoid variables with short names like l |
374 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
459 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
481 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
515 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
518 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
542 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
545 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
591 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
614 |
org/apache/commons/codec/net/RFC1522Codec.java
Rule |
Violation |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
43–188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
106 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
129 |
ShortVariable |
Avoid variables with short names like to |
136 |
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 |
128 |
ShortVariable |
Avoid variables with short names like b |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
168 |
ShortVariable |
Avoid variables with short names like b |
172 |
ShortVariable |
Avoid variables with short names like u |
177 |
ShortVariable |
Avoid variables with short names like l |
178 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
231 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
250 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
277 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
296 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
317 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
320 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
342 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
345 |
Priority 4
org/apache/commons/codec/binary/Hex.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Hex |
38–567 |
org/apache/commons/codec/digest/B64.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like B64 |
31–99 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Rule |
183–228 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Lang |
77–229 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Rule |
85–671 |
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–40 |
org/apache/commons/codec/EncoderException.java
Rule |
Violation |
Priority |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
40–41 |
org/apache/commons/codec/StringEncoderComparator.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like o1 |
3 |
73 |
ShortVariable |
Avoid variables with short names like o2 |
3 |
73 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
80 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
81 |
org/apache/commons/codec/binary/Base16.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
51 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
52 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
53 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_DECODE_TABLE |
3 |
60 |
LongVariable |
Avoid excessively long variable names like UPPER_CASE_ENCODE_TABLE |
3 |
73 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_DECODE_TABLE |
3 |
83 |
LongVariable |
Avoid excessively long variable names like LOWER_CASE_ENCODE_TABLE |
3 |
98 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
3 |
151–200 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
157 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
161 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
3 |
216–241 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
218 |
org/apache/commons/codec/binary/Base32.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
52 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
53 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
54 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
3 |
365–458 |
ShortVariable |
Avoid variables with short names like b |
3 |
375 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
3 |
476–570 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
480 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
490 |
ShortVariable |
Avoid variables with short names like b |
3 |
547 |
org/apache/commons/codec/binary/Base64.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like BITS_PER_ENCODED_BYTE |
3 |
63 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_UNENCODED_BLOCK |
3 |
64 |
LongVariable |
Avoid excessively long variable names like BYTES_PER_ENCODED_BLOCK |
3 |
65 |
LongVariable |
Avoid excessively long variable names like STANDARD_ENCODE_TABLE |
3 |
75 |
LongVariable |
Avoid excessively long variable names like URL_SAFE_ENCODE_TABLE |
3 |
88 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
269 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
393 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the toIntegerBytes method if you want a default access modifier |
3 |
420–442 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
427 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
3 |
643–700 |
ShortVariable |
Avoid variables with short names like b |
3 |
652 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
3 |
724–793 |
ShortVariable |
Avoid variables with short names like in |
3 |
724 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
726 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
736 |
ShortVariable |
Avoid variables with short names like b |
3 |
774 |
org/apache/commons/codec/binary/BaseNCodec.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EOF field if you want a default access modifier |
3 |
59 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_RESIZE_FACTOR |
3 |
85 |
LongVariable |
Avoid excessively long variable names like DEFAULT_BUFFER_SIZE |
3 |
91 |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
3 |
117 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the CHUNK_SEPARATOR field if you want a default access modifier |
3 |
124 |
FieldNamingConventions |
The final field name 'PAD' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
130 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
136 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
151 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the Context nested class if you want a default access modifier |
3 |
181–240 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the ibitWorkArea field if you want a default access modifier |
3 |
187 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the lbitWorkArea field if you want a default access modifier |
3 |
193 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the buffer field if you want a default access modifier |
3 |
198 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the pos field if you want a default access modifier |
3 |
203 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the readPos field if you want a default access modifier |
3 |
208 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the eof field if you want a default access modifier |
3 |
214 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the currentLinePos field if you want a default access modifier |
3 |
220 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the modulus field if you want a default access modifier |
3 |
226 |
ShortVariable |
Avoid variables with short names like b |
3 |
309 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
323 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
324 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
338 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
339 |
LongVariable |
Avoid excessively long variable names like unencodedBlockSize |
3 |
355 |
LongVariable |
Avoid excessively long variable names like chunkSeparatorLength |
3 |
356 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the available method if you want a default access modifier |
3 |
373–375 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
388 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
392 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
408 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the decode method if you want a default access modifier |
3 |
419 |
ShortVariable |
Avoid variables with short names like i |
3 |
419 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
435 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
464 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
484 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the encode method if you want a default access modifier |
3 |
495 |
ShortVariable |
Avoid variables with short names like i |
3 |
495 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
556 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the hasData method if you want a default access modifier |
3 |
611–613 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
639 |
ShortVariable |
Avoid variables with short names like b |
3 |
691 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the readResults method if you want a default access modifier |
3 |
691–706 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
703 |
org/apache/commons/codec/binary/BaseNCodecInputStream.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like r |
3 |
127 |
ShortVariable |
Avoid variables with short names like b |
3 |
132 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
133 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
167 |
ShortVariable |
Avoid variables with short names like c |
3 |
191 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
201 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
201 |
ShortVariable |
Avoid variables with short names like n |
3 |
228 |
ShortVariable |
Avoid variables with short names like b |
3 |
234 |
org/apache/commons/codec/binary/BaseNCodecOutputStream.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like c |
3 |
120 |
ShortVariable |
Avoid variables with short names like i |
3 |
192 |
org/apache/commons/codec/binary/BinaryCodec.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
35–304 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
114 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isEmpty method if you want a default access modifier |
3 |
140–142 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
155 |
LocalVariableNamingConventions |
The final local variable name 'l_ascii' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
159 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
186 |
LocalVariableNamingConventions |
The final local variable name 'l_ascii' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
190 |
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 |
248 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
251 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
300 |
org/apache/commons/codec/binary/CharSequenceUtils.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the regionMatches method if you want a default access modifier |
3 |
54–83 |
ShortVariable |
Avoid variables with short names like cs |
3 |
54 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
57 |
ShortVariable |
Avoid variables with short names like c1 |
3 |
64 |
ShortVariable |
Avoid variables with short names like c2 |
3 |
65 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
78 |
org/apache/commons/codec/binary/Hex.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Hex |
4 |
38–567 |
LongVariable |
Avoid excessively long variable names like DEFAULT_CHARSET_NAME |
3 |
52 |
ShortVariable |
Avoid variables with short names like f |
3 |
109 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
351 |
ShortVariable |
Avoid variables with short names like ch |
3 |
368 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
453 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
456 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
459 |
org/apache/commons/codec/binary/StringUtils.java
Rule |
Violation |
Priority |
Line |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
75 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
94 |
MethodNamingConventions |
The static method name 'getBytesIso8859_1' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
144–146 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
170 |
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–334 |
org/apache/commons/codec/cli/Digest.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
3 |
105 |
LongVariable |
Avoid excessively long variable names like messageDigestAlgorithm |
3 |
142 |
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 |
79 |
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 |
85 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
88 |
org/apache/commons/codec/digest/HmacUtils.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like STREAM_BUFFER_LENGTH |
3 |
60 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
74 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
90 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'stream': resource specifications are implicitly final |
3 |
1060 |
org/apache/commons/codec/digest/MurmurHash2.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like h |
3 |
75 |
ShortVariable |
Avoid variables with short names like k |
3 |
83 |
ShortVariable |
Avoid variables with short names like h |
3 |
183 |
ShortVariable |
Avoid variables with short names like k |
3 |
190 |
org/apache/commons/codec/language/AbstractCaverphone.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
34–72 |
org/apache/commons/codec/language/Caverphone.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
35–97 |
org/apache/commons/codec/language/Caverphone1.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
32–126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
47 |
org/apache/commons/codec/language/Caverphone2.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
32–130 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
47 |
org/apache/commons/codec/language/DaitchMokotoffSoundex.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_END |
3 |
76 |
LongVariable |
Avoid excessively long variable names like MULTILINE_COMMENT_START |
3 |
78 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
124 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
127 |
ShortClassName |
Avoid short class names like Rule |
4 |
183–228 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
185 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
186 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
187 |
LongVariable |
Avoid excessively long variable names like replacementAtStart |
3 |
189 |
LongVariable |
Avoid excessively long variable names like replacementBeforeVowel |
3 |
189 |
LongVariable |
Avoid excessively long variable names like replacementDefault |
3 |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
203 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
209 |
ShortVariable |
Avoid variables with short names like ch |
3 |
215 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
3 |
231 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
242 |
ShortVariable |
Avoid variables with short names like r |
3 |
301 |
ShortVariable |
Avoid variables with short names like sb |
3 |
358 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
415 |
ShortVariable |
Avoid variables with short names like sb |
3 |
445 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
468 |
ShortVariable |
Avoid variables with short names like ch |
3 |
478 |
org/apache/commons/codec/language/DoubleMetaphone.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
37–992 |
LongVariable |
Avoid excessively long variable names like L_R_N_M_B_H_F_V_W_SPACE |
3 |
49 |
LongVariable |
Avoid excessively long variable names like ES_EP_EB_EL_EY_IB_IL_IN_IE_EI_ER |
3 |
51 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
81 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
292 |
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 |
355 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
360 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
408–413 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
769 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
772 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
775 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
778 |
ShortVariable |
Avoid variables with short names like c |
3 |
780 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
790 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
794 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
816 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
828 |
ShortVariable |
Avoid variables with short names like ch |
3 |
848 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
873 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
877 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
889 |
org/apache/commons/codec/language/MatchRatingApproachEncoder.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
33–420 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the cleanName method if you want a default access modifier |
3 |
78–90 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
123 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getFirst3Last3 method if you want a default access modifier |
3 |
154–163 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
160 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getMinRating method if you want a default access modifier |
3 |
178–194 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
209 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
212 |
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 |
218 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
242 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isVowel method if you want a default access modifier |
3 |
272–275 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the leftToRightThenRightToLeftProcessing method if you want a default access modifier |
3 |
290–336 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
333 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeAccents method if you want a default access modifier |
3 |
346–365 |
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 sb |
3 |
351 |
ShortVariable |
Avoid variables with short names like n |
3 |
352 |
ShortVariable |
Avoid variables with short names like c |
3 |
355 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeDoubleConsonants method if you want a default access modifier |
3 |
379–388 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the removeVowels method if you want a default access modifier |
3 |
402–419 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
416 |
org/apache/commons/codec/language/Metaphone.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
55–419 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
88 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
91 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
95 |
ShortVariable |
Avoid variables with short names like n |
3 |
141 |
ShortVariable |
Avoid variables with short names like c |
3 |
333 |
ShortVariable |
Avoid variables with short names like c |
3 |
342 |
ShortVariable |
Avoid variables with short names like n |
3 |
360 |
org/apache/commons/codec/language/Nysiis.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like c |
3 |
105 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
131 |
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 |
142 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
144 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
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 |
156 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
161 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
166 |
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 |
253 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
260 |
org/apache/commons/codec/language/RefinedSoundex.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
49 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
56 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
126 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
126 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the getMappingCode method if you want a default access modifier |
3 |
172–177 |
ShortVariable |
Avoid variables with short names like c |
3 |
172 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
174 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
192 |
org/apache/commons/codec/language/Soundex.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING_STRING |
3 |
61 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_MAPPING |
3 |
69 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_SIMPLIFIED |
3 |
93 |
LongVariable |
Avoid excessively long variable names like US_ENGLISH_GENEALOGY |
3 |
110 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
170 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
225 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
225 |
ShortVariable |
Avoid variables with short names like ch |
3 |
284 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
315 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
319 |
ShortVariable |
Avoid variables with short names like ch |
3 |
327 |
org/apache/commons/codec/language/SoundexUtils.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the clean method if you want a default access modifier |
3 |
40–56 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
42 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
53 |
ShortVariable |
Avoid variables with short names like s1 |
3 |
84 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the difference method if you want a default access modifier |
3 |
84–86 |
ShortVariable |
Avoid variables with short names like s2 |
3 |
84 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the differenceEncoded method if you want a default access modifier |
3 |
108–121 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
111 |
ShortVariable |
Avoid variables with short names like cs |
3 |
137 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the isEmpty method if you want a default access modifier |
3 |
137–139 |
org/apache/commons/codec/language/bm/BeiderMorseEncoder.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
82–187 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
100 |
org/apache/commons/codec/language/bm/Lang.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Lang |
4 |
77–229 |
LongVariable |
Avoid excessively long variable names like languageRulesResourceName |
3 |
140 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
3 |
142–143 |
ShortVariable |
Avoid variables with short names like ls |
3 |
201 |
ShortVariable |
Avoid variables with short names like ls |
3 |
226 |
org/apache/commons/codec/language/bm/Languages.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the merge method if you want a default access modifier |
3 |
173 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
182–184 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
213 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
216 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
225 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
228 |
LongVariable |
Avoid excessively long variable names like languagesResourceName |
3 |
253 |
ShortVariable |
Avoid variables with short names like ls |
3 |
255 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'lsScanner': resource specifications are implicitly final |
3 |
256–257 |
org/apache/commons/codec/language/bm/PhoneticEngine.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like DEFAULT_MAX_PHONEMES |
3 |
58 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the PhonemeBuilder nested class if you want a default access modifier |
3 |
92–177 |
ShortVariable |
Avoid variables with short names like i |
3 |
196 |
ShortVariable |
Avoid variables with short names like i |
3 |
201 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
316 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
378 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
380 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
382 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
392 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
400 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
439 |
org/apache/commons/codec/language/bm/ResourceConstants.java
Rule |
Violation |
Priority |
Line |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the CMT field if you want a default access modifier |
3 |
31 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the ENCODING field if you want a default access modifier |
3 |
32 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EXT_CMT_END field if you want a default access modifier |
3 |
33 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the EXT_CMT_START field if you want a default access modifier |
3 |
34 |
org/apache/commons/codec/language/bm/Rule.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Rule |
4 |
85–671 |
LongVariable |
Avoid excessively long variable names like ALL_STRINGS_RMATCHER |
3 |
87 |
LongVariable |
Avoid excessively long variable names like HASH_INCLUDE_LENGTH |
3 |
95 |
ShortVariable |
Avoid variables with short names like c |
3 |
120 |
ShortVariable |
Avoid variables with short names like rs |
3 |
227 |
ShortVariable |
Avoid variables with short names like ls |
3 |
229 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
3 |
231 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'scanner': resource specifications are implicitly final |
3 |
238 |
ShortVariable |
Avoid variables with short names like rt |
3 |
254 |
ShortVariable |
Avoid variables with short names like rt |
3 |
260 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
276 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
280 |
ShortVariable |
Avoid variables with short names like rt |
3 |
297 |
ShortVariable |
Avoid variables with short names like rt |
3 |
316 |
ShortVariable |
Avoid variables with short names like rt |
3 |
332 |
ShortVariable |
Avoid variables with short names like rt |
3 |
350 |
ShortVariable |
Avoid variables with short names like ph |
3 |
362 |
ShortVariable |
Avoid variables with short names like in |
3 |
369 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
372 |
ShortVariable |
Avoid variables with short names like ph |
3 |
377 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
392 |
LongVariable |
Avoid excessively long variable names like inMultilineComment |
3 |
401 |
LongVariable |
Avoid excessively long variable names like hashIncludeScanner |
3 |
434 |
UnnecessaryModifier |
Unnecessary modifier 'final' on resource specification 'hashIncludeScanner': resource specifications are implicitly final |
3 |
434 |
ShortVariable |
Avoid variables with short names like ph |
3 |
448 |
ShortVariable |
Avoid variables with short names like r |
3 |
450 |
ShortVariable |
Avoid variables with short names like sb |
3 |
456 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
494–545 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
499 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
501 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
505 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
509 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
513 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
532 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
536 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
540–541 |
CommentDefaultAccessModifier |
To avoid mistakes add a comment at the beginning of the pattern field if you want a default access modifier |
3 |
548 |
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 |
564 |
ShortVariable |
Avoid variables with short names like i |
3 |
648 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
658 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
664 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
667 |
org/apache/commons/codec/net/BCodec.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like DECODING_POLICY_DEFAULT |
3 |
54 |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
97–100 |
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 |
146 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
165 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
183 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
204 |
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 |
243 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
267 |
org/apache/commons/codec/net/PercentCodec.java
Rule |
Violation |
Priority |
Line |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMax |
3 |
61 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsMin |
3 |
61 |
LongVariable |
Avoid excessively long variable names like alwaysEncodeCharsArray |
3 |
91 |
ShortVariable |
Avoid variables with short names like b |
3 |
106 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
123 |
LongVariable |
Avoid excessively long variable names like expectedEncodingBytes |
3 |
126 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
129 |
ShortVariable |
Avoid variables with short names like bb |
3 |
138 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
167 |
ShortVariable |
Avoid variables with short names like c |
3 |
173 |
ShortVariable |
Avoid variables with short names like c |
3 |
177 |
ShortVariable |
Avoid variables with short names like c |
3 |
181 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
192 |
ShortVariable |
Avoid variables with short names like b |
3 |
197 |
ShortVariable |
Avoid variables with short names like u |
3 |
200 |
ShortVariable |
Avoid variables with short names like l |
3 |
201 |
ShortVariable |
Avoid variables with short names like b |
3 |
218 |
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 |
253 |
org/apache/commons/codec/net/QCodec.java
Rule |
Violation |
Priority |
Line |
CallSuperInConstructor |
It is a good practice to call super() in a constructor |
3 |
129–131 |
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 |
171 |
ShortVariable |
Avoid variables with short names like b |
3 |
183 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
184–188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
190 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
209 |
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 |
248 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
266 |
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 |
311 |
ShortVariable |
Avoid variables with short names like b |
3 |
355 |
org/apache/commons/codec/net/QuotedPrintableCodec.java
Rule |
Violation |
Priority |
Line |
ShortVariable |
Avoid variables with short names like CR |
3 |
93 |
ShortVariable |
Avoid variables with short names like LF |
3 |
95 |
UnnecessaryModifier |
Unnecessary modifier 'final' on method 'encodeQuotedPrintable': private methods cannot be overridden |
3 |
187–194 |
ShortVariable |
Avoid variables with short names like b |
3 |
187 |
ShortVariable |
Avoid variables with short names like b |
3 |
207 |
ShortVariable |
Avoid variables with short names like b |
3 |
225 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
228 |
ShortVariable |
Avoid variables with short names like b |
3 |
241 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
281 |
ShortVariable |
Avoid variables with short names like b |
3 |
294 |
ShortVariable |
Avoid variables with short names like b |
3 |
312 |
ShortVariable |
Avoid variables with short names like b |
3 |
332 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
362 |
ShortVariable |
Avoid variables with short names like b |
3 |
366 |
ShortVariable |
Avoid variables with short names like u |
3 |
373 |
ShortVariable |
Avoid variables with short names like l |
3 |
374 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
459 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
481 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
515 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
518 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
542 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
545 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
591 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
614 |
org/apache/commons/codec/net/RFC1522Codec.java
Rule |
Violation |
Priority |
Line |
AtLeastOneConstructor |
Each class should declare at least one constructor |
3 |
43–188 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
72 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
106 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
129 |
ShortVariable |
Avoid variables with short names like to |
3 |
136 |
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 |
128 |
ShortVariable |
Avoid variables with short names like b |
3 |
136 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
168 |
ShortVariable |
Avoid variables with short names like b |
3 |
172 |
ShortVariable |
Avoid variables with short names like u |
3 |
177 |
ShortVariable |
Avoid variables with short names like l |
3 |
178 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
231 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
250 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
277 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
296 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
317 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
320 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
342 |
OnlyOneReturn |
A method should have only one exit point, and that should be the last statement in the method |
3 |
345 |
|