将内连接分组SQL转换为Linq及现有写法正确性咨询

你的当前Linq写法是正确的，确实能得到品牌键和对应数量总和的结果，但正如你感觉到的，它绕了不必要的弯路——中间两次创建匿名类投影来传递数据，不仅让代码冗余，还增加了不必要的内存开销。我们可以直接对齐原SQL的逻辑，写出更简洁高效的Linq代码。

先回顾你的原SQL核心逻辑：

SELECT b.Brand_key, b.BrandName, Sum(po.Quantity) AS Quantity
FROM Brands b
INNER JOIN Products p ON b.Brand_key = p.Brand_fkey
INNER JOIN ProductOrders po ON p.Product_key = po.Product_fkey
GROUP BY b.Brand_key
ORDER BY Quantity DESC, b.BrandName ASC

核心流程是：关联三张表 → 按Brand_key分组（同时保留BrandName用于排序）→ 求和Quantity → 按数量降序、品牌名升序排序。

优化后的查询语法写法 #

var testModelTwo = (from b in db.Brands
                    // 关联Products表
                    join p in db.Products on b.brandKey equals p.brandKey
                    // 关联ProductOrders表
                    join po in pom on p.productKey equals po.Product_fkey
                    // 直接按品牌键和品牌名分组（避免后续再关联品牌表）
                    group po by new { b.brandKey, b.BrandName } into groupedData
                    // 直接提取分组键的字段并完成求和
                    select new 
                    {
                        Brand_key = groupedData.Key.brandKey,
                        BrandName = groupedData.Key.BrandName,
                        Quantity = groupedData.Sum(item => item.Quantity)
                    })
                    // 补上原SQL中的排序逻辑（你的原始Linq漏掉了这一步）
                    .OrderByDescending(result => result.Quantity)
                    .ThenBy(result => result.BrandName)
                    .ToList();

这个写法的优势：

• 去掉了中间两次冗余的匿名类投影，直接在关联后进行分组，代码更简洁易读
• 分组时把BrandName包含在分组键中，分组后无需再关联Brands表就能获取品牌名称
• 完全对齐原SQL的排序逻辑，结果行为和原SQL一致

如果你习惯方法链语法，也可以这么写 #

var testModelTwo = db.Brands
    // 关联Products表，保留品牌和产品对象
    .Join(db.Products, 
          brand => brand.brandKey, 
          product => product.brandKey, 
          (brand, product) => new { Brand = brand, Product = product })
    // 关联ProductOrders表，保留之前的关联结果和订单对象
    .Join(pom, 
          bp => bp.Product.productKey, 
          order => order.Product_fkey, 
          (bp, order) => new { bp.Brand, Order = order })
    // 按品牌键和品牌名分组
    .GroupBy(item => new { item.Brand.brandKey, item.Brand.BrandName })
    // 投影出最终需要的字段并求和
    .Select(group => new 
    {
        Brand_key = group.Key.brandKey,
        BrandName = group.Key.BrandName,
        Quantity = group.Sum(item => item.Order.Quantity)
    })
    // 执行排序
    .OrderByDescending(result => result.Quantity)
    .ThenBy(result => result.BrandName)
    .ToList();

总结来说，你的原始写法逻辑是正确的，但通过直接在分组时包含所需的非聚合字段，可以大幅简化代码，避免不必要的中间步骤。优化后的写法既保持了逻辑正确性，又更贴近原SQL的直观逻辑。

内容的提问来源于stack exchange，提问作者Richard Whitehouse