3

本节仅从 Excel 电子表格中读取。这部分工作正常，没有性能问题。

IEnumerable<ImportViewModel> so=data.Select(row=>new ImportViewModel{
                  PersonId=(row.Field<string>("person_id")),
                  ValidationResult = ""
                  }).ToList();

在我传递给视图之前，我想设置 ValidationResult，所以我有这段代码。如果我将此注释掉，模型将快速传递给视图。当我使用 foreach 时，它需要一分钟多的时间。如果我硬编码 item.PersonId 的值，那么它运行得很快。我知道我做错了什么，只是不确定从哪里开始以及我应该遵循的最佳实践是什么。

foreach (var item in so)
                {
                    if (db.Entity.Any(w => w.ID == item.PersonId))
                    {
                        item.ValidationResult = "Successful";
                    }
                    else
                    {
                        item.ValidationResult = "Error:  ";
                    }
                 } 

return View(so.ToList());
4

2 回答 2

5

您现在正在对列表中的每个项目执行数据库调用。这对你的数据库和你的性能来说真的很困难。尝试遍历您的 Excel 结果，收集所有用户并在一个查询中选择他们。从此查询结果制作一个列表（否则每次访问该列表时都会执行查询调用）。然后在结果列表和您的 Excel 之间进行匹配。

于 2012-04-22T08:29:58.377 回答
2

你需要做这样的事情：

var ids = so.Select(i=>i.PersonId).Distinct().ToList();
// Hitting Database just for this time to get all Users Ids
var usersIds = db.Entity.Where(u=>ids.Contains(u.ID)).Select(u=>u.ID).ToList();
foreach (var item in so)
                {
                    if (usersIds.Contains(item.PersonId))
                    {
                        item.ValidationResult = "Successful";
                    }
                    else
                    {
                        item.ValidationResult = "Error:  ";
                    }
                 } 

return View(so.ToList());
于 2012-04-22T11:32:03.493 回答