Cerrar conexion con JAVA

[gabrican]

Hola a tod@s ....

Tengo una aplicacion en JAVA que se conecta a una base de datos ORACLE a través de una Datasource... en el pool tiene 25 conexiones permitidas como máximo, pero se me ha estado cayendo y algunas veces indica que las conexiones están completas cuando solo hay 2 o 3 usuarios conectados.

Pensé que depronto me habia quedado faltando en alguna sentencia cerrar la conexion pero veo que siempre está el llamado a la funcion, entonces no se que puede ser.... que me recomiendan???

Este es mi cod para cerrar las conx

Código:

public void cerrarConexion() throws ConnectionException {
	  	try {
		   con.close();
		}catch (SQLException  sqle) {
			throw new ConnectionException("Ha ocurrido un error al intentar cerrar la conexion con Oracle. Error:" + sqle.getMessage());                                   
		}
	}

Tks

[chuidiang]

Sólo falta saber si llamas siempre que es necesario a cerrarConexion().

Se bueno.

[gabrican]

Iniciado por chuidiang

Sólo falta saber si llamas siempre que es necesario a cerrarConexion().

Se bueno.

Si... acabo de revisar todo el código y siempre lo llamo

[Ronruby]

Iniciado por gabrican

Si... acabo de revisar todo el código y siempre lo llamo

¿La aplicación no te muestra ninguna excepción al llamar a cerrarConexion?
¿Te aseguraste que la referencia de la conexion, con, este seteada correctamente y pueda ser accedido desde tu metodo?

[chuidiang]

Las llamadas a cerrarConexion() deberías ponerlas en los finally del try-catch. ¿Lo haces así?

Se bueno.

[gabrican]

Iniciado por Ronruby

¿La aplicación no te muestra ninguna excepción al llamar a cerrarConexion?
¿Te aseguraste que la referencia de la conexion, con, este seteada correctamente y pueda ser accedido desde tu metodo?

Hola...
Con respecto a la excepción no muestra nada... y en el trace del servidor tampoco aparece... y la referencia tambien esta bien seteada:

Código:

public class Conexion {
	
	private static Connection con;
	
	public Connection conectarBD()throws ConnectionException {
        
		try {
			InitialContext initialContext = new InitialContext();
			DataSource ds =(DataSource)initialContext.lookup("jdbc/ds");

			try {
				con = ds.getConnection();
				Statement statement = con.createStatement();
				
			}catch (SQLException sqle) {
				System.out.println("Error obteniendo la conexión en ConexionDAO: "+sqle.getMessage());
			}
		}catch (NamingException ne) {
			System.out.println("Error en el método getConnection() de la clase ConexionDAO: "+ne.getMessage());
		}
		return con;
	}
				
	public static Connection getInstance(){
		return con;
	}

	
	public void cerrarConexion() throws ConnectionException {
	  	try {
		   con.close();
		}catch (SQLException  sqle) {
			throw new ConnectionException("Ha ocurrido un error al intentar cerrar la conexion con Oracle. Error:" + sqle.getMessage());                                   
		}
	}

[gabrican]

Iniciado por chuidiang

Las llamadas a cerrarConexion() deberías ponerlas en los finally del try-catch. ¿Lo haces así?

Se bueno.

Hola...

Mira tal cual estan....

Código:

public Vector consultarEmpresar(String valor) throws ConnectionException{
		Vector empresas= new Vector();
			
		try{				
			EmpresaDTO empresa;
			PreparedStatement ps;
			ResultSet rslt;
			String sql = null;
			conectar();
			
			sql="select * from tabla";
			ps = conexion.prepareStatement(sql);
			
			ps.setString(1, "%"+valor+"%");
			rslt = ps.executeQuery();			
			
			while (rslt.next()) {
				empresa = new EmpresaDTO();
...
...
...
...
...			}
			
			/* Cierra conexiones establecidas */
			ps.close();
			cerrarConexion();
							
		}catch(SQLException e){
			e.printStackTrace();
		}
			
		return empresas;
	}

[chuidiang]

Hola:

Es dificil de entender y ver el error. En el último trozo de código que has puesto obtienes una conexion no se sabe cómo a la que llamas conexion. El cerrrarConexion cierra un static con qu está en la clase Conexion. ¿Está el método consultarEmpresa también en la clase Conexion? ¿Estas totalmente seguro que ese conexion que tienes en consultarEmpresa es el mismo que el que tienes guardado en el static con de la clase Conexion?

Por otro lado, La clase Conexion tiene un método conectarBD no estático que es el que establece conexión y la guarda en con. No veo dónde llamas a eso, pero cada llamada a conexionBD "machaca" la conexión anterior que hubiera almacenada en Conexion.con. Es posible que por ahí se te queden sin cerrar, ya que la llamada a cerrarConexion() cerrrará la última obtenida, pero no puede nunca cerrar las anteriores si han sido "machacadas".

Mi consejo es que no guardes con en static ni en ningún lado. Haz el método conectarBD() estático, que devuelva una Connection y que el que la pida se encargue de cerrarla. Puedes poner el método cerrarConexion en Conexion también static, pero que cierre la conexión que le pasen por parámetro.

Código java:

public void consultaEmpresa () {
   Connection conexion = null;;
   try {
       conexion = Conexion.conectarBD();
       // aquí tus SQL
   } catch (Exception e) {
      e.printStackTrace();
   } finally {
        Conexion.cerrarConexion(conexion);
   }
}
 

Se bueno.