2015-06-12 29 views
0

我最近開始研究由公司的內部開發人員開發的java webapp(JSP/Servlet)。數據庫連接受併發線程影響

這個應用程序隨機不返回數據,並檢查日誌,我發現一些NullPointerExceptions相關的類的成員變量持有數據庫連接。跟蹤堆棧跟蹤,看起來第二個線程在結束任務後關閉連接,離開沒有連接的第一個線程。

由於公司的需要,應用程序使用不同的數據庫,其中一個規則appdata,另一個包含應用程序必須檢索的數據。因此,根據所要完成的任務,連接到主Servlet的每個類都可以連接到一個或多個數據庫。

我不熟悉JavaEE,但看看數據庫連接類,我沒有看到任何東西保護線程互相沖突。

哪個是正確的方式來處理這種連接?

這是數據庫處理程序的代碼:

package it.metmi.mmasgis.utils; 

import java.sql.*; 
import java.util.ArrayList; 
import java.util.HashMap; 

import org.apache.logging.log4j.LogManager; 
import org.apache.logging.log4j.Logger; 

public class DBManager 
{ 
    private String szDatabase; 
    private String szUsername; 
    private String szPassword; 
    private String szError; 
    private Connection db; 
    private boolean bConnected; 
    private Logger logger; 

    public DBManager(String szDBName) 
    { 
     this(szDBName, "", ""); 
    } 

    public DBManager(String szDBName, String szName, String szPass) 
    { 
     szDatabase = szDBName; 
     szUsername = szName; 
     szPassword = szPass; 
     bConnected = false; 
     szError = ""; 
     logger = LogManager.getFormatterLogger(DBManager.class.getName()); 
    } 

    public boolean connect() 
    { 
     logger.entry(); 

     try { 
      Class.forName("com.mysql.jdbc.Driver"); 

      if(!szDatabase.isEmpty()) 
      { 
       String szCon = "jdbc:mysql://localhost/" + szDatabase; 

       if(!szUsername.isEmpty()) 
       { 
        szCon += "?user=" + szUsername; 

        if(!szPassword.isEmpty()) 
         szCon += "&password=" + szPassword; 
       } 

       db = DriverManager.getConnection(szCon); 
       bConnected = true; 
      } else { 
       logger.error("No database name!!"); 
       System.exit(0); 
      } 
     } catch(SQLException | ClassNotFoundException e) { 
      szError = e.getMessage(); 
      e.printStackTrace(); 
      logger.error("Can't connect: %s", e); 
     } 

     return logger.exit(bConnected); 
    } 

    public void disconnect() 
    { 
     logger.entry(); 

     try { 
      db.close(); 
      bConnected = false; 
     } catch(SQLException e) { 
      e.printStackTrace(); 
      logger.error("Can't disconnect: %s", e); 
     } 

     logger.exit(); 
    } 

    public boolean isConnected() 
    { 
     return bConnected; 
    } 

    public String getError() 
    { 
     return szError; 
    } 

    public ArrayList<HashMap<String,String>> query(String szQuery) 
    { 
     logger.entry(szQuery); 

     ArrayList<HashMap<String,String>> aResults = new ArrayList<HashMap<String,String>>(); 

     int iCols = 0; 
     try { 
      Statement stmt = db.createStatement(); 

      logger.info("Query: %s", szQuery); 
      ResultSet rs = stmt.executeQuery(szQuery); 
      ResultSetMetaData rsmd = rs.getMetaData(); 
      iCols = rsmd.getColumnCount(); 

      while(rs.next()) 
      { 
       HashMap<String,String> pv = new HashMap<String,String>(); 
       for(int i = 0; i < iCols; i++) 
       { 
        String szCol = rsmd.getColumnLabel(i + 1); 
        String szVal = rs.getString(i + 1); 
        pv.put(szCol, szVal); 
       } 
       aResults.add(pv); 
      } 
      rs.close(); 
      stmt.close(); 
     } catch(SQLException e) { 
      e.printStackTrace(); 
      szError = e.getMessage(); 
      logger.error("Error executing query: %s", e); 
     } 

     return logger.exit(aResults); 
    } 

