Использую ли я пул соединений JDBC? - PullRequest
Новая
       55

Использую ли я пул соединений JDBC?

29 голосов
/ 29 сентября 2011

Я пытаюсь определить, действительно ли я использую пул соединений JDBC.После некоторого исследования, реализация кажется слишком легкой.На самом деле это проще, чем обычное соединение, поэтому я хотел бы проверить.

Вот мой класс соединения: 

public class DatabaseConnection {

Connection conn = null;

public Connection getConnection() {

    BasicDataSource bds = new BasicDataSource();
    bds.setDriverClassName("com.mysql.jdbc.Driver");
    bds.setUrl("jdbc:mysql://localhost:3306/data");
    bds.setUsername("USERNAME");
    bds.setPassword("PASSWORD");

    try{
        System.out.println("Attempting Database Connection");
        conn = bds.getConnection();
        System.out.println("Connected Successfully");
    }catch(SQLException e){
        System.out.println("Caught SQL Exception: " + e);
    }
    return conn;
}

public void closeConnection() throws SQLException {
    conn.close();
}

}

Является ли этот пул соединений настоящим?Я использую соединение в другом классе как: 

        //Check data against database.
    DatabaseConnection dbConn = new DatabaseConnection();
    Connection conn;
    ResultSet rs;
    PreparedStatement prepStmt;

    //Query database and check username/pass against table.
    try{
        conn = dbConn.getConnection();
        String sql = "SELECT * FROM users WHERE username=? AND password=?";
        prepStmt = conn.prepareStatement(sql);
        prepStmt.setString(1, user.getUsername());
        prepStmt.setString(2, user.getPassword());
        rs = prepStmt.executeQuery();

        if(rs.next()){ //Found Match.
            do{
                out.println("UserName = " + rs.getObject("username") + " Password = " + rs.getObject("password"));
                out.println("<br>");
            } while(rs.next());
        } else {
            out.println("Sorry, you are not in my database."); //No Match.
        }

        dbConn.closeConnection(); //Close db connection.

    }catch(SQLException e){
        System.out.println("Caught SQL Exception: " + e);
    }

Ответы [ 4 ]

54 голосов
/ 29 сентября 2011

Если предположить, что это BasicDataSource от DBCP , то да, вы используете пул соединений.Однако при каждом подключении вы создаете новый пул соединений.Вы не объединяете соединения из одного пула.Вам нужно создать пул соединений только один раз при запуске приложения и получать от него все соединения.Вы также не должны хранить соединение как переменную экземпляраВы также должны закрыть соединение, оператор и набор результатов, чтобы обеспечить правильное закрытие ресурсов, в том числе в случае исключений.* В этом полезна инструкция * 7 * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * - «« 1010 * * 1011 »).

(при необходимости это может быть реорганизовано как абстрактная фабрика для улучшения подключаемости)

и 

private static final String SQL_EXIST = "SELECT * FROM users WHERE username=? AND password=?";

public boolean exist(User user) throws SQLException {
    boolean exist = false;

    try (
        Connection connection = Database.getConnection();
        PreparedStatement statement = connection.prepareStatement(SQL_EXIST);
    ) {
        statement.setString(1, user.getUsername());
        statement.setString(2, user.getPassword());

        try (ResultSet resultSet = preparedStatement.executeQuery()) {
            exist = resultSet.next();
        }
    }       

    return exist;
}

, которые должны использоваться следующим образом: 

try {
    if (!userDAO.exist(username, password)) {
        request.setAttribute("message", "Unknown login. Try again.");
        request.getRequestDispatcher("/WEB-INF/login.jsp").forward(request, response);
    } else {
        request.getSession().setAttribute("user", username);
        response.sendRedirect("userhome");
    }
} catch (SQLException e) {
    throw new ServletException("DB error", e);
}

В реальной среде Java EE вы должны делегировать создание DataSource серверу контейнера / приложения и получить его из JNDI.В случае Tomcat см. Также, например, этот документ: http://tomcat.apache.org/tomcat-6.0-doc/jndi-resources-howto.html

3 голосов
/ 29 сентября 2011

Не похоже, что это объединено. Вы должны хранить DataSource в DatabaseConnection, а не создавать новый при каждом вызове getConnection (). getConnection () должен возвращать datasource.getConnection ().

2 голосов
/ 29 сентября 2011

Похоже на использование DBCP. Если так, то да. Это уже объединено. А вот значение свойства пула по умолчанию для DBCP. 

/**
* The default cap on the number of "sleeping" instances in the pool.
* @see #getMaxIdle
* @see #setMaxIdle
*/
public static final int DEFAULT_MAX_IDLE  = 8;
/**
* The default minimum number of "sleeping" instances in the pool
* before before the evictor thread (if active) spawns new objects.
* @see #getMinIdle
* @see #setMinIdle
*/
public static final int DEFAULT_MIN_IDLE = 0;
/**
* The default cap on the total number of active instances from the pool.
* @see #getMaxActive
*/
public static final int DEFAULT_MAX_ACTIVE  = 8;
1 голос
/ 11 марта 2014

В качестве продолжения решения BalusC ниже приведена реализация, которую я могу использовать в приложении, требующем более одного подключения, или в общей библиотеке, которая не знает заранее свойства подключения ... 

import org.apache.commons.dbcp.BasicDataSource;

import java.sql.Connection;
import java.sql.SQLException;
import java.util.concurrent.ConcurrentHashMap;

public final class Database {

    private static final ConcurrentHashMap<String, BasicDataSource> dataSources = new ConcurrentHashMap();

    private Database() {
        //
    }

    public static Connection getConnection(String connectionString, String username, String password) throws SQLException {

        BasicDataSource dataSource;

        if (dataSources.containsKey(connectionString)) {
            dataSource = dataSources.get(connectionString);
        } else {
            dataSource = new BasicDataSource();
            dataSource.setDriverClassName("com.mysql.jdbc.Driver");
            dataSource.setUrl(connectionString);
            dataSource.setUsername(username);
            dataSource.setPassword(password);
            dataSources.put(connectionString, dataSource);
        }

        return dataSource.getConnection();

    }

}
...