1

Есть класс

class ClassidIntanceid
{
    public string i_classid { get; set; }
    public string ui_id { get; set; }
    public string i_instanceid { get; set; }
    public string i_market_hash_name { get; set; }
    public float ui_price { get; set; }
}

Также есть список, в котором множество таких экземпляров класса.

List<ClassidIntanceid> classintance = new List<ClassidIntanceid>();

Стоит задача сделать Distinct по двум полям - i_classid и i_instance_id

Если сделать classinstance.Distincs - то он начинает искать дубли по всем полям. В сети нашел одно решение через MoreLinq, но хочется сделать это стандартными средствами. Скорее всего как то через IEqualityComparer можно это сделать.

2
  • @Grundy там одна пропертя, а не несколько Commented 21 авг. 2017 в 6:19
  • @PavelMayorov, какая разница-то? :-) суть одна и та же, ответы даже совпадают с теми, что уже тут: IEqualityComparer, GroupBy, MoreLinq
    – Grundy
    Commented 21 авг. 2017 в 6:31

3 ответа 3

4

Попробуйте использовать такое расширение от автора Jon Skeet:

public static IEnumerable<TSource> DistinctBy<TSource, TKey>
    (this IEnumerable<TSource> source, Func<TSource, TKey> keySelector)
{
    HashSet<TKey> seenKeys = new HashSet<TKey>();
    foreach (TSource element in source)
    {
        if (seenKeys.Add(keySelector(element)))
        {
            yield return element;
        }
    }
}

Самое удачное решение Distinct по полю класса, которое я смог найти для себя.

В вашем случае используйте его так:

var query = classintance.DistinctBy(p => new { i_classid, i_instance_id });
2
  • Это по идее и есть DistinctBy из MoreLinq.
    – VladD
    Commented 20 авг. 2017 в 14:52
  • Да, так и есть. Если перейти по ссылке, то там Джон указал, что этот код сейчас в составе библиотеки MoreLinq. Лично я не пользуюсь этой библиотекой, мне хватает стандартных средств в большинстве случаев, но именно функционал DistinctBy порой бывает очень нужен. Надеюсь его добавят в стандартную библиотеку.
    – Chloroform
    Commented 20 авг. 2017 в 14:53
1

Да, вы правы, это можно сделать через IEqualityComparer, например так:

Создайте класс компарера:

class InstanceidComparer : EqualityComparer<ClassidIntanceid>
{
    public override bool Equals(ClassidIntanceid x, ClassidIntanceid y)
    {
        return x.i_classid == y.i_classid && x.i_instanceid == y.i_instanceid;
    }

    public override int GetHashCode(ClassidIntanceid obj)
    {
        return obj.i_classid.GetHashCode() * 17 + obj.i_instanceid.GetHashCode();
    }
}

затем создаете экземпляр этого класса и передаете его в Distinct:

var comparer = new InstanceidComparer();
var result = classintance.Distinct(comparer);

Минус этого подхода - для сравнения по разным свойствам нужны разные классы компарера

1

Ну например так:

classintance.GroupBy(o => (o.i_classid, o.i_instanceid)).Select(g => g.First())

или даже просто

classintance.GroupBy(o => (o.i_classid, o.i_instanceid), (key, g) => g.First())

Это может быть немного менее эффективно, чем решение из MoreLinq.

И да, можно написать кастомный IValueComparer тоже.


Если вы пользуетесь старым компилятором, вам придётся перейти к менее эффективным анонимным классам:

o => new  { o.i_classid, o.i_instanceid }

Но если вы работаете со свежей версией компилятора, проблем быть не должно.

5
  • "System.ValueTuple`2" не определен и не импортирован
    – Zevra
    Commented 20 авг. 2017 в 10:28
  • у меня netf 4.5
    – Zevra
    Commented 20 авг. 2017 в 10:28
  • @Zevra: Ну так импортируйте. ru.stackoverflow.com/a/644251/10105
    – VladD
    Commented 20 авг. 2017 в 10:35
  • @VladD, а можете пояснить, почему вы считаете что кортежи эффективнее классов? В принципе, теперь с новыми фишками для кортежей, нужда в анонимных классах отпадает (они, вроде, тоже неизменяемые, на сколько я помню)? Commented 20 авг. 2017 в 19:34
  • @Андрей: Только тем, что не грузят сборщик мусора, они ж всё-таки value type.
    – VladD
    Commented 20 авг. 2017 в 20:44

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.