Servlet 和同步

Question

Servlet 在多个线程中运行，所以我的问题是：

如果我有很多调用某些实用程序类（DbUtils，例如 DbUtils）的 servlet，

Connection c = DbUtils.getConnection();
//....some action with db here

我是否应该在 DbUtils 内采取额外的同步操作？

实际上我想将 HttpServlet 继承为 DatabaseInvokerServlet 之类的东西：

公共抽象类 DatabaseInvokerServlet

使用方法扩展了 HttpServlet：

public abstract void getResultSets(Connection connection) throws SQLException;
private AbstractUser currentUser;
private HttpServletRequest request;
private HttpServletResponse response;
protected void processData() {}
protected void afterRequestProcessed() throws ServletException, IOException {}
protected void beforeRequestProcessed() throws ServletException, IOException {}

protected void execute() {
    Connection c = null;
    try {
        c = DbUtils.getConnection();
        getResultSets(c);
        processData();
    } catch (SQLException e) {
        e.printStackTrace();
    } finally {
        try {
            if (c != null) {
                c.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
}

public HttpServletRequest getRequest() {
    return request;
}

public HttpServletResponse getResponse() {
    return response;
}

public AbstractUser getCurrentUser() {
    return currentUser;
}

protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    request.setCharacterEncoding("UTF-8");
    response.setContentType("text/html;charset=UTF-8");
    response.setCharacterEncoding("UTF-8");


    this.request = request;
    this.response = response;
    this.currentUser = (AbstractUser) request.getSession().getAttribute("currentUser");

}

然后我只需将 DatabaseInvokerServlet 继承到新的 servlet 即可执行自定义操作。原因是不在很多地方使用 try-catch-finally 复制粘贴数据库调用块，

但正如我所见，由于同步问题，这种方法不起作用，对吗？

Answer 1

如果 DbUtils 在同一线程中创建连接，如下所示：

public static Connection getConnection() throws SQLException {
    return DriverManager.getConnection(url, username, password);
}

那么它是线程安全的。

但如果连接是一个类变量，如：

private static Connection connection = DriverManager.getConnection(url, username, password);

public static Connection getConnection() throws SQLException {
    return connection;
}

那么它绝对不是线程安全的，因为同一个连接将在所有线程之间共享。此外，当它在线程中关闭时，所有后续线程将无法使用该连接，因为它不再打开。此外，当它从未关闭时，数据库迟早会使连接超时，通常是几个小时后，并且您的应用程序将不再工作，因为连接不再打开。

---

至于servlet，

public abstract class DatabaseInvokerServlet extends HttpServlet {
    private AbstractUser currentUser;
    private HttpServletRequest request;
    private HttpServletResponse response;
    // ...
}

它肯定不是线程安全的。您将当前用户、请求和响应分配为实例变量。每个 servlet 类在应用程序的生命周期内只有一个实例。在整个应用程序的生命周期中，该实例在所有访问者/会话之间共享。每个 HTTP 请求都在单独的线程中运行并使用相同的实例。

想象两个同时访问者：访问者 A 将设置当前用户、请求和响应。然而，数据库过程需要很长时间。在访问者 A 的响应返回之前，访问者 B 调用相同的 servlet，因此当前用户、请求和响应将被覆盖。然后，访问者 A 的查询完成并想要写入响应，而是写入访问者 B 的响应！访问者 B 看到访问者 A 的查询结果，而访问者 A 在屏幕上看不到任何内容！

您永远不应该将特定于请求/会话的数据分配为 servlet 的实例变量。您应该将它们的方法（线程）保留在本地。

public abstract class DatabaseInvokerServlet extends HttpServlet {
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        AbstractUser currentUser = request.getSession().getAttribute("user");
        // Keep the variables in the method block! 
        // Do not assign them as instance variable!
    }
}

---

就整体而言，这种方法是笨拙的。数据库访问层应该与 servlet 无关。它应该在自己的独立类中运行，您可以在每个其他 Java 类、任何 servlet 类或带有 main() 的普通应用程序或其他类中构造/调用这些类。您的 servlet 类中不应有任何单行 java.sql.* 导入（如果没有抽象出来，可能会出现 SQLException）。您的数据库类中不应有任何单行 javax.servlet.* 导入。

另请参阅：

• Servlet 实例化和（会话）变量
• 基本 DAO 教程

Answer 2

如果实用程序类具有状态（例如：类或实例变量），则很可能是。

Answer 3

如果我猜对了，DBUtils 将为每次调用 getConnection() 返回新实例。由于 DBUtils 类是一个实用程序类，因此它不应该维护任何状态。在这种情况下，您不需要任何额外的同步工作。

Answer 4

Servlet 在多个线程中运行。

The J2EE spec says there is only one instance per servlet class running in one web container for non single thread servlet. 

Servlet 2.3 规范

Servlet 容器可以发送
并发请求通过
Servlet 的服务方法。到
处理开发者的请求
servlet 必须足够
并发处理的规定
服务中有多个线程
方法。

Servlet 中的同步。

Never have an member variable in a servlet, it is not thread safe.