Why doesn't "return all collumns" include ID?

OwO

Please don't do that, we are not in 1970's C code. Variables should be declared as local as possible. You are not saving up anything and the compiler will optimize it for you anyways.

Rejected because of missing random tests and assertions via string comparison.

If you want to fix this you can fork this translation.

If you look closely you see that there is a little but important difference in what you both loop over.

Because 0 was added as a test case.

Modified description to have :

Not all strings will contain digits and not all inputs will be string. In those cases, return -1.

Should be clear now.

Sorry, did not notice that you already prepared a fix so I fixed it also.

i don't understand this code

Your code is doing out-of-bounds reads and thus has undefined behaviour. You are incrementing string until you read the NULL terminator in the inner loop and then increment string again to check it in the outer loop --> out of bounds.

Not a kata issue.

Very inconsistent. You should have extracted the punctuation also to variables.

It will still run it once for every character under the hood.

It actually seems to be caused by a security feature of Codewars to strip all <style> tags from descriptions so that the styles for the SVG elements are gone when displaying. I tried to circumvent this in this fork by including it with a base64 encoded data URI as image. Sadly it seems to break the interactivity.