Linq: GroupBy, Sum and Count

Ho una collezione di prodotti

public class Product {

   public Product() { }

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

Ora voglio raggruppare la raccolta in base al codice prodotto e restituire un oggetto contenente il nome, il numero o i prodotti per ciascun codice e il prezzo totale per ciascun prodotto.

public class ResultLine{

   public ResultLine() { }

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

Quindi utilizzo GroupBy per raggruppare per ProductCode, quindi calcolo la somma e conto anche il numero di record per ciascun codice prodotto.

Questo è quello che ho finora:

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>();

Per qualche motivo, la somma viene eseguita correttamente ma il conteggio è sempre 1.

Dati di campionamento:

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" });

Risultato con dati di esempio:

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

Il prodotto 1 dovrebbe avere count = 2!

Ho provato a simulare questo in una semplice applicazione console ma lì ho ottenuto il seguente risultato:

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

Product1: dovrebbe essere elencato una sola volta ... Il codice per quanto sopra è disponibile su pastebin: http://pastebin.com/cNHTBSie

Non capisco da dove provenga il primo "risultato con dati di esempio", ma il problema nell'app della console è che stai usando SelectManyper esaminare ogni elemento di ciascun gruppo .

Penso che tu voglia solo:

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();

L'uso di First()qui per ottenere il nome del prodotto presuppone che ogni prodotto con lo stesso codice prodotto abbia lo stesso nome prodotto. Come notato nei commenti, è possibile raggruppare per nome prodotto e codice prodotto, il che darà gli stessi risultati se il nome è sempre lo stesso per un determinato codice, ma apparentemente genera un SQL migliore in EF.

Suggerirei anche di modificare rispettivamente le proprietà Quantitye e i tipi: perché usare una proprietà stringa per i dati che non è chiaramente testuale?Priceintdecimal

La seguente query funziona. Usa ogni gruppo per fare la selezione invece di SelectMany. SelectManyfunziona su ciascun elemento di ogni raccolta. Ad esempio, nella tua query hai un risultato di 2 raccolte. SelectManyottiene tutti i risultati, per un totale di 3, invece di ogni raccolta. Il seguente codice funziona su ciascuno IGroupingnella parte selezionata per far funzionare correttamente le operazioni di aggregazione.

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(),
              };

a volte è necessario selezionare alcuni campi FirstOrDefault()oppure singleOrDefault()è possibile utilizzare la query seguente:

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();