操作
リファクタ #467
完了
[select] 選曲画面・バー表示のリファクタ
リファクタ #467:
[select] 選曲画面・バー表示のリファクタ
説明
親 #179 「TODO/FIXME 棚卸し」からの派生（select 領域）。
対象 TODO¶
-
src/.../select/MusicSelector.java:38— ミラーランダム段位のスコア表示。 -
src/.../select/MusicSelector.java:425— 表名・フォルダ名を PlayerResource 上でも重複実施している。 -
src/.../select/MusicSelector.java:461— 通常オプションもここに入れて渡すか。 -
src/.../select/MusicSelectCommand.java:21— 最終的には全て EventFactory へ移動。 -
src/.../select/BarContentsLoaderThread.java:68— collector を使用してスコアをまとめて取得。 -
src/.../select/ScoreDataCache.java:15— ResourcePool ベースに移行する。 -
src/.../select/SkinDistributionGraph.java:133— 分布計算タイミング（フォルダ選択時のみとした場合の LAMP 更新整合）の仕様判断。 -
src/.../select/bar/GradeBar.java:92— ライバルスコア。 -
src/.../select/bar/ExecutableBar.java:12— ExecutableBar の拡張設計。
対応観点¶
425 の表名/フォルダ名補完は core 側（PlayerResource）と重複しており #179 派生 core チケットと調整。
優先度: 通常
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #179: TODO/FIXME を領域別に棚卸ししてチケット化する を追加
エージェント ちゃん さんが7日前に更新
- ステータス を 未着手 から 実装中 に変更
- 進捗率 を 0 から 10 に変更
実装に着手しました。
- ブランチ:
feature/467 - 対応方針: #464/#465/#466 を踏襲した保守的方針で、select 領域の 9 つの TODO を全て「決定コメント化」し、挙動は完全不変とする（スキン互換性・既存挙動を一切変更しない）。
エージェント ちゃん さんが7日前に更新
- ステータス を 実装中 から 完了 に変更
- 進捗率 を 10 から 100 に変更
MR !143 が develop へマージされ、完了しました。
対応内容¶
#464/#465/#466 を踏襲した保守的方針で、select 領域に残る 9 件の TODO を全て「決定コメント化」（現状の設計判断と見送り理由を明記）した。コメント行のみの変更で挙動は完全に不変。
変更ファイル（7 ファイル / +22 -9）¶
-
MusicSelector.java: ミラーランダム段位スコア表示 / 表名・フォルダ名の core(PlayerResource) 重複 / NONE の通常オプション -
MusicSelectCommand.java: EventFactory への全面移行 -
BarContentsLoaderThread.java: collector による一括スコア取得 -
ScoreDataCache.java: ResourcePool ベースへの移行 -
SkinDistributionGraph.java: 分布計算タイミングの仕様判断 -
GradeBar.java: 段位のライバルスコア（データソース不在） -
ExecutableBar.java: 拡張設計（Javadoc 形式維持）
テスト¶
-
.\gradlew.bat compileJava: 成功（コメントのみ・挙動不変の narrow check） - 挙動不変のため新規テストなし
補足¶
-
MusicSelector.java:425（表名/フォルダ名の core 重複）は #179 派生の core チケットとの整合が必要なため一元化はスコープ外とし、コメントでクロスカット依存を明記。 -
BarContentsLoaderThread.java:68（collector 一括取得）は実装余地があったが、ホットパスの挙動変化リスクを避けるため見送り（決定コメント化）。
マージコミット: e28f2b72
操作