Ad
  • Default User Avatar

    Dear Smileous,

    The logic of your algorithm is quite right, it would generally work absolutely the same way as kanonical BWT, the difference is in the fact that canonical BWT should be expected to work not with letters of text, but with bytes and their values where character 'a' and 'A' are quite different letters having different ACII code values, while your algorithm implement some default C# comparison method with specific comparison logic.

    In order for you to fix your code you shall try to adjust default for C# sort order and to ensure that texts are sorted based on ACII codes of relevant characters.

    I assume that remark regarding sort order would be useful in KATA Description, but it is absolutely not mandatory, and would be not fare to consider it to be KATA issue.

  • Default User Avatar

    Throws "System.ArgumentNullException" when "a" is "null". It's worth adding "a != null"

  • Default User Avatar

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

  • Default User Avatar

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

  • Custom User Avatar

    Not a kata issue. See the list of forbidden chars, /, *, 0 and 1 are in your code, they count even as a comment.

  • Custom User Avatar

    I don't know if it was just me but something was wrong.
    The kata is good nonetheless but the issue is Value error.
    my code was the same as everyone and contained other code examples in comment.
    My code outputted 5 but Value error when there was none.
    I went ahead on other sites with the same problem and they worked.
    Just a heads up.

  • Custom User Avatar

    This was very fun. I've coded a one-liner solution which I thought was very clever, but as I checked other's codes,,, just wow. Mine was garbage

  • Default User Avatar

    haha it was fun

  • Default User Avatar
  • Custom User Avatar

    Don't spoil the solutions please. I marked your message with the spoiler flag.

  • Default User Avatar

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

  • Default User Avatar

    That was awesome. Definitely in the top 5 ;)