Fork me on GitHub

Résultats PMD

Le document suivant contient les résultats de PMD 5.3.5.

Fichiers

fr/paris/lutece/plugins/crmclient/business/CRMItemQueueDAO.java

Violation Ligne
Use block level rather than method level synchronization 124160
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 151
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 224

fr/paris/lutece/plugins/crmclient/business/demand/DemandItem.java

Violation Ligne
Abstract classes should be named AbstractXXX 4350
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 4350

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemCreateByIdCRMUser.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemCreateByUserGuid.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemCreateByUserGuidV2.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemDelete.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemDeleteV2.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemUpdate.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemUpdateV2.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemUserGuid.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 58
Avoid appending characters as strings in StringBuffer.append. 59
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 59

fr/paris/lutece/plugins/crmclient/business/demand/DemandItemV2.java

Violation Ligne
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 4350
Abstract classes should be named AbstractXXX 4350

fr/paris/lutece/plugins/crmclient/business/user/UserItem.java

Violation Ligne
No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 4349
Abstract classes should be named AbstractXXX 4349

fr/paris/lutece/plugins/crmclient/business/user/UserItemAttribute.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 58
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 59
Avoid appending characters as strings in StringBuffer.append. 59

fr/paris/lutece/plugins/crmclient/business/user/UserItemAttributesJson.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/user/UserItemAttributesXml.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 55
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56

fr/paris/lutece/plugins/crmclient/business/user/UserItemGuidByIdCRMUser.java

Violation Ligne
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 56
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 57
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 58
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 59
Avoid appending characters as strings in StringBuffer.append. 59

fr/paris/lutece/plugins/crmclient/service/CRMClientPlugin.java

Violation Ligne
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. 4658

fr/paris/lutece/plugins/crmclient/service/CRMClientService.java

Violation Ligne
Possible God class (WMC=101, ATFD=209, TCC=0.05263157894736842) 34545
Use explicit scoping instead of the default package private level 6162
Rather than using a lot of String arguments, consider using a container object for those values. 77
The method notify() has an NPath complexity of 1250 86101
Rather than using a lot of String arguments, consider using a container object for those values. 86
Rather than using a lot of String arguments, consider using a container object for those values. 125
The method sendUpdateDemand() has an NPath complexity of 200 134158
Rather than using a lot of String arguments, consider using a container object for those values. 134
Rather than using a lot of String arguments, consider using a container object for those values. 161
The method sendCreateDemandByUserGuid() has an NPath complexity of 6250 168190
Rather than using a lot of String arguments, consider using a container object for those values. 168169
Rather than using a lot of String arguments, consider using a container object for those values. 194
Rather than using a lot of String arguments, consider using a container object for those values. 202203
The method sendCreateDemandByIdCRMUser() has an NPath complexity of 6250 202223
Rather than using a lot of String arguments, consider using a container object for those values. 441
The method notifyV2() has an NPath complexity of 6250 441460
The method sendCreateDemandByUserGuidV2() has an NPath complexity of 31250 463486
Rather than using a lot of String arguments, consider using a container object for those values. 463464
The method sendUpdateDemandV2() has an NPath complexity of 1000 506533
Rather than using a lot of String arguments, consider using a container object for those values. 506507

fr/paris/lutece/plugins/crmclient/service/daemon/CRMClientSenderDaemon.java

Violation Ligne
Use block level rather than method level synchronization 56107
StringBuffer constructor is initialized with size 16, but has at least 20 characters appended. 58
StringBuffer (or StringBuilder).append is called 3 consecutive times with literal Strings. Use a single append with a single combined String. 72
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 87
Deeply nested if..then statements are hard to read 9295

fr/paris/lutece/plugins/crmclient/service/processor/CRMClientWSProcessor.java

Violation Ligne
A method/constructor shouldnt explicitly throw java.lang.Exception 105
A method/constructor shouldnt explicitly throw java.lang.Exception 105

fr/paris/lutece/plugins/crmclient/util/http/WebServiceCaller.java

Violation Ligne
StringBuffer constructor is initialized with size 16, but has at least 275 characters appended. 111
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 112
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 112
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 113
Avoid appending characters as strings in StringBuffer.append. 118
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 121
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 121
Do not add empty strings 132
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 134
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 144