Ad
  • Default User Avatar

    Go version is missing package declaration

  • Default User Avatar

    While this solution is clever I cannot see how it warrants the best practice up votes as this problem can easily be solved without any conversions between strings and integers.

  • Default User Avatar

    The go version returns an error as the second value and imports the errors package -- both of these are unnecessary. First, if you don't use the errors package the code fails to compile. Second, I see no reason for the error value as the tests don't test for it and there are no requirements to return errors. In fact it's so useless you can change the signature to Dative(word string) string and the tests still pass.

  • Default User Avatar

    I experienced a similar issue but it was because I was modifiying the input slice with an in-place filter. When I switched to a secondary slice for the return values the problem went away as you would expect.

  • Default User Avatar

    The description has a link to the wikipedia article on summation at the bottom which should help. This kata is far easier than the level suggests so don't over think it.

  • Default User Avatar

    The description for this kata could do with expanding.

    It does state that everything other than the phone number and name may contain erroneous non-alphanumeric characters but it doesn't state which characters are erroneous nor does it state that erroneous characters in the address should be replaced with a space.

    The example does cover it but it would be helpful if the requirements were more clearly defined.

  • Default User Avatar

    It's already been reported but the go version is missing the package statment, i.e it should have the statement package kata as the first line.