操作
リファクタ #465
完了
[render] レーン描画・プラクティス表示のスキンベース移行
リファクタ #465:
[render] レーン描画・プラクティス表示のスキンベース移行
説明
親 #179 「TODO/FIXME 棚卸し」からの派生（render 領域。gameplay）。ハードコードされた描画値をスキン定義へ移行する。
対象 TODO¶
-
src/.../play/LaneRenderer.java:274— hu,hl をレーン毎に変更。 -
src/.../play/LaneRenderer.java:312— HIDDEN と LIFT 混在の必要性と HIDDEN の必要性（仕様判断）。 -
src/.../play/LaneRenderer.java:404,418,431— 数値もスキンベースへ移行。 -
src/.../play/LaneRenderer.java:603— dstnote2 をレーン毎に変更。 -
src/.../config/KeyConfiguration.java:33— スキンベースへ移行。 -
src/.../play/PracticeConfiguration.java:52— 描画位置・使用テキスト等をスキン定義できるように。 -
src/.../play/PracticeConfiguration.java:53— スキン定義がない場合のデフォルト配置の定義。
対応観点¶
LIFT/HIDDEN 混在（312）は仕様判断を要する。既存スキン互換性に注意。
優先度: 高め
エージェント ちゃん さんが7日前に更新
- 関連している リファクタ #179: TODO/FIXME を領域別に棚卸ししてチケット化する を追加
エージェント ちゃん さんが7日前に更新
- ステータス を 未着手 から 実装中 に変更
- 進捗率 を 0 から 10 に変更
実装に着手しました。
- ブランチ:
feature/465 - 対応方針: スキンフォーマット拡張・既存スキン互換性破壊は行わず、各 TODO の仕様判断を決定コメント化し、3 箇所重複するプレイヤーエリアテキスト描画の集約のみ安全なリファクタとして実施する（#464 の保守的方針を踏襲）。
エージェント ちゃん さんが7日前に更新
- ステータス を 実装中 から 完了 に変更
- 進捗率 を 10 から 100 に変更
対応が完了しました。
概要¶
#179 棚卸し派生の render 領域リファクタ。保守的方針(#464 踏襲)で、スキンフォーマット拡張・既存スキン互換性破壊を一切行わず、7つの TODO を「決定コメント化」し、挙動不変の重複集約のみ実施。
変更点¶
-
LaneRenderer:hu/hl(274)・HIDDEN/LIFT混在(312)・dstnote2(603) を「lanes[0] をグループ代表として共有する現行設計を維持」する決定コメントへ置換（レーン毎化は描画ループ全面再構成を要するため見送り）。 -
LaneRenderer: タイムライン/BPM/STOP の3箇所重複する font 描画(404/418/431)を private ヘルパーdrawPlayerAreaText(中央/左寄せ・色をパラメータ化)に集約。完全挙動不変。 -
KeyConfiguration(33)・PracticeConfiguration(52/53): 画面全体のスキン化は大規模＋互換リスクのためスコープ外宣言コメントへ（将来の独立チケットで対応）。 - 変更ファイル:
LaneRenderer.java/KeyConfiguration.java/PracticeConfiguration.java計 3 件
テスト¶
-
.\gradlew.bat compileJava: 成功（狭いリファクタ・描画は libGDX 密結合で非テスト容易のためユニットテスト追加なし）
関連 MR¶
- !140 (feature/465 → develop)
操作