Zwei Objekte mit Abhängigkeiten füreinander. Ist das schlecht?

Question

Ich lerne viel über Designmuster, wenn ich ein eigenes System für meine Projekte erstelle. Und ich möchte Sie zu einer Designfrage fragen, auf die ich keine Antwort finden kann.

Derzeit baue ich einen kleinen Chat-Server-Sockets verwenden, mit mehreren Clients. Im Moment habe ich drei Klassen:

1. Person-Klasse, die Informationen wie Nick, Alter und einem Raum-Objekt enthält.
2. Zimmer-Klasse, die Informationen wie Raumname, Thema und eine Liste von Personen zur Zeit in diesem Raum hält.
3. Hotels-Klasse, das eine Liste von Personen und eine Liste der Zimmer auf dem Server.

ich ein Diagramm habe es zu verdeutlichen:

ich eine Liste von Personen, die auf dem Server der Hotel-Klasse, weil es schön wäre, um zu verfolgen, wie viele es gleich online ist jetzt (ohne durch alle Räume iterieren zu müssen). Die Personen leben in der Hotelklasse, weil ich gerne nach einer bestimmten Person suchen würde, ohne die Zimmer zu durchsuchen.

Ist das schlechte Design? Gibt es einen anderen Weg, es zu erreichen?

Danke.

Answer 1

Die gegenseitige Abhängigkeit Problem unter den Klassen, streng genommen, können mit Schnittstellen (abstrakte Klassen, wenn Sie die Sprache beispielsweise C++ oder Python) gelöst werden IRoom und IPerson; in Pseudo-Code

interface IPerson 
    IRoom getRoom() 
    // etc 

interface IRoom 
    iter<IPerson> iterPerson() 
    // etc 

das macht nur das Schnittstellen gegenseitig voneinander abhängig - das tatsächliche Implementierungen der Schnittstellen benötigen hängen nur von den Schnittstellen.

Dies gibt Ihnen auch viel Spielraum in Bezug auf die Umsetzung, wenn Sie kreisförmige Referenzschleifen vermeiden wollen (die ein Ärgernis sein können z.B.in CPython durch Verlangsamen von Garbage Collections) - Sie könnten schwache Referenzen, eine zugrunde liegende relationale Datenbank mit einer typischen "one to many relationship" -Tabelle usw. verwenden. Und für den ersten einfachen Prototyp können Sie verwenden, was in der von Ihnen gewählten Sprache am einfachsten ist (wahrscheinlich einfach, und ach notwendigerweise kreisförmig, Referenzen [[Zeiger in C++]] mit einem Person Bezugnahme auf ein Room und ein Room zu einem list<Person>).

Answer 2

Ich mag es nicht. Ein Hotel enthält Zimmer und Zimmer enthalten Menschen. Menschen enthalten keine Räume, sie gehören ihnen an.

Sie müssen nicht unbedingt wiederholen Sie Ihre Gastzählimpuls zu bekommen. Sie können einfach eine laufende Zählung ($ Hotel-> total_guests) behalten und sie ändern, während sie sich ändert.

Answer 3

In einem größeren System, wäre es schlecht sein, aber da von dem, was ich Ihre Anwendungen verstehen, diese drei Klassen nur gemeinsam verwenden, ist es ein Problem, nicht viel. Stellen Sie sicher, dass die Mitgliedsvariablen der Person so benannt werden, dass sie einen Verweis auf einen Raum und nicht auf eine Instanz enthalten.

Außerdem, es sei denn, es ist aus Performance-Gründen (z. B. Sie haben eine große Anzahl von Räumen), wäre es wahrscheinlich sauberer, eine Eigenschaft oder Getter, die über die Räume iterieren und sammeln die Personen, anstatt sie zu cachen das Hotel.

Answer 4

Gegenseitige Abhängigkeit ist in seinem eigenen Recht nicht schlecht. Manchmal erfordert die Datennutzung es.

ich darüber nachdenke, in einer anderen Art und Weise. Es wird einfacher sein, Code zu pflegen, der im Allgemeinen weniger Beziehungen hat - gegenseitige Abhängigkeit oder nicht. Halte es einfach so einfach wie möglich. Die einzige zusätzliche Schwierigkeit bei Ihrer Situation besteht darin, dass es manchmal ein Problem mit dem Check und dem Ei gibt, während Sie Sequenzen erstellen und löschen. Du hast mehr Links zum Buchladen.

Wenn Sie gefragt sind, ob Sie in diesem Fall eine Liste von Personen, die auf ein Hotel brauchen, ich glaube, es gibt zwei Antworten. Ich würde damit beginnen, dass Ihre Objekte (im Speicher) diese Beziehungen liefern, aber Sie brauchen KEINE zusätzliche Join-Tabelle zwischen Personen und Hotels in der Datenbank. Wenn Sie Hibernate verwenden, wird automatisch ein effizienter Join für Sie generiert, wenn Sie nach den Personen in einem Hotel fragen (es wird Hotels auf rooms.hotel_id für Sie beitreten).