Ad
  • Custom User Avatar

    hamiknk: that's for Go, while Khath1d3 used C#.

  • Default User Avatar

    you can debug test cases with next lines
    func QueueTime(customers []int, n int) int {

    fmt.Fprintln(os.Stdout, "Input:", customers)
    fmt.Fprintln(os.Stdout, "Tills", n)
    

    ...

  • Custom User Avatar

    thanks i think i got it ,you can confirm for me

  • Custom User Avatar

    The fact that u had to modify means that it not a duplicate

  • Default User Avatar

    This is virtually an exact duplicate of Sort the odd
    , as I only needed to slightly modify my code here to make it pass there.

  • Default User Avatar

    You can study the format / structure / syntax of higher quality kata and see how their random tests are built in your language.

    Also, you can review the available documentation on this subject.

  • Custom User Avatar

    sorry about that.This is my first Kata been trying to find out how to create a random test for hours now still no luck ,would appreciate it if you point me to the right direction.

  • Custom User Avatar
    • Casing still not correct (the current one is camelCase, instead of PascalCase)
    • No random tests
    • Please don't resolve issue when it's not yet fixed, just reply and then resolve when it's done.
  • Custom User Avatar

    Thanks for the input.Will consider the casing

  • Custom User Avatar

    Odd_ascend_Even_Descend is unacceptable as function name: it isn't camelCase, snake_case, PascalCase, or anything.

    Note that by C# convention method names should be in PascalCase, so OddAscendEvenDescend.

  • Custom User Avatar

    Thank you for your feedback I reevaluate it.

  • Custom User Avatar
    • There is no random tests
    • Initial solution does not compile because it's missing some usings
    • throw new Exception() is not the best placeholder. NotImplementedException would be better, or returning a dummy, empty array would be fine too.
    • Indentation and spacing could use some work. Usings are scattered all over the palce.
    • Author's solution seems to be full of errors.
    • It's not clear, and not tested, whether the input list and its contents can be destroyed, or shold be left untouched.
  • Default User Avatar

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

  • Custom User Avatar

    You used Issue label, that's reserved for problems with the kata itself. Use Questionif you want to use a label for cases like this. And I already told you how to print the input to help you debug your code. You can also provide your code following this guideline: https://docs.codewars.com/training/troubleshooting/#post-discourse

  • Custom User Avatar

    i never said the problem is with the kata i just said i need help reviewing my code maybe someone might have a better eye than me ,because i have been doing this for hours.

  • Loading more items...