- marked as enhancement
Fleet.ShipsとShipStatus.Slotをの置き換え
Issue #136
resolved
Fleet.ShipsとFleet.ActualShips、ShipStatus.SlotとShipStatus.AllSlotは用途の違いがあいまいなので可能なら統廃合して、新規のコードを書くときに迷うことがないようにしたい。
廃止 | 存続 | 名前変更 |
---|---|---|
Fleet.Ships | Fleet.ActualShips | Fleet.Ships |
ShipStatus.Slot | ShipStatus.AllSlot | ShipStatus.Items |
ダメコン消費にバグがあってはならないので #153 を先に実施する。
Comments (6)
-
reporter -
reporter -
reporter - edited description
-
reporter ItemStatus[] から int[] に変更すると、夜戦時ダメコン消費と帰投後の戦況表示に影響があるので、引き続きItemStatus[]を保持しつつ、用途別にメソッドを用意する。
今の作業ブランチは閉じて新しく作り直すが、ここまでの作業で出てきたリファクタリング内容を先行して実装しておく。
-
reporter -
reporter - changed status to resolved
- Log in to comment