個人開発 » beatoraja

skin property 定義群の整理方針¶

skin/property 配下を棚卸しした結果と、小さく安全に進めるための方針を以下にまとめる。本チケットは方針策定のみで、実作業は末尾の派生チケットで進める。

1. 現状の責務マッピング¶

2. 重複している責務（共通化・単一化候補）¶

• id→property が 2 形態で存在：enum 登録（ValueType/BooleanType）と手書き switch（getIntegerProperty0/getBooleanProperty0）。新規エントリがどちらにも追加できることが肥大化・不整合の主因。
• name→/id→ 解決が Factory ごとに別実装（線形走査 / StringType.get マップ / キャッシュ配列）。name/id ドリフトのリスク。
• 反転（-id）・キャッシュが型ごとに再実装されている。
• 重い import static SkinProperty.* + 同一 import ブロックが複数ファイルで重複。

3. 互換のために分離必須な責務（触れてはいけない）¶

• SkinProperty.java の id 値（LR2 数値契約 + JSON/Lua の name→id アンカー）。リナンバー/削除は破壊的変更。
• 意図的な重複 id / エイリアス（例：NUMBER_TOTAL_RATE と NUMBER_SCORE_RATE2）。既存テストも duplicateCounts で重複を許容している。
• enum 定数名 = JSON/Lua の公開プロパティ名。typo 名（playtime_totla_saecond, currenttime_saecond 等）も契約なので「修正」禁止。
• 算術範囲（judge/key/player offset）は LR2 連番 id 規約。数式を完全保存。

4. 小さく安全な分割単位（派生チケット）¶

Phase 0 → 1 → 2 → 3 の順（precedes で連結）。各 Phase は独立して develop へマージ可能な粒度とする。

5. 先に追加すべき互換テスト（Phase 0 = #455 で詳述）¶

既存 SkinPropertyFactoryRegistryTest は enum 登録の id/name 解決を網羅済。未カバーの seam は以下：

• getIntegerProperty0 / getBooleanProperty0 の switch フォールバック各 case の解決・値。
• 算術範囲ハンドラ（judge/key/player offset 等）の解決・値。
• SkinPropertyMapper の round-trip（keyJudgeValueId↔getKeyJudgeValuePlayer/Offset、bombTimerId/holdTimerId 境界、isCustomTimerId/isCustomEventId 範囲、getSkinSelectType↔skinSelectTypeId）。
• ゴールデン契約テスト：同梱 skin/default の JSON スキン（追跡済 27 ファイル）を読み、参照される全 property name/id が non-null 解決されることを assert。誤削除・誤リネームを検出できる。