Não vejo problemas em fazer um foreach
para preencher uma ViewModel
, o problema do seu código é que não precisa fazer esse foreach
, é desnecessário, ficando assim um exemplo:
public class Crm_AnaliseViewModel
{
public string TAG { get; set; }
public int ATUALIZACAO { get; set; }
public stringDateTime RELATORIOS { get; set; }
}
public async Task<ActionResult> Index(Crm_AnaliseViewModel model)
{
if (Session["cod_cli"] != null)
{
int cod_cli = Convert.ToInt32(Session["cod_cli"]);
var query = (from s in db.Crm_Analise
where s.cliente_CRM.Equals(cod_cli)
group s by s.TAG into g
select new Crm_AnaliseViewModel
{
TGTAG = g.Key,
ATRELATORIOS = g.Max(t => t.data_creat),
RLATUALIZACAO = g.Count()
})
.ToList();
return View(query);
}
else
{
return RedirectToAction("Login", "Account");
}
}
Observação: retirei algumas variáveis porque o código não precisa, pelo menos o que está na pergunta.