Liste überschreiben Daten auf allen Positionen vor

Question

ToolMy Problem, wenn ich ein neues Element zu einer Liste hinzufügen dieses Element überschreiben alle anderen Elemente vor.

public static List<Tool_OP> list = new List<Tool_OP>(); 
BinaryReader fs = new BinaryReader(File.Open(FileName, FileMode.Open)); 
do 
{ 
    Tool_OP item = new Tool_OP(); 
    if (!item.ReadRecord(fs)) break; 
    list.Add(item); 
} while (true); 
fs.Close(); 

Hier ist ein Teil meiner Record-Klasse. Mein Datensatz ist OK, wenn ich debbug. Ich weiß nicht, warum meine Liste bei jedem Hinzufügen alle Artikel mit meinem tatsächlichen Artikel überschreibt. Ich habe eine ReadString-Methode, um Strings aus meiner Binärdatei zu erhalten.

class Tool_OP 
{ 
    public static string  comment; 
    public static long  op_idn; 

    public static string ReadString(int max, BinaryReader fs) 
    { 
     byte[] Buf = new byte[max]; 
     string NewStr = ""; 
     Buf = fs.ReadBytes(max); 

     for (int ii = 0; ii < max; ii++) 
     { 
      if (Buf[ii] == 0) return NewStr; 
      NewStr = NewStr + Convert.ToChar(Buf[ii]); 
     } 
     return NewStr; 
    } 

    public Boolean ReadRecord(BinaryReader fs) 
    { 
     comment = ReadString(120, fs); 
     op_idn = fs.ReadInt32(); 
    } 
} 

Answer 1

Da sagten Sie, das Problem ist, dass alle Datensätze die gleichen Daten haben (in Ihren Kommentar), das Problem ist die Tatsache, dass Sie die gleiche Instanz eines Objekts modifizieren und dann das Hinzufügen eines weiteren Verweis auf dieses Objekt in deiner Liste.

Ich empfehle einige Änderungen den Code klarer zu machen:

public static List<Tool_OP> list = new List<Tool_OP>(); 
BinaryReader fs = new BinaryReader(File.Open(FileName, FileMode.Open)); 
bool isValidReadRecord = true; 

while(isValidReadRecord) 
{ 
    Tool_OP item = new Tool_OP(); 
    isValidReadRecord = item.ReadRecord(fs); 

    if(isValidReadRecord) 
    { 
     list.Add(item); 
    } 
}; 
fs.Close(); 

Die wichtigsten Änderungen hier ist, dass wir nicht haben, etwas wie eine offene Schleife aussieht. Sie haben eine klare Bedingung, um die Schleife zu beenden. Und im Vergleich zu dem, was ich ursprünglich gesehen habe, ist das Tool_OP item jedes Mal neu instanziiert. Tool_OP item = new Tool_OP();

Wenn jeder Datensatz zu diesem Zeitpunkt die gleichen Daten enthält, müssen Sie sich ansehen, wie Tool_OP Daten speichert. Wenn die Informationen in static Feldern oder Eigenschaften gespeichert werden, haben sie alle die neusten Daten von item.ReadRecord().

In der Tat, mit Ihrer Bearbeitung wird es sehr deutlich! Entfernen Sie die static Stichwort:

class Tool_OP 
{ 
    public string  comment; 
    public long  op_idn; 
} 

Das static Schlüsselwort bedeutet, dass es in der gesamten Laufzeit für Ihre Klasse eine Definition dieses Feldes ist. Sie sind effektiv globale Werte. Verwenden Sie nur static, wenn Sie wirklich Daten für alle Instanzen einer Klasse freigeben möchten.

Answer 2

Vielen Dank an alle ...

Jetzt funktioniert es ohne statisch in der Tool_OP Klasse und der klaren Code!