Draft
Arrays basics
276 of 283goodmove
Loading description...
Arrays
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.
would be nice to have simple test there, also description is not perfectly clear, could use some explonation or background story to make this interesting
Would be worth adding some tests. Also I kind of agree that leaving out the loop skeleton would make this a little more interesting.
Very simple, and no method signature to run tests. Seems incomplete.
I am not able to submit this kata. I copied several previous solutions, but it always gives me an "Unknown error."
EDIT: This may be because I am on a school computer with a very strict wifi. I will try at home and see what happens.
Agree that leaving out the loop skeleton, making the number of arrays arbitrary, and setting up some test cases would all improve this.
You can never be too verbose when introducing an exercise...
Take out the loop skeleton and update it so that the function can add an arbitrary number of empty arrays. Good beginner kata that might look harder than it is.
Better set of tests and actual data for the dynamically created array would be nice.
This comment has been hidden.
Seems to lack further test cases
My suggestion would be to leave out the loop skeleton entirely, requiring the warrior to generate it. I enjoyed this kata!
Very basic kata, comments should be part of the description.
No test cases.
I'm fairly new here, but shouldn't tests be provided?
Didn't really make sense. Why do we need the for loop? Didn't have clear expectations.
not to clear why the for loop is there, i removed it and was able to pass with
var A = [[2],[3],[4]];
This comment has been hidden.
Too simple.
The description isn't really clear. You should switch the comment to the details part.
I would put a more appropriate and less-generic title.
Your tests are only checking that the resulting array contains at least three sub-arrays.
Based on the description it sounds like you're looking for exactly three sub-arrays.
Thanks for the comment. There definitely was the mistake you've pointed at. The issue's fixed