0

我正在尝试构建这个对象：

[
    {
        "color" : "red",
        "category" : "tshirts",
        "Items" : [
            {
                "r" : 1,
                "n" : "name: E624E",
                "s" : 9819217.000
            }, {
                "r" : 2,
                "n" : "name: 00F37",
                "s" : 9791564.000
            }, {
                "r" : 3,
                "n" : "name: 75B02",
                "s" : 9790543.000
            }, {
                "r" : 4,
                "n" : "name: 08864",
                "s" : 9485392.000
            }
        ]
    }, 
    {
        "color" : "red",
        "category" : "shoes",
        "Items" : [
            {
                "r" : 1,
                "n" : "name: 20272",
                "s" : 9949541.000
            }, {
                "r" : 2,
                "n" : "name: 1E496",
                "s" : 9926730.000
            }, {
                "r" : 3,
                "n" : "name: 00F37",
                "s" : 9926493.000
            }, {
                "r" : 4,
                "n" : "name: 48A44",
                "s" : 9923929.000
            }
        ]
    }
]

我构建它的方式是使用 Linq，遍历颜色和类别的集合：

var colors = ctx.Colors.Select(x => x.Color).Distinct().ToList();

foreach (var color in colors)
{
    var categories = ctx.Categories.Where(x => x.Color == color).Select(x => x.Category).Distinct().ToList();

    foreach (var cat in categories)
    {
        List<ItemAttribute> iattr = (from i in ctx.Item
                                     where i.Color == color && i.Category == cat
                                     select new ItemAttribute { 
                                          r = i.R,
                                          n = i.N,
                                          s = i.S
                                      }).ToList();
        this.ItemStuff.Add(iattr);
    }
}

这行得通，我得到了预期的结果，但性能很糟糕而且看起来很冗长。

任何人都可以提供一些改进这一点的见解吗？

4

2 回答 2

3

也许你正在寻找这样的东西：

var results = 
    from i in ctx.Items
    group i by new { i.Color, i.Category } into g
    select new ItemGroup
    {
        color = g.Key.Color,
        category = g.Key.Category
        items =
            (from i in g 
             select new ItemAttribute
             { 
                 r = i.R,
                 n = i.N,
                 s = i.S
             })
            .ToArray()
    };

或流利的语法：

var results = ctx.Items
    .GroupBy(i => new { i.Color, i.Category })
    .Select(g => new ItemGroup
    {
        color = g.Key.Color,
        category = g.Key.Category
        items = g.Select(i => new ItemAttribute
                { 
                    r = i.R,
                    n = i.N,
                    s = i.S
                })
                .ToArray()
    };
于 2013-09-09T15:57:07.270 回答
1

您的代码在多个地方都遵循相同的次优模式：您一次处理单一颜色或单一类别，而不是按颜色或类别进行分组。迭代时在外循环中执行此colors操作，然后在迭代时在嵌套循环中执行相同操作categories。

这会发生ctx多次，每次都有一个单独的查询。在极少数例外情况下，循环查询肯定会影响您的性能。最好获取所有内容，按颜色/类别分组，然后按照您需要的方式构建结果。

这是一种方法：

var items = ctx.Item.Select(i => new ItemAttribute { 
    r = i.R
,   n = i.N
,   s = i.S
})
.GroupBy(i => new { i.Color, i.Category })
.OrderBy(p => p.Key.Color)
.ThenBy(p => p.Key.Category);

此时，您拥有按颜色和类别分组的所有项目的组，在单个查询中检索。现在剩下的是迭代组，并将数据添加到您正在填充的结构中。

于 2013-09-09T16:00:59.827 回答