如何重构这个嵌套的try-catch - How to refactor this nested try-catch

How to refactor this nested try-catch

本文关键字：嵌套 try-catch 何重构重构 | 更新日期: 2023-09-27

我是try-catch的新手，最近一直在重构大量代码，以跟上DRY的原则。然而，我仍然有一些我不能完全解决的场景。例如，在我正在开发的多语言应用程序中，我想让用户通过在URL的查询字符串中传递两个字母的国家代码来设置界面语言，或者如果用户没有请求的话，让应用程序使用默认语言:

// Which language shall we use?
$language = new language();
if(isset($_GET['language_code'])){
    try {
        $language->set_by_language_code($_GET['language_code']);
    } catch(e_language_not_found $e){
        try {
            $language->set_default_language();
            // TODO - use 'e_language_not_found' to display an error in the default language
        } catch(Exception $e){
            exit('No default language found');
        }
    }
} else {
    try {
        $language->set_default_language();
    } catch(Exception $e){
        exit('No default language found');
    }
}

问题是，当用户根本没有要求任何语言时，我重复了set_default_language()块。如何重构代码，以便在应用程序中只调用该方法一次?谢谢!

在我看来，你应该在你的language类中实现这些方法:

language::hasLanguage()返回一个布尔值，指示语言的存在(不抛出异常!)
language::getDefLang()返回默认语言标识符。

// Which language shall we use?
$language = new language();
// fallback
$langToSet = $language->getDefLang();
// use the passed language identifier if it exists
$passedLangId = isset($_GET['language_code']) ? $_GET['language_code'] : NULL;
if ($passedLangId !== NULL && $language->hasLanguage($passedLangId)) {
  $langToSet = $passedLangId;
}
try {
  $language->set_by_language_code($langToSet);
}
catch(Exception $e){
  exit('No default language found');
}

您还应该为catch块指定更详细的Exception类型。

您也可以在if语句中使用language::hasLanguage()来替换try...catch块。

在两个地方调用函数本身并不坏。它不会影响性能，重构它可能会使代码的可读性降低。

这是我对你的代码的想法:

// Which language shall we use?
$language = new language();
try {
  try {
    if(isset($_GET['language_code'])){
      $language->set_by_language_code($_GET['language_code']);
    } else {
      $language->set_default_language();     
    }
  } catch(e_language_not_found $e){    
      $language->set_default_language();    
  }
} catch (Exception $e){
  exit('No default language found');
}