>     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](https://docs.oracle.com/en/java/javase/12/docs/api/java.sql/java/sql/PreparedStatement.html#setLong(int,long)), 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](https://pt.stackoverflow.com/a/63736/132).

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](https://pt.stackoverflow.com/q/172909/132) 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`.