ForeachはforeachからViewModelに各項目を追加しません

Question

私はこれについてあまりにも多く考えていると確信していますが、リストを取得しようとしています。そして、なぜそれがそれをやっているのか知っています。それは、ループの外側に割り当てられ、その1つのアイテムだけを返すforeachの最後のアイテムを取得しています。しかし、私はそれらのすべてを返すしたい。 私は何が間違っていますか？

var cheeses = _repository.GetAllYearSetupIds(); 
var gouda = new CollectionsManagementViewModel(); 
var yearSetupId = 0; 
foreach(var cheddar in cheeses) 
{ 
    yearSetupId = cheddar.YearSetupId; 
    gouda = _repository.GetOverdueBalances(page, pageLength, yearSetupId, balancefilter, sort, direction == Constants.ascending, spreadsheetType); 
    gouda.Title = title + " Management"; 
}  
return View("CollectionsManagement", gouda); 

Answer 1

現在、あなたはループの各反復でCollectionsManagementViewModelという名前goudaの単一のインスタンスを更新しています。ループgoudaの後に、最後の反復の値が入ります。

各反復で新しいインスタンスCollectionsManagementViewModelを作成し、このインスタンスをビューモデルのリストに追加する必要があります。もちろん、意味のあるべき命名：

// list of models, because you want ALL of them 
var managementModels = new List<CollectionsManagementViewModel>(); 
var setupIds = _repository.GetAllYearSetupIds(); 

foreach(var setupId in setupIds) 
{ 
    // new model created for each setup id 
    var managementModel = _repository.GetOverdueBalances(page, pageLength, 
      setupId.YearSetupId, balancefilter, 
      sort, direction == Constants.ascending, 
      spreadsheetType); 

    managementModel.Title = title + " Management"; 
    managementModels.Add(managementModel); // add model to list 
} 

// pass collection to view 
return View("CollectionsManagement", managementModels); 

Answer 2

こんにちはホープ・以下のコードも役立ちます

var cheeses = _repository.GetAllYearSetupIds(); 
var lstgouda = new List<CollectionsManagementViewModel>(); //make a list 
var yearSetupId = 0; 
foreach(var cheddar in cheeses) 
{ 

    var gouda = new CollectionsManagementViewModel(); 
    yearSetupId = cheddar.YearSetupId; 
    gouda = _repository.GetOverdueBalances(page, pageLength, yearSetupId, balancefilter, sort, direction == Constants.ascending, spreadsheetType); 
    gouda.Title = title + " Management"; 
    lstgouda.add(gouda); //add it to list 
}  
return View("CollectionsManagement", lstgouda); 

おかげ カルティク