mbkarle / mind-the-fog

Roguelike JavaScript game--mind the fog!
2 stars 1 forks source link

Breakitup #115

Closed akarle closed 6 years ago

akarle commented 6 years ago

Its time to break up dungeonAdventure.js 😢

Fixes #114

How Done

Given how dungeonAdventure was basically a whole bunch of functions calling each other (placed in no particular order), I finally split the functions up into files according to their purpose:

Going Forward

These aren't hard lines, some functions may get shifted again, but for now, I think this handles a large portion of the divide we needed / the compartmentalization.

We may want to consider removing the large number of global variables / trimming init.js (through things such as #26 ) but that is future work

akarle commented 6 years ago

Played around a bit, only thing untested is NPCs. Everything seems good!