Linq: GroupBy, Sum і Count

У мене є колекція продуктів

public class Product {

   public Product() { }

   public string ProductCode {get; set;}
   public decimal Price {get; set; }
   public string Name {get; set;}
}

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

public class ResultLine{

   public ResultLine() { }

   public string ProductName {get; set;}
   public string Price {get; set; }
   public string Quantity {get; set;}
}

Тому я використовую GroupBy для групування за ProductCode, потім я обчислюю суму і також підраховую кількість записів для кожного коду продукту.

Ось що я маю досі:

List<Product> Lines = LoadProducts();    
List<ResultLine> result = Lines
                .GroupBy(l => l.ProductCode)
                .SelectMany(cl => cl.Select(
                    csLine => new ResultLine
                    {
                        ProductName =csLine.Name,
                        Quantity = cl.Count().ToString(),
                        Price = cl.Sum(c => c.Price).ToString(),
                    })).ToList<ResultLine>();

Чомусь сума робиться правильно, але підрахунок завжди 1.

Дані Sampe:

List<CartLine> Lines = new List<CartLine>();
            Lines.Add(new CartLine() { ProductCode = "p1", Price = 6.5M, Name = "Product1" });
            Lines.Add(new CartLine() { ProductCode = "p1", Price = 6.5M, Name = "Product1" });
            Lines.Add(new CartLine() { ProductCode = "p2", Price = 12M, Name = "Product2" });

Результат із зразковими даними:

Product1: count 1   - Price:13 (2x6.5)
Product2: count 1   - Price:12 (1x12)

Продукт 1 повинен мати кількість = 2!

Я спробував імітувати це в простому консольному додатку, але там я отримав такий результат:

Product1: count 2   - Price:13 (2x6.5)
Product1: count 2   - Price:13 (2x6.5)
Product2: count 1   - Price:12 (1x12)

Продукт1: повинен бути вказаний лише один раз ... Код вищезазначеного можна знайти на пастібіні: http://pastebin.com/cNHTBSie

Я не розумію, звідки береться перший "результат із зразковими даними", але проблема в консольному додатку полягає в тому, що ви використовуєте SelectManyдля перегляду кожного елемента в кожній групі .

Я думаю, ти просто хочеш:

List<ResultLine> result = Lines
    .GroupBy(l => l.ProductCode)
    .Select(cl => new ResultLine
            {
                ProductName = cl.First().Name,
                Quantity = cl.Count().ToString(),
                Price = cl.Sum(c => c.Price).ToString(),
            }).ToList();

Використання First()тут для отримання назви продукту передбачає, що кожен продукт з тим самим кодом товару має однакову назву продукту. Як зазначається в коментарях, ви можете згрупувати за назвою продукту, а також за кодом продукту, який дасть однакові результати, якщо ім’я завжди однакове для будь-якого заданого коду, але, очевидно, генерує кращий SQL в EF.

Я б також запропонував вам змінити Quantityі Priceвластивості, які будуть, intі decimalтипи відповідно - навіщо використовувати властивість рядка для даних, які явно не є текстовими?

Наступний запит працює. Він використовує кожну групу для вибору замість SelectMany. SelectManyпрацює над кожним елементом з кожної колекції. Наприклад, у вашому запиті ви отримали 2 колекції. SelectManyотримує всі результати, всього 3, замість кожної колекції. Наступний код працює на кожну IGroupingчастину вибору, щоб ваші сукупні операції працювали правильно.

var results = from line in Lines
              group line by line.ProductCode into g
              select new ResultLine {
                ProductName = g.First().Name,
                Price = g.Sum(pc => pc.Price).ToString(),
                Quantity = g.Count().ToString(),
              };

іноді вам потрібно вибрати деякі поля за допомогою, FirstOrDefault()або singleOrDefault()ви можете скористатися поданим нижче запитом:

List<ResultLine> result = Lines
    .GroupBy(l => l.ProductCode)
    .Select(cl => new Models.ResultLine
            {
                ProductName = cl.select(x=>x.Name).FirstOrDefault(),
                Quantity = cl.Count().ToString(),
                Price = cl.Sum(c => c.Price).ToString(),
            }).ToList();