web-dev-qa-db-fra.com

Manière correcte de remplacer Equals () et GetHashCode ()

Comme je n'avais jamais vraiment fait cela auparavant, j'espérais que quelqu'un pourrait me montrer comment appliquer correctement un remplacement de Except () et GetHashCode () pour ma classe.

J'essaie de modifier la classe pour pouvoir utiliser la méthode LINQ Except ().

public class RecommendationDTO{public Guid RecommendationId { get; set; }
public Guid ProfileId { get; set; }
public Guid ReferenceId { get; set; }
public int TypeId { get; set; }
public IList<TagDTO> Tags { get; set; }
public DateTime CreatedOn { get; set; }
public DateTime? ModifiedOn { get; set; }
public bool IsActive { get; set; }
public object ReferencedObject { get; set; }
public bool IsSystemRecommendation { get; set; }
public int VisibilityScore { get; set; }

public RecommendationDTO()
{
}

public RecommendationDTO(Guid recommendationid,
                            Guid profileid,
                            Guid referenceid,
                            int typeid,
                            IList<TagDTO> tags,
                            DateTime createdon,
                            DateTime modifiedon, 
                            bool isactive,
                            object referencedobject)
{
    RecommendationId = recommendationid;
    ProfileId = profileid;
    ReferenceId = referenceid;
    TypeId = typeid;
    Tags = tags;
    CreatedOn = createdon;
    ModifiedOn = modifiedon;
    ReferencedObject = referencedobject;
    IsActive = isactive;
}

public override bool Equals(System.Object obj)
{
    // If parameter is null return false.
    if (obj == null)
    {
        return false;
    }

    // If parameter cannot be cast to Point return false.
    RecommendationDTO p = obj as RecommendationDTO;
    if ((System.Object)p == null)
    {
        return false;
    }

    // Return true if the fields match:
    return (ReferenceId == p.ReferenceId);// && (y == p.y);
}

public bool Equals(RecommendationDTO p)
{
    // If parameter is null return false:
    if ((object)p == null)
    {
        return false;
    }

    // Return true if the fields match:
    return (ReferenceId == p.ReferenceId);// && (y == p.y);
}

//public override int GetHashCode()
//{
//    return ReferenceId;// ^ y;
//}}

J'ai jeté un coup d'oeil à http://msdn.Microsoft.com/en-us/library/ms173147.aspx mais j'espérais que quelqu'un pourrait me montrer selon mon propre exemple.

Toute aide serait appréciée.

Je vous remercie

c#equalitygethashcode
90
Nugs

Vous pouvez remplacer Equals () et GetHashCode () sur votre classe de la manière suivante:

public override bool Equals(object obj)
{
    var item = obj as RecommendationDTO;

    if (item == null)
    {
        return false;
    }

    return this.RecommendationId.Equals(item.RecommendationId);
}

public override int GetHashCode()
{
    return this.RecommendationId.GetHashCode();
}
103
Craig
public override bool Equals(System.Object obj)
{
    // Check if the object is a RecommendationDTO.
    // The initial null check is unnecessary as the cast will result in null
    // if obj is null to start with.
    var recommendationDTO = obj as RecommendationDTO;

    if (recommendationDTO == null)
    {
        // If it is null then it is not equal to this instance.
        return false;
    }

    // Instances are considered equal if the ReferenceId matches.
    return this.ReferenceId == recommendationDTO.ReferenceId;
}

public override int GetHashCode()
{
    // Returning the hashcode of the Guid used for the reference id will be 
    // sufficient and would only cause a problem if RecommendationDTO objects
    // were stored in a non-generic hash set along side other guid instances
    // which is very unlikely!
    return this.ReferenceId.GetHashCode();
}
12
Trevor Pilley

Soyez prudent lorsque vous utilisez une clé primaire comme test d'égalité en remplaçant Equals (), car cela fonctionne uniquement APRÈS la persistance de l'objet. Avant cela, vos objets n’ont pas encore de clé primaire et les identifiants de ceux en mémoire sont tous nuls.

J'utilise base.Equals () si l'un des ID d'objet est égal à zéro mais qu'il existe probablement une méthode plus robuste.

9
Eric Nelson