List Class Sort Usage

Ran into a small problem. Trying to sort the planets my planet generator produces by their axis using the built in sort functions from the List class. This is my implementation. I came upon it while searching around on how to do it and this was the implementation I worked out. It seems that the sort gets caught in a loop...and without knowing enough about the how's and why's of it all I'm not smart enough to effectively troubleshoot it...which is why i'm here!

Any insights or even a better way to tackle sorting my planets by their axis would be awesome!

V/R

Brad

Spoiler: Utilities

Code (CSharp):

1. //Static class for the utility functions used by the program...
2. public static class Utilities
3. {
4.     .....
5.     public static int SortByAxis(PlanetNucleus p1, PlanetNucleus p2)
6.     {
7.         return p1.axis.CompareTo(p2.axis);
8.     }
9. }
10.  
11. //The sort call...
12. planetArray.Sort(Utilities.SortByAxis);

 

What sort of type is planetArray, I'm guessing PlanetNucleus? List or array? I would typically create my own CompareTo function then you could just call planetArray.Sort();

Take a look at, https://msdn.microsoft.com/en-us/library/b0zbh7b6(v=vs.110).aspx
Many way of doing it to suite your needs.

 

Yessir, planetArray is a PlanetNucleus and it's a List, sorry.

private List<PlanetNucleus> planetArray = new List<PlanetNucleus>();

that's the List object declaration. And then the axis values are floats.

 

So as far as I can tell I've followed what's in the MSDN article...but the sort still isn't working...

 

I think this is what you should do. I believe this will work (haven't tested the code)

Code (csharp):

1. public class PlanetNucleus : IComparable<PlanetNucleus> {
2.    // Default comparer for PlanetNucleus type.
3.    public int CompareTo(PlanetNucleus o)
4.    {
5.       // A null value means that this object is greater.
6.       if (o == null)
7.          return 1;
8.       else
9.          return this.axis.CompareTo(o.axis);
10.    }
11. }
12.  
13. // To use it:
14. planetArray.Sort();