Ad
  • Custom User Avatar

    betterHand function returns a Hand, so it is not clear what to do if hands are equal.

  • Custom User Avatar

    It's lazier; it works when is is infinite. I suspect (though I don't exactly recall) that's why I wrote it that way in the first place. Of course, the requirement to return the lexicographically least solution prevents doing anything useful with infinite lists, so it got transformed into this solution.

    If you're bothered by the possibility of creating nested loops, you could manually fuse it:

    indices is = unfoldr go (0, is)
      where go (_, []) = Nothing
            go (a, _:bs) = Just (a, (a+1,bs))
    

    But I don't think that is necessary.

    The real problem here is the use of (!!), turning what should be a quadratic algorithm cubic.

  • Custom User Avatar

    OMG. indices is a very brutal way to get N sequential integers. Consider using

    let n = length is
        indices = [0..n]
    
  • Default User Avatar

    Please add more complex tests - as valid as invalid too. Current tests are VERY simple!

  • Custom User Avatar

    Actually, tests for equility need to be added:

    Normal 'x' == Normal 'x'
    Normal 'x' /= Any
    ...
    

    This will detect such cheaters

  • Default User Avatar

    Choose any your own variant, but write it clearly in kata task conditions.

  • Default User Avatar

    I've added some invalid tests, one question is what to do with ""? Is it Nothing or is it Just (Str [])? Now I test for Nothing (as that's what my solution does :D) But I can change that.. What do you think?

  • Default User Avatar

    well spotted. thanks. Fixed.

  • Default User Avatar

    cheers, I will add some invalid tests

  • Custom User Avatar
    "(a|b)*"  -> ZeroOrMore (Str [ Or (Normal 'a') (Normal 'b') ] )
    

    at the end of second paragraph examples should be

    "(a|b)*"  -> ZeroOrMore (Or (Normal 'a') (Normal 'b') )