fix: PlayerManagerの完全static化、退出後の読み上げ停止

book000 commented 1 year ago

PlayerManager.destroy() 後にも継続して managers に当該キーが残り続けてるように見えたので、#209 をさらに追修正する形の内容となります。
読みやすくするため、PlayerManager にあるメソッドを全て static にしました。（内容的にわざわざインスタンスとして持っている理由が無いと判断）
AutoDisconnected 後、必ずひとつ以上の読み上げキューが残る問題を暫定的に修正しました。これは同時に抜けた一般ユーザ分の退出読み上げが残るために起きる問題です。但し、同時に複数人がVCから抜けた場合この問題は必ず発生するので、根本的な設計部分 https://github.com/jaoafa/JDA-VCSpeaker/issues/145#issuecomment-1401153299 を実装し直さなければ直りません。
デバッグ用として /vcspeaker debug queue を追加

絶対設計レベルからなんとかするべき問題がいくつかあるけど、私には時間がない…

yuuahp commented 1 year ago

そういえば https://github.com/jaoafa/VCSpeaker.kt で設計レベルからどうにかしようと勝手に進めてるけど大丈夫ですか

book000 commented 1 year ago

良いけど、進捗が見えないのですっかり忘れていました…。まあユーザサイドからはなるべく互換性を持たせた上で実装してくれれば特別問題はないどす

jaoafa / JDA-VCSpeaker