JSF-страница зависает, когда перезагружается много раз

Question

Я использую этот код для извлечения данных из таблицы базы данных.

public List<Dashboard> getDashboardList() throws SQLException { 

     if (ds == null) { 
      throw new SQLException("Can't get data source"); 
     } 

     //get database connection 
     Connection con = ds.getConnection(); 

     if (con == null) { 
      throw new SQLException("Can't get database connection"); 
     } 

     PreparedStatement ps = con.prepareStatement(
       "SELECT * from GLOBALSETTINGS"); 

     //get customer data from database 
     ResultSet result = ps.executeQuery(); 

     List<Dashboard> list = new ArrayList<Dashboard>(); 

     while (result.next()) { 
      Dashboard cust = new Dashboard(); 

      cust.setUser(result.getString("SessionTTL")); 
      cust.setPassword(result.getString("MAXACTIVEUSERS")); 


      //store all data into a List 
      list.add(cust); 
     } 

     return list; 
    } 

Этот код является частью страницы JSF, которая размещается на сервере Glassfish. Проблема в том, что когда я повторно загружаю страницу JSF много раз (примерно в 8 раз), веб-страница замерзает. Я подозреваю, что пул потоков заполнен и нет места для новых подключений. Как я могу решить проблему? Закройте соединение, когда запрос завершен или есть другой способ?

С наилучшими пожеланиями

Answer 1

Прежде всего: Да, вы должны закрыть соединение, когда ваш сделано явным вызовом метода close(). Закрытие соединения освободит ресурсы базы данных.

ОБНОВЛЕНИЕ: И вы должны закрыть PreparedStatement (с close()). Я также рекомендовал бы обрабатывать SQLExceptions в вашем методе, а не бросать его, так как вам нужно убедиться, что ваш оператор и соединение закрыты, даже если возникает исключение.

Что-то вроде этого:

Connection connection = dataSource.getConnection(); 
try { 
    PreparedStatement statement = connection.prepareStatement(); 
    try { 
     // Work with the statement 
    catch (SQLException e) { 
     // Handle exceptions 
} catch (SQLException e { 
    // Handle exceptions 
    } finally { 
     statement.close(); 
    } 
} finally { 
    connection.close(); 
} 

Кроме того, вы не должны запрашивать базу данных в методе получения полевого бин. В течение каждого запроса Getters можно вызывать несколько раз. Более элегантным способом было бы подготовить DashboardList в конструкторе или @PostConstruct вашего компонента.