问题描述
|
我正在MVC 3 Web应用程序中使用NHibernate将来的查询,并试图使我所有的数据库访问都在我的控制器中而不是在我的视图中进行。该站点是资源(描述性摘要)目录,其中包含多对多的成绩集合和多对多的主题集合。用户选择一个或多个等级以及一个或多个主题,然后获得匹配资源的列表。
为了填充搜索表单,我正在使用将来的查询来获取所有成绩和主题:
public Domain.SearchFormData GetSearchFormData()
{
// Get all grades and all topics using a multiquery.
IEnumerable<Grade> grades = Session.QueryOver<Grade>()
.Future();
IEnumerable<Topic> topics = Session.QueryOver<Topic>()
.Future();
var result = new SearchFormData();
result.Grades = grades;
result.Topics = topics;
return result;
}
这非常简单并且可以正常工作,并返回SearchFormData
,这是一个简单的DTO。该查询是在控制器中执行的,结果不需要任何进一步的处理,因此它将直接传递到视图中以呈现为复选框列表。
但是由于以后查询的延迟执行,直到视图开始迭代列表时才触发数据库访问。某些人(例如NHibernate Profiler)认为这是不可以的,他们说到视图开始渲染时,所有的数据库访问都应该完成。主要原因是,如果不这样做,很容易进行隐藏的SELECT N + 1。
这里不适用;年级和主题都是没有收藏的简单对象。我不太介意该视图将触发数据库访问。但这使我寻找一种干净,清晰的方式来触发所有将来的查询。一种方法是访问结果,例如将“ 2”复制到列表中。但这并不是我真正需要做的。我只想执行排队的查询。
我认为触发发生在上面的查询之外是有意义的。我可能需要其他数据来呈现整个页面视图,并且可能正在使用其他将来的查询来获取它。例如,主页可能还会显示五个最受欢迎的资源和资源总数。然后,控制器将调用多种方法来累积视图所需的内容,并且一次执行所有这些方法将是最有效的。当然,一种方法是将我的GetSearchFormData
扩展为GetAllHomePageData
,并返回一个DTO,其中包含主页上所有内容的字段。但是,我不仅在主页上使用搜索表单。因此,那样会丢失一些很好的模块化。
解决方法
我认为最好的“解决方案”根本不是在视图模型类中使用IEnumerable-为什么不使用简单数组?这将迫使您在控制器中完全填充模型,并且不会丢失Linq功能等。
第二个问题是您直接在SearchFormData中使用域实体(等级,主题),而应该使用DTO。您提到SearchFormData是DTO,但这并不是完全正确的,因为它引用了您的域实体。
因此SearchFormData应该看起来像这样:
public class SearchFormData
{
public GradeDTO[] Grades { get; set; }
public TopicDTO[] Topics { get; set; }
}
您真正的问题是与设计有关的问题,而不是NHibernate期货问题IMO。
, 以下方法需要改进,但无论如何都可以。我希望它对某人有用。我可能稍后再回来清理。
对于LINQ,HQL和SQL查询期货,请使用:
public static void ExecuteFutureQueries(this ISession session)
{
var sessionImpl = (ISessionImplementor) session;
var dummy = sessionImpl.FutureQueryBatch.Results;
}
对于QueryOver和ICriteria期货,请使用:
public static void ExecuteFutureCriteria(this ISession session)
{
var sessionImpl = (ISessionImplementor) session;
var dummy = sessionImpl.FutureCriteriaBatch.Results;
}
不过要小心。如果以后没有该类型的查询,则调用此方法将导致引发异常。
, Daniel Schilling的回答很好,但是自那时以来,NH的API发生了一些变化。在以后的分批处理中不再提供属性“ 8”。
对于NHibernate 5.0,它看起来应该差不多:
var eng = (NHibernate.Engine.ISessionImplementor)sess;
await eng.FutureCriteriaBatch.GetEnumerator<object>().GetEnumerableAsync();
await eng.FutureQueryBatch.GetEnumerator<object>().GetEnumerableAsync();
代替GetEnumerator
,也可以使用GetFutureValue
。
如果不能使用await
,则可以使用task.Result进行阻塞等待,也可以使用现有的同步版本GetEnumerableAsync
/GetValueAsync
。
同样,Daniel \的警告仍然适用:当您尝试执行FutureCriteriaBatch和FutureQueryBatch时,它们都将抛出并且它们为空。
不幸的是,在NH5.0中无法检查它们是否为空,因此唯一的选择是:
尝试捕获忽略
使用反射检查FutureCriteriaBatch.queries.Count
添加一个虚拟的将来查询以确保至少有一个