What the actual fuck… I only got two pages down and already found several red flags that I would never accept in any code review. Not the least of which is that when querying an edgeless graph for the shortest path from node A to node Z, “the empty path of length 0” is the exact opposite of a correct answer.
So thanks for the link, I guess. I’ll keep this as ammunition for the next time someone quotes Uncle Bob.
Damn, indeed. The Uncle Bob people (or, really, any "this book/blog post/whatever says to do technique x" people) are my absolute least favorite. This is a good riposte. Or, alternatively, if they don't understand why it's bad then you know they're a shit coder.
So thanks for the link, I guess. I’ll keep this as ammunition for the next time someone quotes Uncle Bob.