3

私は webcocket を使用してチャットを作成しようとしています。オフライン ユーザーには Java キューを使用しました。ユーザーがオフラインの場合はメッセージをキューに保持し、ユーザーがオンラインになったときにキューが空かどうかを確認し、そうでない場合はループを使用して削除しますキューからの各メッセージ。問題は、すべてのメッセージがキューに入っているにもかかわらず、最後のメッセージをユーザーに送信するだけです。これが私の 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()したがって、サイズを 1 ずつ増やしたり減らしたりiすると、事実上、キュー内の要素の半分しかアクセスできなくなります。1userMsgs

最初に、キューに 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 に答える