Ad
  • Default User Avatar

    I like your solution it takes really small amount of time in comparison with string format operation. My solution takes 3-4 times more than yours 🙂 but current one with string format takes 2-3 times more than mine 🙂. String formatting really heavy operation even in this case when we have to format 10 numbers only

  • Custom User Avatar

    Thank you for this feedback, I've been wondering where the sweet spot of documentation is. I've had the idea of explain like no one has any idea what anything does. But it does seem to annoy seasoned developers. I've been playing with the idea of writing comments on blocks of code instead. My level of discernment is not high enough to figure out what doesn't need be said, yet.

  • Custom User Avatar

    Looking at your solution it looks like you just like to add documentation for the sake of documentation, even when it results in illegible code and a ton of duplication/stating what is obvious even to novices. I won't claim the top solution to be best practices, but less is more in this case when the solution can be whittled down to a single standard lib function call.

  • Custom User Avatar

    Super clever absolutly, Best Practices not even close. Where is your documentation.

  • Custom User Avatar

    I brute forced my way with programming constructs through this one but seeing the answers that took the far superior mathematical approach really opened up my eyes. Excellent kata to showcase a time where math really does matter (specifically knowledge of math formulas).

  • Custom User Avatar

    The golang version only covers array types of []string{} and make([]string, 5, 5) but not for [5]string. If you try to use [5]string the test fails.

  • Custom User Avatar

    Instructions were not clear about the random sets of data containing towns not found in the strng parameter, as well as expecting -1 to be returned rather than 0 when it happens. Was great debugging practice though.

  • Custom User Avatar

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