Yo1k / tic-tac-toe

A multiplayer turn-based game
MIT License
0 stars 0 forks source link

Consider passing `ActionQueue` to `Logic.advance` instead of storing it in `Logic` #21

Closed stIncMale closed 2 years ago

stIncMale commented 2 years ago

Closing, as this actually can only make things weird: if Logic.advance takes all action queues, then so does World.advance, and those queues must be provided by the UI component. This means that the UI component will have to have references to all action queues (the UI event loop calls World.advance). This is worse than what we currently have (when Logic has private references to all action queues), because currently the UI component needs to have references only to those action queues that are actually filled by the UI, i.e., correspond to players behind the peripheral devices.