需要使用 Linq 优化循环的帮助

Question

免责声明：我对 linq 的经验很少。

我的工作任务之一是维护一个电子商务网站。昨天，我们的一位客户开始抱怨当他们尝试为谷歌创建提要文件时会发生超时。事实证明，如果用户要在其 feed 文件中放入超过 9,000 个项目，我们的代码至少需要一分钟才能执行。

我无法通过运行调试器找到问题的根源，所以我启动了一个探查器（ANTS）并让它做它的事情。它找到了我们问题的根源，即包含一些 linq 代码的 foreach 循环。代码如下：

var productMappings = GoogleProductMappingAccess.GetGoogleProductMappingsByID(context, productID);
List<google_Category> retCats = new List<google_Category>(numCategories);
int added = 0;

//this line was flagged by the profiler as taking 48.5% of total run time
foreach (google_ProductMapping pm in (from pm in productMappings orderby pm.MappingType descending select pm))
{
    if (pm.GoogleCategoryId.HasValue && pm.GoogleCategoryId > 0)
    {
        //this line was flagged as 36% of the total time
        retCats.Add(pm.google_Category);
    }

    else if (pm.GoogleCategoryMappingId.HasValue && pm.GoogleCategoryMappingId > 0)
    {
        retCats.Add(pm.google_CategoryMapping.google_Category);
    }
    else
    {
        continue;
    }

    if (++added >= numCategories)
    {
        break;
    }
}

你们中更有经验的开发人员有什么想法吗？我正在尝试用 sql 替换所有 linq，但我不确定这是否是最好的做法（如果它是用 linq 编写的，那么一定有原因）。

Answer 1

如果您可以过滤掉您不想要的结果，那么您的查询应该会更快 - 您正在使用 orderby 因此，所有这些结果都会耗尽查询中的处理，因为它们都必须进行评估：

 productMappings.Where( pm => (pm.GoogleCategoryMappingId.HasValue
                                && pm.GoogleCategoryMappingId > 0)
                              ||(pm.GoogleCategoryMappingId.HasValue && 
                                 pm.GoogleCategoryMappingId > 0)
                      )
                .OrderBy(...)

您也可以应限制查询返回的结果数量，因为您最多只使用 numCategories。因此，将 a 添加

.Take(numCategories)

到您的查询中，而不是在 foreach 循环中进行检查。

Answer 2

retCats.Add(pm.google_Category); 花费如此长的时间的原因是您引用了一个延迟加载的对象，该对象会再次往返服务器。
如果您可以重构它，只获取 Id 的本地副本而不是整个对象，那么这将加快该部分的速度。

如果您确实需要获取整个对象，请研究如何在获取 ProductMappings 时通过单个查询将其下拉。如何执行此操作取决于您在 SQL 上使用的 LINQ 包装器。

Answer 3

不知道你的数据库模式真的很难说。有几个想法：

1) 通过数据库引擎优化顾问运行查询。也许查询需要一些索引？

2）预处理这些信息并将其放入另一个表或文件中。这样当谷歌请求它时它就不会超时。

Answer 4

这可能应该有效：

var productMappings = GoogleProductMappingAccess.GetGoogleProductMappingsByID(context, productID);
var categories = from pm in productMappings
                 where pm.GoogleCategoryId > 0 ||
                       pm.GoogleCategoryMappingId > 0
                 orderby pm.MappingType descending
                 select pm.google_Category ??
                        pm.google_CategoryMapping.google_Category;

return categories.Take(numCategories);

如果 GetGoogleProductMappingsByID 返回一个 IQueryable （如果适用），效果最好。如果是这样，LINQ 会将整个语句转换为 T-SQL 命令，这将比内存中的 LINQ 快得多。

请随意将 .ToList() 添加到最后一个语句，以使其具有与代码中相同的返回类型（并强制执行 LINQ 语句）。

检查 .HasValue 和 > 0没用。检查 ID > 0就够了。
有关详细信息：http://msdn.microsoft.com/en-us/library /2cf62fcy.aspx（运算符）