個人開発 » beatoraja

レビュー結果を受けてチケット化までがタスク。

別タスクである不要なファイルなどの削除はこのチケットで実施する。

Claude にコードベース全体レビューを依頼しました。ファイル編集・削除は行っていません。

高確度の指摘¶

• docs/codebase-overview.md が現状と乖離しています。
  • 存在しない docs/dependencies-inventory.md への参照があります。
  • ソース数・テスト数・パッケージ表が現状より古くなっています。
  • Jar entry point の説明が MainLoader になっていますが、実際の Main-Class は BeatorajaLauncher です。
• リポジトリ直下の beatoraja.0.log 〜 beatoraja.4.log はランタイム生成物の混入候補です。.gitignore には beatoraja_log.xml のみあり、ローテーションログの ignore パターンがありません。
• build.gradle に古い・環境依存の設定があります。
  • localLibs の exclude 対象は現在の lib/ に存在せず、Maven 依存移行前の名残に見えます。
  • verifyFfmpegDecode の既定引数に D:/rei-no-are/010/010_BGA_256_no_sound.mpg という開発者ローカルの絶対パスがあります。
• 肥大化したホットスポットがあります。
  • skin/property/SkinProperty.java
  • skin/lr2/LR2PlaySkinLoader.java
  • play/BMSPlayer.java
  • play/JudgeManager.java
  • audio/AbstractAudioDriver.java
  • MainController.java
• 変更リスクが高い中核にテスト不足があります。
  • play/JudgeManager.java
  • audio driver / PCM 系
  • play/LaneRenderer.java
  • skin loader の詳細な解釈ロジック
  • launcher controller / view model ロジック

人間の確認が必要な候補¶

• folder/default.json、random/default.json、tools/performance-log-summary.ps1 は配布物として意図された追跡対象か確認が必要です。
• ScoreLogDatabaseAccessor と ScoreDataLogDatabaseAccessor はどちらも使用中ですが、責務が近く将来的な整理候補です。
• skin/property/ 配下の巨大な property / event 定義群は整理余地がありますが、JSON/LR2/Lua 互換に関わるため設計判断が必要です。
• *Smoke.java や experimental パッケージは一見削除候補に見えますが、Gradle の検証タスクから使われるため削除しない方針です。
• TODO/FIXME は 78 件あり、特に audio、LaneRenderer、PCM、JudgeManager に集中しています。

Claude の総評¶

コード自体の死蔵ファイルは少なく整理されています。即対応しやすい棚卸対象は、ドキュメントの陳腐化、ログ生成物の混入、build.gradle の古い exclude と環境依存パスです。アーキテクチャ面では、肥大中核ファイルへのテスト追加が費用対効果の高い次アクションです。

Claude レビュー結果から、実施すべきリファクタリング作業を以下の Redmine チケットとして作成しました。

• #171 docs/codebase-overview.md を現状に合わせて更新する
• #172 ランタイムログ生成物の追跡状態と ignore 設定を整理する
• #173 build.gradle の古い localLibs exclude とローカル絶対パスを整理する
• #174 JudgeManager のテスト追加と責務分割方針を検討する
• #175 audio driver / PCM 系のテスト追加方針を整理する
• #176 skin property 定義群の肥大化を整理する方針を作る

削除候補や仕様判断が必要な項目は、今回の「やるべきリファクタリング」起票からは外しています。

確認が必要だった項目について、ユーザー判断を反映しました。

確認結果¶

• folder/default.json と random/default.json は配布物として残す。
• tools/performance-log-summary.ps1 は保守用ツールとして残す。
• ScoreLogDatabaseAccessor と ScoreDataLogDatabaseAccessor は統廃合して整理する。
• skin/property/ 配下の property / event 定義群は分割・整理する。
• *Smoke.java や experimental パッケージは検証で使うため削除せず、保守対象とする。
• TODO/FIXME はカテゴリ分けしてチケット化する。

追加作成したチケット¶

• #177 ScoreLogDatabaseAccessor と ScoreDataLogDatabaseAccessor の統廃合方針を整理する
• #178 Smoke / experimental 検証コードを保守対象として整理する
• #179 TODO/FIXME を領域別に棚卸ししてチケット化する

既存チケットで扱う項目¶

• skin property 定義群の分割・整理は #176 で扱う。
• folder/default.json、random/default.json、tools/performance-log-summary.ps1 は削除対象ではないため、追加チケット化はしない。