The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'AudioConcat' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 53–127 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingShortType | Do not use the short type | 116 |
| AvoidUsingShortType | Do not use the short type | 117 |
| AvoidUsingShortType | Do not use the short type | 135 |
| AvoidUsingShortType | Do not use the short type | 136 |
| Rule | Violation | Line |
|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 54–57 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 65–68 |
| AvoidUsingShortType | Do not use the short type | 93 |
| AvoidUsingShortType | Do not use the short type | 109 |
| AvoidUsingShortType | Do not use the short type | 125 |
| AvoidUsingShortType | Do not use the short type | 141 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 70 |
| Rule | Violation | Line |
|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 80 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'generateAndStoreCaptcha(Locale, String)' is missing an @Override annotation. | 77–83 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'generateAndStoreCaptcha(Locale, String)' is missing an @Override annotation. | 77–83 |
| Rule | Violation | Line |
|---|---|---|
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 55 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'init(FilterConfig)' is missing an @Override annotation. | 77–82 |
| MissingOverride | The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' is missing an @Override annotation. | 98–156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 105 |
| MissingOverride | The method 'destroy()' is missing an @Override annotation. | 161–164 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_response' could be made final; it is only initialized in the declaration or constructor. | 60 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the LuteceGimpyImage constructor if you want a default access modifier | 71–75 |
| MissingOverride | The method 'validateResponse(Object)' is missing an @Override annotation. | 85–88 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_myRandom' could be made final; it is only initialized in the declaration or constructor. | 55 |
| ImmutableField | Private field '_wordToImage' could be made final; it is only initialized in the declaration or constructor. | 56 |
| ImmutableField | Private field '_wordGenerator' could be made final; it is only initialized in the declaration or constructor. | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 66–80 |
| MissingOverride | The method 'getImageCaptcha()' is missing an @Override annotation. | 87–90 |
| MissingOverride | The method 'getImageCaptcha(Locale)' is missing an @Override annotation. | 118–139 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 131 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 136 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'captcha' | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 149 |
| Rule | Violation | Line |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 72 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 82 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_attenuationValue' could be made final; it is only initialized in the declaration or constructor. | 53 |
| ImmutableField | Private field '_backgroundSoundFiles' could be made final; it is only initialized in the declaration or constructor. | 54 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 68 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 71–79 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_wordGenerator' could be made final; it is only initialized in the declaration or constructor. | 63 |
| ImmutableField | Private field '_word2Sound' could be made final; it is only initialized in the declaration or constructor. | 64 |
| ImmutableField | Private field '_myRandom' could be made final; it is only initialized in the declaration or constructor. | 65 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 75–89 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 101–120 |
| MissingOverride | The method 'getSoundCaptcha()' is missing an @Override annotation. | 143–150 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 145 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'soundCaptcha' | 149 |
| MissingOverride | The method 'getSoundCaptcha(Locale)' is missing an @Override annotation. | 157–164 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 159 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'soundCaptcha' | 163 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 185 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'LuteceSoundConfigurator' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=5, WMC=6) | 43–109 |
| ImmutableField | Private field '_location' could be made final; it is only initialized in the declaration or constructor. | 46 |
| ImmutableField | Private field '_name' could be made final; it is only initialized in the declaration or constructor. | 47 |
| ImmutableField | Private field '_pitch' could be made final; it is only initialized in the declaration or constructor. | 48 |
| ImmutableField | Private field '_rate' could be made final; it is only initialized in the declaration or constructor. | 49 |
| ImmutableField | Private field '_volume' could be made final; it is only initialized in the declaration or constructor. | 50 |
| MissingOverride | The method 'getLocation()' is missing an @Override annotation. | 73–76 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 81–84 |
| MissingOverride | The method 'getPitch()' is missing an @Override annotation. | 89–92 |
| MissingOverride | The method 'getRate()' is missing an @Override annotation. | 97–100 |
| MissingOverride | The method 'getVolume()' is missing an @Override annotation. | 105–108 |
| Rule | Violation | Line |
|---|---|---|
| UseUnderscoresInNumericLiterals | Number 22050 should separate every third digit with an underscore | 62 |
| ImmutableField | Private field '_minAcceptedWordLength' could be made final; it is only initialized in the declaration or constructor. | 63 |
| ImmutableField | Private field '_maxAcceptedWordLength' could be made final; it is only initialized in the declaration or constructor. | 64 |
| ImmutableField | Private field '_minWhiteSoundNumber' could be made final; it is only initialized in the declaration or constructor. | 65 |
| ImmutableField | Private field '_maxWhiteSoundNumber' could be made final; it is only initialized in the declaration or constructor. | 66 |
| ImmutableField | Private field '_backgroundSoundMixerConfigurator' could be made final; it is only initialized in the declaration or constructor. | 67 |
| ImmutableField | Private field '_filters' could be made final; it is only initialized in the declaration or constructor. | 68 |
| ArrayIsStoredDirectly | The user-supplied array 'filters' is stored directly. | 88 |
| MissingOverride | The method 'getSound(String)' is missing an @Override annotation. | 104–107 |
| MissingOverride | The method 'getSound(String, Locale)' is missing an @Override annotation. | 116–170 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 128–135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 164 |
| MissingOverride | The method 'getMaxAcceptedWordLenght()' is missing an @Override annotation. | 175–178 |
| MissingOverride | The method 'getMaxAcceptedWordLength()' is missing an @Override annotation. | 183–186 |
| MissingOverride | The method 'getMinAcceptedWordLenght()' is missing an @Override annotation. | 191–194 |
| MissingOverride | The method 'getMinAcceptedWordLength()' is missing an @Override annotation. | 199–202 |
| MissingOverride | The method 'addEffects(AudioInputStream)' is missing an @Override annotation. | 209–249 |
| ShortVariable | Avoid variables with short names like is | 213 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 230 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 242 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 264 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_audioInputStreamArray' could be made final; it is only initialized in the declaration or constructor. | 53 |
| ImmutableField | Private field '_attenuationPerStream' could be made final; it is only initialized in the declaration or constructor. | 59 |
| ImmutableField | Private field '_attenuationFactor' could be made final; it is only initialized in the declaration or constructor. | 65 |
| ImmutableField | Private field '_mixBuffer' could be made final; it is only initialized in the declaration or constructor. | 67 |
| ImmutableField | Private field '_readBuffer' could be made final; it is only initialized in the declaration or constructor. | 68 |
| MissingOverride | The method 'getFrameLength()' is missing an @Override annotation. | 123–142 |
| MissingOverride | The method 'read()' is missing an @Override annotation. | 149–160 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 154 |
| CyclomaticComplexity | The method 'read(byte, int, int)' has a cyclomatic complexity of 12. | 174–275 |
| ExcessiveMethodLength | Avoid really long methods. | 174–275 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 174–275 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 238 |
| MissingOverride | The method 'skip(long)' is missing an @Override annotation. | 287–295 |
| CheckSkipResult | Check the value returned by the skip() method of an InputStream to see if the requested number of bytes has been skipped. | 291 |
| MissingOverride | The method 'available()' is missing an @Override annotation. | 304–314 |
| MissingOverride | The method 'close()' is missing an @Override annotation. | 320–323 |
| MissingOverride | The method 'mark(int)' is missing an @Override annotation. | 331–337 |
| MissingOverride | The method 'reset()' is missing an @Override annotation. | 345–351 |
| MissingOverride | The method 'markSupported()' is missing an @Override annotation. | 358–369 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_audioInputStreamList' could be made final; it is only initialized in the declaration or constructor. | 54 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 67 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'bAnotherStreamAvailable' | 90 |
| MissingOverride | The method 'getFrameLength()' is missing an @Override annotation. | 96–117 |
| MissingOverride | The method 'read()' is missing an @Override annotation. | 124–158 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 171–205 |
| MissingOverride | The method 'available()' is missing an @Override annotation. | 212–215 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'init(FilterConfig)' is missing an @Override annotation. | 81–83 |
| UncommentedEmptyMethodBody | Document empty method body | 82–83 |
| MissingOverride | The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' is missing an @Override annotation. | 99–149 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 106 |
| AddEmptyString | Do not add empty strings | 140 |
| MissingOverride | The method 'destroy()' is missing an @Override annotation. | 154–157 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_decay' could be made final; it is only initialized in the declaration or constructor. | 45 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 59–64 |
| MissingOverride | The method 'getRemainingSize()' is missing an @Override annotation. | 72–81 |
| MissingOverride | The method 'reset()' is missing an @Override annotation. | 86–94 |
| MissingOverride | The method 'filter(byte, int, int, int)' is missing an @Override annotation. | 109–148 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_soundFilter' could be made final; it is only initialized in the declaration or constructor. | 49 |
| ImmutableField | Private field '_sampleSizeInBits' could be made final; it is only initialized in the declaration or constructor. | 51 |
| ShortVariable | Avoid variables with short names like in | 61 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 82–127 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'PitchFilter' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=4, WMC=7) | 43–136 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 57–61 |
| MissingOverride | The method 'filter(byte, int, int, int)' is missing an @Override annotation. | 75–78 |
| MissingOverride | The method 'getAudioFormat(AudioInputStream)' is missing an @Override annotation. | 87–95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 70 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'generateAndStoreCaptcha(Locale, String)' is missing an @Override annotation. | 3 | 77–83 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'generateAndStoreCaptcha(Locale, String)' is missing an @Override annotation. | 3 | 77–83 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'init(FilterConfig)' is missing an @Override annotation. | 3 | 77–82 |
| MissingOverride | The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' is missing an @Override annotation. | 3 | 98–156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 105 |
| MissingOverride | The method 'destroy()' is missing an @Override annotation. | 3 | 161–164 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_response' could be made final; it is only initialized in the declaration or constructor. | 3 | 60 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the LuteceGimpyImage constructor if you want a default access modifier | 3 | 71–75 |
| MissingOverride | The method 'validateResponse(Object)' is missing an @Override annotation. | 3 | 85–88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_myRandom' could be made final; it is only initialized in the declaration or constructor. | 3 | 55 |
| ImmutableField | Private field '_wordToImage' could be made final; it is only initialized in the declaration or constructor. | 3 | 56 |
| ImmutableField | Private field '_wordGenerator' could be made final; it is only initialized in the declaration or constructor. | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 66–80 |
| MissingOverride | The method 'getImageCaptcha()' is missing an @Override annotation. | 3 | 87–90 |
| MissingOverride | The method 'getImageCaptcha(Locale)' is missing an @Override annotation. | 3 | 118–139 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 3 | 131 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 136 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'captcha' | 3 | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 149 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'AudioConcat' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 53–127 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 72 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 82 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_attenuationValue' could be made final; it is only initialized in the declaration or constructor. | 3 | 53 |
| ImmutableField | Private field '_backgroundSoundFiles' could be made final; it is only initialized in the declaration or constructor. | 3 | 54 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 68 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 71–79 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_wordGenerator' could be made final; it is only initialized in the declaration or constructor. | 3 | 63 |
| ImmutableField | Private field '_word2Sound' could be made final; it is only initialized in the declaration or constructor. | 3 | 64 |
| ImmutableField | Private field '_myRandom' could be made final; it is only initialized in the declaration or constructor. | 3 | 65 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 75–89 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 101–120 |
| MissingOverride | The method 'getSoundCaptcha()' is missing an @Override annotation. | 3 | 143–150 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 145 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'soundCaptcha' | 3 | 149 |
| MissingOverride | The method 'getSoundCaptcha(Locale)' is missing an @Override annotation. | 3 | 157–164 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 159 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'soundCaptcha' | 3 | 163 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 185 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'LuteceSoundConfigurator' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=5, WMC=6) | 3 | 43–109 |
| ImmutableField | Private field '_location' could be made final; it is only initialized in the declaration or constructor. | 3 | 46 |
| ImmutableField | Private field '_name' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| ImmutableField | Private field '_pitch' could be made final; it is only initialized in the declaration or constructor. | 3 | 48 |
| ImmutableField | Private field '_rate' could be made final; it is only initialized in the declaration or constructor. | 3 | 49 |
| ImmutableField | Private field '_volume' could be made final; it is only initialized in the declaration or constructor. | 3 | 50 |
| MissingOverride | The method 'getLocation()' is missing an @Override annotation. | 3 | 73–76 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 81–84 |
| MissingOverride | The method 'getPitch()' is missing an @Override annotation. | 3 | 89–92 |
| MissingOverride | The method 'getRate()' is missing an @Override annotation. | 3 | 97–100 |
| MissingOverride | The method 'getVolume()' is missing an @Override annotation. | 3 | 105–108 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseUnderscoresInNumericLiterals | Number 22050 should separate every third digit with an underscore | 3 | 62 |
| ImmutableField | Private field '_minAcceptedWordLength' could be made final; it is only initialized in the declaration or constructor. | 3 | 63 |
| ImmutableField | Private field '_maxAcceptedWordLength' could be made final; it is only initialized in the declaration or constructor. | 3 | 64 |
| ImmutableField | Private field '_minWhiteSoundNumber' could be made final; it is only initialized in the declaration or constructor. | 3 | 65 |
| ImmutableField | Private field '_maxWhiteSoundNumber' could be made final; it is only initialized in the declaration or constructor. | 3 | 66 |
| ImmutableField | Private field '_backgroundSoundMixerConfigurator' could be made final; it is only initialized in the declaration or constructor. | 3 | 67 |
| ImmutableField | Private field '_filters' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| ArrayIsStoredDirectly | The user-supplied array 'filters' is stored directly. | 3 | 88 |
| MissingOverride | The method 'getSound(String)' is missing an @Override annotation. | 3 | 104–107 |
| MissingOverride | The method 'getSound(String, Locale)' is missing an @Override annotation. | 3 | 116–170 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 128–135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 164 |
| MissingOverride | The method 'getMaxAcceptedWordLenght()' is missing an @Override annotation. | 3 | 175–178 |
| MissingOverride | The method 'getMaxAcceptedWordLength()' is missing an @Override annotation. | 3 | 183–186 |
| MissingOverride | The method 'getMinAcceptedWordLenght()' is missing an @Override annotation. | 3 | 191–194 |
| MissingOverride | The method 'getMinAcceptedWordLength()' is missing an @Override annotation. | 3 | 199–202 |
| MissingOverride | The method 'addEffects(AudioInputStream)' is missing an @Override annotation. | 3 | 209–249 |
| ShortVariable | Avoid variables with short names like is | 3 | 213 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 230 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 242 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 264 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_audioInputStreamArray' could be made final; it is only initialized in the declaration or constructor. | 3 | 53 |
| ImmutableField | Private field '_attenuationPerStream' could be made final; it is only initialized in the declaration or constructor. | 3 | 59 |
| ImmutableField | Private field '_attenuationFactor' could be made final; it is only initialized in the declaration or constructor. | 3 | 65 |
| ImmutableField | Private field '_mixBuffer' could be made final; it is only initialized in the declaration or constructor. | 3 | 67 |
| ImmutableField | Private field '_readBuffer' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| MissingOverride | The method 'getFrameLength()' is missing an @Override annotation. | 3 | 123–142 |
| MissingOverride | The method 'read()' is missing an @Override annotation. | 3 | 149–160 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 154 |
| CyclomaticComplexity | The method 'read(byte, int, int)' has a cyclomatic complexity of 12. | 3 | 174–275 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 174–275 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 3 | 174–275 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 238 |
| MissingOverride | The method 'skip(long)' is missing an @Override annotation. | 3 | 287–295 |
| CheckSkipResult | Check the value returned by the skip() method of an InputStream to see if the requested number of bytes has been skipped. | 3 | 291 |
| MissingOverride | The method 'available()' is missing an @Override annotation. | 3 | 304–314 |
| MissingOverride | The method 'close()' is missing an @Override annotation. | 3 | 320–323 |
| MissingOverride | The method 'mark(int)' is missing an @Override annotation. | 3 | 331–337 |
| MissingOverride | The method 'reset()' is missing an @Override annotation. | 3 | 345–351 |
| MissingOverride | The method 'markSupported()' is missing an @Override annotation. | 3 | 358–369 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_audioInputStreamList' could be made final; it is only initialized in the declaration or constructor. | 3 | 54 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 67 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'bAnotherStreamAvailable' | 3 | 90 |
| MissingOverride | The method 'getFrameLength()' is missing an @Override annotation. | 3 | 96–117 |
| MissingOverride | The method 'read()' is missing an @Override annotation. | 3 | 124–158 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 3 | 171–205 |
| MissingOverride | The method 'available()' is missing an @Override annotation. | 3 | 212–215 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'init(FilterConfig)' is missing an @Override annotation. | 3 | 81–83 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 82–83 |
| MissingOverride | The method 'doFilter(ServletRequest, ServletResponse, FilterChain)' is missing an @Override annotation. | 3 | 99–149 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 106 |
| AddEmptyString | Do not add empty strings | 3 | 140 |
| MissingOverride | The method 'destroy()' is missing an @Override annotation. | 3 | 154–157 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_decay' could be made final; it is only initialized in the declaration or constructor. | 3 | 45 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 59–64 |
| MissingOverride | The method 'getRemainingSize()' is missing an @Override annotation. | 3 | 72–81 |
| MissingOverride | The method 'reset()' is missing an @Override annotation. | 3 | 86–94 |
| MissingOverride | The method 'filter(byte, int, int, int)' is missing an @Override annotation. | 3 | 109–148 |
| AvoidUsingShortType | Do not use the short type | 1 | 116 |
| AvoidUsingShortType | Do not use the short type | 1 | 117 |
| AvoidUsingShortType | Do not use the short type | 1 | 135 |
| AvoidUsingShortType | Do not use the short type | 1 | 136 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_soundFilter' could be made final; it is only initialized in the declaration or constructor. | 3 | 49 |
| ImmutableField | Private field '_sampleSizeInBits' could be made final; it is only initialized in the declaration or constructor. | 3 | 51 |
| ShortVariable | Avoid variables with short names like in | 3 | 61 |
| MissingOverride | The method 'read(byte, int, int)' is missing an @Override annotation. | 3 | 82–127 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'PitchFilter' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=4, WMC=7) | 3 | 43–136 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 57–61 |
| MissingOverride | The method 'filter(byte, int, int, int)' is missing an @Override annotation. | 3 | 75–78 |
| MissingOverride | The method 'getAudioFormat(AudioInputStream)' is missing an @Override annotation. | 3 | 87–95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 54–57 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 65–68 |
| AvoidUsingShortType | Do not use the short type | 1 | 93 |
| AvoidUsingShortType | Do not use the short type | 1 | 109 |
| AvoidUsingShortType | Do not use the short type | 1 | 125 |
| AvoidUsingShortType | Do not use the short type | 1 | 141 |
