Vérifier si un objet existe déjà dans la collection

Lors de la comparaison, il est préférable d'utiliser la comparaison par rapport à un identificateur, qui dans votre cas est le PartyGroupId. Si vous utilisez contient alors la valeur par défaut de surcharge de contains (), alors la valeur de hachage de l'objet de la liste est utilisé pour la comparaison.

Donc, plutôt que de laisser la comparaison .NET, vous pouvez créer un IEqualityComparer mise en œuvre ou l'utilisation de Linq clause where comme suit.

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.   
    } 
} 

Une autre suggestion est de renommer PartyGroup membres de la classe:

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

OriginalL'auteur Devendra D. Chavan

Votre code a certaines questions. Tout d'abord, vous réutilisez le même objet pour chacun de vos itérations.

Envisager

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

Vous remarquerez que votre liste va avoir 2 articles, mais ils font référence au même objet. Si vous itérer sur la liste et cochez Bar, chacun sera de retour "Beta".

Vous souhaitez créer un nouveau Foo pour chaque élément.

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);

Dans la boucle termes, que, fondamentalement signifie simplement créer l'objet à l'intérieur de la boucle au lieu de l'extérieur.

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

Concernant la vérification pour voir si la collection contient déjà l'objet, avez-vous correctement remplacé Equals et GetHashCode? Lorsque vous traitez les classes, le comportement par défaut est tout simplement de vérifier si l'objet de références sont égaux. Si vous êtes préoccupé par la valeurs les objets sont de l'encapsulation, alors vous devez fournir la logique pour vous-même. Dans votre classe, vous devez remplacer la Equals et GetHashCode méthodes à mettre en œuvre de votre choix de la méthode de détermination de l'égalité.

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;
    }
}

Maintenant, quand Contains cherche à déterminer si l'objet est déjà dans la liste, il sera basé sur les valeurs contenues dans l'objet plutôt que de la mémoire de référence.

OriginalL'auteur Anthony Pegram

Essayer cette

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); 
    } 
} 

OriginalL'auteur Tester101