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

Question

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

LINQ 生成的文件正在使用泛型和反射将 LINQ 生成的类映射到 SQL 对象(在我们的例子中是表和 View )。

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

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<DVD>(1231)

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

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

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

谢谢

Answer 1

这里有一种方法可以使这种情况更通用一些。 Rince 并重复其他 CRUD 操作。对于某些情况，性能可能是 Not Acceptable 。在那些情况下，我会重组程序的那部分以调用非通用版本。

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" ) );

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