PMD Results

The following document contains the results of PMD

Files

org/opencpf/service/dao/hibernate/LinkHibernateDAO.java

Violation Line
Each class should declare at least one constructor 32

org/opencpf/service/dao/hibernate/ComponentHibernateDAO.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 44

org/opencpf/service/dao/hibernate/ReportHibernateDAO.java

Violation Line
Each class should declare at least one constructor 37

org/opencpf/service/dao/hibernate/TeamHibernateDAO.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 50

org/opencpf/service/dao/hibernate/SponsorHibernateDAO.java

Violation Line
Each class should declare at least one constructor 34

org/opencpf/service/dao/hibernate/UserHibernateDAO.java

Violation Line
Each class should declare at least one constructor 34

org/opencpf/service/dao/hibernate/GalleryHibernateDAO.java

Violation Line
Each class should declare at least one constructor 34

org/opencpf/service/dao/hibernate/EventHibernateDAO.java

Violation Line
Each class should declare at least one constructor 36
A method should have only one exit point, and that should be the last statement in the method 54

org/opencpf/service/dao/hibernate/LocationHibernateDAO.java

Violation Line
Each class should declare at least one constructor 25

org/opencpf/service/dao/hibernate/MemberHibernateDAO.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 53

org/opencpf/service/spring/EventServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 36

org/opencpf/service/spring/LocationServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 10

org/opencpf/service/spring/SponsorServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 31
Perhaps 'sponsorList' could be replaced by a local variable. 35
Perhaps 'nextID' could be replaced by a local variable. 40

org/opencpf/service/spring/UserServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 41

org/opencpf/service/spring/ViewerConfig.java

Violation Line
Each class should declare at least one constructor 22

org/opencpf/service/spring/TeamServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 36

org/opencpf/service/spring/ReportServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 35

org/opencpf/service/spring/LinkServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 32

org/opencpf/service/spring/GuestBookServiceSpringImpl.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 46
Avoid empty catch blocks 77

org/opencpf/service/spring/GalleryServiceSpringImpl.java

Violation Line
Each class should declare at least one constructor 56
It is a good practice to call super() in a constructor 64
Perhaps 'imagePath' could be replaced by a local variable. 92
Perhaps 'thumbPath' could be replaced by a local variable. 94
Avoid assigments in operands 218
Avoid assigments in operands 244
Avoid assigments in operands 265

org/opencpf/service/spring/MemberServiceSpringImpl.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 46