3

我正在尝试使用 webcocket 编写聊天，对于离线用户，我使用了 java Queue，如果用户离线，我将消息保存在队列中，当用户上线时，我检查队列是否为空，如果不是，使用循环我删除来自队列的每条消息。问题是即使所有消息都在队列中，它只是向用户发送最后一条消息，这是我的 onOpen 方法：

@ServerEndpoint(value = "/chat/{room}/{user}", encoders = ChatMessageEncoder.class, decoders = ChatMessageDecoder.class)
 public class ChatEndpoint {


private final Logger log = Logger.getLogger(getClass().getName());
private static final Map<String, Queue<ChatMessage>> userMessageBuffer = new HashMap<>();

@OnOpen
public void open(final Session session,
        @PathParam("room") final String room,
        @PathParam("user") final String userId) {

    log.info("session openend and bound to room: " + room);
    // session.getUserProperties().put("room", room);
    session.getUserProperties().put("user", userId);

    Queue<ChatMessage> userMsgs = userMessageBuffer.get(userId);
    ChatMessage sendChat = new ChatMessage();

    if (userMsgs != null && !userMsgs.isEmpty()) {

        for (int i = 0; i < userMsgs.size(); i++) {

            sendChat = userMsgs.remove();
            System.out.println("size!!!!!! " + sendChat.getMessage());
            try {

                    session.getBasicRemote().sendObject(sendChat);

            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            } catch (EncodeException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }

        }
    }

}}

有谁知道问题出在哪里？

4

1 回答 1

4

我没有检查完整的代码，但肯定

      for (int i = 0; i < userMsgs.size(); i++)

是你的问题。因为您在 for 循环中使用i++和检查userMsgs.size()作为条件。因此，您将大小增加i和1减少userMsgs1，实际上您将只能访问队列中一半的元素。

假设您最初在队列中有 8 个元素（想象这就像Admiral General Aladeen解释圆形导弹）

               i=0 and userMsgs.size()=8
               i=1 and userMsgs.size()=7
               i=2 and userMsgs.size()=6
               i=3 and userMsgs.size()=5
               i=4 and userMsgs.size()=4 // comes out of loop.

你应该改用while循环，

  while(!userMsgs.isEmpty()){
   .....
  }

您说您只能向用户发送最后一条消息，可能是因为队列中只有 2 条消息。我知道这很少见，但根据您的代码应该是这种情况。

于 2015-07-07T10:48:53.893 回答