Właściwy sposób wywoływania źródła danych JNDI w Tomcat

Question

Używam aplikacji WWW Java na serwerze Tomcat i chciałbym wiedzieć, co jest "najlepszą praktyką" pod względem dostępu do połączenia z bazą danych z JNDI Tomcat?

Obecnie jest to w zasadzie to, co robię za każdym razem, kiedy potrzeba, aby uzyskać dostęp do bazy danych:

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(); 
     } 
    } 
} 

Jednakże, jest to właściwy sposób wyszukać połączenie z JNDI za każdym razem, gdy chcesz uzyskać dostęp do bazy danych ? Czy powinienem zamiast tego odwoływać się do kontekstu lub źródła danych?

Answer 1

JNDI wyszukiwań są zasadniczo mapie wyszukiwań więc ich koszt jest minimalny. Ale lepiej jest, aby DataSource raz i "cache", że. Więc jeśli cokolwiek - napisanie metody zwracania DataSource jest idealne, aby nie wiązać zbyt wiele kodu do elementów wewnętrznych J2EE i ułatwić testowanie kodu.

Answer 2

Twój kod wyszukiwania wygląda dobrze
W twoim kontekście problemowym buforowanie źródła danych wydaje się w porządku, o ile nie buforujesz rzeczywistego obiektu połączenia.

Nie stosowałem tego podejścia od jakiegoś czasu. Te dni, co najmniej, używam sprężynę wstrzyknąć źródło danych/zainicjować JdbcTemplate

Answer 3

można zrobić coś takiego też: -

... 

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

... 

Answer 4

new InitialContext() jest drogie w każdym kontenerze aplikacji, powinien być static final i stworzył w static {} bloku, skutecznie czyni go Singleton. Powinieneś tylko raz utworzyć to odwołanie i użyć go ponownie w dowolnym miejscu.

DataSource również powinien być static. Powinieneś mieć metodę public static Connection getConnection(); do pobierania obiektów Connection, kod nigdy nie powinien mieć do czynienia bezpośrednio z DataSource. Działa to szczególnie dobrze z implementacjami PooledDataSource.