数据访问层 - LINQ-To-SQL 和泛型。我可以优化这个吗？

Question

我们正在努力改进 DAL，它是用 LINQ 编写的，可与 MS SQL 数据库通信。我们的目标是用尽可能少的代码实现良好的可重用性。

LINQ 生成的文件利用泛型和反射将 LINQ 生成的类映射到 SQL 对象（在我们的例子中为表和视图）。

请参阅现有访问器的示例。该方法驻留在包含自定义构造函数、访问器和修改器的分部类中：

public clsDVD getDVD(int dvdId)
{
   try
   {
      using (DataContext dvdDC = new DataContext(ConnectionStringManager.getLiveConnStr()))
      {
                    // Deferred loading
                    dvdDC.DeferredLoadingEnabled = false;

                    var tDVD = dvdDC.GetTable<DVD>();

                    return (from t in tDVD 
                            // Filter on DVD Id
                            where t.DVDId == (dvdId)
                            select t).Single();                   
      }
   catch (Exception e)
   {
       Logger.Log("Can't get requested DVD.", e);
       throw;
   }
 }

我相信这非常容易维护，因为大部分工作都是在 var tDVD 之后完成的，

建议不要这样做声明 tDVD 并使用 dataContext.TableName，但在幕后它仍然调用 GetTable。

我认为改进这一点的唯一方法是将这个部分类分成 4 个（CRUD）部分类。例如，

clsDVD_Select, clsDVD_Update, clsDVD_Insert, clsDVD_Delete

在这种情况下，每个类将代表一组行为。

我们正在讨论的想法是看看是否可以在 LINQ 泛型之上使用泛型。

例如，我们不会使用部分类，而是通过使用针对 SQL 数据库的反射来动态找出类的属性。我首先关心的是性能影响。将会有多么重大的意义。

我们将使用以下内容代替 ClsDVD.getDVD(1231)： GenericDC.Select(1231)

.Select方法将找出主键并对该表运行选择查询。我很难理解这是如何运作的。假设我们可以让它适用于简单的选择，即使用主键过滤器进行选择，但是当我们开始进行复杂的连接和分组选择时会发生什么。当我们希望每个 DVD 类别有多个选择时会发生什么？

我最后关心的是良好实践。以前有人告诉我，拥有一致的代码是件好事。例如，如果我决定使用数据表，那么我应该在整个项目中坚持使用数据表。让项目的一半包含数据表，另一半包含用户定义的类，这是一个坏主意。你同意这一点吗？

我认为现有的实现非常好，但也许我错过了一些非常明显的东西，并且有一种更简单、更面向对象的方法来实现相同的结果？

谢谢

Answer 1

这是使这种情况更加通用的一种方法。冲洗并重复其他 CRUD 操作。对于某些情况，性能可能不可接受。在这些情况下，我会重组程序的该部分以调用非通用版本。

public T GetSingleItem(Func<T,bool> idSelector ) where T : ??? // forgot what type it needs to be off the top of my head
{ 
    try 
    { 
        using (DataContext context = new DataContext(ConnectionStringManager.getLiveConnStr())) 
        { 
            context.DeferredLoadingEnabled = false; 
            return context.GetTable<T>().Single( item => idSelector( item );
        } 
    }
    catch (Exception e) 
    { 
        Logger.Log("Can't get requested item.", e); 
        throw; 
    } 
} 

这就是您获得该物品的方式。不太优雅，因为您必须告诉通用函数您将使用哪一列。

GenericDC.GetSingleItem<DVD>( dvd => dvd.ID == 1231 )

为了使其更加通用，将其限制为具有 ID 的单个项目...

public IEnumerable<T> GetItems(Func<T,bool> selectFunction ) where T : ??? // forgot what type it needs to be off the top of my head
{ 
    try 
    { 
        using (DataContext context = new DataContext(ConnectionStringManager.getLiveConnStr())) 
        { 
            context.DeferredLoadingEnabled = false; 
            return context.GetTable<T>().Select( item => selectFunction( item );
        } 
    }
    catch (Exception e) 
    { 
        Logger.Log("Can't get requested item.", e); 
        throw; 
    } 
} 

那么您可以这样调用它：

GenericDC.GetItems<DVD>( dvd => dvd.Title == "Title" && dvd.Cast.Contains( "Actor" ) );

另一种可能的解决方案是创建一个自定义代码生成器，您可以在一个地方对其进行修改并创建类似的例程对于所有其他类型。如果您遇到性能问题，这可能是一个很好的解决方案。您可能希望限制对您使用的模板代码段的更改。