[Resolvido] NullPointerException em comparação de Strings?

Programação Java
#1

Olá,

O seguinte exercício consiste em fazer uma espécie de agenda com um array de objetos.
O código está gerando uma exceção NullPointerException no IF que verifica se a pessoa que está sendo incluída no array já existe.

public class TestaAgenda {
	public static void main(String[] args) {
		Agenda novaAgenda = new Agenda(5);
                boolean foiIncluso;
		for(int x = 0;x <= 10;x++){
			Pessoa novaPessoa = new Pessoa();
			novaPessoa.setNome(JOptionPane.showInputDialog(null,"Nome"));
			foiIncluso = novaAgenda.incluiPessoa(novaPessoa);
		}

	}

A Class acima chama o seguinte método da Class Agenda:

public boolean incluiPessoa(Pessoa novaPessoa) {
		int capacidadeAntiga = 0;
		boolean foiIncluso      = false;		
		
		//Se a agenda já contém registros
		if(quant > 0){
			//Verifica se a pessoa já existe			
			for(int x = 0;x <= this.agenda.length;x++){				
				if(novaPessoa.getNome().equals(this.agenda[x].getNome())){
					foiIncluso = false;
					return foiIncluso;
				}
			}
			
			//Se a agenda não tiver mais posições disponíveis
			if(agenda.length < (quant + 1)){
				capacidadeAntiga   = this.agenda.length;
				Pessoa agendaTemp[] = new Pessoa[this.agenda.length];
				//Copia agenda para lista temporária
				for(int x = 0;x < this.agenda.length;x++){
					agendaTemp[x] = this.agenda[x];
				}
				//Instância novamente a agenda com 10 posições a mais
				this.agenda = new Pessoa[capacidadeAntiga+10];
				//Copia lista temporária para agenda nova
				for(int x = 0;x < capacidadeAntiga;x++){
					this.agenda[x] = agendaTemp[x];
				}			
			}
			
			//Adiciona novo registro na agenda: umaP
			this.agenda[quant] = novaPessoa;
			quant++;
			foiIncluso = true;					
		}
		else{
			this.agenda[0] = novaPessoa;
			quant++;
			foiIncluso = true;
		}
		
		return foiIncluso;
	}

}

Comentando o IF que verifica se a pessoa já existe o código funciona.
Mas realmente não sei o porque da exceção.

Valeu!

#2

Oi,

Para descobrir quem esta null faça isso:

//Se a agenda já contém registros if(quant > 0){ //Verifica se a pessoa já existe for(int x = 0;x <= this.agenda.length;x++){ System.out.println(novaPessoa.getNome()); System.out.println(this.agenda[x]); System.out.println(this.agenda[x].getNome()); if(novaPessoa.getNome().equals(this.agenda[x].getNome())){ foiIncluso = false; return foiIncluso; } }

Post o resultado.

#3

vai no for e coloca -1 no tamanho da sua length!!

for(int x = 0;x <= this.agenda.length -1;x++){

#4

[quote=Gustavo Sperandio]vai no for e coloca -1 no tamanho da sua length!!

for(int x = 0;x <= this.agenda.length -1;x++){ [/quote]

Oi,

Na real seria só retirar o <= e deixar só <

Tchauzin!

#5

Trocando <= por < a Exception continuou.

O resultado dos system.out.print foram.

Matheus
João
null
Exception in thread “main” java.lang.NullPointerException
at Ex4.Agenda.incluiPessoa(Agenda.java:31)
at Ex4.TestaAgenda.main(TestaAgenda.java:16)

Na real eu me perdi agora. =S

#6

:oops: é nem tinha me tocado nessa idéia!!!

#7

Troque a linha 9 da sua primeira postagem que está assim

if(novaPessoa.getNome().equals(this.agenda[x].getNome())){

Por isso:

if(this.agenda[x] != null && novaPessoa.getNome().equals(this.agenda[x].getNome())){
#8

Fechou.

A solução era testa se != null e também trocar o <= por <.

Obrigado

#9