PMD Results

The following document contains the results of PMD

Files

org/tmforum/mtosi/message/body/impl/xmlbeans/InventoryRetrievalRequestBodyImpl.java

ViolationLine
It is a good practice to call super() in a constructor49

org/tmforum/mtosi/message/body/InventoryRetrievalRequestBody.java

ViolationLine
Each class should declare at least one constructor40

org/tmforum/mtosi/message/body/MessageBody.java

ViolationLine
Each class should declare at least one constructor41
Use explicit scoping instead of the default package private level43

org/tmforum/mtosi/message/header/ActivityStatus.java

ViolationLine
It is a good practice to call super() in a constructor38
Use explicit scoping instead of the default package private level42
Use explicit scoping instead of the default package private level43
Use explicit scoping instead of the default package private level44
Ensure you override both equals() and hashCode()50

org/tmforum/mtosi/message/header/CommunicationPattern.java

ViolationLine
It is a good practice to call super() in a constructor38
Use explicit scoping instead of the default package private level44
Use explicit scoping instead of the default package private level45
Use explicit scoping instead of the default package private level46
Use explicit scoping instead of the default package private level47
Ensure you override both equals() and hashCode()53

org/tmforum/mtosi/message/header/CommunicationStyle.java

ViolationLine
It is a good practice to call super() in a constructor38
Use explicit scoping instead of the default package private level44
Use explicit scoping instead of the default package private level45
Ensure you override both equals() and hashCode()51

org/tmforum/mtosi/message/header/MessageHeader.java

ViolationLine
Avoid unnecessary constructors - the compiler will generate these for you74
It is a good practice to call super() in a constructor74

org/tmforum/mtosi/message/header/MessageType.java

ViolationLine
It is a good practice to call super() in a constructor39
Ensure you override both equals() and hashCode()52

org/tmforum/mtosi/message/header/RequestResponseHeader.java

ViolationLine
These nested if statements could be combined73

org/tmforum/mtosi/message/InventoryRetrievalRequest.java

ViolationLine
It is a good practice to call super() in a constructor49

org/tmforum/mtosi/message/InventoryRetrievalResponse.java

ViolationLine
It is a good practice to call super() in a constructor44

org/tmforum/mtosi/message/MessageResources.java

ViolationLine
It is a good practice to call super() in a constructor44
A method should have only one exit point, and that should be the last statement in the method49

org/tmforum/mtosi/message/MTOSIMessage.java

ViolationLine
Each class should declare at least one constructor47
Use explicit scoping instead of the default package private level50
Use explicit scoping instead of the default package private level56
Use explicit scoping instead of the default package private level58