5

Como utilizar o ForEach() do List na implementação abaixo

using System.Collections.Generic;

public class Program
{
    public static void Main()
    {   
        List<Pessoa> pessoas = new List<Pessoa>();
        pessoas.Add(new Pessoa(){Nome = "José" , Sexo = "M"});
        pessoas.Add(new Pessoa(){Nome = "Pedro", Sexo = "M"});
        pessoas.Add(new Pessoa(){Nome = "João" , Sexo = "M"});
        pessoas.Add(new Pessoa(){Nome = "Maria", Sexo = "F"});

        bool temMulher = new VerificaSeHaMulher(pessoas).Verifica();
    }
}

class VerificaSeHaMulher
{
    List<Pessoa> pessoas;

    public VerificaSeHaMulher(List<Pessoa> pessoas)
    {
        this.pessoas = pessoas;
    }

    public bool Verifica()
    {
        foreach(Pessoa pessoa in pessoas)
          if(pessoa.Sexo.Equals("F"))
            return true;

        return false;
    }
}

class Pessoa
{
    public string Nome;
    public string Sexo;
}

No caso, quero deixar mais limpa a implementação do método Verifica.

5

1 Resposta 1

2

Eu mudaria algumas coisas mais por estilo. Mas é o meu gosto, não quer dizer que é claramente melhor.

Fora isto eu pensaria se este método não deveria estar dentro de outra classe. Tem utilidade em usar desta forma, dependendo dos seus requisitos, mas se você não conseguir dizer qual o motivo disto ser uma classe, está enfeitando o pavão.

using System.Collections.Generic;

public class Program {
    public static void Main() {   
        var pessoas = new List<Pessoa>() {
            new Pessoa() {Nome = "José" , Sexo = "M"},
            new Pessoa() {Nome = "Pedro", Sexo = "M"},
            new Pessoa() {Nome = "João" , Sexo = "M"},
            new Pessoa() {Nome = "Maria", Sexo = "F"}
        };
        bool temMulher = new VerificaSeHaMulher(pessoas).Verifica();
    }
}

class VerificaSeHaMulher {
    private readonly List<Pessoa> pessoas;
    public VerificaSeHaMulher(List<Pessoa> pessoas) => this.pessoas = pessoas;

    public bool Verifica() {
        foreach (var pessoa in pessoas) if (pessoa.Sexo == "F") return true;
        return false;
    }
}

class Pessoa {
    public string Nome;
    public string Sexo;
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Se quiser usar LINQ basta fazer isto:

return pessoas.Any(p => p.Sexo == "F");

Veja funcionando no .NET Fiddle.

Documentação do Any().

1
  • Cara, era isso que eu precisava, obrigado. 16/10/2015 às 23:23

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 .