Java异常工厂较高水平异常

我有一个连接到不同的数据存储和这些数据存储抛出一些例外的服务。我不想将这些例外返回给服务的用户，而是返回更高级别的例外 storeServiceTransientReadException

当连接到数据存储时，我将使用try，catch抛出模式;例如，对于卡桑德拉连接：

public ResultSet safeExecute(Statement statement) throws StoreServiceException { 
    try { 
     return session.execute(statement); 
    } 
    catch(QueryExecutionException ex){ 
     log.error(ex); 
     StoreServiceException storeException = StoreServiceExceptionFactory.getTransientException(ex); 
     throw storeException; 
    }

在卡桑德拉的例子，我想该工厂创建一个读或写这取决于是否异常是ReadFailureException，ReadTimeoutException，WriteFailureException或WriteTimeoutException

对于其他数据storeServiceException例外商店我想要遵循相同的模式，那么服务的用户只需要担心服务错误而不是特定的数据存储错误。

对于工厂我想沿（伪）线的东西：

public class ExceptionsFactory { 

public StoreServiceTransientException getTransientException(Exception ex){ 

    if ReadException 
     return StoreServiceTransientException("read exception ") 


    if WriteException 
     return StoreServiceTransientException("write exception ") 

} 

public StoreServiceNonTransientException getTransientNonException(Exception ex){ 

    if ReadException 
     return StoreServiceNonTransientException("read exception ") 


    if WriteException 
     return StoreServiceNonTransientException("write exception ") 


}

但我无法找到这里面我担心许多网上的例子。这是一个非常糟糕的主意？我应该只有更多特定的catch块返回我想要的storeServiceException？

来源

2017-07-07 JJHolloway

在Java中，通常的做法是类名以大写字母开头。 'StoreServiceTransientException'将是一个快速但巨大的改进 – byxor

@byxor好点 - 当我尝试使示例更通用 – JJHolloway

这是一个非常糟糕的主意？在我看来，是的。这是一个坏主意。使用Exception（s）的昂贵部分正在填充堆栈跟踪。如果您预先创建并保存了您所抛出的Exception，那么堆栈跟踪将变得毫无意义（或至少大大降低了值）。您还没有登录当前堆栈跟踪，所以我也会改变

log.error(ex);

到

log.error("Caught exception: " + ex.getMessage(), ex);

而且具有相似的根本原因实例化的例外 -

throw new storeServiceException("Exception in storage.", ex);

而且名称应遵循正常的命名约定。 Java类名称以大写字母开头 - “你”看起来像一个变量。

来源

2017-07-07 12:42:28

我不好 - 更新示例并删除了错字时，我的部分错别字。谢谢（你的）信息！ – JJHolloway

Java异常工厂较高水平异常

回答

相关问题