在 C++ 中重构代码 [关闭]

Question

类路由的构造函数最初包含以下代码，用于检查文件中是否存在元素（“gpx”、“rte”等）。哪个运行正常。

  if (! elementExists(source,"gpx"))
  {
      oss << endl << "no gpx tag";
      constructorReport = oss.str();
      constructorSucceeded = false;
      return;
  }
  if (! elementExists(source,"rte"))
  {
      oss << endl << "no rte tag";
      constructorReport = oss.str();
      constructorSucceeded = false;
      return;
  }

我尝试引入一个函数来替换这些 if 语句。程序运行良好。

void Route::constCheck(string source, string type)
{

    if (! XML_Parser::elementExists(source, type))
    {
        std::ostringstream oss;
        oss << std::endl << "no" << type <<" tag";
        constructorReport = oss.str();
        constructorSucceeded = false;
        return;
    }
}

我已经更改了它检查产生错误的 gpx 文件，但是使用我添加的功能，它会继续运行，就好像没有错误一样。

感谢任何帮助，如果您需要更多信息，请告诉我。我试图按照指南保持代码简洁。

Answer 1

在您的原始代码中，您在其中一个测试第一次失败时从函数返回，您不会继续尝试其他测试。

既然您已将测试移至函数中，调用者无法知道测试是否失败，因此它将执行所有测试，并且在其中一个测试失败时永远不会从其函数返回。您需要此函数返回一个布尔值，指示它是否失败。

bool Route::constCheck(string source, string type)
{

    if (! XML_Parser::elementExists(source, type))
    {
        std::ostringstream oss;
        oss << std::endl << "no" << type <<" tag";
        constructorReport = oss.str();
        constructorSucceeded = false;
        return false;
    }
    return true;
}

那么原始代码的替换将如下所示：

if (!constCheck(source, "gpx")) {
    return;
}
if (!constCheck(source, "rte")) {
    return;
}