IList.Add() sovrascrivere i dati esistenti

Question

Sto affrontando un problema di aggiungere dati a un IList ma il problema è ogni volta ho aggiunto i dati i dati esistenti viene sovrascritto con quello attuale il mio codice è il seguente:

Test test = new Test(); 
IList<Test> myList = new List<Test>(); 

foreach (DataRow dataRow in dataTable.Rows) 
{ 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); 
} 

Qual è la ragione di questo?

Answer 1

mossa creazione oggetto di prova all'interno del ciclo

IList<Test> myList = new List<Test>(); 

foreach (DataRow dataRow in dataTable.Rows) 
{ Test test =new Test(); 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); 
} 

quello che attualmente facendo è l'aggiornamento stesso istante del test all'interno del ciclo e aggiungere più e più volte lo stesso ..

Answer 2

È necessario spostare il la creazione dell'oggetto Test all'interno del loop. Il motivo è che l'oggetto new Test() viene istanziato una sola volta e il riferimento allo stesso oggetto continua ad essere aggiunto all'elenco nel loop.

Answer 3

Perché il Test test viene copiato dal riferimento. Hai bisogno di spostarlo all'interno del ciclo.

fare in questo modo

IList<Test> myList = new List<Test>(); 
foreach (DataRow dataRow in dataTable.Rows) 
{ 
    Test test =new Test(); 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); 
} 

Answer 4

Perché non si crea nuova istanza nel ciclo foreach, correggere il codice:

Test test = null; 
IList<Test> myList = new List<Test>(); 

foreach (DataRow dataRow in dataTable.Rows) 
{ 
    test = new Test(); 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); 
} 

Answer 5

State usando la stessa variabile di riferimento (test) tutto il tempo. Prova a crettare il nuovo Test() nel tuo loop.

Answer 6

Si sta creando una singola istanza di Test

Test test = new Test(); // This is your instance 
IList<Test> myList = new List<Test>(); 

foreach (DataRow dataRow in dataTable.Rows) 
{ 
    // Here you change the values of the existing instance each time you loop 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); // but you are still just adding the same reference to the list multiple times 
} 

E poi dal momento che non si è mai creando un nuovo Test esempio, si aggiunge lo stesso riferimento alla lista più volte. Ciò significa che in pratica si sta semplicemente memorizzando lo stesso oggetto: se si apportano modifiche a un elemento nell'elenco sarà immediatamente visibile in tutti gli altri perché sono essenzialmente lo stesso oggetto

La soluzione è spostare l'istanza di prova all'interno del ciclo

IList<Test> myList = new List<Test>(); 

foreach (DataRow dataRow in dataTable.Rows) 
{ 
    Test test = new Test(); // Each loop iteration will now create a new instance of Test 
    test.PatientID = Convert.ToInt64(dataRow.ItemArray[0]); 
    test.LastName = dataRow.ItemArray[1].ToString(); 
    test.FirstName = dataRow.ItemArray[2].ToString(); 
    myList.Add(test); 
} 

Se avete bisogno di capire meglio questo, guardare i tipi di riferimento e il valore in .NET e il passaggio per riferimento/valore

tipi di valore e Ref in .NET: http://msdn.microsoft.com/en-us/library/t63sy5hs.aspx

Alcune informazioni su puntatori su Wikipedia http://en.wikipedia.org/wiki/Pointer_(computer_programming)