Ad
  • Custom User Avatar

    Why only 1 random test case? Please add more test cases.

  • Custom User Avatar

    Ok i see moust of the votes go for 6 kuy. I will change the expected rank to 6 kyu.

  • Default User Avatar

    i think its 6 kyu

  • Custom User Avatar

    Thank you i fixed the non-integral inputs and the function is now in camelCase standart.

    It has a section for the specific languages, but i just realized it's not working. It is just showing them all.
    I changed the names of the sections to java , python and javascript

    And it is now working for Java and Python, but when JavaScript is selected it shows Java too.
    How can I fix that ?

    I will probably remove that restriction (x+y<=27) and say that the matrix builds only with the alphabet chars.
    Because it's more like a note to tell people there is no need to check just start from 'a' and go on to 'z'. Just like you did in your solution.

  • Custom User Avatar

    It passes non-integral numbers as inputs, it doesn't implement the specs ( x+y<=27 ), the description is still bad, the examples aren't language-specific, the initial code has a spelling error in the funtcion name, said function name doesn't adhere to camelCase standard, the example tests are too limited to catch solutions that switch x and y, the output datatype should be a 2D array of strings instead of a string.

    All in all, not too good. Neither the kata itself nor the translation. Consider focusing on solving two month old issues instead of creating new translations.

  • Custom User Avatar

    JavaScript translation added. Any feedback is welcome.

  • Custom User Avatar
  • Custom User Avatar

    Changed the random cases. This is the problem with random rows and columns generation. It should be fixed now. Please verify it.

  • Custom User Avatar

    Should be clear from this message:

    .... should equal '[[aaa, 
    ....
    ....
    ....
    [jjj, jkj, jlj, jmj, jnj, joj, jpj, jqj, jrj, jsj, jtj, juj, jvj, jwj, jxj, jyj, jzj, j{j, j|j, j}j, j~j, j\x7fj, j\x80j, j\x81j]]'
    

    ord('\x81') - ord('a') = 32(where a was mapped to 0)

  • Custom User Avatar

    x + y <= 27.

    This requirement is not satisfied in python.

  • Custom User Avatar

    Approved. Thank you for the translation.

  • Custom User Avatar

    Please review the random test cases. of my translation.

  • Custom User Avatar
  • Custom User Avatar

    You are right. Im fixing that.

    Formatting added.
    Hope it looks better now.

    Thank you for your advices.

  • Custom User Avatar

    Row 0 -> "a", row 1 -> "b", row 2 -> "c" ... row 26 -> "z"

    Here row 0 is "a" means "z" should be row 25 as you started with 0.

    Also you should format the example above

    MatrixPalindromes.createMatrix(3,3)
    Should return a string representation of the matrix : "[[aaa, aba, aca], [bbb, bcb, bdb], [ccc, cdc, cec]]".

    To somewhat like

    MatrixPalindromes.createMatrix(3,3) --> "[[aaa, aba, aca], [bbb, bcb, bdb], [ccc, cdc, cec]]".
    

    because without formatting it looks..... kind of Boring?

  • Loading more items...