在 DAO 中捕获错误有好处吗？

Question

public boolean checkInd() {
    int dis_ind = 2;
    HashMap parmMap = new HashMap();
    //line below can generate errors
    getSqlMapClientTemplate().queryForList("authentication.checkInd", parmMap);
    List results = (List) parmMap.get("Result0");
    HashMap resultMap;
    if (result.size()>0)
        resultMap = (HashMap)resultMap.get(0); 
    dis_ind = (Integer)resultMap.get("PK_VALUE");
    if (dis_ind == 1)
        return true;
    else
        return false;
}

代码的第 5 行可能会产生错误。仅举几例，错误可能是...映射中提到的存储过程不存在，映射中提到的列名不存在..等等。

如果我没有捕获错误并且出现问题，则日志跟踪将写入server.log 并且 Web 应用程序停止在我的 web.xml 中提到的错误页面

如果我捕获错误，那么 Web 应用程序不会停止在错误页面并继续其快乐方式（谁知道现在会停在哪里）。

所以现在我想不要在这里发现错误。

问题是我不想在 server.log 中写入堆栈跟踪（如上所述，现在默认情况下会发生这种情况）。相反，我想将堆栈跟踪写入名为 myapp.log 的文件（我们使用的是 log4j）。但是，我不知道如何在不实际捕获错误的情况下将堆栈跟踪显式写入 myapp.log 。下面是我要编写的代码（但应用程序不会停止，因为我捕获了错误并将继续运行，这不好）。

OurLogger log = OurLogger.getLogger("myapp", MyClassName.class);
public boolean checkInd() {
    int dis_ind = 2;
    try {
    HashMap parmMap = new HashMap();
    //line below can generate errors
    getSqlMapClientTemplate().queryForList("authentication.checkInd", parmMap);
    List results = (List) parmMap.get("Result0");
    HashMap resultMap;
    if (result.size()>0)
        resultMap = (HashMap)resultMap.get(0); 
        dis_ind = (Integer)resultMap.get("PK_VALUE");
    }
    catch (BadSqlGrammarException e) {log.error(e, e.getMessage());}
        if (dis_ind == 1)
            return true;
        else
            return false;
}

在这种情况下我有哪些选择以及最佳实践是什么？

PS：这个问题与我的其他问题相关 但我尝试在这个问题中添加示例并寻求最佳实践建议。

Answer 1

捕捉并不需要是“永久的”。只需捕获它，记录它，然后重新抛出错误。就像这样：

catch (BadSqlGrammarException e) {
  log.error(e, e.getMessage());
  throw e;
}