Yes, I found that one after I wrote my Kata. For some strange reason, it wasn't shown when I searched for "Hamming" before I wrote the Kata. But I'll make sure not to write a duplicate.
Thank you for your feedback (also to my other Kata)! Happy to hear that! I will keep your suggestion in mind and also extend the random tests for this one.
Thanks. But I would also remove the "index 31 for MSB" for two reasons: For 32-bit integers, if you start with index 1 for 1st bit, the 32nd bit would be index 32, not 31. And for dynamically typed languages such as Python, the parameter depends on the type passed to the function - could also be a 64-bit long and therefore the MSB would be index 64. Not a real issue, but could be confusing, I think.
As others reported already, the parameter 'list' should be renamed. And I always think it's a good idea to have an example in the description, even for simple tasks.
Yes, I found that one after I wrote my Kata. For some strange reason, it wasn't shown when I searched for "Hamming" before I wrote the Kata. But I'll make sure not to write a duplicate.
I mark the issue as resolved now.
It would be most appreciated!
Approved your translations. Thank you for your work! :-)
Thanks for your feedback. I changed the description.
Thank you for your feedback (also to my other Kata)! Happy to hear that! I will keep your suggestion in mind and also extend the random tests for this one.
Ok, now I think I have it right. Thanks again for your help!
Have another look.
Good point :-D I will find out how to and do that. Thanks for the advice!
Good to know it wasn't my problem ;-) Thanks for your answer!
Thank you! :-)
Thanks. But I would also remove the "index 31 for MSB" for two reasons: For 32-bit integers, if you start with index 1 for 1st bit, the 32nd bit would be index 32, not 31. And for dynamically typed languages such as Python, the parameter depends on the type passed to the function - could also be a 64-bit long and therefore the MSB would be index 64. Not a real issue, but could be confusing, I think.
This comment is hidden because it contains spoiler information about the solution
I would change the description for the bit index a bit: Something like "the index of the least significant bit is 1".
As others reported already, the parameter 'list' should be renamed. And I always think it's a good idea to have an example in the description, even for simple tasks.
Loading more items...