Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/folderlisting/business/FolderListingDatabaseHome.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. 46114

fr/paris/lutece/plugins/folderlisting/business/FolderListingHome.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. 55132
Avoid variables with short names like f 72
When instantiating a SimpleDateFormat object, specify a Locale 85
Do not add empty strings 91
When instantiating a SimpleDateFormat object, specify a Locale 118

fr/paris/lutece/plugins/folderlisting/business/portlet/FolderListingPortlet.java

Violation Ligne
It is a good practice to call super() in a constructor 7376

fr/paris/lutece/plugins/folderlisting/business/portlet/FolderListingPortletDAO.java

Violation Ligne
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.IPortletInterfaceDAO' 36
Avoid unused imports such as 'fr.paris.lutece.portal.business.portlet.IPortletInterfaceDAO' 36
Avoid variables with short names like p 77
Avoid variables with short names like p 118

fr/paris/lutece/plugins/folderlisting/business/portlet/FolderListingPortletHome.java

Violation Ligne
Avoid using redundant field initializer for '_singleton' 52
Possible unsafe assignment to a non-final static field in a constructor. 52
It is a good practice to call super() in a constructor 5763
Singleton is not thread safe 7275

fr/paris/lutece/plugins/folderlisting/service/Folder.java

Violation Ligne
Singleton is not thread safe 5761

fr/paris/lutece/plugins/folderlisting/service/FolderLoaderDatabase.java

Violation Ligne
Use explicit scoping instead of the default package private level 50

fr/paris/lutece/plugins/folderlisting/service/FolderLoaderProperties.java

Violation Ligne
Private field '_strFolderFilesPath' could be made final; it is only initialized in the declaration or constructor. 62
Avoid variables with short names like is 135

fr/paris/lutece/plugins/folderlisting/service/FolderService.java

Violation Ligne
A class which only has private constructors should be final 44139

fr/paris/lutece/plugins/folderlisting/web/FolderListingDatabaseJspBean.java

Violation Ligne
Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. 102
It is a good practice to call super() in a constructor 104107
Do not add empty strings 153
Avoid declaring a variable if it is unreferenced before a possible exit point. 269
Avoid declaring a variable if it is unreferenced before a possible exit point. 271
Avoid reassigning parameters such as 'strFolderPath' 302
Avoid reassigning parameters such as 'strPath' 322
This call to String.startsWith can be rewritten using String.charAt(0) 324

fr/paris/lutece/plugins/folderlisting/web/FolderListingFileJspBean.java

Violation Ligne
Avoid unnecessary constructors - the compiler will generate these for you 7981
Document empty constructor 7981
Return an empty array rather than null. 105153
Avoid variables with short names like in 121

fr/paris/lutece/plugins/folderlisting/web/portlet/FolderListingPortletJspBean.java

Violation Ligne
System.out.println is used 93
System.out.println is used 96
Avoid declaring a variable if it is unreferenced before a possible exit point. 165
Avoid declaring a variable if it is unreferenced before a possible exit point. 166
Use explicit scoping instead of the default package private level 240249