Criação de métodos get e set para uma array de classe

Primeiro eu sugiro que você altere a nomenclatura dos métodos. O método setEmpregados, na verdade, deveria se chamar addEmpregado. Ele recebe apenas um empregado e adiciona na empresa. Se ele de fato recebesse um array com todos os empregados da empresa, aí sim seria correto chamá-lo de setEmpregados. O mesmo ocorre com o método getEmpregados. Você está retornando apenas 1 empregado, portanto o correto seria chamá-lo de getEmpregado.

Perceba que um array precisa ser inicializado com um tamanho fixo. Você não inicializou ele em nenhum momento, por isto você recebe uma NullPointerException. Você pode inicializar na própria declaração:

private Funcionario[] empregados = new Funcionario[10];

Isto lhe permitiria adicionar 10 empregados. Pra você não ficar limitado a um tamanho fixo, sugiro que utilize, ao invés do array, um Set ou um List. Eles não tem esta limitação.

Um outro problema está no seu algoritmo de busca de uma posição nula (setEmpregados). Se você inserir os 10 registros, vai receber um IndexArrayOutOfBoundsException quando tentar inserir o 11. Você precisa buscar somente dentro do limite de tamanho do array.

Entendo, vou corrigir o algoritmo e verificar como ficou. Obrigado wagnerfrancisco.

Ok, o problema de NullPointerException foi resolvido, não estou recebendo este erro ao compilar o programa,
mas meus métodos set(que seria o add) e o get não estão funcionando corretamente, aparentemente os valores não estão sendo setados
e o get não está retornando os dados do funcionário, o que estou fazendo de errado?

Segue abaixo os métodos:

public void setEmpregados(Funcionario f) {
		int x = 0;
		boolean cond = true;
		while (cond == true) {
			if (empregados[x] == null) {
				this.empregados[x] = f;								
				cond = false;				
			} else {
				x++;			
			}
		}		
	}

public Funcionario getEmpregados (int posicao) {
		return this.empregados[posicao];
	}

Obs: As classes e métodos são os mesmos de quando criei o tópico no forum.

wagnerfrancisco, não corrigi as nomenclaturas porque pretendo adicionar mais funcionários, estou realizando o teste com apenas 1 para corrigir os erros. E referente ao erro IndexArrayOutOfBoundsException ao adicionar mais de 10 funcionários, não há problema, estou fazendo este algoritmo apenas para aprendizado, estou aprendendo a trabalhar com encapsulamento e construtores por enquanto.

Obrigado.

JorjaoCorreia:

Ok, o problema de NullPointerException foi resolvido, não estou recebendo este erro ao compilar o programa,
mas meus métodos set(que seria o add) e o get não estão funcionando corretamente, aparentemente os valores não estão sendo setados
e o get não está retornando os dados do funcionário, o que estou fazendo de errado?

Segue abaixo os métodos:

public void setEmpregados(Funcionario f) {
		int x = 0;
		boolean cond = true;
		while (cond == true) {
			if (empregados[x] == null) {
				this.empregados[x] = f;								
				cond = false;				
			} else {
				x++;			
			}
		}		
	}

public Funcionario getEmpregados (int posicao) {
		return this.empregados[posicao];
	}

Obs: As classes e métodos são os mesmos de quando criei o tópico no forum.

wagnerfrancisco, não corrigi as nomenclaturas porque pretendo adicionar mais funcionários, estou realizando o teste com apenas 1 para corrigir os erros. E referente ao erro IndexArrayOutOfBoundsException ao adicionar mais de 10 funcionários, não há problema, estou fazendo este algoritmo apenas para aprendizado, estou aprendendo a trabalhar com encapsulamento e construtores por enquanto.

Obrigado.

Já debugou para ver se esta caindo no seu if?

if (empregados[x] == null) { this.empregados[x] = f; cond = false; }

jeffev, não estou usando nenhuma IDE, estou usando o compilador do Java mesmo por enquanto.

