VPF::Как правильно использовать пул соединения с БД

di109

Дата 24.12.2012, 16:05 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 24
Регистрация: 26.9.2012

Репутация: нет
Всего: нет

Ниже приведен код который используя шаблон 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 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 491
Регистрация: 8.5.2008

Репутация: 1
Всего: 10

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

Это сообщение отредактировал(а) Старовъръ - 25.12.2012, 21:56

--------------------

My GitHub blog, Java Professionals Course, Blog on quality & management, Lib for randomized testing

di109

Дата 30.12.2012, 13:54 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 24
Регистрация: 26.9.2012

Репутация: нет
Всего: нет

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

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

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Java EE (J2EE) и Spring \| Следующая тема »