Linq удалить дубликаты

Question

оптимального качества я всегда использую эту технику, чтобы иметь дело с дубликатами от List<T> с помощью Linq

var distrinctPricings = pricings.OrderByDescending(x => x.ValidFrom).GroupBy(x => x.Quantity).Select(x => x.First()); 

pricings коллекции списка пользовательских типа не реализаций любого рода сопоставимых.

Здесь я беру только самые последние принцы, когда есть один с тем же количеством.

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

Answer 1

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

var d = new Dictionary<int, Pricing>(): // Assuming .Quantity is an int 

foreach(var p in pricings) 
{ 
    if (!d.ContainsKey(p.Quantity) || p.ValidFrom > d[p.Quantity].ValidFrom) 
    { 
     d[p.Quantity] = p; 
    } 
} 

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