Loading collection data...
Collections are a way for you to organize kata so that you can create your own training routines. Every collection you create is public and automatically sharable with other warriors. After you have added a few kata to a collection you and others can train on the kata contained within the collection.
Get started now by creating a new collection.
This comment is hidden because it contains spoiler information about the solution
Maurice,
Ahah! You are right. I'm pretty sure I meant to flip that back and compose it with
not
.Either way, really appreciate the comment!
Best,
Jeff
Thanks ;) Just fiddling around trying to learn things!
Changed the teamname!
Thanks for the comment
It's a less elegant attempt at what you do with the Hits and BattleOutcomes. Glad to see my implementation does have a context-specific advantage.
Thank you for pointing that out! I'll have a look ...
Thanks! My goal was to make the entrance code as clean as possible and really easy to read.
Thanks four your feedback. I didn't look at it in this way. :)
Thanks. Your points are valid. However, it was my goal to make only number-based scoring. Kata requirements are met and I did not intend to add things which are unnecessary.
Hi there!
Many thanks for reading my code and giving usefull critics and suggestions. I agree on every point...
...I just don't understand the "added bonus" part. If I convert the enum value to a string wouldn't that be a number anyway (visually talking)? For example:
Result.Win.ToString() should be "0" no? Maybe I just don't get what you wrote.
Thanks again for the kind reply!
Nicola
Thank you for the comment.
I agree with you regarding the CompareWith() method. That's what happens when you just want/need things done quickly.
It would definetely need more work if it were to be used in the future.
Good point, its indeed a sum and not an assignment.
Same solution? Ha! Ur looks fine to me as well :-)!
Thanks for the feedback, good point. I wasn't aware of that.
Thank you kindly for these remarks, I'll be sure to keep an eye out for these issues in the future :)