Comment réparer "Constructor Calls Overridable Method"

@Voo dit,

votre question est à propos de l'appel d'une méthode virtuelle sur un déjà
entièrement construit de l'objet. Bien connu des chutes de l'appel
méthodes virtuelles sur le construit de l'objet sont bien connus, mais ne pas
appliquer ici

De Efficace Java 2nd EditionPoint 17: Conception et document pour l'héritage, ou bien de l'interdire:

Il y a un peu plus de restrictions qu'une classe doit respecter pour permettre
de l'héritage. Les constructeurs ne doivent pas invoquer des méthodes substituables,
directement ou indirectement. Si vous ne respectez pas cette règle, le programme de l'échec
résultat. Le constructeur de la superclasse s'exécute avant de la sous-classe
constructeur, de sorte que la méthode de remplacement dans la sous-classe sera invoquée
avant la sous-classe constructeur a été exécuté. Si la méthode de remplacement
dépend de l'initialisation effectuée par le constructeur de sous-classe,
la méthode ne fonctionnera pas comme prévu.

Invocation d'une méthode remplaçable cours de construction de l'objet peut entraîner l'utilisation de données non initialisées, conduisant à des exceptions d'exécution ou à des résultats imprévus.

Constructeurs doivent appeler uniquement les méthodes qui sont définitives ou privé

Vous pouvez utiliser statique usine méthodes pour résoudre le problème, vous devez créer vos objets à partir de la Bar class.

Efficace Java, Article 1: Envisager statique usine méthodes à la place des constructeurs

Le chemin normal pour une classe de permettre à un client d'obtenir une instance de la
lui-même est de fournir un constructeur public. Il existe une autre technique
que devrait être une partie de chaque kit de programmation. Une classe peut
fournir un public statique méthode de fabrique, qui est simplement un statique
méthode qui renvoie une instance de la classe.

Donc, vous allez avoir l'interface :

public interface Foo {
     void doFoo();
}

et la mise en œuvre:

public class FooImpl implements Foo {
   @Override
   public void doFoo() {
   //.. Do important code
   }
}

Pour créer votre classe avec votre méthode de fabrique qui vous permettrait de travailler de cette manière:

• Utiliser l'interface pour définir la variable de votre classe private Foo fi au lieu de private FooImpl fil'utilisation d'interfaces plus de types de béton est la clé pour une bonne encapsulation et pour le couplage de votre code.

• Faire votre constructeur par défaut privé pour empêcher l'instanciation de la classe à l'extérieur.

  privé Bar() {
  //Empêche l'instanciation
  }

• Supprimer tous les appels à remplacer les méthodes qui sont présents dans votre constructeur.

• Créer votre statique méthode de fabrique

Finalement, vous obtenez une classe Bar avec une méthode de fabrique, comme :

public class Bar {
    private Foo fi;

    private Bar() {//Prevents instantiation
        fi = new FooImpl();
    }

    public static Bar createBar() {
        Bar newBar = new Bar();
        newBar.fi.doFoo(); 

        return newBar;
    }
}

Mon Patron dit: “le Sonar mises en garde sont sur symptômespas sur le maladie. C'est mieux quand vous pouvez traiter la maladie.” !

Votre IDE est vous dire que, parce qu'il est potentiellement dangereux. Vous pouvez fournir des implimentation ou doFoo et de faire toutes les Barre d'objets de différentes choses sur le démarrage. Cela ressemble à un mauvais choix de conception dans la plupart des cas.

Il semble que vous soyez à l'aide d'un modèle de stratégie, dans un constructeur. Il n'est pas sage d'utiliser une stratégie ou tout autre overidable comportement dans le constructeur. L'utiliser ailleurs.