Ad
  • Default User Avatar

    Thanks for your feedback. I've taken them into consideration, and even started to edit the Kata accordingly, but soon found myself being redundant.

    1. The requirements on the input are clearely stated as "non-negative integer". "Valid" means that the input complies with the requirements specified. There is nothing to add. Non-negative integer means n >= 0, so adding that is redundant.
    2. I made a change to specify the bytes are represented by a string of '0' and '1' of length 8.
    3. Examples are given by the exmaple test cases. To repeat the test cases is redundant. A code warrior should be able to code to test cases. That's part of a programmers job and part of this kata's challenge. Suggest a higher kyu if you think this adds to the difficulty.
    4. I chose to add a random test. If anything in the description was not clear, this test case provides the necessary details. The test case does not provide the solution; it only validates the solution once found.

    I'm marking this issue as resolved.

  • Default User Avatar

    There are a couple random string tests implemented, not names though, that would be much harder :)

  • Default User Avatar

    I fixed it! The function should return None when given an empty string, or a string less than length 2.

  • Custom User Avatar

    Python PEP8:

    Function names should be lowercase, with words separated by underscores as necessary to improve readability.

    mixedCase is allowed only in contexts where that's already the prevailing style (e.g. threading.py), to retain backwards compatibility.

    Example:

    def string_color(name):
    
  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Default User Avatar

    Really?! What part is ambiguous or not clear? What exactly did you misunderstand? Do you have any suggestions for better text?

  • Custom User Avatar

    Could use a better description, rather than just leaving all the explaining to the test cases.

  • Custom User Avatar

    The description could use some proofreading. I noticed some syntax errors, such as double negetives, and things could be better worded. Also, I am wondering if it is possible to do tests that have multiple possible solutions, instead of flat out stating that you are not doing it.