csherin/mazes — Issue #4: Constants/variables instead of accessing the hash directly
Author: PragTob
Created: 2012-01-20 19:06:28+00:00
State: open

Overall in orthogonal and sigma I'd rather see you define the hashes/arrays as constants, and then access the constants. If they are variable at first put the in a variable and then access them. Directly accessing the constructed hash looks a bit weird to me.

For reference, I mean things like this:

{ :north => :south,
:south => :north,
:east => :west,
:west => :east }[direction]