Überprüfen, ob ein Objekt existiert bereits in Sammlung

Beim Vergleich ist es besser zu nutzen-Vergleich mit Bezug auf einen Bezeichner, der in Ihrem Fall ist die PartyGroupId. Wenn Sie verwenden enthält, wird die Standard-überlastung von Contains (), dann wird der hash-Wert des Objekts in der Liste wird für den Vergleich verwendet.

Also anstatt lassen die Vergleich zum .NET Sie können einen eigenen IEqualityComparer der Umsetzung oder der Verwendung von Linq Where-Klausel wie folgt.

using (AseDataReader reader = command.ExecuteReader()) 
{ 
    while (reader.Read()) 
    {  
        int groupId = Convert.ToInt32(reader["party_group_id"]);

        if (partyGroupsList.Where(partyGroup => partyGroup.PartyGroupID == groupId).Any() == false)
        {
           PartyGroup newPartyGroup = new PartyGroup()
                                      {
                                          PartyGroupID = groupId,
                                          PartyGroupName = reader["party_group_name"].ToString(),
                                          PersonList = myPersonList
                                      };

           partyGroupsList.Add(newPartyGroup);                 
        } 

        //If object already exists in the list then do not add, continue 
        //to the next row.   
    } 
} 

Ein weiterer Vorschlag ist, auf umbenennen, um PartyGroup Klasse Mitglieder:

class PartyGroup
 {
   public int ID { get; set; }
   public string Name { get; set; }
   public IList PersonList { get; set; }
}

InformationsquelleAutor Devendra D. Chavan

Dein code hat einige Probleme. Zunächst sind Sie wiederverwenden das gleiche Objekt für jeden Ihrer Iterationen.

Betrachten

List<Foo> foos = new List<Foo>();
Foo foo = new Foo();
foo.Bar = "Alpha";
foos.Add(foo);
foo.Bar = "Beta";
foos.Add(foo);

Werden Sie feststellen, dass Ihre Liste haben, 2 Elemente, aber Sie werden das gleiche Objekt referenzieren. Wenn Sie die Liste Durchlaufen und prüfen Bar jeder zurück "Beta".

Den Sie erstellen möchten eine neue Foo für jedes Element.

List<Foo> foos = new List<Foo>();
Foo foo = new Foo();
foo.Bar = "Alpha";
foos.Add(foo);
Foo anotherFoo = new Foo();
anotherFoo.Bar = "Beta";
foos.Add(anotherFoo);

Looping-Begriffe, die im Grunde nur bedeutet, dass das Objekt innerhalb der Schleife anstelle von außerhalb.

while (someCondition)
{
    Foo foo = new Foo();
    //do your work, populate the object, etc.
    //then check contains 
    if (!myList.Contains(foo))
        myList.Add(foo);
}

In Bezug auf überprüfen, um zu sehen, ob die Sammlung enthält bereits das Objekt, haben Sie richtig überschrieben Equals und GetHashCode? Beim Umgang der Klassen, die Standard-Verhalten ist einfach zu überprüfen, wenn die Objektreferenzen gleich sind. Wenn Sie sind besorgt über die Werte die Objekte sind gekapselt, dann müssen Sie die Logik für, sich selbst. In Ihrer Klasse, die Sie außer Kraft setzen müssen Equals und GetHashCode Methoden, um Ihre gewünschte Methode der Bestimmung der Geschlechter.

class Foo
{
    public string Bar { get; set; }

    public override int GetHashCode()
    {
        return this.Bar.GetHashCode();
    }

    public override bool Equals(object other)
    {
        Foo otherFoo = other as Foo;
        if (otherFoo == null)
            return false;
        else
            return this.Bar == otherFoo.Bar;
    }
}

Nun, wenn Contains sucht, um zu bestimmen, ob das Objekt bereits in der Liste enthalten, wird es tun, basierend auf die darin enthaltenen Werte in das Objekt, anstatt den Speicher auf.

InformationsquelleAutor Anthony Pegram

Versuchen, diese

while (reader.Read()) 
{ 
    partyGroup.PartyGroupID = Convert.ToInt32(reader["party_group_id"]); 
    partyGroup.PartyGroupName = reader["party_group_name"].ToString(); 
    partyGroup.PersonList = myPersonList; 

    if (!myPartyGroupList.Contains(partyGroup)) 
    { 
        myPartyGroupList.Add(partyGroup); 
    } 
} 

InformationsquelleAutor Tester101