7 kyu
Return pyramids
641 of 1,381josemanuel.rincon
Loading description...
Strings
Algorithms
ASCII Art
Puzzles
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.
This comment has been hidden.
updated in this fork
Resolved
python new test framework is required. updated in this fork
Approved
Description is inaccurate: "an extra line feed character is needed at the end of the string." should be "a line feed character is needed at the end of the string." Because there are never two line feed characters at the end of a string.
Done in this fork
JS: Node v12 should be used along with its appropiate assertion tools (Mocha + Chai). Refer to this and this
Updated to Node v12
Ruby 3.0 should be enabled.
Enabled in this fork
After testing it shows expected result totally similar as an actual but still don't accept it, does somebody knows where can be the problem?
Make sure the string ends with a newline.
make sure you dont pushing empty space in every line try to pop_back first ' ' in each line
make sure you dont pushing empty space in every line try to pop_back first ' ' in each line
This comment has been hidden.
C translation kumited
@josemanuel.rincon
please scrutinize for approval, thank you
domo!
Enjoyed this kata very much but think its difficulty is more of a 6kyu )
The rankings for kata are all over the place. What you have to know to actually code the solution for this is 8 kyu as it is a simple string return. But figuring out the amount of each is harder. A lot of the Kata's are very algorithm specific. If you have solved a similar problem before they are trivial, if you have to come up with a solution they can be very difficult.
Kata ranks cannot be changed.
Sorry to insist, but... Translations are here and you only approved one :D
i just now have approved all translations
Lovely, thanks.
If you can also remove the -1 to my posts [not telling it's you], even better :D
JS, Ruby, Crystal and c++ translations kumited; thanks for your kata :)
If that special case with n = 0 is not removed then it should be in translations too.
Let's see if the author will give me some feed in this regard, then.
Actual tests has
test.it("should have the value 'codewars' inside of it")
, which makes no sense. Please replace it with something that makes sense ;-)removed
This comment has been hidden.
All done.
Note: the default return statement for
n=0
is now"\n"
(not""
, to be consistent with the other cases)EDIT: note about the last line feed char added to the description.
approved to 7 kyu according to the average, but not sure it's a good idea, considering that those are 6 kyu:
How? It's just the opposite: 0 lines but 1 '\n'.
Hi,
Mmmmh, yeah, you're actually right. Well, even if it's weird, it's part of the specifications, now. Let's see what the author decide about that: if he prefers this behavior corrected, I'll do it (or anyone else).
Hi! Its right, and fix it
ok, I just changed the internal solution and the tests... But it seems the server is not up to date yet. :/
Mmmmh, well. Jose changed the range of the tests to exclude the
0
value, actually. So now, both solutions are good.Fixed
Kind Regards