Чтение нескольких строк из базы данных - Где я ошибаюсь?

Question

string connectionString = ConfigurationManager.AppSettings["AllRttpDBConnectionString"]; 
MySqlConnection connection = new MySqlConnection(connectionString); 
MySqlCommand command = connection.CreateCommand(); 

command.CommandText = "Select * from test where ServiceName like 'T%' " ; 

try 
{ 
    connection.Open(); 
} 
catch (Exception e) 
{ 
    Console.WriteLine(e.ToString()); 
} 

try 
{ 
    MySqlDataReader reader; 
    reader = command.ExecuteReader(); 

    while (reader.Read()) 
    { 
    Player.Name = reader["Name"].ToString(); 
    Player.Number = Convert.ToInt32(reader["Number"].ToString()); 

    //push to list 
    PlayerList.Add(Player); 
    } 

    connection.Close(); 
} 
catch (Exception e) 
{ 
    connection.Close(); 
    logger.Info(e.ToString()); 
} 

Выше приведен код, который я использую для чтения нескольких строк из базы данных в список. Однако все мои элементы списка имеют одни и те же данные (последняя строка базы данных).

Я знаю, что это, вероятно, очень простая, глупая ошибка, но я просто не вижу ее.

Answer 1

Похоже, что вы добавляете один и тот же объект снова и снова, меняя значения прочитанного fromt текущей строки. Вам необходимо использовать:

Player player = new Player() 
player.Name = reader["Name"].ToString(); 
player.Number = Convert.ToInt32(reader["Number"].ToString()); 

//push to list 
PlayerList.Add(player); 

затем добавьте его в коллекцию.

Answer 2

Похоже, вы постоянно изменяете один экземпляр Player.

Чтобы это исправить, необходимо создать новый экземпляр для каждой записи:

while (reader.Read()) 
{ 
    // I'm guessing about the type here 
    Player player = new Player(); 
    player.Name = reader["Name"].ToString(); 
    player.Number = Convert.ToInt32(reader["Number"].ToString()); 

    //push to list 
    PlayerList.Add(player); 
} 

Answer 3

Так что происходит, вы не делаете новый Player каждый раз, когда вы добавляете его в список.

Ваш код, когда вы добавляете элементы в список, должен выглядеть примерно так.

PlayerClass NewPlayer = new PlayerClass; 
NewPlayer.Name = reader["Name"].ToString(); 
NewPlayer.Number = Convert.ToInt32(reader["Number"].ToString()); 

//push to list 
PlayerList.Add(NewPlayer); 

Answer 4

Позвольте мне предложить вам небольшое улучшение на ваш код:

public IEnumerable<Player> GetPlayers() 
{ 
    string connectionString = ConfigurationManager.AppSettings["AllRttpDBConnectionString"]; 
    using (var conn = new MySqlConnection(connectionString)) 
    using (var cmd = conn.CreateCommand()) 
    { 
     conn.Open(); 
     cmd.CommandText = "SELECT Name, Number FROM test WHERE ServiceName LIKE 'T%';"; 
     using (var reader = cmd.ExecuteReader()) 
     { 
      while (reader.Read()) 
      { 
       yield return new Player 
       { 
        Name = reader.GetString(0), 
        Number = reader.GetInt32(1) 
       }; 
      } 
     } 
    } 
} 

И когда вам нужно создать список:

List<Player> playersList = GetPlayers().ToList(); 

Вы также должны убедиться, правильно распоряжаться все доступные ресурсы, как в моем примере.