Ad
  • Default User Avatar

    I think this is not realy best practice. This solution violates CleanCode rules. It looks quiet smart, because the solution solves the problem inside one function. But in my opinion best practice would be to avoid the repetition and using a second function instead like in the solution of zombie87.
    In addition I'm not sure if the use of Math.Max and Math.Min is the best choice for performance with the focus of speed an needed memory.