从 linq 语句中调用函数

Question

只是想知道这是否是最有效的方法？有没有一种方法可以将所有 linq 包含在一个语句中，而不是调用方法，例如子选择或其他方法？

newEmployee = (from emp
               in db.employees
               select new 
               {
                   a.EmployeeID,
                   a.Username,
                   Status = emp.GetEmployeeCurrentStatus(a.Username)
               }).ToList();

这是返回员工状态的 GetEmployeeCurrentStatus：

 public string GetEmployeeCurrentStatus(string username)
        {
            using (Entities db = new Entities())
            {
                var times = (from d in db.TimeTables
                             where d.DateTime == DateTime.Today &&
                             d.Employee.Username == username
                             select d)
                             .OrderByDescending(d => d.TimeID).FirstOrDefault();

                return (x.ClockOut == null ? "IN" : "OUT");                                
            }
        }

Answer 1

怎么样：

newEmployee = (db.employees.Select(emp => new
                  {
                      emp.EmployeeID,
                      emp.Username,
                      Status = db.TimeTables
                        .Where(d => d.Employee.Username == emp.Username
                          && d.DateTime == DateTime.Today)
                          .Select(x => x.ClockOut == null ? "IN" : "OUT")
                          .FirstOrDefault()
                  })).ToList();

您的尝试可能看起来更干净并且功能正常。但是，它正在启动辅助数据库调用。这将不利于可扩展性和性能。我发布的版本使用相同的初始数据库连接，并且将使连接成为 1-1。这将导致更严格、更快的查询以及更低的资源使用。

Answer 2

您无法真正在查询（或将使用数据库执行的查询的一部分）内调用自定义方法。您基本上有两个选择：

• 在执行需要调用该方法的select之前调用ToList（这样，将在内存数据上调用该方法）

• 编写如果可能的话，查询可以全部在 SQL Server 上运行。这可以使用 谓词生成器 中的 AsExpandable 扩展来完成。有关其工作原理的详细信息，另请参阅我的博客文章。

Answer 3

对于小数据（员工计数）来说这很好，但由于每个 GetEmployeeCurrentStatus 都需要一个 sql 新连接，因此这不是最佳实践。
我个人将获取所有员工（一次数据库访问），然后获取所有员工状态（一次数据库访问），所以我将它们全部兑现，现在我将在本地加入他们

希望这有帮助

Answer 4

不管效率如何，使用 GetEmployeeCurrentStatus(...) 作为方法会使代码更清晰且更可重用。

Answer 5

假设您使用 LINQ to SQL 或 EF，我将重构您的查询以使用 Join。这样，您将在数据库上执行单个高效的 SQL 查询，而不是两个单独的查询。