Loading collection data...
Collections are a way for you to organize kata so that you can create your own training routines. Every collection you create is public and automatically sharable with other warriors. After you have added a few kata to a collection you and others can train on the kata contained within the collection.
Get started now by creating a new collection.
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.