如何按照Bob叔叔的规则（建议）正确使用一个尝试块，以一种方法正确地使用一种方法？

Question

例如，我有一个方法

void process(String userId) {
  if(userId == null) throw new IlligalArgumentException("User ID is required");

  User user = userService.findUserById(userId);

  if(user == null) throw new UserNotFoundException("User with ID: "+ userId +" not found");
  
  try {
    DataResponse response = analyticsAPI.loadAnalytics(userId, user.getDob(), user.getFirstName());  

    //logic
  } catch(AnalyticsAPIException e) {
     //logic
  }
}

1. illigalArgumentException是未检查 exception
2. usernotfoundexception是 未检查的例外
3. aNalyticsapiexception /code>是检查例外，

我读到，从尝试结束的方法开始使用该方法是一种最佳实践，而不是在一种方法中乘以try-catch块。

更喜欢异常而不是错误代码，我们更喜欢异常而不是错误代码 因为它们更明确。在处理尝试 /捕获时，我们 不应该在函数中添加比尝试 /捕获块更多的逻辑，因此 该功能有一件事：处理错误。建议：不要使用 嵌套的尝试 /捕获。< / p>

这样的事情：

void process(String userId) {
  try {
    if(userId == null) throw new IlligalArgumentException("User ID is required");
  
    User user = userService.findUserById(userId);
  
    if(user == null) throw new UserNotFoundException("User with ID: "+ userId +" not found");
    
    DataResponse response = analyticsAPI.loadAnalytics(userId, user.getDob(), user.getFirstName());  
    // logic
  } catch(AnalyticsAPIException e) {
       //logic
  }
}

但是看起来很奇怪。我在TryCatch块的内部抛出了一个例外，并希望它不会在捕获量中处理。我希望它将被投入到称为该方法的服务上。

我可以接下来：

public void process(String userId) {
  if(userId == null) throw new IlligalArgumentException("User ID is required");

  User user = userService.findUserById(userId);

  if(user == null) throw new UserNotFoundException("User with ID: "+ userId +" not found");
  
  DataResponse response = callApi(userId, user.getDob(), user.getFirstName());  

  //logic
 }
    
private DataResponse callApi(String userId, Date dob, String firstName){
  try {
    return analyticsAPI.loadAnalytics(userId, user.getDob(), user.getFirstName());  
  } catch(AnalyticsAPIException e) {
    //logic
  }
}

但这并不总是有效。那么，什么越好？

Answer 1

叔叔提出的是，您没有尝试或捕获块中的语句列表。相反，您应该将“正常”情况与异常情况分开。目标是分开不同级别的抽象。

为了避免姓名碰撞，我经常将“正常”案例与“ try”单词前缀。
我也经常以自己的方法将尝试/捕捉以保持专注。例如，

“过程”方法应集中在处理用户（用户ID）的含义上。这是一个抽象级别，如果您将其与其他方法分开，则更容易阅读和理解。

void process(String userId) {
    User user = getUserById(userId);

    loadAnalytics(user);
}

GetusById仅关注您要通过ID来获得用户时所需的逻辑。

void void getUserById(String userId){
    if(userId == null) throw new IlligalArgumentException("Usesr ID is required");
    
    User user = userService.findUserById(userId);
    if(user == null) throw new UserNotFoundException("User with ID: "+ userId +" not found");
    
    return user;
}

要了解“过程”方法，不必理解
加载分析可能会导致特殊状态，甚至会导致其处理方式。

但是，当您深入研究LoadAnalytics方法时，您想知道它的工作原理。现在，您可以立即看到加载分析可能会导致异常状态。该方法专注于异常处理，因为它仅包含尝试/捕获，您还专注于可能发生的例外。

void loadAnalytics(User user){
    try {
     tryLoadAnalytics(user);
    } catch(AnalyticsAPIException e) {
     handleAnalyticsError(e);
    }
}

我经常使用“尝试”前缀来避免名称碰撞，并清楚地表明方法可能会失败。

void tryLoadAnalytics(){
    DataResponse response = callApi(userId, user.getDob(), user.getFirstName());  
    
    //logic
}

与“正常”情况一样，异常处理也分开，因此您可以专注于如何处理特定异常。

void handleAnalyticsError(AnalyticsAPIException e){
    //logic
}