Exception时信息的记录

系统总有出现异常的时候，那么出现异常时应该如何处理？

 

一直以来，我都以为这么处理就足够的：

• 在日志中打印Exception的堆栈信息，以便排查原因
• 反馈给用户系统xxx出现问题

package com.nicchagil.util.requestlogger;

import java.io.IOException;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;

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

import org.apache.log4j.Logger;

public class RequestDemoServlet extends HttpServlet {
    
    private final Logger logger = Logger.getLogger(RequestDemoServlet.class);
       
    public RequestDemoServlet() {
        super();
    }

    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    
        ResultSet rs = null;
        PreparedStatement pstmt = null;
        Connection conn = null;
        try {
            Class.forName("oracle.jdbc.driver.OracleDriver");
            conn = DriverManager.getConnection(
                    "jdbc:oracle:thin:@hostname:port:sid", "username",
                    "password");
            
            pstmt = conn.prepareStatement("select * from t_xxx t where t.id = ? ");
            
            pstmt.setString(1, "paramter");
            rs = pstmt.executeQuery();
            
            while (rs.next()) {
                System.out.println("'" + rs.getString("result1") + "' - " + "'" + rs.getString("result2") + "'");
            }
            
        } catch (ClassNotFoundException e) {
            // Print exception logs
            logger.error("Failed to query xxx", e);
            
            // Prompt that system error
            response.getWriter().write("Failed to query xxx!");
            
        } catch (SQLException e) {
            // Print exception logs
            logger.error("Failed to query xxx", e);
            
            // Prompt that system error
            response.getWriter().write("Failed to query xxx!");
            
        } catch (Throwable t) {
            // Print exception logs
            logger.error("System error", t);
            
            // Prompt that system error
            response.getWriter().write("System error!");
            
        } finally {
            try {
                if (rs != null) {
                    rs.close();
                    rs = null;
                }
                if (pstmt != null) {
                    pstmt.close();
                    pstmt = null;
                }
                if (conn != null) {
                    conn.close();
                    conn = null;
                }
            } catch (SQLException e) {
                // Print exception logs
                logger.error("System error", e);
                
                // Prompt that system error
                response.getWriter().write("System error!");
            }
        }
    
    }

    protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        this.doGet(request, response);
    }

}

 

日前，得到一位资深同事Glen的指点，深知以上的操作是不足够的。

简单来说，我们需要记录，谁在什么时候做了什么事情。

尽量详细地记录这些信息，让我们拥有更详细的信息用于排查问题（不知道请求参数的情况下，确实难以定位问题）；在和其他团队作联合调试时，我们也可以提供足够的信息，不致陷于尴尬、被动地位。

• 当前登录用户的ID或代号（在系统需要登录并用户已登录的前提下）
• 当前请求的URL
• 当前的请求参数

 

一般情况下，Web Project是在Servlet（或使用框架时所称的Action、Controller、Ctrl）处理Exception的。

除了“当前的请求参数”外，其他的都较好处理。因为每个请求的参数不尽一致，在每个Servlet都hardcode来打印各个参数，这是一个累人的活。

这里，分享一个静态方法用于打印HttpServletRequest的全部请求参数，这样就不致于每次都hardcode打印请求参数了。

package com.nicchagil.util.requestlogger;

import java.util.Iterator;
import java.util.Map;

import javax.servlet.http.HttpServletRequest;

public class RequestLogger {
    
    /**
     * toString for HttpServletRequest Parameters
     * @param request
     * @return
     */
    public static String toString(HttpServletRequest request) {
        
        Map map = request.getParameterMap();
        
        /* Since there are String[] in the map, can not return map.toString() directly. */
        
        if (map == null || map.isEmpty()) {
            return "";
        }
        
        StringBuffer sb = new StringBuffer();
        Object key = null;
        String[] value = null;
        Iterator iterator = map.keySet().iterator();
        
        while (iterator.hasNext()) {
            key = iterator.next();
            value = (String[])map.get(key);
            
            sb.append(key.toString()).append(" : ").append(toString(value, true));
            sb.append("\n");
        }
        
        return sb.toString();
    }
    
    /**
     * toString for String Array 
     * @param stringArray
     * @return
     */
    public static String toString(String[] stringArray, boolean alwaysArray) {
        
        if (stringArray == null || stringArray.length == 0) {
            return "";
        }
        
        if (!alwaysArray) {
            if (stringArray.length == 1) {
                return stringArray[0];
            }
        }
        
        StringBuffer sb = new StringBuffer();
        sb.append("[");
        
        for (int i = 0; i < stringArray.length; i++) {
            sb.append(stringArray[i]);
            
            if (i < stringArray.length - 1) {
                sb.append(", ");
            }
        }
        
        sb.append("]");
        
        return sb.toString();
    }

}