Motif de conception approprié pour une application de discussion simple

Je concevons une simple application de discussion (juste pour le coup de pied). Je me suis demandé d'un design simple pour cette application de discussion. Pour vous donner une vue d'ensemble .. Voici les règles:

1. L'utilisateur anonyme Entrez chat avec un surnom. (ID utilisateur) est vraisemblablement attribué par système en arrière-plan. Li>
2. Ils peuvent rejoindre (S'abonner à) une conversation de discussion. Et il verra le texte de chat d'autres utilisateurs figurant sur la zone désignée. li>
3. Ils peuvent répondre à une conversation particulière et que tous les autres devraient voir cela. Li> ol>

   c'est ça! (Voir je vous ai dit que c'était une simple application de discussion). Donc, mon intention n'est pas vraiment la demande; Mais le modèle de conception et les objets utilisés. P>

   Voici comment je l'ai conçu. (Je suis codé dans Java .. au cas où cela compte vraiment) p>

   1. Objet utilisateur - Deux attributs ID et pseudo Li>
   2. Objet de message - une interface de message simple et une implémentation (pour l'instant) en tant que SimpleMessage, avec une chaîne en tant qu'attribut pour contenir le message. LI>
   3. Objet de la fenêtre de discussion - essentiellement composition de l'utilisateur et des messages. Comme il a un objet utilisateur et une liste de messages. LI>
   4. Séance de discussion - Composition à nouveau. Fondamentalement, cela aura une liste de fenêtres de discussion. Chaque fenêtre de discussion enregistre une session de discussion. La session de discussion est responsable d'informer toutes les fenêtres de discussion lorsqu'un nouveau message apparaît. (Motif d'observateur quiconque?) Li> OL>

      D'accord .. Donc, maintenant, j'ai mis en œuvre un modèle d'observateur en faisant de la mise en œuvre de Chatwindow "Chatlistener" Patter qui a une méthode appelée "notification (message)". Donc, la Chatsession notifie chaque chatwindow enregistré. P>

      Maintenant voici quelques choses que je veux clarifier / vouloir votre opinion sur. 1. J'ai besoin d'avoir une méthode de désenregistrer également pour toutes les fenêtres de discussion, si une fenêtre de discussion est fermée et ne veut pas obtenir plus de notifications. Cela signifie potentiellement que, soit je devrais avoir un gestionnaire d'enregistrement central "statique" qui n'a qu'une instance, puis une fenêtre de discussion devrait pouvoir se désinscrire en fournissant une "session de discussion". Pour cette raison, chaque session de discussion devrait avoir un identifiant. (Inclus ici). Ou je pourrais maintenir une instance de chatsession dans la fenêtre de discussion aussi, pour toujours avoir une instance prête. (Je déteste les singletons comme je pense qu'ils vont contre oups). Une autre manière ne serait pas de ne pas avoir de contrôle de désenregistrement de la fenêtre de discussion, avec la fenêtre de discussion, la notification de la fermeture de la fenêtre doit plutôt venir directement à la discussion et qu'elle devrait faire, ce qu'il devrait faire! P>

      2. Cette conception a du sens du tout? Si vous dites que c'est un morceau de merde et donnez-moi une approche encore meilleure; Vous obtiendrez certainement un grand merci de moi. Outre le modèle d'observateur, quels modèles pourraient être utilisés ici pour le simplifier plus ou le rendre meilleur. Aussi .. Tout point faible de cette conception, s'il est approprié mais peut être amélioré. P> li>

      3. Aussi lorsque un utilisateur tape un nouveau message dans sa propre fenêtre de discussion; Il doit être propagé à toutes les fenêtres de discussion, quelle est la session de discussion, mais en même temps; Est-ce que cela signifie que la session de chat a-t-elle besoin d'obtenir un message avec "ID de la fenêtre de discussion" et message? Et puis il le propocine à toutes les fenêtres, y compris celui qui est le propriétaire du message? Qu'est-ce qu'un meilleur moyen de gérer cela. Je veux dire, une manière où la fenêtre permet à la session de discussion du message puis de discuter avec toutes les autres fenêtres. (Je pense que cela aura besoin de certains si ... ne les aime pas non plus) p>

         Quoi qu'il en soit, laissez-moi savoir vos commentaires. Aussi s'il vous plaît rem. L'application de travail n'est pas l'intention, je suis à la recherche d'une bonne discussion, de bonnes pratiques de modèle de conception et d'utilisation. p> li> OL>

         Code complet ci-dessous, si cela vous donne une grande ... N'hésitez pas à la déchirer et à modifier des problèmes liés à presque toutes les sémantiques. P>

         package com.oo.chat;
         
         public class User {
         
             private Long userId;
             private String nickname;
         
             public User(Long userId, String nickname) {
                 this.userId = userId;
                 this.nickname = nickname;
             }
         
             public void setUserId(Long userId) {
                 this.userId = userId;
             }
         
             public void setNickname(String nickname) {
                 this.nickname = nickname;
             }
         
             public Long getUserId() {
                 return userId;
             }
         
             public String getNickname() {
                 return nickname;
             }
         
             public boolean equals(Object objectToCompare) {
                 if (!(objectToCompare instanceof User)) {
                     return false;
                 }
                 User incoming = (User) objectToCompare;
                 if (incoming.getNickname() != null && incoming.getUserId() != null) {
                     if (incoming.getNickname().equalsIgnoreCase(this.nickname)
                             && incoming.getUserId().equals(this.userId))
                         return true;
                 }
                 return false;
             }
         }
         
         
         package com.oo.chat;
         
         public interface Message {
         
             public String getValue();
         
             public void setValue(String value);
         
         }
         
         package com.oo.chat;
         
         public class SimpleMessage implements Message {
         
             private String value;
         
             public SimpleMessage() {
         
             }
         
             public SimpleMessage(String value) {
                 this.value = value;
             }
         
             public String getValue() {
                 return value;
             }
         
             public void setValue(String value) {
                 this.value = value;
             }
         }
         
         package com.oo.chat;
         
         public interface ChatListener {
         
             public void notify(Message newMessage);
         
         }
         
         package com.oo.chat;
         
         import java.util.ArrayList;
         import java.util.List;
         
         public class ChatWindow implements ChatListener {
         
             private User user;
             private List<Message> messageList;
             private Long id;
         
             public User getUser() {
                 return user;
             }
         
             public List<Message> getMessageList() {
                 return messageList;
             }
         
             public void setUser(User user) {
                 this.user = user;
             }
         
             public void setMessageList(List<Message> messageList) {
                 this.messageList = messageList;
             }
         
             public void addMessageToList(Message newMessage) {
                 if (this.messageList == null) {
                     this.messageList = new ArrayList<Message>();
                 }
                 this.messageList.add(newMessage);
             }
         
             public void notify(Message newMessage) {
                 addMessageToList(newMessage);
             }
         
             public Long getId() {
                 return id;
             }
         
             public void setId(Long id) {
                 this.id = id;
             }
         }
         
         package com.oo.chat;
         
         import java.util.ArrayList;
         import java.util.List;
         
         public class ChatSession {
         
             private List<ChatListener> registeredChatListeners;
         
             public void register(ChatWindow chatWindow) {
                 if (registeredChatListeners == null)
                     registeredChatListeners = new ArrayList<ChatListener>();
                 registeredChatListeners.add(chatWindow);
             }
         
             public List<ChatListener> getRegisteredChatListeners() {
                 return registeredChatListeners;
             }
         
             public void setRegisteredChatWindows(
                     List<ChatListener> registeredChatListeners) {
                 this.registeredChatListeners = registeredChatListeners;
             }
         
             public void incomingMessage(Long chatListenerId, Message message) {
                 publish(message);
             }
         
             protected void publish(Message messageToPublish) {
                 if (registeredChatListeners != null) {
                     for (ChatListener eachListener : registeredChatListeners) {
                         eachListener.notify(messageToPublish);
                     }
                 }
             }
         }