Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/calculetteqf/business/ITrancheDAO.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 127
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 129

fr/paris/lutece/plugins/calculetteqf/business/IUserDataDAO.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 90

fr/paris/lutece/plugins/calculetteqf/business/Tranche.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_strLibelle' doesn't match '[a-z][a-zA-Z0-9]*' 122

fr/paris/lutece/plugins/calculetteqf/business/UserData.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_computeDate' doesn't match '[a-z][a-zA-Z0-9]*' 97
FormalParameterNamingConventions The method parameter name '_validUntil' doesn't match '[a-z][a-zA-Z0-9]*' 107
FormalParameterNamingConventions The method parameter name '_file' doesn't match '[a-z][a-zA-Z0-9]*' 117

fr/paris/lutece/plugins/calculetteqf/business/UserDataDAO.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 129

fr/paris/lutece/plugins/calculetteqf/daemon/PurgeDaemon.java

Rule Violation Line
FieldNamingConventions The field name 'formatter' doesn't match '_[a-z][a-zA-Z0-9]*' 52

fr/paris/lutece/plugins/calculetteqf/pdf/PDFImageElement.java

Rule Violation Line
FieldNamingConventions The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' 56
FieldNamingConventions The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' 57
FieldNamingConventions The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' 58

fr/paris/lutece/plugins/calculetteqf/pdf/PDFQRCodeElement.java

Rule Violation Line
FieldNamingConventions The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' 65
FieldNamingConventions The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' 66
FieldNamingConventions The final field name '_foregroundColor' doesn't match '[a-z][a-zA-Z0-9]*' 67
FieldNamingConventions The final field name '_matrix' doesn't match '[a-z][a-zA-Z0-9]*' 68
FieldNamingConventions The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' 69

fr/paris/lutece/plugins/calculetteqf/pdf/PDFService.java

Rule Violation Line
FieldNamingConventions The public constant name 'formatter' doesn't match '[A-Z][A-Z_0-9]*' 117
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 320
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 402

fr/paris/lutece/plugins/calculetteqf/utils/CryptoUtils.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 62146

fr/paris/lutece/plugins/calculetteqf/utils/StringUtils.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 4390

fr/paris/lutece/plugins/calculetteqf/web/AbstractManageUserDataJspBean.java

Rule Violation Line
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 4247

fr/paris/lutece/plugins/calculetteqf/web/FormBean.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_allocataireCaf' doesn't match '[a-z][a-zA-Z0-9]*' 51
FormalParameterNamingConventions The method parameter name '_numAllocataire' doesn't match '[a-z][a-zA-Z0-9]*' 61
FormalParameterNamingConventions The method parameter name '_codePostal' doesn't match '[a-z][a-zA-Z0-9]*' 71
FormalParameterNamingConventions The method parameter name '_attestationNonCaf' doesn't match '[a-z][a-zA-Z0-9]*' 81
FormalParameterNamingConventions The method parameter name '_numFiscal' doesn't match '[a-z][a-zA-Z0-9]*' 91
FormalParameterNamingConventions The method parameter name '_referenceAvisImposition' doesn't match '[a-z][a-zA-Z0-9]*' 101
FormalParameterNamingConventions The method parameter name '_fromJS' doesn't match '[a-z][a-zA-Z0-9]*' 111

fr/paris/lutece/plugins/calculetteqf/web/UserDataJspBean.java

Rule Violation Line
FieldNamingConventions The final field name '_userDataHome' doesn't match '[a-z][a-zA-Z0-9]*' 82

Priority 2

fr/paris/lutece/plugins/calculetteqf/business/TrancheDAO.java

Rule Violation Line
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 320

fr/paris/lutece/plugins/calculetteqf/pdf/PDFService.java

Rule Violation Line
GuardLogStatement Logger calls should be surrounded by log level guards. 290

fr/paris/lutece/plugins/calculetteqf/utils/StringUtils.java

Rule Violation Line
AvoidReassigningParameters Avoid reassigning parameters such as 'bytes' 60

Priority 3

fr/paris/lutece/plugins/calculetteqf/CalculetteQFPlugin.java

Rule Violation Line
CommentRequired Header comments are required 4380
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 65

fr/paris/lutece/plugins/calculetteqf/business/ITrancheDAO.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 129

fr/paris/lutece/plugins/calculetteqf/business/Tranche.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 117120
CommentRequired Public method and constructor comments are required 122125

fr/paris/lutece/plugins/calculetteqf/business/TrancheDAO.java

Rule Violation Line
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 249252

fr/paris/lutece/plugins/calculetteqf/business/UserData.java

