Проблемы с сортировкой пузырьков

Question

Я продолжаю получать исключение NullReferenceException, это была необработанная ошибка при попытке запустить этот код и не знаю почему. Я пытаюсь сортировать по имени человека Я работаю только с C# около 4 Мт.

public static student[] players = new student[30]; 
    public struct student 
    { 
     public string lastname; 
     public string firstname; 
     public string likes; 
    } 

    public static void read() 
    { 
     StreamReader sr = new StreamReader(@"names.txt"); 
     int i = 0; 
     while (!sr.EndOfStream) 
     { 
      players[i].lastname = sr.ReadLine(); 
      players[i].firstname = sr.ReadLine(); 
      players[i].likes = sr.ReadLine(); 
      i++; 
     } 
     sr.Close(); 
    } 

     public static void sort() 
     { 
      //alphabetically lists players 
      student temp; 

      for (int i = 0; i < players.Length - 1; i++) 
      { 
       for (int pos = 0; pos < players.Length - 1; pos++) 
       { 

        if (players[pos + 1].lastname.CompareTo(players[pos].lastname) < +0) 
        { 
         temp = players[pos + 1]; 
         players[pos + 1] = players[pos]; 
         players[pos] = temp; 
        } 
       } 
      } 

     } 

Answer 1

Перед тем, как сравнить lastname, вам необходимо проверить элемент массива не null.

private static void Sort(Player[] players) 
{ 
    //alphabetically lists players 

    for (int i = 0; i < players.Length; i++) 
    for (int pos = 0; pos < players.Length - 1; pos++) 
    { 
     var currentPlayer = players[pos] != null ? players[pos].LastName : null; 
     var nextPlayer = players[pos + 1] != null ? players[pos + 1].LastName : null; 

     if (Compare(currentPlayer, nextPlayer) > 0) 
     { 
      var temp = players[pos + 1]; 
      players[pos + 1] = players[pos]; 
      players[pos] = temp; 
     } 
    } 
} 

private static int Compare<T>(T current, T next) 
    where T: IComparable 
{ 
    if (current == null) return -1; 
    if (next == null) return 1; 
    return current.CompareTo(next); 
} 

Answer 2

я бы поймал NullReference

while (!sr.EndOfStream) 
{ 
    player[i] = new student(); 
    players[i].lastname = sr.ReadLine(); 
    players[i].firstname = sr.ReadLine(); 
    players[i].likes = sr.ReadLine(); 
    i++; 
} 

Answer 3

Сортировка уже построен в рамках .NET с помощью Enumerable.OrderBy и EnumerableOrderByDescending. В вашем случае вы можете значительно уменьшить функцию:

public static IOrderedEnumerable<student> sort() 
{ 
    IOrderedEnumerable<student> orderedPlayers = players.OrderBy(p => p.lastname); 
    return orderedPlayers; 
} 

Это упорядочивает результаты не бросать исключение, даже если lastName является нулевым.