Merge pull request #24 from DevFactory/release/Utility-class-should-not-have-public-const

Fixing squid:S1118 -  Utility classes should not have public constructors

Author: janithb

src/main/java/org/audit4j/core/Configurations.java

@@ -52,9 +52,9 @@ public class Configurations {
     static final String ENVIRONMENT_CONFIG_VARIABLE_NAME = "AUDIT4J_CONF_FILE_PATH";
 
     /**
-     * Instantiates a new configurations.
+     *  
      */
-    public Configurations() {
+    private Configurations() {
 
     }
 

src/main/java/org/audit4j/core/CoreConstants.java

@@ -147,5 +147,9 @@ public final class CoreConstants {
 
     /** The Constant ENCODE_UTF8. */
     public static final String ENCODE_UTF8 = "UTF-8";
+    
+    private CoreConstants(){
+    	
+    }
 
 }

src/main/java/org/audit4j/core/ErrorGuide.java

@@ -59,7 +59,14 @@ public final class ErrorGuide {
 
     /** The Constant LAYOUT_ERROR. */
     public static final String OPTION_ERROR = ERROR_URL + "optionError";
-    
+   
+    /**
+     * private constructor to avoid instantiation of this class
+     */
+    private ErrorGuide(){
+    	
+    }
+ 
     /**
      * Gets the guide.
      *

src/main/java/org/audit4j/core/TroubleshootManager.java

@@ -38,6 +38,13 @@ public final class TroubleshootManager {
 
     /** The Constant MAX_PORT_NUMBER. */
     public static final int MAX_PORT_NUMBER = 49151;
+    
+    /**
+     * private constructor to avoid instantiation of this class
+     */
+    private TroubleshootManager(){
+    	
+    }
 
     /**
      * Troubleshoot event.

src/main/java/org/audit4j/core/annotation/DeIdentifyUtil.java

@@ -29,6 +29,14 @@
  */
 public class DeIdentifyUtil {
 
+
+    /**
+     * private constructor to avoid instantiation of this class
+     */
+    private DeIdentifyUtil(){
+    	
+    }
+
 	/**
 	 * Deidentify left.
 	 *

src/main/java/org/audit4j/core/dto/AuditLevel.java

@@ -29,5 +29,12 @@ public class AuditLevel {
 
 	/** The Constant DEFAULT_LEVEL. */
 	public static final String DEFAULT_LEVEL = "default";
+	
+	 /**
+     * private constructor to avoid instantiation of this class
+     */
+    private AuditLevel(){
+    	
+    }
 
 }

src/main/java/org/audit4j/core/extra/scannotation/ClasspathUrlFinder.java

@@ -18,6 +18,13 @@
 public class ClasspathUrlFinder
 {
 
+	 /**
+     * private constructor to avoid instantiation of this class
+     */
+    private ClasspathUrlFinder(){
+    	
+    }
+    
    /**
     * Find the classpath URLs for a specific classpath resource.  The classpath URL is extracted
     * from loader.getResources() using the baseResource.

src/main/java/org/audit4j/core/extra/scannotation/IteratorFactory.java

@@ -6,6 +6,14 @@
 
 public class IteratorFactory {
     private static final ConcurrentHashMap<String, DirectoryIteratorFactory> registry = new ConcurrentHashMap<String, DirectoryIteratorFactory>();
+    
+    /**
+     * private constructor to avoid instantiation of this class
+     */
+    private IteratorFactory(){
+    	
+    }
+    
     static {
         registry.put("file", new FileProtocolIteratorFactory());
     }

src/main/java/org/audit4j/core/handler/file/archive/FTPArchiveClient.java

@@ -32,6 +32,13 @@
  * @author <a href="mailto:[email protected]">Janith Bandara</a>
  */
 public final class FTPArchiveClient {
+	
+	/**
+     * private constructor to avoid instantiation of this class
+     */
+    private FTPArchiveClient(){
+    	
+    }
 
     /** The Constant USAGE. */
     public static final String USAGE = "Usage: ftp [options] <hostname> <username> <password> [<remote file> [<local file>]]\n"

src/main/java/org/audit4j/core/jmx/JMXUtils.java

@@ -10,6 +10,13 @@
  * @since 2.4.0
  */
 public class JMXUtils {
+	
+	/**
+     * private constructor to avoid instantiation of this class
+     */
+    private JMXUtils(){
+    	
+    }
 
     /**
      * Gets the object name.