Skip to main content
2 de 2
adicionou 1 caractere ao conteúdo
Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189
at br.com.drogaria.dao.FabricanteDAO.excluir(FabricanteDAO.java:33)

Essa linha é a seguinte:

    comando.setLong(1, f.getCodigo());

Sabemos que comando não é null e que f também não é null. Logo, o suspeito é que o resultado de f.getCodigo() seja null.

De acordo com o javadoc, o segundo parâmetro é do tipo long. Assim sendo, se você passar null no lugar deste parâmetro, o resultado será um NullPointerException de autounboxing. Veja mais sobre isso no terceiro caso dessa resposta.

Considerando que você chamou o método setCodigo(Long) no seu objeto Fabricante, mas mesmo assim o método getCodigo() retornou null, então a origem do seu NullPointerException não está em sua classe FabricanteDAO, e sim em sua classe Fabricante.

Apesar disso, a sua classe FabricanteDAO tem ainda assim um monte de problemas. Recomendo fortemente que você utilize o try-with-resources. Veja nessa outra pergunta o porquê de fazer isso e como fazer.

Ah, finalmente isso daqui:

    StringBuilder sql = new StringBuilder();
    sql.append("INSERT INTO fabricante ");
    sql.append("(descricao) ");
    sql.append("VALUES (?)");

É algo desnecessário. Você está montando e remontando sempre a mesma SQL pedaço por pedaço todas as vezes que o método salvar executa. O melhor seria fazer assim:

private static final String INSERT_SQL = "INSERT INTO fabricante (descricao) VALUES (?)";

Pois dessa forma, você constrói a SQL apenas um única vez quando a classe é inicializada e o custo de construí-la é praticamente zero, uma vez que ela já vai estar disponibilizada pronta e inteira dentro do bytecode. O mesmo pode ser dito da sua SQL do método excluir.

Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189