Ad
  • Custom User Avatar

    Good call. I've also added an easy way to print the list so people can debug it more easily.

  • Custom User Avatar

    I think the better solution is to do it in place, but it is not important. The only thing that is not allowed (per the way the Node is coded) you cannot swap values around.

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    The random tests are now replacing mapped characters correctly.

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    I have addressed this by adding more tests (including the example you provided). I have also updated the kata description to be slightly more instructive. I would be curious to know what edge cases you found that could randomly pass even if they were wrong. Those probably indicate that my description of the requirements was incomplete.

    Thanks for the feedback!

  • Custom User Avatar

    Good catch! I have unpublished this kata while I explore the additional tests. I have a feeling that I may have to update my solution as well. Thanks for the feedback :)