退出時にメッセージを送信する＆動作しない問題を修正

zoosm3 commented 8 years ago

退出時にメッセージを送信する動作確認用に#20を暫定修正

ihcomega56 commented 8 years ago

@zoosm3 すみませんまだ途中です。全部見終わったらチャットで連絡します。m(__)m

ihcomega56 commented 8 years ago

ありがとうございます〜！実行時の例外に関する修正はマージしてOKそうですね。ただ、JKD8u72とか8u74とかでもがんがんエラーはきました(T_T)結局8u25使って動かせてます…

その他、修正していただきたい点いくつかコメントしました！それとmasterに変更が入っているのでとりこみお願いします。あとそもそも仕様っぽい部分ですが、メッセージの表示を画面上で行いたいですよね。

コード見ただけで全然調べられてないので今から考えますが、

this.stompClient.send('/app/message', {}, name+"さんが退室しました");
this.stompClient.disconnect();
this.stompClient = null;

これ一緒にやっちゃうとだめ(?)ってことですよね、、画面で出すにはどうするか、ちょっと考えましょう〜！

取り急ぎ指摘の修正と、masterのとりこみおねがいします！🗿

zoosm3 commented 8 years ago

@ihcomega56 遅くなって申し訳ありません。

実行時の例外に関する修正はマージしてOKそうですね。マージお願いします。

ただ、JKD8u72とか8u74とかでもがんがんエラーはきました(T_T)結局8u25使って動かせてます

ここはJDK1.8.25で固定して、参加者の方にも準備してもらうことになると思います。

それとmasterに変更が入っているのでとりこみお願いします。

取り込みました。取り込むにあたってメッセージが名前：本文の形式になっているので、都合上、メッセージを送るときの名前を付けました。 `const CHAT_MASTER_NAME = 'ぬし';

あとそもそも仕様っぽい部分ですが、メッセージの表示を画面上で行いたいですよね。他の人に退出メッセージを送るのには↓の部分が必要です。 `this.stompClient.send('/app/message', {}, name+"さんが退室しました");

ただこのあと、自分の接続が切れてしまうと自分自身が退出メッセージを受け取れないので、画面での表示処理が必要かと思います。まだpushしていませんが、画面での表示処理はすでにあるメソッドonSubscribeGreetingを呼べば出来そうです。いかがでしょうか？

java-women / ChatToolForBeginners