Ad
  • Custom User Avatar

    Yeah for the JS kata I'm 100% sure my code correctly does what it's intended to. But the test cases are broken. If I make the result year less than the year they ask for, I get "expected 3 to equal 4" and if I make it less than or equal to, I get "expected 51 to equal 50". So, forget it. Seems like author put the wrong expect value in the tests. Or maybe there's some secret catch to the problem that I missed in the instructions. It's a shame Codewars doesn't show us the test inputs so it's impossible to debug why it failed. Anyway I agree with you, moving on!

  • Custom User Avatar
  • Custom User Avatar

    Hopefully someone else can approve this, I don't know jack about C#.

  • Custom User Avatar

    I haven't been using codewars lately, but I don't really see how performance test cases can be reliable in the kind of environment codewars provides. You can't predict the power or specs of the servers the test cases are going to be running on, or outside variables going on in the infrastructure that might have an effect on the performance. It's not a controlled environment.

    From the email notifications I've been getting, the other implementations seem to have more issue with the test cases not being reliable in general as well. JS version covers its intended scope.

  • Custom User Avatar

    How would bigger arrays make it harder? More dimensions would be interesting though!

  • 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

  • Custom User Avatar

    FYI the random tests will most likely generate some "words" to check that are longer than the maximum possible word length (so i.e. if the board is 4x4, there will be some 17+ letter words thrown in). The contents before the maximum length may or may not be part of the board. Also FYI the board size is random in the random tests, it can be anywhere from 4x4 to 10x10. I didn't have any issues with my solution after running the random test cases many times, so I would like to see yours. If it's not your solution then I will look into the test cases, maybe the random ones are formulated with an incorrect data type somewhere.

  • Custom User Avatar

    How exactly are the random test cases not working? Also can you also show me your code via pastebin or repl.it or something?

  • Custom User Avatar

    Since the ruby version was adding the depth method to the array class, I decided to do it the same way for javascript... however it would be more straight forward if it were just a function that took an array parameter.

  • Custom User Avatar

    Alright, I guess I just wait then ;)

  • Custom User Avatar

    I created a javascript translation, how do I get it added to the kata?

  • Custom User Avatar

    Great solution, I think it's the most readable one so far. voted best practices.

  • Custom User Avatar

    You're right, I didn't know the terminology for it! Glad you enjoyed it

  • Custom User Avatar

    I think it would be too ambiguous otherwise

  • Loading more items...