    public boolean update(String szQuery) 
    { 
     logger.entry(szQuery); 

     boolean bResult = false; 

     try { 
      Statement stmt = db.createStatement(); 

      logger.info("Query: %s", szQuery); 
      stmt.executeUpdate(szQuery); 

      bResult = true; 
      stmt.close(); 
     } catch(SQLException e) { 
      e.printStackTrace(); 
      szError = e.getMessage(); 
      bResult = false; 
      logger.error("Error executing query: %s", e); 
     } 
     return logger.exit(bResult); 
    } 
} 

類任務,所有的Servlet類是基於,是一個簡單的抽象類:

package it.metmi.mmasgis.servlet; 

import javax.servlet.http.HttpServletRequest; 
import javax.servlet.http.HttpServletResponse; 

public abstract class Task 
{ 
    public abstract void doTask(HttpServletRequest request, HttpServletResponse response); 
} 

會拋出NullPointerException異常類它是在調用db.disconnect()期間的這一個。這個類通過AJAX快速調用4或5次從JS中編寫的接口。

package it.metmi.mmasgis.servlet.params; 

import it.metmi.mmasgis.servlet.Task; 
import it.metmi.mmasgis.utils.Const; 
import it.metmi.mmasgis.utils.DBManager; 
import it.metmi.mmasgis.utils.Query; 
import it.metmi.mmasgis.utils.Utility; 

import javax.servlet.http.HttpServletRequest; 
import javax.servlet.http.HttpServletResponse; 

import java.io.IOException; 
import java.io.PrintWriter; 
import java.util.ArrayList; 
import java.util.HashMap; 

import com.google.gson.Gson; 
import com.google.gson.GsonBuilder; 

import org.apache.logging.log4j.LogManager; 
import org.apache.logging.log4j.Logger; 

public class ClassType extends Task 
{ 
    private DBManager db = null; 
    private Logger logger = LogManager.getFormatterLogger(ClassType.class.getName()); 

    @Override 
    public void doTask(HttpServletRequest request, HttpServletResponse response) 
    { 
     logger.entry(request, response); 

     String szCensimento = Utility.getParameter(request, "censimento"); 
     String szCategoria = Utility.getParameter(request, "category"); 
     ArrayList<HashMap<String,String>> aClasses = new ArrayList<HashMap<String,String>>(); 
     PrintWriter out = null; 

     logger.debug("Census: %s", szCensimento); 
     logger.debug("Category: %s", szCategoria); 

     db = new DBManager(szCensimento, Const.DB_USER, Const.DB_PASS); 

     if(db.connect()) 
     { 
      String szQuery = String.format(Query.classes, szCategoria, szCategoria); 
      aClasses = db.query(szQuery); 

      db.disconnect(); 
     } 

     try { 
      out = response.getWriter(); 
      jsonEncode(aClasses, out); 
     } catch(IOException e) { 
      e.printStackTrace(); 
      logger.error("Failed to encode JSON: %s", e); 
     } 

     logger.exit(); 
    } 

    private void jsonEncode(ArrayList<HashMap<String,String>> aData, PrintWriter out) 
    { 
     HashMap<String,Object> result = new HashMap<String,Object>(); 
     result.put("results", aData); 
     result.put("success", true); 

     Gson gson = new GsonBuilder().create(); 
     gson.toJson(result, out); 
    } 
} 

如果Web應用程序將使用一個數據庫,它可以被改寫爲一個單例,但這種方式我對如何處理不同的數據庫不同的連接不知道。 如何避免這些異常?

+0

基本上這不是手動處理這種情況的正確決定。爲什麼不使用一些常見的做法,例如,使用Spring和Spring JDBC來處理所有這些東西。你正在做許多你不應該做的手動工作。看看這個:https://spring.io/guides/gs/relational-data-access/ –

+0

我剛剛閱讀你鏈接的指南。 那麼我需要重寫整個webapp才能使用Spring? 此外,這個應用程序不是基於Gradle,Maven或其他。 – HelLViS69

+0

如果您計劃您的應用程序將變得越來越大,這只是可能的和流行的方法。但如果不是,請繼續閱讀此帖。 http://stackoverflow.com/questions/5003142/jsp-using-mvc-and-jdbc/5003701#5003701 –

回答

0

問題是連接對象被聲明爲成員。 在解析的方法內移動變量。