c++ - "Cleaning up"嵌套的 if 语句

Question

在我正在创建的控制台程序中，我有一些代码可以通过文件进行解析。解析每一行后，检查语法错误。如果存在语法错误，程序将停止读取文件并转到程序的下一部分。问题是，它非常困惑，因为到目前为止我唯一的解决方案是一系列嵌套的 if 语句或一行 if 语句。嵌套 ifs 的问题是它很快就会变得非常困惑，一系列 if 语句让程序测试一些不需要测试的东西。这是我的问题的一些 sudo 代码(注意我没有使用返回语句)

显示的是伪代码而不是真实代码，因为它非常大

嵌套如果:

open file;
read line;
//Each if is testing something different
//Every error is different
if (line is valid)
{
    read line;
    if (line is valid)
    {
        read line;
        if (line is valid)
        {
            do stuff;
        }
        else
            error;
    }
    else
        error;

}
else
    error;
code that must be reached, even if there was an error;

非嵌套 ifs:

bool fail = false;
open file;
read line;
//Each if is testing something different
//Every error is different
if (line is valid)
    read line;
else
{
    error;
    fail = true;
}
if (!error && line is valid)
    read line;
else
{
    error;
fail = true;
}
if (!error && line is valid)
    do stuff;
else
    error;
//Note how error is constantly evaluated, even if it has already found to be false
code that must be reached, even if there was an error;

我看过许多不同的网站，但他们的判决与我的问题不同。这段代码在运行时确实有效，但如您所见，它不是很优雅。有没有人对我的问题有更可读/更有效的方法？任何帮助表示赞赏:)

Answer 1

我想到了两个选项:

选项 1:链读取和验证

这类似于 std::istream 提取运算符的工作方式。你可以这样做:

void your_function() {
  std::ifstream file("some_file");
  std::string line1, line2, line3;
  if (std::getline(file, line1) &&
      std::getline(file, line2) &&
      std::getline(file, line3)) {
    // do stuff
  } else {
    // error
  }
  // code that must be reached, even if there was an error;
}

方案二:拆分成不同的功能

这可能会有点长，但如果你把事情分开(并给所有东西一个合理的名字)，它实际上是非常可读和可调试的。

bool step3(const std::string& line1,
           const std::string& line2,
           const std::string& line3) {
  // do stuff
  return true;
}

bool step2(std::ifstream& file,
           const std::string& line1,
           const std::string& line2) {
  std::string line3;
  return std::getline(file, line3) && step3(line1, line2, line3);
}

bool step1(std::ifstream& file,
           const std::string& line1) {
  std::string line2;
  return std::getline(file, line2) && step2(file, line1, line2);
}

bool step0(std::ifstream& file) {
  std::string line1;
  return std::getline(file, line1) && step1(file, line1);
}

void your_function() {
  std::ifstream file("some_file");
  if (!step0(file)) {
    // error
  }
  // code that must be reached, even if there was an error;
}

这个示例代码有点太简单了。如果在每个步骤中发生的行验证比 std::getline 的返回值更复杂(在进行实际输入验证时通常是这种情况)，那么这种方法的好处是更具可读性。但如果输入验证像检查 std::getline 一样简单，那么应该首选第一个选项。