1

ドキュメントまたはドキュメントとページの識別子として使用するクラスでは、GetHashCode の次の実装を使用します。それは正しいと「感じました」が、この方法でドメイン固有の条件付けを実際に見たことがなかったので、そうしない理由があるかどうか疑問に思っていました. もちろん、ToString メソッドにも同じ条件があります。

public override int GetHashCode ()
{
    int hash = 0;

    unchecked
    {
        hash = 17;
        hash = hash * 23 + this.ProductManufacturer.Value.GetHashCode();
        hash = hash * 23 + this.ProductName.Value.GetHashCode();
        hash = hash * 23 + this.ProductVersion.Value.GetHashCode();
        hash = hash * 23 + this.Guid.Value.GetHashCode();

        if (this.Type != IdentifierType.Document)
        {
            hash = hash * 23 + this.PageNumber.Value.GetHashCode();
            hash = hash * 23 + this.PageCount.Value.GetHashCode();
        }
    }

    return (hash);
}

回答とエリックのリンクに基づいて更新されたコード:

public bool Equals (Identifier other)
{
    return (this.Equals(other, this.Type));
}

public override bool Equals (object obj)
{
    return ((obj is HouseOIdentifier) && (this.Equals(obj as Identifier)));
}

public bool Equals (Identifier other, IdentifierType type)
{
    bool result = false;

    if (object.ReferenceEquals(this, other))
    {
        result = true;
    }
    else if (!object.ReferenceEquals(other, null))
    {
        result
            = (this.Type == other.Type)
            && (this.ProductManufacturer.Key == other.ProductManufacturer.Key)
            && (this.ProductManufacturer.Value == other.ProductManufacturer.Value)
            && (this.ProductName.Key == other.ProductName.Key)
            && (this.ProductName.Value == other.ProductName.Value)
            && (this.ProductVersion.Key == other.ProductVersion.Key)
            && (this.ProductVersion.Value == other.ProductVersion.Value)
            && (this.Guid.Key == other.Guid.Key)
            && (this.Guid.Value == other.Guid.Value)
            ;

        if (type == IdentifierType.Page)
        {
            result
                &= (this.PageNumber.Key == other.PageNumber.Key)
                && (this.PageNumber.Value == other.PageNumber.Value)
                && (this.PageCount.Key == other.PageCount.Key)
                && (this.PageCount.Value == other.PageCount.Value)
                ;
        }
    }

    return (result);
}

public override int GetHashCode ()
{
    int hash = 0;

    unchecked // Overflow is fine, just wrap.
    {
        hash = 17;
        hash = hash * 23 + this.Type.GetHashCode();
        hash = hash * 23 + this.ProductManufacturer.Key.GetHashCode();
        hash = hash * 23 + this.ProductManufacturer.Value.GetHashCode();
        hash = hash * 23 + this.ProductName.Key.GetHashCode();
        hash = hash * 23 + this.ProductName.Value.GetHashCode();
        hash = hash * 23 + this.ProductVersion.Key.GetHashCode();
        hash = hash * 23 + this.ProductVersion.Value.GetHashCode();
        hash = hash * 23 + this.Guid.Key.GetHashCode();
        hash = hash * 23 + this.Guid.Value.GetHashCode();

        if (this.Type == HouseOfSynergy.FastForm.Core.Identifier.EnumType.Page)
        {
            hash = hash * 23 + this.PageNumber.Key.GetHashCode();
            hash = hash * 23 + this.PageNumber.Value.GetHashCode();
            hash = hash * 23 + this.PageCount.Key.GetHashCode();
            hash = hash * 23 + this.PageCount.Value.GetHashCode();
        }
    }

    return (hash);
}

public override string ToString ()
{
    return ("whatever");
}
4

1 に答える 1

3

等しいオブジェクトが同じハッシュ コードを持っている限り、実装は問題ありません。それが唯一の要件です。ハッシュ コードの計算方法、つまり if ステートメントを使用するかどうかは重要ではありません。

(また、ハッシュ コードを使用するほとんどすべてのデータ構造が壊れてしまうため、オブジェクトの有効期間中にハッシュ コードが変更されないことが望ましいです。最適には、オブジェクトは不変である必要があります。)

于 2013-10-12T11:59:07.437 に答える