Skip to main content
adicionou 15 caracteres ao conteúdo
Fonte Link
novic
  • 36,7mil
  • 4
  • 32
  • 70

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.

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 string 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
                    {
                        TG = g.Key,
                        AT = g.Max(t => t.data_creat),
                        RL = 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.

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 DateTime 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
                    {
                        TAG = g.Key,
                        RELATORIOS = g.Max(t => t.data_creat),
                        ATUALIZACAO = 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.

Fonte Link
novic
  • 36,7mil
  • 4
  • 32
  • 70

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 string 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
                    {
                        TG = g.Key,
                        AT = g.Max(t => t.data_creat),
                        RL = 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.