Як користуватися IEqualityComparer

У моїй базі даних є дзвоники з такою ж кількістю. Я хочу отримати їх усі без дублювання. Я створив клас порівняння, щоб виконати цю роботу, але виконання функції викликає велику затримку функції без різниці, від 0,6 сек до 3,2 сек!

Чи правильно я це роблю, чи мені доводиться використовувати інший метод?

reg.AddRange(
    (from a in this.dataContext.reglements
     join b in this.dataContext.Clients on a.Id_client equals b.Id
     where a.date_v <= datefin && a.date_v >= datedeb
     where a.Id_client == b.Id
     orderby a.date_v descending 
     select new Class_reglement
     {
         nom  = b.Nom,
         code = b.code,
         Numf = a.Numf,
     })
    .AsEnumerable()
    .Distinct(new Compare())
    .ToList());

class Compare : IEqualityComparer<Class_reglement>
{
    public bool Equals(Class_reglement x, Class_reglement y)
    {
        if (x.Numf == y.Numf)
        {
            return true;
        }
        else { return false; }
    }
    public int GetHashCode(Class_reglement codeh)
    {
        return 0;
    }
}

Ваша GetHashCodeреалізація завжди повертає одне і те ж значення. Distinctпокладається на хорошу хеш-функцію для ефективної роботи, оскільки вона внутрішньо створює хеш-таблицю .

Під час реалізації інтерфейсів класів важливо прочитати документацію , щоб знати, який контракт ви повинні реалізувати. ¹

У своєму коді, рішення направити GetHashCodeна Class_reglement.Numf.GetHashCodeі реалізувати її належним чином там.

Окрім цього, у вашому Equalsметоді повно непотрібного коду. Його можна переписати наступним чином (однакова семантика, ¼ коду, більш читабельна):

public bool Equals(Class_reglement x, Class_reglement y)
{
    return x.Numf == y.Numf;
}

Нарешті, ToListдзвінок є непотрібним і трудомістким: AddRangeприймає будь-які, IEnumerableтому перетворення в a Listне потрібно. AsEnumerableтут також зайвий, оскільки обробка результату в AddRangeвсе одно спричинить це.

¹ Написання коду, не знаючи, що він насправді робить, називається програмою культового вантажу . Це напрочуд широко поширена практика. Це принципово не працює.

Спробуйте цей код:

public class GenericCompare<T> : IEqualityComparer<T> where T : class
{
    private Func<T, object> _expr { get; set; }
    public GenericCompare(Func<T, object> expr)
    {
        this._expr = expr;
    }
    public bool Equals(T x, T y)
    {
        var first = _expr.Invoke(x);
        var sec = _expr.Invoke(y);
        if (first != null && first.Equals(sec))
            return true;
        else
            return false;
    }
    public int GetHashCode(T obj)
    {
        return obj.GetHashCode();
    }
}

Прикладом його використання може бути

collection = collection
    .Except(ExistedDataEles, new GenericCompare<DataEle>(x=>x.Id))
    .ToList(); 

Просто код, з реалізацією GetHashCodeта NULLвалідацією:

public class Class_reglementComparer : IEqualityComparer<Class_reglement>
{
    public bool Equals(Class_reglement x, Class_reglement y)
    {
        if (x is null || y is null))
            return false;

        return x.Numf == y.Numf;
    }

    public int GetHashCode(Class_reglement product)
    {
        //Check whether the object is null 
        if (product is null) return 0;

        //Get hash code for the Numf field if it is not null. 
        int hashNumf = product.hashNumf == null ? 0 : product.hashNumf.GetHashCode();

        return hashNumf;
    }
}

Приклад: список Class_reglement, відмінний за Numf

List<Class_reglement> items = items.Distinct(new Class_reglementComparer());

Включення вашого класу порівняння (або, точніше, AsEnumerableвиклику, який вам потрібно було використати, щоб він працював) означало, що логіка сортування перейшла від того, що базується на сервері бази даних, до того, що він знаходиться на клієнті бази даних (ваша програма). Це означало, що вашому клієнту тепер потрібно отримати та потім обробити більшу кількість записів, що завжди буде менш ефективним, ніж виконання пошуку в базі даних, де можна використовувати відповідні індекси.

Вам слід спробувати розробити речення where, яке відповідає вашим вимогам, див. Застосування IEqualityComparer із посиланням LINQ to Entities Except для отримання додаткової інформації.

Якщо ви хочете загальне рішення без боксу:

public class KeyBasedEqualityComparer<T, TKey> : IEqualityComparer<T>
{
    private readonly Func<T, TKey> _keyGetter;

    public KeyBasedEqualityComparer(Func<T, TKey> keyGetter)
    {
        _keyGetter = keyGetter;
    }

    public bool Equals(T x, T y)
    {
        return EqualityComparer<TKey>.Default.Equals(_keyGetter(x), _keyGetter(y));
    }

    public int GetHashCode(T obj)
    {
        TKey key = _keyGetter(obj);

        return key == null ? 0 : key.GetHashCode();
    }
}

public static class KeyBasedEqualityComparer<T>
{
    public static KeyBasedEqualityComparer<T, TKey> Create<TKey>(Func<T, TKey> keyGetter)
    {
        return new KeyBasedEqualityComparer<T, TKey>(keyGetter);
    }
}

використання:

KeyBasedEqualityComparer<Class_reglement>.Create(x => x.Numf)

IEquatable<T> це може бути набагато простішим способом зробити це за допомогою сучасних фреймворків.

Ви отримуєте приємну просту bool Equals(T other)функцію, і ви не можете возитися з кастингом або створенням окремого класу.

public class Person : IEquatable<Person>
{
    public Person(string name, string hometown)
    {
        this.Name = name;
        this.Hometown = hometown;
    }

    public string Name { get; set; }
    public string Hometown { get; set; }

    // can't get much simpler than this!
    public bool Equals(Person other)
    {
        return this.Name == other.Name && this.Hometown == other.Hometown;
    }

    public override int GetHashCode()
    {
        return Name.GetHashCode();  // see other links for hashcode guidance 
    }
}

Зверніть увагу, що вам НЕ потрібно реалізовувати, GetHashCodeякщо ви використовуєте це у словнику або щось подібне Distinct.

PS. Я не думаю, що будь-які власні методи Equals працюють з фреймворком сутності безпосередньо на стороні бази даних (я думаю, ви це знаєте, тому що робите AsEnumerable), але це набагато простіший спосіб зробити простий Equals для загального випадку.

Якщо речі, здається, не працюють (наприклад, помилки повторюваних ключів при виконанні ToDictionary), поставте точку зупинки всередину Equals, щоб переконатися, що вона потрапила, і переконатися, що ви GetHashCodeвизначили (із ключовим словом override).