Problema com java.lang.NullPointerException

27 de janeiro de 2011 3 respostas

chavenco 27 de janeiro de 2011

gente estou com um problemao aqui com uma nullPointer nao consigo achar o pq ela esta ocorrendo me ajudem

Obrigado

esse e o meu metodo

public int inserir(Object obj) {

Convenio convenio = (Convenio) obj;
    Connection conexao = this.conectar();

    try {
        
        Statement comando = conexao.createStatement();
        return comando.executeUpdate("insert into convenio values (" + convenio.getId() + ", '" + convenio.getNome() + "')");
    } catch (SQLException sqle) {
        return -1;
    }finally{
        this.desconectar(conexao);

    }
}

e aqui e onde esta ocorrendo a nullpointer, num trecho de codigo de JSP

<% String parametroNome = request.getParameter(“nome”);

if (parametroNome == null) {
                    parametroNome = "";
                }


    %>
    <table border="1">
        <tr>
            <td style="text-align: center"><b>Código</b></td>
            <td style="text-align: center"><b>Nome</b></td>
        </tr>

        <%
                    if (operacao != null && operacao.equals("4")) {

                        exames.dao.ConvenioDaoJdbc cDaoj = new exames.dao.ConvenioDaoJdbc();

                        java.util.ArrayList convenios = cDaoj.listar(parametroNome);

                        for (int i = 0; i < convenios.size(); i++) {
                            exames.dominio.Convenio convenio = (exames.dominio.Convenio)convenios.get(i);
        %>

3 Respostas

antonioedirane 27 de jan. de 2011

Primeiro, tenta editar o post e colocar o código dentro da tag [code] fica mais fácil pra gente poder te ajudar.
Depois tenta ver na mensagem de erro qual a linha está dando erro.
E ai responde com a linha que está dando erro.

iogui 27 de jan. de 2011

chavenco, boa noite…

Primeira coisa… tah difícil de ler o que vc postou, use tags code e indente o código para facilitar a leitura.
Segundo, cole a stacktrace no post para que possamos ver.
Terceira coisa… isto não parece ser “Java Avançado”!

Obs.: É má prática de programação, colocar este monte de código scriptlet em uma página jsp.

[]s

ViniGodoy 27 de jan. de 2011

Use PreparedStatement para evitar aquelas concatenações ali. Evita também ter problemas com aspinhas e caracteres especiais, já pensou se o nome do convenio do cara for MacDonald’s? Esse apóstrofe faria uma catástrofe no seu código;
JAMAIS ignore exceptions. Retornar -1 é uma péssima forma de trata-las. Você perde informação muito importante, a do porque o método não deu certo. Ou você loga essa exception, ou você a dispara novamente como uma RuntimeException;
Certifique-se de fechar o ResultSet, o Statement e a conexão no bloco finally;
Se seu método trabalha com convenios, receba um convênio como parâmetro. Deixe que quem conhece a origem do objeto faça o cast (a classe que chama esse método).
Se o convenio não pode ser nulo, certifique-se de testar ess condição no início do método, e lançar exception caso o objeto fornecido seja;
Especifique o tipo do seu List, para evitar casts:

java.util.List&lt;exames.dominio.Convenio&gt; convenios = cDaoj.listar(parametroNome);

Como já disseram, use a tag code. Se não sabe fazer isso, leia aqui:
http://www.guj.com.br/java/50115-voce-e-novo-no-guj-vai-criar-um-topico-e-colar-seu-codigo-fonte-leia-aqui-antes-por-favor

Criado 27 de janeiro de 2011

Ultima resposta 27 de jan. de 2011

Respostas 3

Participantes 4

3 Respostas

Topicos relacionados