負の遺産を浄化する

[ChenCMD]

負の遺産とは？

ここで言う負の遺産とはTheSkyBlock(以下、TSBと省略する)の実装上の妥当では無い実装のことを指す。
現在TSBにはいくつかのこの負の遺産が存在する。それらを将来の為にも浄化(すなわち妥当な実装に変更)する必要がある。
2021年12月14日時点で負の遺産と思われるものは以下のものである。

アセットのファイル名の採番

実装が妥当では無い理由

• ソート順序の問題によりreviewの負担を増加させる
• コードが冗長になりやすい
• 分岐が存在する場合、必ずしもその順番通りにfunctionが推移しない

妥当な実装

思考中

asset_manager名前空間の神namespace化

実装が妥当では無い理由

• 一名前空間で非常に多くの実装を持つ
• mob_managerやplayer_managerがありながらにしてそれらにほぼ処理を渡さずにasset_managerが処理をしている
• ぶっちゃけTSBの本体といっても過言ではない

妥当な実装

よしなに各namespaceに処理を分割する必要がある

コーディング規約

実装が妥当では無い理由

• 複雑すぎる
• 利用実態にそぐわない規約が存在する
• reviewでの指摘が困難

妥当な実装

簡易化する。必要なものに関してはGitHubActionsで自動化するべき。

[anitmyan]

間違えた！ごめんなさい！