markszcs / starcraft_themed_maze_game

"# starcraft_themed_maze_game"

starcraft_themed_maze_game's Issues

Clean up the code

The current code is quite hard to understand, the variable naming doesnt follow naming conventions.
eg.: the id property of an object should be unique, there are plenty of theese in the code.

I can directly define attributes to the td and then set its value like td.__variableName = "example value",
i dont have to create a whole new class to represent the td in order to keep track of the state of the game.

Fix the movement system

The current movement system is quite dumb while also is complicated.

The fixed version should probably use the object.nextSibling attribute or the td itself could have a __posX and __posY attribute in order to identify the neighbour of a td. And the event listener on the td should call the function of the movements system with the argument of the clicked td in order to decide whether or not the movement is legal.