Beta
Passing through jugglers
Loading description...
Puzzles
Algorithms
Graph Theory
View
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
https://www.codewars.com/kata/reviews/5db0c65918d8df0001818183/groups/5dc8b4ab2d55c30001fe5a96 This solution sometimes fails at the "tiny" random tests. A corresponding fixed test should be added.
The actual tests doesn't seed the RNG with current time so the test cases are actually deterministic.
Fixed
using namespace std;
is still required in sample tests.Fixed
Inputs are vulnerable to user functions Should calculate expected result before sending to user functions
Thanks for the feedback, this is my first Kata and I must admit that I don't quite know how this testing stuff really works ^^
Dont relay on includes by user code This should be also put in the test part
Thanks!