Ad
  • Default User Avatar
  • Default User Avatar

    I enjoyed this kata since I'm a big fan of regular expressions, But it's more suited at 5kyu instead of 4.

  • Custom User Avatar

    That was fun :-)

  • Custom User Avatar

    Ruby version kumited, please check and comment or approve.

  • Custom User Avatar

    Which language did you solve in?

  • Default User Avatar

    There is a strange requirement: "true123 is an identifier, as opposed to boolean followed by integer".. Looks like foreign for this task. Actually I ignored it (did not check and maintain this case). And my solvation passed all tests! May be you should remove this requirement from kata description?

  • Default User Avatar

    Oh, you are right.

  • Custom User Avatar

    ... yielding Token items until there are no more items to yield, at which point it should do whatever the appropriate action is for your chosen language.

    I don't know Java, but a quick search reveals that next() should throw an exception when there's nothing more to return. Since this is the official spec of the interface, I'd say it's rather more obvious than not.

  • Default User Avatar

    To pass "testException" in Java you have to throw an exception in "next()" if the string is empty, which is not obvious. Returning null or new Token(null, null) in "next()" or false in "hasNext()" doesn't work.

  • Custom User Avatar

    approved

  • Custom User Avatar

    Rust translation ready for review.

  • Custom User Avatar

    Also (probably) needs new fixed tests to match new spec.

  • Custom User Avatar

    Also (probably) needs improved fixed tests to match new spec.

  • Custom User Avatar

    You seem to have outdated sample tests. You can either:

    • Reset the trainer (removes your code, so save it)
    • Ignore the sample tests and just hit "attempt"
  • Custom User Avatar

    I have gathered input and tuned the details of my fork, then went ahead and approved it.

    • The description is now updated for all languages.
    • Python has new fixed tests and added random tests and will probably invalidate a few solutions
    • Other languages need to be updated and brought into alignment, but they already have separate issues.
  • Loading more items...