操作
リファクタ #457
完了skin property: SkinProperty の id 定数をカテゴリ別に整理する(Phase 3)
リファクタ #457:
skin property: SkinProperty の id 定数をカテゴリ別に整理する(Phase 3)
説明
#176 の整理方針に基づく Phase 3。最後に着手する(他フェーズと独立しても可)。
背景¶
SkinProperty.java(1081 行)は IMAGE / TIMER / NUMBER / OPTION / STRING / EVENT / BUTTON / SLIDER / GRAPH などの public static final int 定数がフラットに並ぶ名前空間で、見通しが悪い。この id 群は JSON name→id アンカー / LR2 数値 op 契約 / Lua から参照されるため、値と import static SkinProperty.* の公開面は一切変えられない。
対応内容¶
- 定数をカテゴリ(Timer / Number / Option / String / Event 等)ごとにグループ化し、コメント区切り・並び順の整理、もしくはドメイン別 partial 定数インターフェースへの分割を検討する。
-
値は一切変えず、
import static SkinProperty.*で参照している既存コードがそのままコンパイルできることを保証する。
確認観点¶
- id 値の重複(エイリアス互換)を保持し、リナンバー・削除は行わない。
- Phase 0 のゴールデン契約テストを green に保つ。
- 定数の整理のみで振る舞いを変えない。
操作