Que tal verificar se sala é null antes de tudo?

“BreckDor:

public boolean validar(Sala sala, ArrayList salasAlocadas)

é porque eu tenho que mostrar qual o motivo do erro. pois isso será passado por um arquivo de testes. entao se o id for nulo eu obrigatoriamente preciso lançar uma IdentificacaoInvalidaException por ela ter no construtor getMessage = “Identificacao Invalida” Veja aqui o log de erro. se eu naol ançar uma escesao com esta mensagem ““Identificacao Invalida”” que no meu caso será IdentificacaoInvalidaException, mas acaba caindo em NullPointerException.

At line 86: Expected the error message , but the error message was <(no message: exception = java.lang.NullPointerException)>
At line 88: Expected , but no error occurred.
At line 90: Expected , but no error occurred.
At line 92: Expected , but no error occurred.
At line 94: Expected the error message , but the error message was <(no message: exception = java.lang.NullPointerException)>
At line 96: Expected the error message , but the error message was <(no message: exception = java.lang.NullPointerException)>
At line 98: Expected , but no error occurred.
At line 100: Expected <Sala não existe.>, but no error occurred.

Não entendi… esta classe é uma classe p/ realizar testes?

NullPointerException significa que você tentou acessar o membro de um objeto não inicializado.
Significa basicamente que seu código não foi implementado da forma correta.

isso é um problema pois no teste ele receberá os parametos exatamente desta forma
expectError “Identificacao Invalida” adicionarSala id= capacidade=30 finalidade=“Sala de Aula” tipo="Normal"
Onde id nao sera informado(por então nao pode ser inicializado) por isso eu havia feito
if ((sala.getId() == null)){ throw new IdentificacaoInvalidaException(); }' entao como estava sendo lançado o NullPointerException. eu imaginei que algo mais ou menos assim poderia resolvertry{
if ((sala.getId() == null)){
throw new IdentificacaoInvalidaException();
}
} catch(NullPointerException e){
// usando algum polimofismo
NullPointerException Idie = new IdentificacaoInvalidaException
e = idie;
System.out.println(e.getMessage());
}`

então eu queria algo mais ou menos assim. caso desse NullPointerException ele lancasse IdentificacaoInvalidaException?

@BreckDor vamos organizar sua dúvida, fala qual é seu propósito e quais tecnologias esta usando. Já percebi que isto é uma classe de teste unitário. É com JUnit mesmo? Já posta a versão. Assim o pessoal poderá entender melhor e sugerir soluções. Abrçs.

não uso JUniti. estou usando apenas o easyAccepty( Testes de aceitação ). no caso o easyAcepty espera que em um error onde o id da sala nao é informado eu lhe mostre a mensagem “Identificacao Invalida”. por isso criei a Excesao

public class IdentificacaoInvalidaException extends Exception {

public IdentificacaoInvalidaException(){

// este é a frase de error que o easy acepty espera.

super(“Identificacao Invalida”);

}

}
mas quando fiz

if ((sala.getId() == null)){
        throw new IdentificacaoInvalidaException();
    }

ele lança NullPointerException em vez da excesao que defini na clausula throw new.
entao eu preciso de uma forma que se id de sala for nulo ele lance IdentificacaoInvalidaException e nao NullPointerException pois nãoé este o error que o easyAccepty espera.
aqui a parte do script do easyAcepty que estou tentando resolver:

Entradas invalidas

expectError “Identificacao Invalida” adicionarSala id= capacidade=30 finalidade=“Sala de Aula” tipo=“Normal”

expectError “Identificacao Invalida” adicionarSala id="" capacidade=30 finalidade=“Sala de Aula” tipo=“Normal”

sejamos diretos,

só vou reformular o que os camaradas já falaram:

// se o id for vazio
if ((sala.getId().equals("")) ) {
    throw new IdentificacaoInvalidaException();
}

// se o id for nulo
if ((sala.getId() == null)){
    throw new IdentificacaoInvalidaException();
}

qual o propósito de verificar == null se você já acessou a instancia? inverta a sequência, se o id for nulo e você tentar chamar um .equals você terá uma NullPointerException.

Se é realmente nesse código que está lançando NullPointerException, é porque você não inicializou a variável sala e, neste caso, não faz sentido tratar NullPointerException.

Coloca um teste:

if (sala == null) {
    throw new SalaNaoInformadaException();
}

Realmente igorventurelli era esse o codigo que eu procurava quando criei esse topico. tambem concordo que nao era uma boa ideia mas graças a vcs percibi que minhs solução era so tratar o parameto antes da criação do objeto sala. valeu a todos vcs foram de grande ajuda.