汚いコード直す

[Quarri6343]

・return null 廃止
・エラーメッセージがマジックナンバーになってるの廃止
・guiメッセージがマジックナンバーになってるの廃止
他にもあったら追記

[Quarri6343]

switch文廃止

[Quarri6343]

aikisoコマンド削除

[Quarri6343]

こことか、大量のif-elseをなんとかする
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/decoration/BloodExpression.java#L113-L149

[Quarri6343]

直sendmessage廃止
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/gui/EquipmentGroupModifyItemGui.java#L49

[Quarri6343]

この鯖の名前は「ShortLife」です
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/listener/CommonListener.java#L177

[Quarri6343]

重要な分岐変数がマジックナンバーになっているのを修正
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/gui/item/MatchModeSelectItem.java#L130

[Quarri6343]

単一責任原則が明らかに破られている長いメソッドたちをなんとかする
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/listener/CommonListener.java#L186-L226

[Quarri6343]

↑イベントのレシーバーメソッドとその処理を分離する
例えばonWeaponPreReload(武器再装填時のイベント)は「イベントの受信という役割だけを担うメソッド」なので、「武器リロード時にどんなメソッドが呼び出されるか」だけを書くべき。
実際のリロード処理を書いてしまうとメソッド名と内容の乖離が起きる

[Quarri6343]

定数だけを列挙するクラスは全てenumにする
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/match/map/MapMarkerPoints.java#L11

[Quarri6343]

warningがちらほらあるので潰せる限り潰す

[Quarri6343]

でかすぎるクラスを分割
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/match/Match.java#L1770

[Quarri6343]

Optional#get() 廃止
https://github.com/TeamFelnull/ShortLifePlugin/blob/41f891d39f152ce6f671d886daf85bcf75a1aa99/src/main/java/dev/felnull/shortlifeplugin/match/Match.java#L424

https://qiita.com/BlueRayi/items/ef46496ef2ef36b9cbb7

[Quarri6343]

メソッドの戻り値の@nullableを原則廃止（optionalで代用可）