Suggérer un modèle de design?

Question

J'ai réfléchi à la meilleure façon d'implémenter un certain type de fonctionnalité ... étant des objets dérivés référençant d'autres objets dérivés avec la même base (y compris le référencement mutuel). Ci-dessous, le meilleur exemple (simple) que je puisse imaginer pour illustrer le genre de problème: une maison composée de 3 pièces (bien qu'en pratique elle pourrait être des centaines) chacune représentée par sa propre classe dérivée. Bien sûr cela ne fonctionnera pas tel quel, cela provoquerait un débordement de pile (de manière appropriée!)

En supposant que chaque pièce de l'exemple est spécifique à une pièce d'une maison imaginaire, il ne devrait pas y avoir plus d'une instance de toute sous-classe. Construire beaucoup d'instances, même sans la récursivité, serait désordonné. Cela me ferait penser à un singleton si ce n'était pas un tabou ... quelqu'un peut-il suggérer quelque chose de plus approprié?

Merci!

public abstract class Room 
{ 
    abstract public void DoAction(); 
    public Room[] ConnectedRooms; 
} 

public class Kitchen : Room 
{ 
    public Kitchen() { ConnectedRooms = new Room[] { new Hall() }; } 
    public override void DoAction() { MakeTea(); } 
} 

public class Bedroom : Room 
{ 
    public Bedroom() { ConnectedRooms = new Room[] { new Hall() }; } 
    public override void DoAction() { Sleep(); } 
} 

public class Hall : Room 
{ 
    public Hall() { ConnectedRooms = new Room[] { new Hall(), new Bedroom() }; } 
    public override void DoAction() { LookOutWindow(); } 
} 

Answer 1

Vos chambres doivent contenir références à une instance de l'autre pièces - pas de nouvelles instances.

par exemple. créez un nouveau Kitchen, Hall, Bedroom. Puis passez une référence du Hall au Bedroom, au Kitchen etc. De cette façon vous avez vos trois objets, et ils ont des références aux pièces voisines.

Kitchen k = new Kitchen(); 
Bedroom b = new Bedroom(); 
k.addNeighbour(b); 

etc.

(normalement je construis un Kitchen avec des références appropriées, mais vous semblez avoir des références circulaires, de sorte que ce n'est pas si facile)

Si vous avez vraiment besoin seulement une instance de chacun, puis consultez le singleton pattern. Cependant, je ne le ferais pas à moins que cela ne soit strictement nécessaire. Après tout, beaucoup de maisons ont plus d'une chambre à coucher!

Answer 2

Tout ce que vous devez faire est d'éviter d'instancier une pièce dans le constructeur d'une autre. Vous aurez besoin d'instancier toutes vos pièces (Cuisine k = nouvelle cuisine(), Hall h = nouvelle Hall();), puis ajoutez les connexions (k.ConnectedRooms [0] = h; h.ConnectedRooms [0] = k;). Ou, pour être intelligent, créer une méthode qui les relie mutuellement:

public abstract class Room 
{ 
public void Connect(Room r) 
{ 
this.ConnectedRooms.Add(r); 
r.ConnectedRooms.Add(this); 
} 
} 

En supposant que vous modifiez vos tableaux de connexion à des listes, ce qui est probablement une bonne idée.

Answer 3

Vous avez vraiment besoin d'encapsuler (est-ce le bon mot?) Votre modèle plus loin.

public class House 
{ 
    private readonly Hall hall = new Hall(); 
    private readonly Kitchen kitchen = new Kitchen(); 

    // etc 

    public House() 
    { 
     hall.AddAdjacentRoom(kitchen); 
     // etc 
    } 

}