Coloca um print só pra ver se entra nesse if.

E ae Jorjao, seguinte, aparentemente seus métodos set e get de Empregados estão ok, faz o seguinte:

empresa.getEmpregados(0);

Atribui o código acima à uma variável e em seguinte da um system.out.printl, assim:

Funcionario[] funcionario = empresa.getEmpregados(0);
System.out.println(funcionario.getNome());

E ve se vai imprimir Jorge, ai está tudo ok com seus métodos

jcandinho.junior, assim são retornados os valores, acredito que o problema esteja no método get então pois o set está setando os valores corretamente.
Alguma dica de como eu posso fazer este get funcionar?

Obrigado.

O método get está retornando null? Tá dando algum erro?

Posta mais detalhado para entendermos melhor.

wagnerfrancisco, ele não retorna nada. É como se eu não estivesse chamando o método pra apresentar o resultado.

Tu fez o que eu falei?
Atribuir o get para uma variável e depois dar um system.out no na variável?

Na classe Funcionario, linha 8

private static int identificador;

identificador é static você tem que atribuir um valor para ele.

private static int identificador = 0;

jcandinho.junior, sim, conforme eu disse ali se eu atribuir o get pra uma variável e imprimir utilizando o System.out os valores são impressos,
mas somente com o método get não é impresso nada. Estou fazendo algo errado no método get, mas não sei o que.

Alguém tem uma ideia do que pode ser?

JorjaoCorreia:

jcandinho.junior, sim, conforme eu disse ali se eu atribuir o get pra uma variável e imprimir utilizando o System.out os valores são impressos,
mas somente com o método get não é impresso nada. Estou fazendo algo errado no método get, mas não sei o que.

Alguém tem uma ideia do que pode ser?

Mas tu sempre precisa do System.out.println pra imprimir. Só o get não vai imprimir mesmo, o que tu pode é ao invés de atribuir a uma variável, passar o método get como parâmetro diretamente.

wagnerfrancisco, eu achei que só criado o método get os dados seriam retornados todos desorganizados, mas seriam retornados.
Ja corrigi o método get, agora os resultados estão sendo retornados corretamente. Deixei desta maneira:

public Funcionario getEmpregados (int posicao) {
		System.out.println("Salario: " 					  + empregados[posicao].getSalario());
		System.out.println("Nome: " 				          + empregados[posicao].getNome());
		System.out.println("Esta na empresa? "			  + empregados[posicao].isEstaNaEmpresa());
		System.out.println("Departamento: " 		                  + empregados[posicao].getDepartamento());
		System.out.println("Dia/Mes/Ano de contratacao: "         + empregados[posicao].dataEntrada.getDia() + "/" + empregados[posicao].dataEntrada.getMes() + "/" +                                     empregados[posicao].dataEntrada.getAno());
		return this.empregados[posicao];
	}

Ai é só passar a posição que eu quero verificar ao chamar o método get que os dados são retornados certinho. Obrigado pessoal, especialmente o wagnerfrancisco

JorjaoCorreia:

jcandinho.junior, sim, conforme eu disse ali se eu atribuir o get pra uma variável e imprimir utilizando o System.out os valores são impressos, mas somente com o método get não é impresso nada. Estou fazendo algo errado no método get, mas não sei o que.

Alguém tem uma ideia do que pode ser?

O atributo identificador usa o modificador "static", entao voce deveria fazer o metodo assim

    public static int getIdentificador() {
		return identificador;
    }
 
 


Também mudaria o setEmpregados


 
    public boolean addEmpregados(Funcionario f) {  
             
        try {
        	this.empregados[this.i] = f;
        	this.i++;
        	return true;
        } catch (IndexOutOfBoundsException e){
        	System.out.printf ("Tamanho Array %s \"Status Cheia\" " , e.getMessage());
        	return false;
        }      
       
    }

Me corrijam se eu estiver errado, pois tbm sou iniciante em Java.