Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Java EE (J2EE) и Spring > Как правильно использовать пул соединения с БД


Автор: di109 24.12.2012, 16:05
Ниже приведен код который используя шаблон Singliton создает пул соединения с БД, в сервлете в методе ИНИТ  создается это соединение. На практике работает (ошибок нет)

1.    Правильно ли написан код
2.    Как, кога и где закрыть этот пул
3.    Может мне показалось, но с пулом он стал работать медленнее, чем без пула


Код

package test;

import java.sql.Connection;
import java.sql.SQLException;

import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
import javax.sql.*;

public class ConnectionFactory {

    private static DataSource pool;
    private static Context env = null;    
    private static ConnectionFactory cdb;

    private ConnectionFactory() {

    }

    //Singleton
    public static ConnectionFactory getInstance() {
        if(cdb == null) {
            cdb = new ConnectionFactory();
        }
        return cdb;
    }
    
    //pool
    static {
      try{
          
            env = (Context) new InitialContext().lookup("java:comp/env");
            pool  = (DataSource) env.lookup("datasource");
            if (pool == null)
            throw new NamingException("'datasource' is an unknown DataSource");
            
       } catch (NamingException e) { 
       
           e.printStackTrace();
    
       }
    }
    
      //metod return pool
      public Connection getConnection() throws SQLException{
            Connection conn = pool.getConnection();
            return conn;
          }

}




Код

package test;

import java.io.IOException;
import java.io.PrintWriter;

import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import java.sql.*;
import java.util.logging.Logger;
import javax.sql.*;

import model.Model;

/**
 * Servlet implementation class Data
 */
@WebServlet("/Data")
public class Data extends HttpServlet {
    private Logger log = Logger.getLogger(Data.class.getName());
    
     DataSource pool;
     Connection conn = null;
     Model m = new Model();
       
    /**
     * @see HttpServlet#HttpServlet()
     */
    public void init() throws ServletException {
     
     ConnectionFactory c = ConnectionFactory.getInstance();    
      
        try {
                conn = c.getConnection();
                
            } catch (SQLException e) {
                
                log.info("Error!");
            } 
            /*finally {
                try{ conn.close(); } catch(Exception e) { }
            }*/

    }

    /**
     * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
     */
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        
        response.setContentType("text/html");
        PrintWriter out = response.getWriter();
        
        out.println("<html>");
        out.println("<head>");
        out.println("<meta http-equiv=\"refresh\" content=\"15\">");
        out.println("<title>POOL</title>");
        out.println("<link REL=\"stylesheet\" TYPE=\"text/css\" HREF=\"css/style.css\">");
        out.println("</head>");
        out.println("<body>");

        out.println("Hello!");
        out.println("<BR>");
        
        // Вывод данных из БД
        out.println(m.show(conn, "user"));
        
        
        out.println("</body>");
        out.println("</html>");
        out.close();     
     
    }

}






Автор: Старовъръ 25.12.2012, 21:54
1. Код выглядит правильно, однако Синглтон нужно инициализировать прям в статическом поле, а не при первом вызове. Иначе чревато проблемами в многопоточной среде, подробней о Singleton можно прочесть http://articles.javatalks.ru/articles/6. Плюс посмотри в сторону Dependency Injection фреймворков: Guice, Spring.
2. Думаю корректней сказать - завершить пул. Пул - это не соединение, он не закрывается. Разрушаться/завершаться он должен самим контейнером раз уж используется JNDI.
3. Без пула работать быстрей не может, т.к. соединение каждый раз создается новое. Собственно пул для этого и нужен. Когда приложение и сервер находятся на одной машине это не так ощутимо, однако все равно разница будет. http://articles.javatalks.ru/articles/8 пример настройки пула для MySQL, однако там используется DI framework а не JNDI, надеюсь это не собъет с толку.

Автор: di109 30.12.2012, 13:54
А если возникнет такая ситуация: есть сервлет (контроллер), модель (java класс), и вид (jsp файл),
сервлет (контроллер) - используя методы модели, (заносит, удаляет, обновляет данные в БД), в качестве соединения используется  pool,
который создается в методе init() и завершается в методе destroy().

Будет ли считаться за ошибку если я в виде (jsp файл) опять создам pool и используя метод модели, который возвращает данные, выведу данные из БД в браузер?

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)