Beta
Generate keywords
Loading description...
Algorithms
Strings
Arrays
Parsing
Fundamentals
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.
getShortTextis never tested except in random tests.It is not specified how words with the same frequency should be ordered in
getKeywords.The expected and actual values are reversed.
Fun kata , however i feel like the instructions are not complete. 'most frequently used words in the text, then less used.' What if words occur the same amount of times?
Also in your first testcase the word 'the' occurs 11 times but is not found in your expected result.
To the description should be added: "Include only words 4 or more characters long that appear at least 2 times in the string"
A good subject of problem ... but seems to need more tests (random tests could be interesting way to avoid people (like me ;-D ) to 'circumvent' the solution.
Resolved. Thank you for a good advice :)