Rule Violation Line
DataClass The class 'UserData' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=12, WMC=16) 44140
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static 4852
CommentRequired Enum comments are required 4852
CommentRequired Public method and constructor comments are required 8285
CommentRequired Public method and constructor comments are required 8790
MethodReturnsInternalArray Returning '_file' may expose an internal array. 114
ArrayIsStoredDirectly The user-supplied array '_file' is stored directly. 117
CommentRequired Public method and constructor comments are required 132139

fr/paris/lutece/plugins/calculetteqf/business/UserDataDAO.java

Rule Violation Line
DefaultPackage Use explicit scoping instead of the default package private level 6061
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _encryptionService field if you want a default access modifier 61

fr/paris/lutece/plugins/calculetteqf/daemon/PurgeDaemon.java

Rule Violation Line
CommentRequired Header comments are required 4674
ImmutableField Private field 'formatter' could be made final; it is only initialized in the declaration or constructor. 52
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _userDataHome field if you want a default access modifier 54
DefaultPackage Use explicit scoping instead of the default package private level 54
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 70

fr/paris/lutece/plugins/calculetteqf/pdf/IPDFService.java

Rule Violation Line
CommentRequired Header comments are required 4249
CommentRequired Public method and constructor comments are required 45
CommentRequired Public method and constructor comments are required 47

fr/paris/lutece/plugins/calculetteqf/pdf/PDFImageElement.java

Rule Violation Line
CommentRequired Header comments are required 52118
CommentRequired Public method and constructor comments are required 6065

fr/paris/lutece/plugins/calculetteqf/pdf/PDFQRCodeElement.java

Rule Violation Line
CommentRequired Header comments are required 59159
UseUnderscoresInNumericLiterals Number 0.029375f should separate every third digit with an underscore 63
CommentRequired Public method and constructor comments are required 7190
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 77
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 87

fr/paris/lutece/plugins/calculetteqf/pdf/PDFService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34550
CommentRequired Header comments are required 83550
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Fonts': nested enums are implicitly static 130137
CommentRequired Enum comments are required 130137
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 224
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 288
ShortVariable Avoid variables with short names like tt 332
ConfusingTernary Avoid if (x != y) ..; else ..; 337346
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 415
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 465
ShortVariable Avoid variables with short names like qf 506
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 509
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 510
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 511
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 512
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 513
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 514
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 515
TooFewBranchesForASwitchStatement A switch with less than three branches is inefficient, use a if statement instead. 527533

fr/paris/lutece/plugins/calculetteqf/service/EncryptionService.java

Rule Violation Line
CommentRequired Header comments are required 4692
CommentRequired Public method and constructor comments are required 6366

fr/paris/lutece/plugins/calculetteqf/service/IEncryptionService.java

Rule Violation Line
CommentRequired Header comments are required 3647
CommentRequired Public method and constructor comments are required 39
CommentRequired Public method and constructor comments are required 41
CommentRequired Public method and constructor comments are required 43
CommentRequired Public method and constructor comments are required 45

fr/paris/lutece/plugins/calculetteqf/utils/CryptoUtils.java

Rule Violation Line
CommentRequired Header comments are required 62146
ShortVariable Avoid variables with short names like qf 74
CommentRequired Public method and constructor comments are required 7480
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 76
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 76
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 77
ShortVariable Avoid variables with short names like iv 84
CommentRequired Public method and constructor comments are required 8996
UseUnderscoresInNumericLiterals Number 65536 should separate every third digit with an underscore 93
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'secret' 95
CommentRequired Public method and constructor comments are required 98113
ShortVariable Avoid variables with short names like iv 103
CommentRequired Public method and constructor comments are required 115129
ShortVariable Avoid variables with short names like iv 120
CommentRequired Public method and constructor comments are required 131144
ShortVariable Avoid variables with short names like md 135

fr/paris/lutece/plugins/calculetteqf/utils/StringUtils.java

Rule Violation Line
CommentRequired Header comments are required 4390
ShortVariable Avoid variables with short names like ci 66

fr/paris/lutece/plugins/calculetteqf/web/AbstractManageTranchesJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 50117
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

fr/paris/lutece/plugins/calculetteqf/web/AbstractManageUserDataJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 4247

fr/paris/lutece/plugins/calculetteqf/web/CalculetteQFApp.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34341
CommentRequired Header comments are required 79341
CommentRequired Public method and constructor comments are required 117131
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 173
CommentRequired Public method and constructor comments are required 174187
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 189
NPathComplexity The method 'getInfos(HttpServletRequest)' has an NPath complexity of 240 190233
CyclomaticComplexity The method 'getInfos(HttpServletRequest)' has a cyclomatic complexity of 12. 190233
CommentRequired Public method and constructor comments are required 190233
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 192
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 317
CommentRequired Public method and constructor comments are required 318340

fr/paris/lutece/plugins/calculetteqf/web/FormBean.java

Rule Violation Line
DataClass The class 'FormBean' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) 36115
CommentRequired Header comments are required 36115

