java - 适用于简单聊天应用程序的设计模式

Question

我正在设计一个简单的聊天应用程序(只是为了好玩)。我一直想知道该聊天应用程序的简单设计。给你概述..这里是规则:

1. 匿名用户仅使用昵称进入聊天。 (User id) 大概是由系统在后台分配的。
2. 他们可以加入(订阅)聊天对话。他会在指定区域看到其他用户的聊天文本。
3. 他们可以回复特定对话，其他人都应该看到。

就是这样! (看我告诉过你这是一个简单的聊天应用程序)。所以，我的意图并不是真正的应用；但是其中使用的设计模式和对象。

现在我是这样设计的。 (我正在用 Java 编码......以防万一)

1. 用户对象——两个属性id和nickname
2. 消息对象 - 一个简单的消息接口(interface)和实现(目前)作为 SimpleMessage，以字符串作为包含消息的属性。
3. 聊天窗口对象 - 基本上由用户和消息组成。因为它有一个用户对象和消息列表。
4. 聊天 session - 再次组合。基本上它会有一个聊天窗口列表。每个聊天窗口都注册到一个聊天 session 。聊天 session 负责在出现新消息时通知所有聊天窗口。 (观察者模式有人吗？)

好的.. 所以现在我已经通过让 ChatWindow 实现“ChatListener”模式来实现观察者模式，该模式具有名为“notify(Message)”的方法。因此 ChatSession 会通知每个已注册的 ChatWindow。

现在我想澄清几件事/想听听您的意见。 1.我还需要为所有聊天窗口取消注册方法，以防聊天窗口关闭并且不想再收到任何通知。这可能意味着，要么我应该有一个只有一个实例的“静态”中央注册管理器，然后任何聊天窗口都应该能够通过提供“聊天 session ”ID 来自行注销。因此，每个聊天 session 都应该有一个 ID。 (包括在此)。或者我也可以在聊天窗口中维护一个 ChatSession 实例，以始终准备好一个实例。 (我讨厌单例，因为我认为他们反对 oops)。另一种方法是不对聊天窗口进行注销控制，有聊天窗口，而是窗口关闭的通知应该直接传给 ChatSession，它应该做什么，它应该做什么!

1. 这个设计是否有意义？如果您说这是废话并给我一个更好的方法；我一定会非常感谢你。除了观察者模式之外，这里可以使用所有模式来进一步简化或改进它。另外.. 这个设计的任何弱点，如果合适但可以改进。

2. 还有当用户在他自己的聊天窗口中键入新消息时；它需要同时传播到所有聊天窗口，这就是聊天 session 所做的；这是否意味着.. 聊天 session 需要获取带有“聊天窗口 ID”和消息的消息？然后它将它传播到所有窗口，包括作为消息所有者的窗口？有什么更好的方法来处理这个问题。我的意思是，窗口让聊天 session 知道消息然后与所有其他窗口聊天 session 的方式。 (我想它需要一些假设……也不喜欢它们)

   无论如何...请让我知道您的意见。也请雷姆。工作应用程序不是意图，我正在寻找一个好的讨论，好的设计模式实践和用法。

下面是完整的代码，如果它给您带来高分...请随意将其拆开并提出与几乎所有语义相关的问题。

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);
            }
        }
    }
}

提前感谢所有贡献者。干杯

Answer 1

我觉得基本设计不错。显然要完成此操作，您需要添加更多功能。当前的设计将所有消息无限期地保存在内存中，但在某些时候您将需要代码来清除旧消息。

我确实看到的几个重要的设计问题是:

1. 消息界面不链接回消息的发件人 - 大多数聊天显示谁说了什么，如果消息中没有用户字段，这将很困难。
2. 消息接口(interface)没有时间属性。这将使清除旧消息变得更加困难。