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
tmbémtambé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
.