Должен ли я использовать конкатенацию моих строковых полей в качестве хэш-кода?

у меня есть класс адресов в C#, который выглядит так:

public class Address
{            
    public string StreetAddress { get; set; }
    public string RuralRoute { get; set; }
    public string City { get; set; }
    public string Province { get; set; }
    public string Country { get; set; }
    public string PostalCode { get; set; }
}

я реализую равенство, и поэтому мне нужно переопределить хэш-код. Сначала я собирался использовать формулу хэш-кода из EJ, но затем я подумал: Это все строковые поля, не могу ли я просто использовать StringBuilder, чтобы объединить их и вернуть хэш-код из этой строки?

что есть:

var str = new StringBuilder();
str.Append(StreetAddress)
   .Append(RuralRoute)
   ...

return str.ToString().GetHashCode();

каковы преимущества / недостатки этого? Почему бы и нет?

14
c# equality hashcode iequatable string

4 ответов

Я бы избегал этого просто на том основании, что он создает кучу строк бессмысленно - хотя точка Kosi2801 о том, чтобы сделать коллизии простыми, также актуальна. (Подозреваю, что нет!--6-->на самом деле создайте много столкновений, из-за природы полей, но...)

Я бы пошел на" простой и легкий, чтобы получить право " алгоритм, который я ранее в ответ (спасибо, что посмотрели его lance :) - и который указан в эффективном Java, как вы сказали. В этом случае это закончится как:

public int GetHashCode()
{
    int hash = 17;
    // Suitable nullity checks etc, of course :)
    hash = hash * 23 + StreetAddress.GetHashCode();
    hash = hash * 23 + RuralRoute.GetHashCode();
    hash = hash * 23 + City.GetHashCode();
    hash = hash * 23 + Province.GetHashCode();
    hash = hash * 23 + Country.GetHashCode();
    hash = hash * 23 + PostalCode.GetHashCode();
    return hash;
}

Это не null-безопасный, конечно. Если вы используете C# 3, Вы можете рассмотреть метод расширения:

public static int GetNullSafeHashCode<T>(this T value) where T : class
{
    return value == null ? 1 : value.GetHashCode();
}

затем вы можете использовать:

public int GetHashCode()
{
    int hash = 17;
    // Suitable nullity checks etc, of course :)
    hash = hash * 23 + StreetAddress.GetNullSafeHashCode();
    hash = hash * 23 + RuralRoute.GetNullSafeHashCode();
    hash = hash * 23 + City.GetNullSafeHashCode();
    hash = hash * 23 + Province.GetNullSafeHashCode();
    hash = hash * 23 + Country.GetNullSafeHashCode();
    hash = hash * 23 + PostalCode.GetNullSafeHashCode();
    return hash;
}

вы мог бы создайте утилиту метода массива параметров, чтобы сделать это еще проще:

public static int GetHashCode(params object[] values)
{
    int hash = 17;
    foreach (object value in values)
    {
        hash = hash * 23 + value.GetNullSafeHashCode();
    }
    return hash;
}

и называют его:

public int GetHashCode()
{
    return HashHelpers.GetHashCode(StreetAddress, RuralRoute, City,
                                   Province, Country, PostalCode);
}

в большинстве типов участвуют примитивы, так что будет выполнять бокс несколько излишне, но в этом случае у вас будут только рекомендации. Конечно, вы в конечном итоге создадите массив без необходимости, но вы знаете, что говорят о преждевременной оптимизации...

15

Не делайте этого, потому что объекты могут быть разными, хотя хэш-код один и тот же.

думать

"StreetAddress" + "RuralRoute" + "City"

vs

"Street" + "AddressRural" + "RouteCity"

оба будут иметь один и тот же хэш-код, но разное содержимое в полях.

5

для такого рода вещей, вы можете реализовать IEqualityComparer<Address>:

public class Address : IEqualityComparer<Address>
{        
    //
    // member declarations
    //

    bool IEqualityComparer<Address>.Equals(Address x, Address y)
    {
        // implementation here
    }

    int IEqualityComparer<Address>.GetHashCode(Item obj)
    {
        // implementation here
    }
}

вы также можете реализовать IComparable<Address> чтобы сделать заказ...

0
public string getfourDigitEncryptedText(string input) { 
    int hashCode = input.hashCode(); 
    string hstring = (new StringBuilder()).append(hashCode).append("").toString(); 
    string rev_hstring = (new StringBuilder(hstring)).reverse().toString(); 
    string parts[] = rev_hstring.trim().split(""); 
    int prefixint = 0; 
    for(int i = 1; i <= parts.length - 3; i++) 
        prefixint += integer.parseInt(parts[i]);   
    string prefixstr = "0"; 
    if((new integer(prefixint)).toString().length() < 2) 
        prefixstr = (new StringBuilder()).append((new integer(prefixint)).toString()).append("5").toString(); 
    else if((new integer(prefixint)).toString().length() > 2) 
        prefixstr = "79"; 
    else 
        prefixstr = (new integer(prefixint)).toString(); 
    string finalstr = (new StringBuilder()).append(prefixint).append(rev_hstring.substring(3, 5)).toString();
    return finalstr; 
}
-4