问题描述
|
我这里有这组数据。事件的属性为“ 0”,类型为“ 1”
List<Events> e;
List<Groups> g;
// Get the data from the database using dapper
using( var con = DataAccessMaster.GetopenConnection( ) ) {
using( var multi = con.QueryMultiple( sprocname,new { StartDate = fromDate,EndDate = toDate },commandType:CommandType.StoredProcedure ) ) {
e = multi.Read<Events>( ).ToList( );
g = multi.Read<Groups>().ToList();
}
}
// Only put the groups that belong to one another within the related event so that when we goto bind it will be painless
foreach ( var ev in e ) {
ev.EventGroups = new List<Groups>();
foreach ( Groups group in g.Where( Groups => ( ev.EventID == Groups.EventID ) ) ) {
ev.EventGroups.Add( group );
}
}
return e;
我觉得最后一块可以比原来更干净地重写。我该怎么做才能使它更清洁?
解决方法
您可以使用Enumerable.ToList扩展方法将IEnumerable 转换为新的List :
foreach (var ev in e)
{
ev.EventGroups = g.Where(groups => ev.EventID == groups.EventID)
.ToList();
}
, 您可以用ToList()
折叠内部循环。
foreach ( var ev in e ) {
ev.EventGroups = g.Where( Groups => ( ev.EventID == Groups.EventID ) ).ToList();
}
外循环已经达到了LINQy的水平,因为它是一个副作用循环,而那些并不是LINQy。
, 举例来说
ev.EventGroups = g.Where( Groups => ( ev.EventID == Groups.EventID )).ToList();
浮现在脑海。