fr/paris/lutece/plugins/calculetteqf/web/TrancheJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 66330
ImmutableField Private field '_trancheHome' could be made final; it is only initialized in the declaration or constructor. 114
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 130
DefaultPackage Use explicit scoping instead of the default package private level 148155
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 149155
ConfusingTernary Avoid if (x != y) ..; else ..; 175

fr/paris/lutece/plugins/calculetteqf/web/UserDataJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 56137

Files

fr/paris/lutece/plugins/calculetteqf/CalculetteQFPlugin.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4380
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 65

fr/paris/lutece/plugins/calculetteqf/business/ITrancheDAO.java

Rule Violation Priority Line
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 127
CommentRequired Public method and constructor comments are required 3 129
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 129

fr/paris/lutece/plugins/calculetteqf/business/IUserDataDAO.java

Rule Violation Priority Line
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 90

fr/paris/lutece/plugins/calculetteqf/business/Tranche.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 117120
CommentRequired Public method and constructor comments are required 3 122125
FormalParameterNamingConventions The method parameter name '_strLibelle' doesn't match '[a-z][a-zA-Z0-9]*' 1 122

fr/paris/lutece/plugins/calculetteqf/business/TrancheDAO.java

Rule Violation Priority Line
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 249252
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 320

fr/paris/lutece/plugins/calculetteqf/business/UserData.java

Rule Violation Priority Line
DataClass The class 'UserData' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=12, WMC=16) 3 44140
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static 3 4852
CommentRequired Enum comments are required 3 4852
CommentRequired Public method and constructor comments are required 3 8285
CommentRequired Public method and constructor comments are required 3 8790
FormalParameterNamingConventions The method parameter name '_computeDate' doesn't match '[a-z][a-zA-Z0-9]*' 1 97
FormalParameterNamingConventions The method parameter name '_validUntil' doesn't match '[a-z][a-zA-Z0-9]*' 1 107
MethodReturnsInternalArray Returning '_file' may expose an internal array. 3 114
ArrayIsStoredDirectly The user-supplied array '_file' is stored directly. 3 117
FormalParameterNamingConventions The method parameter name '_file' doesn't match '[a-z][a-zA-Z0-9]*' 1 117
CommentRequired Public method and constructor comments are required 3 132139

fr/paris/lutece/plugins/calculetteqf/business/UserDataDAO.java

Rule Violation Priority Line
DefaultPackage Use explicit scoping instead of the default package private level 3 6061
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _encryptionService field if you want a default access modifier 3 61
FormalParameterNamingConventions The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' 1 129

fr/paris/lutece/plugins/calculetteqf/daemon/PurgeDaemon.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4674
FieldNamingConventions The field name 'formatter' doesn't match '_[a-z][a-zA-Z0-9]*' 1 52
ImmutableField Private field 'formatter' could be made final; it is only initialized in the declaration or constructor. 3 52
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _userDataHome field if you want a default access modifier 3 54
DefaultPackage Use explicit scoping instead of the default package private level 3 54
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 70

fr/paris/lutece/plugins/calculetteqf/pdf/IPDFService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4249
CommentRequired Public method and constructor comments are required 3 45
CommentRequired Public method and constructor comments are required 3 47

fr/paris/lutece/plugins/calculetteqf/pdf/PDFImageElement.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 52118
FieldNamingConventions The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' 1 56
FieldNamingConventions The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' 1 57
FieldNamingConventions The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' 1 58
CommentRequired Public method and constructor comments are required 3 6065

fr/paris/lutece/plugins/calculetteqf/pdf/PDFQRCodeElement.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 59159
UseUnderscoresInNumericLiterals Number 0.029375f should separate every third digit with an underscore 3 63
FieldNamingConventions The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' 1 65
FieldNamingConventions The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' 1 66
FieldNamingConventions The final field name '_foregroundColor' doesn't match '[a-z][a-zA-Z0-9]*' 1 67
FieldNamingConventions The final field name '_matrix' doesn't match '[a-z][a-zA-Z0-9]*' 1 68
FieldNamingConventions The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' 1 69
CommentRequired Public method and constructor comments are required 3 7190
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 77
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 77
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 87

fr/paris/lutece/plugins/calculetteqf/pdf/PDFService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34550
CommentRequired Header comments are required 3 83550
FieldNamingConventions The public constant name 'formatter' doesn't match '[A-Z][A-Z_0-9]*' 1 117
UnnecessaryModifier Unnecessary modifier 'static' on enum 'Fonts': nested enums are implicitly static 3 130137
CommentRequired Enum comments are required 3 130137
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 224
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 288
GuardLogStatement Logger calls should be surrounded by log level guards. 2 290
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 1 320
ShortVariable Avoid variables with short names like tt 3 332
ConfusingTernary Avoid if (x != y) ..; else ..; 3 337346
AvoidThrowingRawExceptionTypes Avoid throwing raw exception types. 1 402
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 415
LooseCoupling Avoid using implementation types like 'HashMap'; use the interface instead 3 465
ShortVariable Avoid variables with short names like qf 3 506
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 509
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 510
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 511
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 512
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 513
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 514
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 515
TooFewBranchesForASwitchStatement A switch with less than three branches is inefficient, use a if statement instead. 3 527533

