Forma correcta de invocar el origen de datos JNDI dentro de Tomcat

Question

Estoy usando una aplicación web Java en un servidor Tomcat y me gustaría saber cuál es la "mejor práctica" en términos de acceder a una conexión de base de datos desde el JNDI de Tomcat?

Actualmente esto es básicamente lo que estoy haciendo cada vez que necesite acceder a la base de datos:

Context envContext = null; 
DataSource dataSource = null; 
try { 
    envContext = (Context)ctx.lookup("java:/comp/env"); 
    dataSource = (DataSource)envContext.lookup("jdbc/datasource"); 
    return dataSource.getConnection(); 
} catch (Exception e){ 
    e.printStackTrace(); 
    return null; 
}finally { 
    if(envContext != null){ 
     try{ 
      envContext.close(); 
     } catch (NamingException e){ 
      e.printStackTrace(); 
     } 
    } 
} 

Sin embargo, es esta la forma correcta para buscar una conexión desde JNDI cada vez que quiera acceder a la base de datos ? ¿Debo hacer una referencia al Contexto o al Datasource en su lugar?

Answer 1

Las búsquedas JNDI están Mapa esencialmente búsquedas de manera que su cabeza es mínimo. Pero es preferible obtener el DataSource una vez y "caché". En todo caso, escribir un método para devolver DataSource es ideal para no vincular demasiado código a las partes internas de J2EE y hacer que el código sea más fácil de probar.

Answer 2

su código de búsqueda se ve bien
En el contexto del problema, almacenamiento en caché de la fuente de datos parece bien, siempre que no se esté almacenamiento en caché el objeto de conexión real.

No he usado este enfoque desde hace tiempo. En estos días, como mínimo, que utilizan la primavera para inyectar el origen de datos/inicializar JdbcTemplate

Answer 3

Usted puede hacer algo como esto también: -

... 

Context initContext = new InitialContext(); 
DataSource dataSource = (DataSource) initContext.lookup("java:comp/env/jdbc/datasource"); 

... 

Answer 4

new InitialContext() es cara en cada contenedor de aplicaciones, que debe ser un static final y creado en un bloque static {}, con eficacia haciendo un Singleton. Solo debe crear esta referencia una vez y volver a usarla donde la necesite.

DataSource debe ser un static también. Debería tener un método public static Connection getConnection(); para recuperar objetos Connection, el código nunca debería tener que tratar con DataSource directamente. Esto funciona especialmente bien con las implementaciones PooledDataSource.