Erro no fechamento de conexões JDBC

Já parou para pensar que se ocorrer um erro no primeiro IF o segundo não será finalizado?

if(rs != null) rs.close(); if(ps != null) ps.close(); if(closeCon){ con.close();

Apesar de você ter rasão no que disse, nos meus testes isso não ocorreu.
Então penso que possa ser outra coisa. Já aconteceu com você algo parecido?

wagnerluis.silva:

Apesar de você ter rasão no que disse, nos meus testes isso não ocorreu.
Então penso que possa ser outra coisa. Já aconteceu com você algo parecido?

Ok, mas como foram os testes? O que ocorreu neles? Que exception é lançada e, além disto, até que ponto do finally o código é executado?

Ele executou todos os if contidos no finally.
Consigo verificar o erro quando ele chega no if do código que utiliza o método.

try {             
    operacao = operacaoDao.findOperacao(numSerial,true);  
} catch (Exception e) {  
    log.error("Erro:"+e.getMessage());  
}  
  
if(operacao == null )  
    throw new Exception("Nao foi possivel validar a operação");  // Acontece o erro e as sessões continuam abertas

wagnerluis.silva:

Ele executou todos os if contidos no finally. Consigo verificar o erro quando ele chega no if do código que utiliza o método.

try {             
    operacao = operacaoDao.findOperacao(numSerial,true);  
} catch (Exception e) {  
    log.error("Erro:"+e.getMessage());  
}  
  
if(operacao == null )  
    throw new Exception("Nao foi possivel validar a operação");  // Acontece o erro e as sessões continuam abertas

Sim consigo.

neste ponto aqui que ele acontece

try {               
    operacao = operacaoDao.findOperacao(numSerial,true);    
} catch (Exception e) {    
    log.error("Erro:"+e.getMessage());    
}    
    
if(operacao == null )    
    throw new Exception("Nao foi possivel validar a operação");  // Acontece o erro e as sessões continuam abertas

wagnerluis.silva:

Sim consigo.

neste ponto aqui que ele acontece

try {               
    operacao = operacaoDao.findOperacao(numSerial,true);    
} catch (Exception e) {    
    log.error("Erro:"+e.getMessage());    
}    
    
if(operacao == null )    
    throw new Exception("Nao foi possivel validar a operação");  // Acontece o erro e as sessões continuam abertas

Isso. A mensagem que é exibida é a do if .
E a conexão fica aberta.
Vocês conhecem alguma coisa que possa fazer com que a sessão seja liberada? Acho que isso já me ajudaria.

Onde fecha a conexão ele testa essa váriável booleana closeCon

if(closeCon){  
     con.close();  
}

Vá no modo debug de sua IDE e veja nessa parte se a váriavel closeCon está chegando como false;
É melhor retirar esse if e colocar direto con.close(), uma vez que no inicio do método você já testa if closed abre uma nova;

Uma estutura segura envolvendo transações é essa:

public class Dao{
   private Connection con;
   
   
   public void executaNoBanco(){
   try{
      //obtenha sua coneção this.con = ..., preferencialmente de uma classe separada
      String query = "select * from tabela";
      PreparedStatement p = con.prepareStatement();
      ResultSet rs = p.executeQuery();
      while(rs.next()){
          //guarde valores em uma lista
      }
      rs.close();
      p.close();
   }catch(Exception e){
      //log
   }finally{
      try{
         con.close();
      }catch(Exception ex){
         //log
      }
   }
   }
}

Essa estrutura foi proposta pelo livro do Deitel java como programar.

Se puder usar o Java 7, use isto aqui:

try (
    PreparedStatement p = con.prepareStatement();  
    ResultSet rs = p.executeQuery();  
) {
    while(rs.next()){  
        //guarde valores em uma lista  
    }  
}

que fecha as conexões e prepared statements adequadamente, se elas forem declaradas e obtidas dentro dos parênteses do try (procure por “try-with-resources”).

Aliás, a menos que você tenha um motivo muito forte, não trate “Exception” dentro de seu código em um método como esses. Trate apenas as exceções adequadas em vez de ficar pegando tudo - isso se chama “preguiça”. Quem tem de pegar tudo é uma outra parte do seu código.

Olá entanglement,

Com exceção de sua definição de “melhor prática”, faço minhas as suas palavras nesse post: http://www.guj.com.br/java/285902-duvida-com-metodos-estaticos

"
Muito código que posto aqui reconhecidamente não obedece às melhores práticas, a menos que eu diga explicitamente isso.

Muitas vezes eu tento escrever o código mais claro que faz uma determinada coisa - e então você vai ver certas coisas, por exemplo, como o uso de Null Layout em Swing, que não recomendo para ninguém. Ou então acessar diretamente os campos de uma classe, sem usar setters e getters. Não aconselho que as pessoas façam copy & paste do meu código, justamente porque

• Eu uso um estilo muito peculiar, fácil de ser detectado pelos professores
• Pretendo sempre mostrar como se faz mas não qual é a melhor prática (até porque a “melhor prática” é uma superstição inventada por aquelas pessoas que não pensam antes de fazer
  "

Fora esse problema com as exceções, vocês podem me dizer uma forma de “matar” as sessões que estão ficando abertas.
Preciso que elas fiquem disponíveis para outros usuários.