I like this one
it's very helpful, but not required. You can solve the kata without recursion if you wish.
This comment is hidden because it contains spoiler information about the solution
This was fun! Thanks
Very rarely, the random tests will generate mazes with no possible exit path, e.g.
[ [ '_', '_', '_', '_', '_' ],
[ '_', 'W', 'W', 'W', 'W' ],
[ 'W', 'W', 'W', 'W', '_' ],
[ '_', '_', '_', '_', 'x' ],
[ 'W', 'W', 'W', 'W', 'W' ] ]
which isn't supposed to happen.
nice kata tho :)
Inspections are disqualified for no reason
I burned for three hours
Seems very much a duplicate of other existing maze katas. Values array seems totally unnecessary. When I messed up values format, it gave me some cryptic message and didn't really help at all.
If you modify your input, you deserve whatever you get.
That said, it would be better to explicitly bite off your head for it.
Yes you're right. These may be considered extra by requirements but those are really the best practices IMHO. I've cited safe coding & old browser support as reasons.
I may understand you wrong, but to me it seems you want to add code, that is not covered by the requirements. If there is quoted that you have an array with something in, you should not check for null.
I know array is full of junk but for safe coding atleast there should be a null check else this code will crash. Also some old browsers like IE8 will not support .indexOf() method on array.
You aren't really testing anything in your requirements. The tests woefully inadequate.