fr/paris/lutece/plugins/calculetteqf/service/EncryptionService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4692
CommentRequired Public method and constructor comments are required 3 6366

fr/paris/lutece/plugins/calculetteqf/service/IEncryptionService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 3647
CommentRequired Public method and constructor comments are required 3 39
CommentRequired Public method and constructor comments are required 3 41
CommentRequired Public method and constructor comments are required 3 43
CommentRequired Public method and constructor comments are required 3 45

fr/paris/lutece/plugins/calculetteqf/utils/CryptoUtils.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 62146
CommentRequired Header comments are required 3 62146
ShortVariable Avoid variables with short names like qf 3 74
CommentRequired Public method and constructor comments are required 3 7480
ConsecutiveLiteralAppends StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. 3 76
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 76
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 77
ShortVariable Avoid variables with short names like iv 3 84
CommentRequired Public method and constructor comments are required 3 8996
UseUnderscoresInNumericLiterals Number 65536 should separate every third digit with an underscore 3 93
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'secret' 3 95
CommentRequired Public method and constructor comments are required 3 98113
ShortVariable Avoid variables with short names like iv 3 103
CommentRequired Public method and constructor comments are required 3 115129
ShortVariable Avoid variables with short names like iv 3 120
CommentRequired Public method and constructor comments are required 3 131144
ShortVariable Avoid variables with short names like md 3 135

fr/paris/lutece/plugins/calculetteqf/utils/StringUtils.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 4390
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 4390
AvoidReassigningParameters Avoid reassigning parameters such as 'bytes' 2 60
ShortVariable Avoid variables with short names like ci 3 66

fr/paris/lutece/plugins/calculetteqf/web/AbstractManageTranchesJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 50117
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

fr/paris/lutece/plugins/calculetteqf/web/AbstractManageUserDataJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 4247
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 1 4247

fr/paris/lutece/plugins/calculetteqf/web/CalculetteQFApp.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34341
CommentRequired Header comments are required 3 79341
CommentRequired Public method and constructor comments are required 3 117131
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 173
CommentRequired Public method and constructor comments are required 3 174187
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 189
NPathComplexity The method 'getInfos(HttpServletRequest)' has an NPath complexity of 240 3 190233
CyclomaticComplexity The method 'getInfos(HttpServletRequest)' has a cyclomatic complexity of 12. 3 190233
CommentRequired Public method and constructor comments are required 3 190233
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 192
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 317
CommentRequired Public method and constructor comments are required 3 318340

fr/paris/lutece/plugins/calculetteqf/web/FormBean.java

Rule Violation Priority Line
DataClass The class 'FormBean' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) 3 36115
CommentRequired Header comments are required 3 36115
FormalParameterNamingConventions The method parameter name '_allocataireCaf' doesn't match '[a-z][a-zA-Z0-9]*' 1 51
FormalParameterNamingConventions The method parameter name '_numAllocataire' doesn't match '[a-z][a-zA-Z0-9]*' 1 61
FormalParameterNamingConventions The method parameter name '_codePostal' doesn't match '[a-z][a-zA-Z0-9]*' 1 71
FormalParameterNamingConventions The method parameter name '_attestationNonCaf' doesn't match '[a-z][a-zA-Z0-9]*' 1 81
FormalParameterNamingConventions The method parameter name '_numFiscal' doesn't match '[a-z][a-zA-Z0-9]*' 1 91
FormalParameterNamingConventions The method parameter name '_referenceAvisImposition' doesn't match '[a-z][a-zA-Z0-9]*' 1 101
FormalParameterNamingConventions The method parameter name '_fromJS' doesn't match '[a-z][a-zA-Z0-9]*' 1 111

fr/paris/lutece/plugins/calculetteqf/web/TrancheJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 66330
ImmutableField Private field '_trancheHome' could be made final; it is only initialized in the declaration or constructor. 3 114
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 130
DefaultPackage Use explicit scoping instead of the default package private level 3 148155
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier 3 149155
ConfusingTernary Avoid if (x != y) ..; else ..; 3 175

fr/paris/lutece/plugins/calculetteqf/web/UserDataJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 56137
FieldNamingConventions The final field name '_userDataHome' doesn't match '[a-z][a-zA-Z0-9]*' 1 82