Loading collection data...
Collections are a way for you to organize kata so that you can create your own training routines. Every collection you create is public and automatically sharable with other warriors. After you have added a few kata to a collection you and others can train on the kata contained within the collection.
Get started now by creating a new collection.
pretty bad description, you say if pockets == nil i.e. if the first argument is nil, but what you really mean is if all the values in the hash are nil or empty arrays, or if the hash itself is empty.
This comment is hidden because it contains spoiler information about the solution
Eh, sorry, I wasn't getting notifications on this post for some reason and approved it before seeing your first comment. I published it with the power user average ranking (6kyu), since the overall average seemed a bit too high (5kyu). Are you sure it's too late to change the nil/empty array thing, though? The author's solution would have to be updated, but aside from that it's a minor change.
no, that won't change anything since currently that's the user who's converting the nils to an empty array (I mean... If the user coded it in a clever way... 'XD ). But yeah, that's what beta phase is made for, so go ahead. That way, I won't have second thoughts when approving it to 7 kyus. ;)
EDIT: mmh damn... already approved... :( Then, keep it that way instead.
Agreed. I thought of doing that, but it would invalidate pretty much all of the existing solutions. Would that be ok?
what do you think about removing pockets being nil and replace them with an empty array instead? That would make way more sense, imo.
As far as the kata is right now the design is bad. Values of two semantics are mixed together with no way to tell between the both (not to mention exluding the option to have a price deduction of less than a dollar).
Bad kata design is still bad. Re-raising issue.
The result is suspect to floating point errors because no rounding is specified. e.g:
The
Order value should be rounded to two decimal points
part is not tested.Untrue. Gonna re-raise issue.
Copied.
Loading more items...