メソッド名を改善 #27

[ihcomega56]

結局ご指摘いただいた箇所はそのままとしました。

何故かと言うと…

ご指摘の通りthis.disconnectButton.disabled = !connected;だけ取り出すと読みづらいんですが 全体を読むと

• 接続された→接続ボタンは無効、切断ボタンは有効
• 切断された→切断ボタンは無効、切断ボタンは有効 これだけなので、今の状態が1番シンプルかなーと思いました。

あと、いただいた案

sendButtonのステータスに現在と反対の値を設定する

について、 今のステータスをひっくり返すには今のステータスが分からないといけないはずなんですが クラス変数(…とは言わないよねJSよくわからんけどまぁスコープが広いサムシン)とかにステータスもたせると煩雑になるし this.connectButton.disabled = !this.connectButton.disabled; とかはさすがにカオスすぎるし…改善が浮かばないのも正直なところです。

何かコードの案があったらいただけると嬉しいです><

で、メッセージ入力に応じボタンのステータス切り替えるメソッドは名前かえてみました。

[mahya8585]

かしこまりです。 どうやらコンフリクトしているようなので、、、、直してからマージということで・・・ （今手元にGitの環境ないので・・・） @ihcomega56 もしすぐにコンフリクト修正できるようならセルフマージしちゃって構いません。

[ihcomega56]

コンフリクト解消完了したのでマージしちゃいます〜