操作
リファクタ #179
完了
TODO/FIXME を領域別に棚卸ししてチケット化する
リファクタ #179:
TODO/FIXME を領域別に棚卸ししてチケット化する
説明
Redmine #165 のコードベースレビューとユーザー確認からの派生タスクです。
背景¶
TODO/FIXME が 78 件あり、audio、LaneRenderer、PCM、JudgeManager などに集中しています。まとめて削除・修正するのではなく、カテゴリ分けして必要な作業を個別チケット化します。
対応内容¶
- TODO/FIXME を全件抽出する。
- 領域別に分類する。例: audio、play/render、judge、skin、database、launcher、その他。
- 既に解消済み・不要なコメント、仕様判断が必要なコメント、実装が必要なコメントを分ける。
- 実装が必要なものは個別 Redmine チケットとして作成する。
- 既存チケットと重複するものは新規作成せず関連付け・コメントで整理する。
確認観点¶
- TODO/FIXME の意味を変えずに分類すること。
- 一つのチケットが大きくなりすぎないこと。
- リスクの高い gameplay/audio/skin から優先度を付けられること。
いづ さん さんが25日前に更新
- 関連している リファクタ #165: コードベースの棚卸 を追加
エージェント ちゃん さんが7日前に更新
- ステータス を 未着手 から 実装中 に変更
- 進捗率 を 0 から 10 に変更
着手しました（棚卸し・チケット化タスク）。
- 種別: コード変更を伴わない調査・Redmine起票タスク（ブランチ・MR・ビルドなし）
- 対応方針: TODO/FIXME 全78件（実質）を領域別に分類し、実装/仕様判断が必要なものを領域別チケット（約10件）へ分割。未使用クラス等のデッドコードは削除チケットとして起票。既存 #405 重複分は関連付けのみ。棚卸しの全文は本チケットのコメントに記録する。
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #462: [棚卸し] 未使用クラス・旧方式コードのデッドコード削除 を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #463: [audio] PCMデコード・オーディオドライバのリファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #464: [play/judge] 判定処理の仕様判断・リファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #465: [render] レーン描画・プラクティス表示のスキンベース移行 を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #466: [skin] スキンローダ・プロパティの汎用化・未実装対応 を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #467: [select] 選曲画面・バー表示のリファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #468: [score/db] スコア・データアクセスの仕様判断・リファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #469: [input] 入力処理のリファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している 機能 #470: [launcher] ランチャー未実装機能（最小入力間隔・スキンプレビュー） を追加
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #471: [core/misc] コア・BGA・パターン生成・起動処理のリファクタ を追加
エージェント ちゃん さんが7日前に更新
- 関連している 機能 #405: ペット性格システム を追加
エージェント ちゃん さんが7日前に更新
TODO/FIXME 棚卸し結果（全文）¶
*.java 全体を走査し 79件を抽出（うち 1 件は BytePCMTest の説明コメントで実 TODO でないため除外、実質 78件 / 47ファイル）。意味を変えずに領域別・処分別に分類した。
起票一覧（領域別 10 チケット）¶
| チケット | 領域 | 優先度 | 主な対象 |
|---|---|---|---|
| #462 | デッドコード削除 | 通常 | GdxAudioDeviceDriver×10, GdxVideoProcessor×3, RankingData:127, RankingDataCache:101, GradeBar:20/25 |
| #463 | audio | 高め | PCM:77/82/171, GdxSoundDriver:147, PreviewMusicProcessor:139 |
| #464 | play/judge | 高め | JudgeManager:24/412/614/615, KeyInputProccessor:149, SkinHitErrorVisualizer:189 |
| #465 | render | 高め | LaneRenderer:274/312/404/418/431/603, KeyConfiguration:33, PracticeConfiguration:52/53 |
| #466 | skin | 高め | BooleanPropertyFactory:67/70/73, IntegerIndexPropertyTypes:153, SkinNoteDistributionGraph:26/130/153, LR2系×4, JsonPlaySkinObjectLoader:39 |
| #467 | select | 通常 | MusicSelector:38/425/461, MusicSelectCommand:21, BarContentsLoaderThread:68, ScoreDataCache:15, SkinDistributionGraph:133, GradeBar:92, ExecutableBar:12 |
| #468 | score/db | 通常 | PlayDataAccessor:25/26, MusicResult:85, ScoreData:15, RivalDataAccessor:84 |
| #469 | input | 通常 | BMSPlayerInputProcessor:49(FIXME), BMControllerInputProcessor:17, KeyInputLog:72 |
| #470 | launcher（機能） | 通常 | InputConfigurationView:29, PlayConfigurationView:34 |
| #471 | core/misc | 通常 | PixmapResourcePool:82, PlayerResource:159/178, PlayerConfigInitializer:17, BMSPlayerMode:10, BGAProcessor:31, SkinGauge:138, ExtraNoteModifier:15, ModeModifier:20 |
いずれも本 #179 と relates で関連付け済み。gameplay/audio/skin（#463–#466）を優先度「高め」とした。
処分別の内訳¶
- A. 解消済み・不要・デッドコード（→ #462）: GdxAudioDeviceDriver（全メソッド未実装スタブ、MainController:416 のコメントアウトのみで未使用）、GdxVideoProcessor（参照ゼロ）、RankingData:127（旧方式明記）、RankingDataCache:101（空 catch）、GradeBar:20/25（「不要かも」）。
- B. 仕様判断が必要: JudgeManager:24/412（#464）、LaneRenderer:312（#465）、PlayDataAccessor:25/26・MusicResult:85・ScoreData:15（#468）、SkinDistributionGraph:133（#467）、PlayerConfigInitializer:17・BMSPlayerMode:10（#471）— 各チケット内で「仕様判断」として明記。
- C. 実装/リファクタが必要: 上記以外。領域チケットに分配。
-
既存チケットとの重複: PetGrowthService:35 はすでに
TODO(#405)表記あり。新規起票せず #405 を relates で関連付けのみ。
棚卸し全件（ファイル:行）¶
- BMSPlayerMode.java:10 / KeyConfiguration.java:33 / GdxSoundDriver.java:147
- GdxAudioDeviceDriver.java:15,21,27,33,39,45,51,57,63,69
- BMControllerInputProcessor.java:17 / BMSPlayerInputProcessor.java:49(FIXME)
- PCM.java:77,82,171 / KeyInputLog.java:72
- InputConfigurationView.java:29 / PlayConfigurationView.java:34
- RankingDataCache.java:101 / RankingData.java:127
- ExtraNoteModifier.java:15 / ModeModifier.java:20
- PetGrowthService.java:35(→#405) / PlayDataAccessor.java:25,26
- PixmapResourcePool.java:82 / PlayerConfigInitializer.java:17
- BGAProcessor.java:31 / JudgeManager.java:24,412,614,615
- GdxVideoProcessor.java:11,22,28 / PlayerResource.java:159,178
- KeyInputProccessor.java:149 / LaneRenderer.java:274,312,404,418,431,603
- RivalDataAccessor.java:84 / ScoreData.java:15
- PracticeConfiguration.java:52,53 / SkinGauge.java:138
- BarContentsLoaderThread.java:68 / MusicSelectCommand.java:21
- PreviewMusicProcessor.java:139 / MusicSelector.java:38,425,461
- SkinDistributionGraph.java:133 / ScoreDataCache.java:15
- GradeBar.java:20,25,92 / ExecutableBar.java:12
- JsonPlaySkinObjectLoader.java:39 / LR2SelectSkinLoader.java:22 / LR2SkinHeaderLoader.java:49 / LR2PlaySkinLoader.java:945 / LR2SkinCSVLoader.java:338
- SkinNoteDistributionGraph.java:26,130,153 / SkinHitErrorVisualizer.java:189
- BooleanPropertyFactory.java:67,70,73 / IntegerIndexPropertyTypes.java:153
- result/MusicResult.java:85
本タスクはコード変更を伴わない棚卸し・起票タスクのため、ブランチ・MRはなし。
エージェント ちゃん さんが7日前に更新
- ステータス を 実装中 から 完了 に変更
- 進捗率 を 10 から 100 に変更
棚卸し・領域別チケット化が完了したため完了とする。
- 実質78件を領域別 10 チケット（#462–#471）へ起票・関連付け済み。
- PetGrowthService:35 は #405 へ関連付けのみ（重複起票なし）。
- 実際の実装/仕様判断は各派生チケットで対応する。
- コード変更・MRなし。
操作