リファクタ #466
いづ さん さんが7日前に更新
親 #179 「TODO/FIXME 棚卸し」からの派生(skin 領域)。LR2/JSON ローダのコピペ整理とプロパティの未実装対応。
## 対象 TODO
- `src/.../skin/property/BooleanPropertyFactory.java:67,70,73` — select, decide 時の実装・未実装。
- `src/.../skin/property/IntegerIndexPropertyTypes.java:153` — 64-69 を target2, target3 に割り当てたい。
- `src/.../skin/SkinNoteDistributionGraph.java:26` — 各 Texture を 1 枚にまとめ glBindTexture 回数を削減。
- `src/.../skin/SkinNoteDistributionGraph.java:130` — スキン定義側で分岐できないか。
- `src/.../skin/SkinNoteDistributionGraph.java:153` — 高速化のアイデア。
- `src/.../skin/lr2/LR2SelectSkinLoader.java:22` — コピペのリファクタリング。
- `src/.../skin/lr2/LR2SkinCSVLoader.java:338` — 固有実装の汎用化。
- `src/.../skin/lr2/LR2PlaySkinLoader.java:945` — 固有実装の汎用化。
- `src/.../skin/lr2/LR2SkinHeaderLoader.java:49` — header 読み込みに失敗したら null を返す。
- `src/.../skin/json/JsonPlaySkinObjectLoader.java:39` — プレイスキン固有の値設定。note 未定義時は通らないため場所要検討。
## 対応観点
スキン互換性を崩さないこと。 スキン互換性を崩さないこと。高速化(26/153)とリファクタは分けて進めてもよい。
優先度: 高め(一部未実装)
戻る