epsilon-phase / Symmetricality

More symmetry for everyone :)
GNU General Public License v3.0
4 stars 0 forks source link

Refactor PlanRenderer #31

Closed epsilon-phase closed 8 years ago

epsilon-phase commented 8 years ago

PlanRenderer is a monolithic class whose functionality is frustrating to extend. This is due, in part, to the fact that it has to update all of the different rendering layers by itself. These objects could, with relative ease, be shoved into their own classes.

PlanFacet would be a good name for the base class, the logic could be rolled into a subclass such as BuildingFacet and DesignationFacet.

It also is painful to fix the display because of how poorly the Hud class and the PlanRenderer class interface with each other.