search

jukkae / rascal

A roguelike.
Other
1 stars 0 forks source link

Map creation should return a new Map instance #224

Closed jukkae closed 5 years ago

jukkae commented 5 years ago

This will probably be a somewhat larger refactor, but it'll absolutely be worth it. There are several bugs related to level creation and this should fix that automatically.

jukkae commented 5 years ago

Map vs. World? Instantiation and initialization in GameplayState vs. Map vs. MapUtils?

jukkae commented 5 years ago

Briefly, I think World should be a representation of any given level as a whole: Map, as well as NPCs and items, environmental conditions, etc. Map, then, should only just be a representation of the permanent features of the environment: Walls, floors, water, etc. Consequently, MapUtils should more properly be called WorldUtils – though, to be fair, this whole "utils" thing is crap and should be refactored into something that actually makes sense.