2

Tenho uma tabela com o seguinte esquemas:

GarantiaImovel | imovelID | GarantiasTipos
12             |    3     | Caução2
12             |    4     | Depósito
12             |    5     | Caução2
12             |    5     | Fiador
12             |    5     | Seguro Fiança

Então tenho a seguinte Query:

var garantias = (from p in db.T_GarantiasLocaticias
                         where p.imovelID == 5
                         select p.GarantiasTipos).ToList();

A minha intenção nesta Query é retornar as três linhas na tabela acima com a coluna GaantiasTipo, mas eu usando o foreach e ele só me retorna a ultima linha como posso retornar na seguinte estrutura:

foreach (var garantia in garantias)
{
     lblGarantiaLocaticias.Text = garantia + ", ";
}

Como proceder?

1
  • Era só usar um +=. Assim: lblGarantiaLocaticias.Text += garantia + ", ";. O foreach vai ser produzido de qualquer maneira, seja explicitamente ou no string.Join.
    – Jéf Bueno
    Commented 16/08/2017 às 13:07

2 Respostas 2

1

Se não precisa fazer um foreach é só usar String.Join com o retorno daquele expressão , exemplo:

var garantias = (from p in db.T_GarantiasLocaticias
                  where p.imovelID == 5
                  select p.GarantiasTipos)
               .ToList();

lblGarantiaLocaticias.Text = String.Join(", ", garantias);

Exemplo Online

Observação: até pode utilizar um for ou foreach ou até um while, mas, não há necessidade já visto que String.Join já faz isso de forma transparente. Uma outra observação é sobre o que estava acontecendo, porque faltou acumular os valores já obtidos nas interações e que no foreach da sua pergunta não fazia, no caso seria só colocar um sinal de mais (+) antes do igual (=) (que representa um operador de atribuição de adição) e um ajuste para que no último elemento não tivesse a virgula (,) desta forma:

lblGarantiaLocaticias.Text = "";
foreach (var garantia in garantias)
{
     if (lblGarantiaLocaticias.Text != "") 
         lblGarantiaLocaticias.Text += ", ";
     lblGarantiaLocaticias.Text += garantia;
}

Referencias:

1
  • 1
    Exatamente isso! Eu de tanto olhar outros recursos não atentei para o String.Join Commented 16/08/2017 às 2:14
1

Pode usar o Aggregate(...) (vulgo reduce):

var garantias = (from p in db.T_GarantiasLocaticias
                 where p.imovelID == 5
                 select p.GarantiasTipos)
                .ToList()
                .Aggregate(string.Empty, (actual, novo) => 
                {
                    // caso a string concatenada nao esteja vazia,
                    // adiciona o separador
                    if(!string.IsNullOrEmpty(actual)) {
                        actual += ","
                    }

                    return actual + novo;
                });

Pode ver um exemplo do Aggregate aqui.

3
  • Vendo assim, parece que seria um boa opção, mas, além dos erros de codificação e falta de codificação ele acaba fazendo da mesma forma que o meu fez na primeira solução, portanto, "Se não quiser percorrer a colecção duas vezes, pode usar" não é um argumento válido.
    – novic
    Commented 16/08/2017 às 21:28
  • @VirgilioNovic depende, se o OP usar os objectos trazidos para mais alguma coisa, não há nada de errado em usar o string.Join. Caso queira apenas construir a string, evita materializar a colecçao com o ToList e depois percorrer a mesma novamente com o string.Join(....). Mesmo que nao considere o argumento válido, o Aggregate continua a ser uma alternativa viável. E obrigado por indicar os erros ;)
    – jpmnteiro
    Commented 16/08/2017 às 22:01
  • O Aggregate só vai funcionar depois da materialização da camada ORM (ou seja, no Linq para Objetos) utilizada, ou seja, vai dar na mesma, não disse que essa solução não é viável ele é viável sim, só disse que o argumento utilizado não confere para utilizar a solução.
    – novic
    Commented 16/08/2017 às 22:14

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .