個人開発 » beatoraja

実装に着手しました。

• ブランチ: feature/473
• 対応方針: VS Code(Eclipse JDT) の Problems 全 374 件のうち beatoraja 本体 353 件を解消する。別リポ(jbms-parser/jbmstable-parser)の 21 件は対象外。
• 内訳: 未使用import 82 / raw type 73 / switch case不足 43 / 非推奨API 39 / unchecked系 50 / 未使用フィールド 21 / リソースリーク 13 / raw-typeメソッド 10 / 未使用変数・dead code 12 / unlikely-argument(実バグ) 3 / その他5。
• 本チケットは挙動不変方針(#462-471系)は適用せず、抑制で逃げず可能な限り正しく修正する。unlikely-argument 3件は実バグとして修正する。

対応が完了しました。

概要¶

VS Code / Eclipse JDT の Problems 全 374 件のうち beatoraja 本体 353 件を解消。別リポ(jbms-parser 9 / jbmstable-parser 12 = 21件)は対象外。抑制で逃げず可能な限り正しく修正する方針で、型安全化・非推奨API移行・リソースリーク解消に加え、JDT が指摘した潜在バグ2件も修正した。

変更点¶

• 未使用 import 82 / 未使用フィールド 21 / 未使用ローカル・dead code 12 / 未使用メソッド 3 を削除。
• raw type / unchecked 系約145件: ジェネリクス正規化。不可能箇所のみ最小スコープの @SuppressWarnings＋コメント。
• deprecation 39: libGDX Array.toArray(Class) → toArray(T[]::new)、Thread.getId() → Thread.threadId()。挙動同一。
• missing-switch-case 43: enum switch に挙動保存の no-op default を追加。
• resource-leak 13: song-search テストを try-with-resources 化（アサーション保持）。
• 潜在バグ2件を修正（挙動が変わる）: KeyInputLog のスクラッチ/BSS 判定（Arrays.asList(int[]).contains が常に false だった）と、ResourceConfigurationView の難易度表URL重複チェック（Collection<TableInfo>.contains(String) が常に false だった）。
• TODO マーカー2件: リテラル TODO トークンを除去（#405 / #468 参照は保持）。
• JsonSelectSkinObjectLoader: 到達可能な return null; を復元し、真の dead code(else)を除去。
• 変更ファイル: src/bms/player/beatoraja/skin/json/JSONSkinLoader.java ほか約55ファイル（KeyInputLogTest 新規追加を含む）。

テスト¶

• .\gradlew.bat test : 成功（BUILD SUCCESSFUL、実 exit 0）。KeyInputLogTest 追加。

関連 MR¶

• !148 (feature/473 → develop)