Beta
simpleLog
18wthit56
Loading description...
Algorithms
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.
The example solution seems to special case for the possibility that the
valueOf()
anObject
isnull
. This is not tested with.Primitive numbers are, apparently, treated the same as primitive booleans. But they're not specified anywhere.
Also, the part about how strings should be formatting needs serious reworking. Even running the real test with dummy code and observing the test cases is more helpful than the current descriptions.
I've added an explanation regarding primitive numbers.
Is there any feedback you can give me about the description beyond "fix it"?
The rule is very simple: wrap strings around a pair of
"
, and escape all"
and\
in the original string. Somehow it turns out cryptic like in the current description.Needs example tests and random tests
Tests check for anonymous function as [anonymous] while description says it must be [anonymous function]
Thanks for the heads-up. I've fixed the description to say
[anonymous]()
.