List<T> и GroupBy

[Farlep]

Добрый день

Есть List<User> Users { get; set; };

В нем идут дубликаты номеров телефона

Использую поиск по мобильному номеру и формирую новый список без дублей.

Users = Users.GroupBy(x => x.CalledNumber).Select(x => x.First()).ToList();

Но задача стоит более сложная, как для меня

Нужно перед тем как убрать дубль из файла, просуммировать поля объекта

01.12.2021 11:30:22 +38067*** Входящий звонок 608 10 11
01.12.2021 11:48:40 +38067*** Входящий звонок 162 15 5

//длительность вызова/кол-во м.б
public int CallDuration { get; set; }
//стоимость вызова
public double CallCostWithTaxes { get; set; }
//стоимость вызова с учетом скидки
public double CallCostWithDiscount { get; set; }

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

[pu4koff]

использовать такую перегрузку метода:

Users = Users.GroupBy(x => x.CalledNumber, (num, gr) =>
{
  return new User ...
}).ToList()

для каждой группы в итоге вызовется указанный делегат и на вход в num попадёт CalledNumber как ключ группы, а в gr - коллекция элементов этой группы, т.е. все User с данным CallerNumber. Вот на основе этих данных можно создать новый User или объект любого нужного типа и заполнить нужными данными.

[Farlep]

pu4koff, спасибо за предложение но я так и не осилил ваш запрос.

Пошел другим путем, через SortedDictionary. Если ключ в словаре (номер телефона) то перебирая свой лист суммирую нужные поля. Да и сортировка есть уже, не пришлось реализовывать самостоятельно интерфейс для класса юзеров IEnumerable и IEnumerator.

[pu4koff]

С GroupBy по сути то же самое получается, просто запись короче.

Users = Users.GroupBy(x => x.CalledNumber, (num, gr) =>
{
  var user = new User(); // создаём новый объект, который вернём вместо группы
  user.CalledNumber = num; // указываем номер
  // в свойства прописываем суммы значений из всей группы объектов User
  user.CallDuration = gr.Sum(u => u.CallDuration);
  user.CallCostWithTaxes = gr.Sum(u => u.CallCostWithTaxes);
  user.CallCostWithDiscount = gr.Sum(u => u.CallCostWithDiscount);
  return user; // возвращаем новый объект, заполненный суммарными значениями
}).ToList()

ну, можно там ещё наверно проверку добавить, если в gr один элемент, то его же и возвращать, не выполняя лишнюю работу.