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.
C#
In Solution section: typo in the method comment : "// retrun" > "// return".
JavaScript translation that enforces in-place mutation
the kata would be made more interesting if it enforced in-place mutation of the list instead of allowing pure funtions to pass (that circumvents the bug entirely)
I suggest adding test cases where the date of birth is exactly n years before the current date and is in a leap year
please consider improving the description by providing more specific details. the current instructions are unclear and could be refined for better understanding.
wording is too C#-specific (and is probably not an appropriate type for C#)
additionally, description does not state that the given positions are for a knight and a bishop, the description thus does not state what the problem to solve is. those pieces could be pawns for all I know from reading the description
it is also a very strange and indirect way of stating that only two pieces exist on the board
It could instead read something along the lines of:
You will be provided with the positions of a knight and a bishop on a board that is otherwise empty (e.g.
[4, "C"]
and[6, "D"]
).Not the case for Ruby
This comment is hidden because it contains spoiler information about the solution
Groovy translation
I feel like the method name for the Groovy translation should be
whatIsTheTime
in camelCase instead of PascalCaseThis comment is hidden because it contains spoiler information about the solution
The JavaScript translation uses PascalCase for the name of the function (SubtractSum), but according to JavaScript naming conventions it should be using camelCase for this (subtractSum), this should be corrected both in the default code and in the tests.
EDIT: I also got it to work by including an additional default (parameterless) constructor.
I had a lot of issues with this one trying to use a constructor to initialize the name property.
i.e, I had to remove:
public PersonEventArgs(string name)
{
Name = name;
}
And pass the name by doing:
ContactNotify?.Invoke(this, new PersonEventArgs(){Name=name});
Any attempt to pass name resulted in a CS7036 error depicting there was no parameter 'name' of 'PersonEventArgs.PersonEventArgs(string)'
C#: method name should be
PascalCase
(Please refer to implementation of backward compatibility here )Actual and expected are swapped in PHP
Loading more items...