操作
リファクタ #462
完了[棚卸し] 未使用クラス・旧方式コードのデッドコード削除
リファクタ #462:
[棚卸し] 未使用クラス・旧方式コードのデッドコード削除
説明
親 #179 「TODO/FIXME 棚卸し」からの派生(A群: 解消済み・不要・デッドコード)。
対象 TODO¶
-
src/.../audio/GdxAudioDeviceDriver.java:15,21,27,33,39,45,51,57,63,69— 全メソッドが Auto-generated 未実装スタブ。実体参照は MainController:416 のコメントアウトのみで未使用。 -
src/.../play/bga/GdxVideoProcessor.java:11,22,28— Auto-generated スタブ。参照ゼロで未使用。 -
src/.../ir/RankingData.java:127— 「旧方式のため後で削除」と明記されたデッドコード。 -
src/.../ir/RankingDataCache.java:101— Auto-generated の空 catch block。ログ出力等へ整理。 -
src/.../select/bar/GradeBar.java:20,25— 「不要かも」。未使用確認の上で削除判断。
対応方針¶
未使用を再確認のうえ削除。将来実装予定のドライバー(GdxAudioDeviceDriver/GdxVideoProcessor)は、残す場合は意図をコメント明記するか Issue 化してからスタブを残すこと。
優先度: 通常(低リスク・クリーンアップ)
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #179: TODO/FIXME を領域別に棚卸ししてチケット化する を追加
エージェント ちゃん さんが7日前に更新
- ステータス を 未着手 から 実装中 に変更
- 進捗率 を 0 から 10 に変更
実装に着手しました。
- ブランチ:
feature/462 - 対応方針: 未使用スタブ2クラス(GdxAudioDeviceDriver/GdxVideoProcessor)を削除し、RankingDataの旧方式分岐を除去。空catchはログ整理、GradeBarのmscore/rscoreは実使用中のためTODOを決定コメント化して残置。
エージェント ちゃん さんが7日前に更新
- ステータス を 実装中 から 完了 に変更
- 進捗率 を 10 から 100 に変更
対応が完了しました。
概要¶
親 #179「TODO/FIXME 棚卸し」由来の低リスククリーンアップ。未使用スタブクラスと旧方式のデッドコードを削除し、空 catch をログ整理した。
変更点¶
-
audio/GdxAudioDeviceDriver.javaを削除(未使用スタブ、参照はMainControllerのコメントアウトのみ)。MainControllerの死んだcase AudioDeviceコメントとAudioConfigのコメント済AudioDeviceenum 項目も除去。 -
play/bga/GdxVideoProcessor.javaを削除(参照ゼロ)。 -
ir/RankingData.java:旧方式の「空プレイヤー名 → PLAYER_YOU」フォールバック分岐と未使用ヘルパーisEmptyPlayerNameを削除(現行は ID 照合hasSamePlayerIdのみ)。 -
ir/RankingDataCache.java:空 catch(printStackTrace)をLogger.getGlobal().warningへ整理。 -
select/bar/GradeBar.java:mscore/rscoreの「不要かも」TODO を決定コメント化(実使用中のため削除せず)。 - 変更ファイル: 削除2・編集5の計7ファイル。
テスト¶
-
.\gradlew.bat compileJava: 成功(警告は既存の deprecation/unchecked のみ)。削除シンボルへの残存参照ゼロをgit grepで確認。純粋な削除±微整理のため新規テスト追加はなし。
関連 MR¶
- !142 (feature/462 → develop)
操作