3

Qual a sintaxe correta para chamar o método Add_Pessoa da classe ClsPessoa por uma Lista?

public class ClsPessoa
{
    private static string PSTR_Nome = "";
    private static DateTime PDT_DataNascimento;
    private static string PSTR_Email = "";

    public ClsPessoa()
    {
    }

    static public string Nome
    {
        get { return PSTR_Nome; }
        set { PSTR_Nome = value; }
    }

    static public DateTime DataNascimento
    {
        get { return PDT_DataNascimento; }
        set { PDT_DataNascimento = value; }
    }

    static public string Email
    {
        get { return PSTR_Email; }
        set { PSTR_Email = value; }
    }       

    public void Add_Pessoa(string PV_Nome, DateTime PV_DataNascimento, string PSTR_Email)
    {
        Nome = PV_Nome;
        DataNascimento = PV_DataNascimento;
        Email = PSTR_Email;
    }

Algo assim?

var pessoas = new List<ClsPessoa>();
pessoas[0].Add_Pessoa(TXT_Nome.Text, DTP_DataNascimento.DisplayDate, TXT_Email.Text);

1 Resposta 1

5

Não confunda o objeto Pessoa e o objeto List<Pessoa>. São coisas distintas. Então você não pode "adicionar uma pessoa em uma pessoa". Olha como a frase não faz sentido. Você pode "adicionar uma pessoa à uma lista de pessoas". Então as operações são separadas.

Se vai adicionar, você não escolhe em que posição vai colocar. E assim como no array, você tem criar uma instância da classe desejada e então adicionar a instância:

var pessoas = new List<ClsPessoa>();
pessoas.Add(new ClsPessoa() {Nome = TXT_Nome.Text,
                             DataNascimento = DTP_DataNascimento.DisplayDate,
                             Email = TXT_Email.Text});

Se quiser mudar o conteúdo da posição 0 na lista aí usaria:

pessoas[0] = new ClsPessoa() {Nome = TXT_Nome.Text,
                              DataNascimento = DTP_DataNascimento.DisplayDate,
                              Email = TXT_Email.Text});

Ou se quiser mudar uma propriedade:

pessoas[0].Email = TXT_Email.Text;

Mas tem um jeito melhor de montar a classe toda:

public class Pessoa {
    public Pessoa(string nome, DateTime dataNascimento, string email) {
        Nome = nome ?? ""; //esquisito, mas é o que tinha no código original
        DataNascimento = dataNascimento;
        Email = email ?? "";
    }
    public string Nome { get; set;}
    public DateTime DataNascimento { get; set; }
    public string Email { get; set; }
}

Coloquei no GitHub para referência futura.

Olha como ficou mais simples. Com nomes melhores, sem usar notação húngara que é horrível e não ajuda nada, sem usar coisas fora do padrão. Com sintaxe simplificada de propriedades automáticas. Usando o construtor da forma correta e evitando um método auxiliar que não servia para nada. E principalmente consertando o problema dos membros estáticos. Se vai criar uma classe que precisa ser instanciada, ou seja, vai criar um objeto com dados nela, os membros do objeto não podem ser estáticos.

Ainda dá para melhorar várias coisas nesta classe. Mas vamos por partes. Uma delas seria nem ter um construtor. Neste caso ele não está sendo especialmente útil. Só não digo que ele é completamente desnecessário porque não conheço o requisito. Consigo enxergar algum motivo para tê-lo.

Se realmente precisa ter um classe para abstrair melhor uma lista de pessoas, então outra classe precisa ser criada. Mas isto já é outro assunto.

Se fizer isto, aí pode mudar um pouco a sintaxe acima, embora não vá fazer muita diferença. Eu acho a primeira forma mais legível. Embora dê para deixar a forma abaixo mais legível também. Mas deixa para depois, não vou encher de novidades na sua cabeça.

pessoas.Add(new Pessoa(Nome.Text,
                       DataNascimento.DisplayDate,
                       Email.Text));

Sei que te apresentei várias coisas novas, dê uma pesquisada, se não encontrar nada aqui, faça uma pergunta sobre o que ficou com dúvida (uma pergunta por assunto) e responderemos.

Saiba mais sobre membros estáticos em:

6
  • Apenas uma pequena nota: com esse construtor, as propriedades Nome e DataNascimento deveriam ter o set privado.
    – ramaral
    Commented 11/10/2015 às 21:00
  • @ramaral Até acho que é uma possibilidade. Mas não obrigatoriedade. Ele só deveria ser privado se não quiser deixar as propriedades serem alteradas publicamente. Em geral, não é o que se deseja. A não ser que deseja que o objeto seja imutável. Mas aí também teria que mudar mais algumas coisas.
    – Maniero
    Commented 11/10/2015 às 22:04
  • Pô, muito obrigado pela resposta ... entendi perfeitamente o que quer dizer ... =D facilitou muito a minha aprendizagem.
    – Ayo
    Commented 11/10/2015 às 22:41
  • @Ayo se a resposta de bigown responde à sua dúvida, sinta-se à vontade para aceitá-la clicando no marcador à esquerda da resposta dele.
    – OnoSendai
    Commented 12/10/2015 às 0:33
  • @bigown Só mais uma pergunta. Por que usar var em vez de List<ClsPessoa> pessoas = new List<ClsPessoa>();
    – Ayo
    Commented 12/10/2015 às 13:01

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 .