重构复杂的linq查询

Question

我创建了这个linq管道，该管道返回的dto非常适合我的需求。 但是，当我尝试保持代码整洁时，我看不到别人比我容易读懂它。 有什么干净的方法可以做到吗？

    public static IEnumerable<SubscriptionOfferList> GetSubscriptionOffers(this IEnumerable<Product> products, IEnumerable<Plan> plans) =>
         products
             .GroupBy(p => p.Metadata["SubscriptionType"])
             .Select(productGroup => new SubscriptionOfferList
             {
                Name = productGroup.Key,
                Offers = productGroup.Select(p => new SubscriptionOffer
                {
                    Id = p.Id,
                    Name = p.Name,
                    Price = p.Metadata["SubscriptionPrice"],
                    Plans = plans
                    .Where(plan => plan.ProductId == p.Id)
                    .Select(plan => new PaymentPlan
                    {
                        Name = plan.Nickname,
                        Id = plan.Id,
                        Price = plan.Tiers ?? new List<PlanTier>
                        {
                            new PlanTier
                            {
                                UnitAmount = plan.Amount.GetValueOrDefault(),
                                UpTo = null
                            }
                        },
                    }).ToList()
                }).ToList(),
             });

Answer 1

我不知道为什么注释不好，只是将其分解为较小的功能

private static List<Plan> MakePlans(IEnumerable<Plan> plans, int pid)
{
    return plans
            .Where(plan => plan.ProductId == pid)
            .Select(plan => new PaymentPlan
            {
                Name = plan.Nickname,
                Id = plan.Id,
                Price = plan.Tiers ?? new List<PlanTier>
                {
                    new PlanTier
                    {
                        UnitAmount = plan.Amount.GetValueOrDefault(),
                        UpTo = null
                    }
                },
            }).ToList();
}

public static IEnumerable<SubscriptionOfferList> GetSubscriptionOffers(this IEnumerable<Product> products, IEnumerable<Plan> plans) =>
     products
         .GroupBy(p => p.Metadata["SubscriptionType"])
         .Select(productGroup => new SubscriptionOfferList
         {
            Name = productGroup.Key,
            Offers = productGroup.Select(p => new SubscriptionOffer
            {
                Id = p.Id,
                Name = p.Name,
                Price = p.Metadata["SubscriptionPrice"],
                Plans = MakePlans(plans, p.Id)
            }).ToList(),
         });

Answer 2

在该方法的顶部加上注释，并在整个linq部分中添加一些注释，以确保它们工作正常，只要查询工作正常即可。