VEGO-Engine / Engine

2 stars 0 forks source link

Generalize helper functions #71

Open Nanogamer7 opened 4 months ago

Nanogamer7 commented 4 months ago

Some functions like AssetManager::calculateSpawnPosition() were originally just a helper function for one single use case, but probably have more than one use case. This needs to be evaluated for each helper function individually. Furthermore, helper functions that can be reused also need to be documented, and helper functions that cannot be reused should be set private.

ineslelin commented 1 week ago

found only one other helper function that could have multiple use cases during my search, calculateType() in AssetManager. It's used specifically for Powerups, but could probably also be used for other items or entities