The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 127 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 154 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 127 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 136 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 138 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 127 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'steps' doesn't match '_[a-z][a-zA-Z0-9]*' | 19 |
| FieldNamingConventions | The field name 'output' doesn't match '_[a-z][a-zA-Z0-9]*' | 20 |
| FieldNamingConventions | The field name 'session' doesn't match '_[a-z][a-zA-Z0-9]*' | 21 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name 'sessions' doesn't match '_[a-z][a-zA-Z0-9]*' | 64 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 93 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–73 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'action' doesn't match '_[a-z][a-zA-Z0-9]*' | 5 |
| FieldNamingConventions | The field name 'question' doesn't match '_[a-z][a-zA-Z0-9]*' | 6 |
| FieldNamingConventions | The field name 'botId' doesn't match '_[a-z][a-zA-Z0-9]*' | 7 |
| FieldNamingConventions | The field name 'botSessionId' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| FieldNamingConventions | The field name 'audioFile' doesn't match '_[a-z][a-zA-Z0-9]*' | 9 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_proxyAddress' doesn't match '[a-z][a-zA-Z0-9]*' | 74 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constant' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 8–17 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 13–83 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name 'fileStoreService' doesn't match '_[a-z][a-zA-Z0-9]*' | 27 |
| FieldNamingConventions | The constant name 'embeddingStores' doesn't match '[A-Z][A-Z_0-9]*' | 28 |
| FieldNamingConventions | The constant name 'embeddingModel' doesn't match '[A-Z][A-Z_0-9]*' | 29 |
| Rule | Violation | Line |
|---|---|---|
| SystemPrintln | System.out.println is used | 28 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 237–240 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 50–362 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 237–240 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 237–240 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 155–158 |
| CommentRequired | Public method and constructor comments are required | 160–163 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 226–229 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 46–155 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 226–229 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 217–220 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 19 |
| ImmutableField | Private field 'steps' could be made final; it is only initialized in the declaration or constructor. | 19 |
| ImmutableField | Private field 'output' could be made final; it is only initialized in the declaration or constructor. | 20 |
| CommentRequired | Public method and constructor comments are required | 23–27 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 40 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 115 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 125 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 70–98 |
| CommentRequired | Public method and constructor comments are required | 103–110 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like ID | 45 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 3–61 |
| DataClass | The class 'RequestData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=10, WMC=10) | 3–61 |
| MethodReturnsInternalArray | Returning 'audioFile' may expose an internal array. | 43 |
| ArrayIsStoredDirectly | The user-supplied array 'audioFile' is stored directly. | 46 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 141 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 151 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 154 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 25–111 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 26–111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 60–71 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PersistentChatMemoryStore nested class if you want a default access modifier | 89–109 |
| CommentRequired | Header comments are required | 89–109 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–117 |
| CommentRequired | Header comments are required | 37–117 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 38–117 |
| CommentRequired | Public method and constructor comments are required | 40–68 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 57–67 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 72–83 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the StreamingAssistant nested class if you want a default access modifier | 113–116 |
| CommentRequired | Header comments are required | 113–116 |
| CommentRequired | Public method and constructor comments are required | 115 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 13–83 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 45 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 25–148 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 26–148 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 28 |
| UseIndexOfChar | String.indexOf(char) is faster than String.indexOf(String). | 76 |
| Rule | Violation | Line |
|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 13–31 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 18 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 22 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 50–117 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 116 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 116 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 66–340 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 129 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 147–154 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 148–154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 174 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 302 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 303 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 167 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 66–326 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 123 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 171 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 171 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 204 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 204 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 225 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 225 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 280 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 280 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 167 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 68–330 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 132 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 150–157 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 151–157 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 177 |
| CommentRequired | Public method and constructor comments are required | 282–289 |
| CommentRequired | Public method and constructor comments are required | 292–298 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 62–301 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 167 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 167 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Bot | 43–237 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like Bot | 4 | 43–237 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 237–240 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 50–362 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 237–240 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 237–240 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 155–158 |
| CommentRequired | Public method and constructor comments are required | 3 | 160–163 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 226–229 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 127 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 46–155 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 154 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 127 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 136 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 138 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 127 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 226–229 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 217–220 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The field name 'steps' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 19 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 19 |
| ImmutableField | Private field 'steps' could be made final; it is only initialized in the declaration or constructor. | 3 | 19 |
| ImmutableField | Private field 'output' could be made final; it is only initialized in the declaration or constructor. | 3 | 20 |
| FieldNamingConventions | The field name 'output' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 20 |
| FieldNamingConventions | The field name 'session' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 21 |
| CommentRequired | Public method and constructor comments are required | 3 | 23–27 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 40 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 115 |
| AvoidPrintStackTrace | Avoid printStackTrace(); use a logger call instead. | 3 | 125 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The static field name 'sessions' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 64 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–98 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 93 |
| CommentRequired | Public method and constructor comments are required | 3 | 103–110 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–73 |
| ShortVariable | Avoid variables with short names like ID | 3 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 3–61 |
| DataClass | The class 'RequestData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=10, WMC=10) | 3 | 3–61 |
| FieldNamingConventions | The field name 'action' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 5 |
| FieldNamingConventions | The field name 'question' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 6 |
| FieldNamingConventions | The field name 'botId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 7 |
| FieldNamingConventions | The field name 'botSessionId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| FieldNamingConventions | The field name 'audioFile' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 9 |
| MethodReturnsInternalArray | Returning 'audioFile' may expose an internal array. | 3 | 43 |
| ArrayIsStoredDirectly | The user-supplied array 'audioFile' is stored directly. | 3 | 46 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 141 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 151 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 154 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 25–111 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 26–111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 60–71 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PersistentChatMemoryStore nested class if you want a default access modifier | 3 | 89–109 |
| CommentRequired | Header comments are required | 3 | 89–109 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–117 |
| CommentRequired | Header comments are required | 3 | 37–117 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 38–117 |
| CommentRequired | Public method and constructor comments are required | 3 | 40–68 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 57–67 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 72–83 |
| LocalVariableNamingConventions | The local variable name '_proxyAddress' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 74 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the StreamingAssistant nested class if you want a default access modifier | 3 | 113–116 |
| CommentRequired | Header comments are required | 3 | 113–116 |
| CommentRequired | Public method and constructor comments are required | 3 | 115 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constant' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 8–17 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 13–83 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 13–83 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 25–148 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 26–148 |
| FieldNamingConventions | The static field name 'fileStoreService' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 27 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 28 |
| FieldNamingConventions | The constant name 'embeddingStores' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 28 |
| FieldNamingConventions | The constant name 'embeddingModel' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 29 |
| UseIndexOfChar | String.indexOf(char) is faster than String.indexOf(String). | 3 | 76 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 13–31 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 18 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 22 |
| SystemPrintln | System.out.println is used | 2 | 28 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 50–117 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 116 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 116 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 66–340 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 129 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 147–154 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 148–154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 174 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 302 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 303 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 167 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 66–326 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 123 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 171 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 171 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 204 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 204 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 225 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 225 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 280 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 280 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 167 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 68–330 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 132 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 150–157 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 151–157 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 177 |
| CommentRequired | Public method and constructor comments are required | 3 | 282–289 |
| CommentRequired | Public method and constructor comments are required | 3 | 292–298 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 62–301 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 167 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 62–300 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 122 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 140–147 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 141–147 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 167 |
