Ad
  • Custom User Avatar

    The Side property is not found neither the OutputBoard method. Can you update the description?

  • Custom User Avatar
  • Custom User Avatar

    What...? 8/
    You publish a translation while you perfectly know it doesn't contain all the tests? That's a joke... Honestly, I'm tempted to reject it, now, just to make the point.

    Well, do your job...

  • Custom User Avatar

    thx.

    I made a last check and something seems weird, tho: only 13 tests in the test caeses while there are supposed to be 24. I checked and you have only 13 assertions, no multiple assertions in the same test. How is that?

  • Custom User Avatar

    Good for me. Err... I didn't see there was equals too. Could you remove == and != from the description too?

  • Custom User Avatar

    Might be, but again, the idea is to make the versions consistent.

    I'm sorry but, to me, it's out of question to approve your translation as long as your Pos structure does some work for the user (meaning: holding the data, ok. Equality, hashing, difference, why not. But difference/addition: that's out of question). But now, considere this: if you do not provide the functionnalities for Pos, what will do the user? Build is own structure, so yours has no use anymore. That's why I provided ints only in the java version.

    Designing appropriate tools is part of the job of the user, especially when it comes to harder katas. Why not giving him raw data, so?