寻求有关我的 LINQ 表达式设计的反馈 [关闭]

Question

场景：我有一个在 .NET 4.5 上使用 EF 6.1 和 SQL Server 2012 的 Windows 控制台应用程序。

问题：我想得到一些反馈，以下哪个 LINQ 表达式是从数据库加载数据的首选方式或更有效的方式：

    // Option 1: Using List<>
    public List<Employee> EmployeeList 
    {
         get { return Context.Employees.Select(e => e).ToList(); }
    }

    // Option 2: Using IList<>
    public IList<Employee> EmployeeList2
    {
         get { return Context.Employees.Select(e => e).ToList(); }
    }

    // Option 3: Using IEnumerable<>
    public IEnumerable<Employee> EmployeeList3
    {
         get { return Context.Employees.Select(e => e); }
    }

    // Option 4: Using IQueryable
    public IQueryable<Employee> EmployeeList4
    {
         get { return Context.Employees.Select(e => e); }
    }

    // Option 5: Using IQueryable & AsNoTracking()
    public IQueryable<Employee> EmployeeList5
    {
         get { return Context.Employees.Select(e => e).AsNoTracking(); }
    }

    // Option 6: Using "async/await"
    public async Task<List<Employee>> GetEmployeeList6()
    {
         return await Context.Employees.Select(x => x).ToListAsync(); 
    }

   // Option 7: Using "using" statement
    public List<Employee> GetEmployeeList7()
    {
         using (Context context = new Context())
         {
             return Context.Employees.Select(e => e).ToList();
         }
     }

请注意，我不需要 EF 来跟踪任何更改。我只需要将员工列表放入内存中以便我可以操作它们，这就是我将扩展方法 AsNoTracking() 添加到选项 5 的原因。

Answer 1

你在比较苹果和橘子。您列出的每种方法都有优点和缺点，因此无法回答哪种方法是“首选方法”。

同样，这些示例中的许多将立即返回，因为它们使用延迟执行，因此它们在技术上会运行得更快，但实际上不会让您的程序运行得更快。

但是，如果您只是想获取所有员工的内存列表，并且您不关心更改跟踪，正如您所说，我更喜欢这样：

public IReadOnlyCollection<Employee> EmployeeList
{
     get { return Context.Employees.AsNoTracking().ToList(); }
}

• 您不需要.Select(e => e)：它没有任何作用。
• AsNoTracking() 会给你带来轻微的性能提升。
• .ToList() 确保在方法返回之前将值实际加载到内存中。
• 我个人喜欢使用IReadOnlyCollection<>s，当我想表明我正在提供已加载到内存中的内容时，但我不想给人这样的印象，即使用代码会从我归还的收藏。