Cool kata...

cpp test 3:
Assert::That(name({"give", "your", "kata", "a", "name"}), Equals(13)); // giveyourkataaname
I believe "giveyourkataaname" has 17 characters, the comment is wrong.

How can you have 20 strings with unique lengths between 1 and 10 ??

Luckily, your friend Tóḿáś has ...

Yep, fixed -- thanks!

Sorry I misread your question. You're right, that was a typo. It's been fixed :)

The length of "givekataaname" is indeed the correct solution. Although "giveyourkataaname" is longer, it doesn't meet the given requirements, because its weight is 189 while its length is 17, and 189 </= 170.

The comment for this test is perhaps wrong and misleading:

@test.it("Test 3")
    def test_case():
        test.assert_equals(name(["give", "your", "kata", "a", "name"]), 13) # giveyourkataaname

Don't believe the comments ;-)

• giveyourkataaname - length: 17, value: 189 (> 10 * len) --> incorrect
• givekataaname - length: 13, value: 110 (< 10 * len) --> correct

So that comment is wrong/misleading

why 'your' included? 'y' = 25, 'o' = 15, 'u' = 21, 'r' = 18, lenght 'kata' = 4:
25 + 15 + 21 + 28 > 10^4
can someone explane?
also why included 'give', 's', 'k', 't' in sample tests on Python

Interesting kata!

How is it possible the name in Test3 pass with a length of 13?
In the comment it is assumed the whole string which has size 17.

Assert::That(name({"give", "your", "kata", "a", "name"}), Equals(13)); // giveyourkataaname

Pretty interesting Kata.
Am I the only one that thinks that this actually is a bit harder then 6 KYU in general?
Atleast from c++ pov I'd rate it 5

All in all, you should lower the number of tests, not kata expectations;

C++ fork with fixes

• Resolves this issue
• More sane initial solution
• Improved random data generation
• Improved code formatting
• Added missing headers
• Added assertion messages
• Removed useless preloaded part
• Removed using namespace std; from test cases

Please review and reject/approve.