Ticket #61 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

prototype: State refactoring

Reported by: jelinson Owned by: rthomas
Priority: minor Milestone: milestone3
Component: component1 Version:
Keywords: Cc:
Blocking: Blocked By:

Description

returnUpdateStats is confusing. It should haven't return in its name. Either return the spec as is or define a getter method. Second, is the negative 50 the dollar amount you get each turn? If so, that should be a constant. Lastly, why does it make a new spec instead of using its own?

Change History

Changed 2 years ago by ecoughlan

  • priority changed from major to minor

Changed 2 years ago by rthomas

  • milestone changed from milestone2 to milestone3

Changed 2 years ago by jelinson

  • owner changed from somebody to rthomas
  • status changed from new to assigned

Changed 2 years ago by rthomas

  • status changed from assigned to closed
  • resolution set to fixed

The reason it doesn't use its own spec is because the game state does the calculating and it doesn't have the changes from during the turn. So it creates a change spec for the turn state to use.

Note: See TracTickets for help on using tickets.