Comment rendre un constructeur disponible uniquement pour la classe d'usine?

Question

Ok, la question n'est peut-être pas très claire. Permettez-moi de donner quelques détails:

Disons que j'ai une classe de fabrique d'objets Shoe (CShoe) appelée CFactory. CFactory est une classe singleton qui crée et stocke toutes les chaussures instanciées à l'aide d'une hashmap simple. Il est ensuite accédé par des méthodes statiques pour utiliser les objets créés.

Existe-t-il un moyen de forcer le constructeur CShoe de sorte qu'il peut seulement appelé par l'usine? (en d'autres termes, assurez-vous que la création de chaussures ne peut être faite que par la classe singleton factory et non par d'autres classes)

Answer 1

Vous pourriez faire un Shoeinner class de ShoeFactory:

public class ShoeFactory { 

    public static class Shoe { 
     private String name; 

     private Shoe() { 
     } 

     private Shoe(String name) { 
      this.name = name; 
     } 
    } 

    public static Shoe createShoe(String shoeName) { 
     return new Shoe(shoeName); 
    } 
} 

Je pense que cela couvre à peu près tous les cas, sauf .... réflexion:

public class SmellyShoe { 

    public static void main(String[] args) { 
     try { 
      java.lang.reflect.Constructor c = Shoe.class.getDeclaredConstructors()[0]; 
      c.setAccessible(true); 
      Shoe smelly = (Shoe)c.newInstance(null); 
      // grr 
     } catch (InstantiationException e) { 
      e.printStackTrace(); 
     } catch (IllegalAccessException e) { 
      e.printStackTrace(); 
     } catch (IllegalArgumentException e) { 
      e.printStackTrace(); 
     } catch (InvocationTargetException e) { 
      e.printStackTrace(); 
     } 
    } 
} 

Answer 2

Vous pouvez donner l'accès au package constructeur CShoe et mettre CShoe et CFactory dans le même package .

Answer 3

Puisque vous voulez ajouter chaque objet à la carte, vous pouvez aussi déplacer cette logique vers le constructeur CShoe - l'objet va s'ajouter.

Answer 4

Vous pourriez peut-être passer l'objet usine dans le constructeur?

public CShoe(CFactory factory) 
{ 
    if (factory == null || 
      !factory.isValid()) // or whatever 
    { 
     throw new IllegalArgumentException(); 
    } 
} 

Answer 5

Couldn » t vous venez de passer l'instance de l'appelant comme argument d'un membre statique de la chaussure et de faire une vérification comme "isInst anceOf "qui appelle le constructeur si vrai?

Answer 6

Tout d'abord, si vous conservez toutes les instances créées, cela s'appelle une fuite. Je vais continuer en supposant que vous voulez dire une référence non-forte, limitée ou un tel cache, et aussi que Shoe est immuable. Utilisez simplement une méthode static pour renvoyer l'usine.

public final class Shoe implements Footwear { 
    private static final FootwearFactory<Shoe,Something> FACTORY = 
     new FootwearFactory<Shoe,Something>() { 
      ... 
      public Shoe get(Something value) { 
       value = new Something(value); 
       ... 
       return new Show(value); 
      } 
     }; 
    private static FootwearFactory<Shoe,Something> getFactory() { 
     return FACTORY; 
    } 

    private final Something value; 
    private Shoe(Something value) { 
     this.value = value; 
    } 
    ... 
}