Beta
month name story
Loading description...
Fundamentals
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.
I saw that the issue had been addressed earlier in the discourse, however I noticed that you still have a typographical error in the Details of the kata ('converted' appears as 'cvonverted').
Fixed.
2 typos on the description.
and the number of the (day) will be cvonverted back to the name.
thanks fixed it !
If you don't want people to use
if else
andswitch
you can't just tell people to do that, you need to enforce it.You need to read user code and check if they've used one:
i tset if the user used if else switch with the test : Test.assertEquals(month("September", 12),"undefined 8");
That doesn't work.
Just to prove the point:
https://www.codewars.com/kumite/new?group_id=59b3e79b237fb349d80009bd&review_id=59b3d093d4f411aa2d000c12
its not working for me
https://www.codewars.com/kata/reviews/59b3d093d4f411aa2d000c12/groups/59b3e79b237fb349d80009bd
The issue was handled in the pre load stage.
Needs random tests.
This comment has been hidden.
You check the result against a reference solution.
but the solution is based on the inpute, if i randomly create the values i can not know what would be the result
Test.assertEquals(month("September", 12),"undefined 8");
Invalid input is completely out of specs, and there is no mention how to handle them.
This comment has been hidden.
No, that's totally unrelated. Any uninitialized variables becomes
undefined
when turned to string.You really need to practice JS basic language features more before making a kata.
Just to prove the point:
https://www.codewars.com/kumite/new?group_id=59b3e79b237fb349d80009bd&review_id=59b3d093d4f411aa2d000c12
I know but i dont think there is a batter solution. if you have one please show it. i also ask the person not to use those practices so i think it pair use. the chanses someone will inticipate this is un real and the link is not working for me
Okay, that wasn't the correct link. This is:
https://www.codewars.com/kata/reviews/59b3d093d4f411aa2d000c12/groups/59b3e79b237fb349d80009bd
That seems fixed.