feat: Add Systems #4
Conversation
- Add SystemManager - Add PositionSystem - Add PhyiscsSystem
star-dash/star-dash/.DS_Store
Outdated
There was a problem hiding this comment.
is it possible to remove .DS_Stores from being committed? It should be in the .gitignore, not sure why it's being tracked here
| var dispatcher: EventModifiable { get set } | ||
| var entityManager: EntityManager { get set } | ||
|
|
||
| func update(by deltaTime: TimeInterval) |
There was a problem hiding this comment.
Can I clarify what is the role of this method? I'm a bit confused because I noticed that not all systems (like PositionSystem) override this method. And in PhysicsSystem, the update method basically sets all the force vectors to 0. Under what circumstance is this method called?
There was a problem hiding this comment.
It more like for timed components in the game, e.g. those that after a certain time will stop working like powerups
|
|
||
| protocol System { | ||
| var isActive: Bool { get set } | ||
| var dispatcher: EventModifiable { get set } |
There was a problem hiding this comment.
What does this dispatcher do ah? Does the dispatcher represent an object that can receive requests/events and route them to appropriate methods? If so, I think it has a different set of functionalities from an EventModifiable. Perhaps a new protocol called EventDispatcher can be created.
There was a problem hiding this comment.
Also, do all systems require a dispatcher? If not, perhaps this property can be optional
There was a problem hiding this comment.
yep can make an EventDispatcher. Can make it optional too
Changes made thus far