Masquer le constructeur des classes utilitaires : Les classes utilitaires ne doivent pas avoir de constructeur public ou par défaut.

Question

Je reçois cet avertissement sur Sonar. Je veux une solution pour supprimer cet avertissement sur Sonar Qube.

Ma classe est comme ça :

public class FilePathHelper {
    private static String resourcesPath;
    public static String getFilePath(HttpServletRequest request) {
        if(resourcesPath == null) {
            String serverpath = request.getSession().getServletContext().getRealPath("");               
            resourcesPath = serverpath + "/WEB-INF/classes/";   
        }
        return resourcesPath;       
    }
}

Je veux une solution appropriée pour supprimer cet avertissement sur le sonar.

Answer 1

Je recommande de désactiver cette règle dans Sonar, il n'y a aucun avantage réel à introduire un constructeur privé, juste des caractères redondants dans votre base de code que d'autres personnes doivent lire et que l'ordinateur doit stocker et traiter.

Answer 2

Ajouter un constructeur privé :

private FilePathHelper(){
    super();
}

Answer 3

public class LmsEmpWfhUtils {    
    private LmsEmpWfhUtils() 
    { 
    // prevents access default paramater-less constructor
    }
}

Cela empêche le constructeur sans paramètre par défaut d'être utilisé ailleurs dans votre code.

Answer 4

Documentation SonarQube recommande d'ajouter static à la déclaration de la classe.

C'est-à-dire que le changement public class FilePathHelper a public static class FilePathHelper .

Vous pouvez également ajouter un constructeur privé ou protégé.

public class FilePathHelper
{
    // private or protected constructor
    // because all public fields and methods are static
    private FilePathHelper() {
    }
}

Answer 5

Rendre la classe utilitaire finale et ajouter un constructeur privé