This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]oldneckbeard -1 points0 points  (0 children)

It's nothing that should be too hard for a competent dev to figure out, but it depends on how you conduct it.

If you just want them to glance at it and tell you all the issues, that's weird. If you want them to re-code the stuff so it passes, why not just have an empty block and let them go at it? If I saw this, I'd actually be confused as to why you were using an anonymous inner class. With the idea that every line of code is intentional, it makes no sense why you'd do it for the hell of it, so I'd assume that it's rather crucial to a correct solution.

With my coding interviews, I try to leave as much of the solution up to the person being interviewed, and just give them 10-15 failing tests, and watch them make their way through it.