Opinião sobre o código para me orientar sobre as regras de OO

Poste o código da sua classe funcionário.

Compile e rode isso:

f1.data.dia = -1000;

Essa classe permite isso, não?

Portanto, os dados não estão encapsulados permitindo que o objeto assuma um “estado” inválido. Primeira opinião

Até mais,

phph:

Poste o código da sua classe funcionário.

Compile e rode isso:

f1.data.dia = -1000;

Essa classe permite isso, não?

Portanto, os dados não estão encapsulados permitindo que o objeto assuma um “estado” inválido. Primeira opinião

Até mais,

A classe esta no outro PC, mas não, não fiz o encapsulamento pra mermitir que só entre dados entre 1 e 31, mas o que eu quero é que vocês olhem esse código, pois estou aprendendo ainda array.

Quando ao break, você pode usar um return caso o que você já tenha executado a operação que o método é responsável.

Mas em geral não adianta muito se preocupar com OO se seu código tem problemas de lógica. Por exemplo:

...	
	void adiciona(Funcionario f) {
		funcionarios = new Funcionario[10];
		...

Neste caso você está iniciando seu array toda vez que adiciona um funcionário, ou seja, só guardará o último que tentou adicionar.

s4nchez:

Quando ao break, você pode usar um return caso o que você já tenha executado a operação que o método é responsável.

Mas em geral não adianta muito se preocupar com OO se seu código tem problemas de lógica. Por exemplo:

...	
	void adiciona(Funcionario f) {
		funcionarios = new Funcionario[10];
		...

Neste caso você está iniciando seu array toda vez que adiciona um funcionário, ou seja, só guardará o último que tentou adicionar.

Entendi, quero tentar consertar isso, mas como ficaria no caso para toda vez que iniciar ele supostamente “guardar” a posição?

Tirar o laço?
Não estanciar o objeto do array?

Fiquei com essas dúvidas agora.

cgomesnet:

phph:

Poste o código da sua classe funcionário.

Compile e rode isso:

f1.data.dia = -1000;

Essa classe permite isso, não?

Portanto, os dados não estão encapsulados permitindo que o objeto assuma um “estado” inválido. Primeira opinião

Até mais,

A classe esta no outro PC, mas não, não fiz o encapsulamento pra mermitir que só entre dados entre 1 e 31, mas o que eu quero é que vocês olhem esse código, pois estou aprendendo ainda array.

O que ele quis dizer foi sobre os métodos gets e sets… É uma má prática no java deixar variáveis públicas expostas… O certo para obter a data seria chamar um método público que retornava ela, como getData() por exemplo.

cgomesnet:

s4nchez:

Quando ao break, você pode usar um return caso o que você já tenha executado a operação que o método é responsável.

Mas em geral não adianta muito se preocupar com OO se seu código tem problemas de lógica. Por exemplo:

...	
	void adiciona(Funcionario f) {
		funcionarios = new Funcionario[10];
		...

Neste caso você está iniciando seu array toda vez que adiciona um funcionário, ou seja, só guardará o último que tentou adicionar.

Entendi, quero tentar consertar isso, mas como ficaria no caso para toda vez que iniciar ele supostamente “guardar” a posição?

Tirar o laço?
Não estanciar o objeto do array?

Fiquei com essas dúvidas agora.

Instancia o array no construtor da classe. Ou melhor, crie um construtor e instancia o array lá.

class Empresa {  
    private String nome;  
    private String cnpj;  
    private Funcionario[] funcionarios;  
    
   //construtor
    public Empresa() {
      //será instanciado o array somente qdo o objeto for criado a 1ª vez. 
      this.funcionarios = new Funcionario[10];

    } 
   
   //aqui vc faria os métodos get e set para recuperar/alterar os atributos da classe.
   public String getNome(){...}
   public voit setNome(String n){...}
   public String getCnpj(){...}
   public voit setCnpj()(String n){...}

   //como o RenatoCustodio disse, é bom proteger tanto os atributos qto os métodos.
    public void adiciona(Funcionario f) {  
        funcionarios = new Funcionario[10];  
        System.out.println ("O tamanho do array é: "+ funcionarios.length);       
        fim:  
        for (int i = 0; i < funcionarios.length; i++) {  
            if (this.funcionarios[i] == null) {  
                this.funcionarios[i] = f;                 
                System.out.println("Registro efetuado com sucesso na posicao "+i+".");                    
                break fim;  
            }  
            //como este bloco executa somente qdo o anterio dá falso, é melhor usar 'else'.            
            else {  
                System.out.println("Posição "+i+" ocupada.");  
            }             
        }   
    }     
}

Abraços

Muito obrigado pessoal.

Estou ancioso para voltar pra casa hoje a noite e estudar as novas sugestões, creio que esse é o melhor método para aprender (faço, mesmo que errado) e depois efetuo a correção.

Foi assim que aprendi a ler e a escrever, rsrsrs Não tem porque não ser assim com o Java