Ad
  • Custom User Avatar

    Local tests are not (or wrongly?) implemented.
    Edge cases are not in description.
    Return value for edge cases does not make sense (should be distinguishable from not-a-collision).
    Zero radius circles are simply points, which can collide (I do not see your point. nested circles collide, don't they? what's different when the inner circle has a zero radius?).

    Also, unnecessarily hard to get at values. Adds nothing to kata logic.

    When all the fluff is handled, it's a nice little kata that showcases a basic piece of math. But there's more fluff than business logic here.

  • Custom User Avatar

    Java version needs random tests. Also, some of the tests appear to say interesting instead of intersecting.