Duvida sobre Herança

Olá

enum TipoPessoa {
    FISICA, JURIDICA;
}

class Pessoa {
    private String nome;
    private TipoPessoa tipo;
    private String cadastroNacional;
}

podes fazer um cast tipo:

Pessoa pessoa = new PessoaFisica(nome, cpf);
PessoaFisica pf = (PessoaFisica)pessoa;
pf.getCpf();

tnaires:

Olá Sugestão de design, livrando-se da herança - e livrando-se também do instanceof, que fatalmente você precisaria usar:

enum TipoPessoa {
    FISICA, JURIDICA;
}

class Pessoa {
    private String nome;
    private TipoPessoa tipo;
    private String cadastroNacional;
}

Dessa forma você pode colocar métodos de validação, formatação, e tudo o que for específico para o tipo da pessoa no enum.

Muito obrigado, não tinha pensado em usar enum para o caso, mas mesmo assim poderia explicar como eu queria fazer? Gosto de saber os metodos bons de fazer alguma coisa, mas para isso também gosto de conhecer os ruins para saber o motivo do desenvolvimento dessa solução de design obrigado!

a solução do amigo tnaires é melhor, mas outra solução seria criar uma interface Pessoa:

interface Pessoa {
    String getCadastroNacional();
}

class PessoaFisica implements Pessoa {
    String cpf;

    public PessoaFisica(String cpf) {
        this.cpf = cpf;
    }

    public String getCadastroNacional() {
        return cpf;
    }
}

class PessoaJuridica implements Pessoa {
    String cnjp;

    public PessoaJuridica(String cnpj) {
        this.cnpj = cnpj;
    }

    public String getCadastroNacional() {
        return cnjp;
    }
}

public class Test {
    public static void main(String[] args) {
        Test t = new Test();
        Pessoa pessoa1 = new PessoaFisica("[CPF removido]");
        t.imprimeCadastroNacional(pessoa1);
        Pessoa pessoa2 = new PessoaJuridica("123.123.123/1234-12");
        t.imprimeCadastroNacional(pessoa2);
    }

    public void imprimeCadastroNacional(Pessoa pessoa) {
        System.out.println(pessoa.getCadastroNacional());
    }
}

marcosharbs:

podes fazer um cast tipo:

Pessoa pessoa = new PessoaFisica(nome, cpf); PessoaFisica pf = (PessoaFisica)pessoa; pf.getCpf();

teria que ser algo como:

Pessoa pessoa = new PessoaFisica(nome, cpf);

if (pessoa instanceof PessoaFisica) {
    PessoaFisica pf = (PessoaFisica)pessoa;
    pf.getCpf();
}

Como falei, você precisaria do instanceof para saber o tipo de pessoa que você está lidando. Observe o código do wbdsjunior no post anterior.

Como falei, você precisaria do instanceof para saber o tipo de pessoa que você está lidando. Observe o código do wbdsjunior no post anterior.

Muito obrigado, entendi sim, no caso eu estava usando herança em uma hora errada, já que pessoaFisica e pessoaJuridica são tipos, no caso a herança seria valida se eu criasse uma classe funcionario e extendesse ela a pessoa!
Aproveitando o tópico alguem tem alguma indicação de livro ou material sobre padrões de projeto, design e afins? Sou novo em OO e as vezes me pego com essas duvidas! Obrigado!

Outra duvida no enum eu crio metodos de set e get do cadastro nacional? e depois atribuo ele a variavel no objeto pessoa?

Santhell:

Aproveitando o tópico alguem tem alguma indicação de livro ou material sobre padrões de projeto, design e afins? Sou novo em OO e as vezes me pego com essas duvidas! Obrigado!

Porém eu achei o livro Head First Design Patterns mais digerível:
http://www.amazon.com/First-Design-Patterns-Elisabeth-Freeman/dp/[telefone removido]/ref=sr_1_2?ie=UTF8&s=books&qid=[telefone removido]&sr=8-2

Santhell:

Outra duvida no enum eu crio metodos de set e get do cadastro nacional? e depois atribuo ele a variavel no objeto pessoa?

public enum TipoPessoa {
    FISICA {
        boolean validar(String cadastroNacional) {
            // Retorna false se o CPF for inválido
        }
    },
    JURIDICA {
        boolean validar(String cadastroNacional) {
            // Retorna false se o CNPJ for inválido
        }
    };

    abstract boolean validar(String cadastroNacional);
}

public class Pessoa {
    // Definições mostradas anteriormente

    public boolean validar(String cadastroNacional) {
        return this.tipoPessoa.validar(cadastroNacional);
    }
}