LINQを使用してメソッドを改善する

Question

次の方法を記述する方法はありますか？多分LINQを使用する？ 新しいリストを宣言してからforeachを使用してリストを作成するのではなく、LINQを使用して1行でこれを行うより良い方法がありますか？あなたがIEnumerable<RoleAreasModel>を返すので

public IEnumerable<RoleAreasModel> GetRolesWithRoleAreasModelByUserGUID(Guid guid) 
    { 
     IList<RoleAreasModel> rolesWithRoleAreas = new List<RoleAreasModel>(); 
     foreach (AspnetRolesRecord role in GetByUserGUID(guid)) 
     { 
     RoleAreasRecord roleAreaRecord = RoleAreasService.Instance.GetRecord(RoleAreasRecord.Fields.ID, role.ID); 
     rolesWithRoleAreas.Add(new RoleAreasModel(roleAreaRecord.Area, new RolesMapper().MapToModel(role), getAreaControllersData(roleAreaRecord.ID))); 
     } 
     return rolesWithRoleAreas; 
    } 

Answer 1

public IEnumerable<RoleAreasModel> Foo(Guid id) 
{ 
    var mapper = new RolesMapper(); 
    return from role in GetByUserGUID(id) 
      let areaRecord = RoleAreasService.Instance.GetRecord(RoleAreasRecord.Fields.ID, role.ID) 
      select new RoleAreasModel(areaRecord.Area, mapper.MapToModel(role), getAreaControllersData(areaRecord.ID)); 
} 

はそれが役に立てば幸い読みください。

Answer 2

、なぜ一時リストを作成するのではなく、yield returnを使うのか？ところで

public IEnumerable<RoleAreasModel> GetRolesWithRoleAreasModelByUserGUID(Guid guid) 
{ 
    foreach (AspnetRolesRecord role in GetByUserGUID(guid)) 
    { 
     RoleAreasRecord roleAreaRecord = RoleAreasService.Instance.GetRecord(RoleAreasRecord.Fields.ID, role.ID); 
     yield return new RoleAreasModel(roleAreaRecord.Area, new RolesMapper().MapToModel(role), getAreaControllersData(roleAreaRecord.ID)); 
    } 
} 

、エリックリッペルトはイテレータとそれらがどのようにC＃で働くに関するブログ記事の非常に興味深いシリーズを書きました。非常に価値が

http://blogs.msdn.com/b/ericlippert/archive/2009/07/09/iterator-blocks-part-one.aspx

http://blogs.msdn.com/b/ericlippert/archive/tags/iterators/

Answer 3

これはいかがですか？

public IEnumerable<RoleAreasModel> GetRolesWithRoleAreasModelByUserGUID(Guid guid) 
{ 
    return GetByUserGUID(guid).Select(role => 
    { 
     var roleAreaRecord = RoleAreasService.Instance.GetRecord(RoleAreasRecord.Fields.ID, role.ID); 
     return new RoleAreasModel(roleAreaRecord.Area, new RolesMapper().MapToModel(role),getAreaControllersData(roleAreaRecord.ID))); 
    }); 
}