10

我有一个访问数据库的 Jersey 资源。基本上它在资源的初始化中打开一个数据库连接。对资源的方法执行查询。

我观察到，当我不使用@Singleton 时，每次请求都会打开数据库。而且我们知道打开连接真的很昂贵，对吧？

所以我的问题是，我应该指定资源是单例的，还是根据请求保留它真的更好，尤其是在资源连接到数据库时？

我的资源代码如下所示：

//Use @Singleton here or not?
@Path(/myservice/)
public class MyResource {

    private ResponseGenerator responser;
    private Log logger = LogFactory.getLog(MyResource.class);

    public MyResource() {
        responser = new ResponseGenerator();
    }

    @GET
    @Path("/clients")
    public String getClients() {

        logger.info("GETTING LIST OF CLIENTS");

        return responser.returnClients();
    }

    ...
    // some more methods
    ...

}

我使用类似于以下的代码连接到数据库：

public class ResponseGenerator {
    private Connection conn;
    private PreparedStatement prepStmt;
    private ResultSet rs;

    public ResponseGenerator(){
        Class.forName("org.h2.Driver");
        conn = DriverManager.getConnection("jdbc:h2:testdb");
    }

    public String returnClients(){
        String result;
        try{
           prepStmt = conn.prepareStatement("SELECT * FROM hosts");

           rs = prepStmt.executeQuery();

           ...
           //do some processing here
           ...
        } catch (SQLException se){
            logger.warn("Some message");
        } finally {
            rs.close();
            prepStmt.close();
            // should I also close the connection here (in every method) if I stick to per request
            // and add getting of connection at the start of every method
            // conn.close();
        }

        return result
    }

    ...
    // some more methods
    ...

}

对代码最佳实践的一些评论也会有所帮助。

4

2 回答 2

0

与其考虑使资源成为单例，不如更专注于管理后端、服务类型的对象，例如ResponseGenerator作为单例的类，显然不应该在每个请求中都实例化。

将资源设置为单例也是作为单例进行管理的一种方式ResponseGenerator，但这不是唯一或最好的方式，请参阅访问 Jersey Resource 类中的外部对象和如何将协作者连接到 Jersey 资源中？将其注入非单例资源的方法。

请注意，您的ResponseGenerator类在作为单例运行之前需要工作，无论是注入到每个请求资源中还是在单例资源中实例化。它不是线程安全的，您将在启动时打开一个连接并在请求之间重用它，这是行不通的，您应该使用连接池来完成高效+安全地重用请求之间的连接的繁重工作。

对代码最佳实践的一些评论也会有所帮助。

你会在http://codereview.stackexchange.com上得到更好的回应，但是：

  • ResponseGenerator对于一个类来说是一个糟糕的名字（Web 应用程序中的几乎所有东西都是响应生成器）。

  • 不要使用 String 作为你的服务和对象的返回类型，使用正确的类型对象（例如，听起来你正在返回一个java.util.List东西）。

  • 不要吞下您的 SQLException，将其冒泡以允许 Jersey 在您的资源中生成 5xx 系列响应代码。

  • 使用最终成员变量。

  • 您的日志对象应该是静态的。

于 2016-09-17T22:48:43.463 回答
-4

您最好的选择是使用 Spring with Jersey 之类的框架，我在类似的帖子中对此进行了概述。唯一的区别是，不是注入服务 bean，而是注入一个池化的 DataSource，这可以使用c3p0轻松配置。

示例 applicationContext.xml，注意“范围”设置为原型，相当于 Spring 用语中的单例。

<bean id="pooledDataSource" scope="prototype" class="com.mchange.v2.c3p0.ComboPooledDataSource" destroy-method="close">
    <property name="jdbcUrl" value="${jpa.url}" />
    <property name="user" value="${jpa.username}" />
    <property name="password" value="${jpa.password}" />
    <property name="initialPoolSize" value="1" />
    <property name="minPoolSize" value="1" />
    <property name="maxPoolSize" value="3" />
    <property name="idleConnectionTestPeriod" value="500" />
    <property name="acquireIncrement" value="1" />
    <property name="maxStatements" value="50" />
    <property name="numHelperThreads" value="1" />
</bean>

在您的 MyResource.java 中，您只需添加以下内容，Spring 就会适当地注入它。

private DataSource pooledDataSource;
public void setPooledDataSource(DataSource pooledDataSource) {
    this.pooledDataSource = pooledDataSource;
}

然后您可以更改您的 ResponseGenerator 以接受 DataSource 并使用它来查询数据库。

于 2010-05-20T21:05:14.543 回答