java - 处理以下异常的有效方法

Question

我有以下代码:

public boolean perform(CronJob job)
   final Date lastQueryDateTime = new Date();
   QueryMode queryMode = job.getQueryModeEnum();
   try {
      if (queryMode.getCode().equals("FULL")) 
      {
         calculateCoverage();    

      } else if (queryMode.getCode().equals("INCREMENTAL")) 
      {
          calculateCoverage();
      }        
       job.setLastQueryDate(lastQueryDateTime);
       job.save(job);

       return true;

      } catch (Exception e) {
       LOG.error("Error while calculating the coverage for products.", e);
       return false;
      }
}

private void calculateCoverage()
{

// Some Logic

}

我怀疑中间是否确实有太多逻辑需要处理，以及 try/catch 范围是否太宽。有更好的方法来做到这一点吗？我想在calculateCoverage中处理try/catch，但在这种情况下，我需要将job作为参数传递给calculateCoverage，我认为不是那么干净。有什么建议吗？

谢谢!

Answer 1

看起来这是一个 Quartz 作业(或类似的作业)，它不应该允许抛出异常。

因此，您并不是期望某个特定方法抛出特定类型的异常(在这种情况下，您可能希望缩小该方法调用的范围)。相反，您希望确保如果此方法内出现任何问题，您不会将 Cron Job 置于错误状态。

由于在这种情况下 try/catch 的唯一目的是记录引发的任何异常，并且您不打算以任何特殊方式处理它们，所以我认为这是完全合理的让 block 包含方法的全部内容。

不过，请注意整体逻辑。目前，计算覆盖率时出现异常将导致作业无法再次运行。如果您希望作业稍后运行，即使它现在失败了，您可能需要在那里有一个 try/finally block 。

try {
    final Date lastQueryDateTime = new Date();
    QueryMode queryMode = job.getQueryModeEnum();
    String code = queryMode.getCode();
    try{
        if (code.equals("FULL") || code.equals("INCREMENTAL")) 
        {
            calculateCoverage();
        }   
    } finally {
        job.setLastQueryDate(lastQueryDateTime);
        job.save(job);
    }
    return true;
} catch (Exception e) {
    LOG.error("Error while calculating the coverage for products.", e);
    return false;
}

这仍然会导致捕获并记录方法中的任何异常，但如果在计算覆盖率时发生错误，您仍然会尝试将作业设置为再次运行。