c++ - 这种回调的使用是惯用的吗？

Question

我注意到我的一些代码中有一个共同的模式

std::string line;
if (!in || !std::getline(in, line)) {
  throw line_read_error(in,line_counter);
}
++line_counter;
std::istringstream sin{line};

// ...read from sin...

if (!sin.eof()) {
  sin.clear();
  throw incomplete_read_error(in,line_counter,sin.tellg());j
}

我从线路上读取的内容在每个位置都不同，但设置和读取后检查是相同的。

我把它分解出来，创建一个对象来保存我的 in 流和 line_counter，并传递一个回调对于 body :

class LineReader {
  std::istream& in;
  size_t line_counter;
public:
  template <typename Function>
  void with_next_line(Function callback) {
    std::string line;
    if (!in || !std::getline(in, line)) {
      throw line_read_error(in,line_counter);
    }
    ++line_counter;
    std::istringstream sin{line};

    callback(sin);

    if (!sin.eof()) {
      sin.clear();
      throw incomplete_read_error(in,line_counter,sin.tellg());j
    }
  }
  // ...
}

改变我的用途

line_reader.with_next_line([/*...*/](auto& sin){
  // ...read from sin...
});

这样重复肯定少了，但是还是有点别扭。

然而，我真正关心的是其他人是否容易人们要关注，因为我真的在努力使我的代码尽可能清晰。

像这样的东西我会过得更好吗

auto sin = line_reader.get_next_line();

// ...read from sin...

line_reader.check_done(std::move(sin));

Answer 1

执行设置 + 清理的正常方法是拥有一个对象，其构造函数执行设置而析构函数执行清理 (RAII)。

但是，如果您还没有读到最后，您要做的清理工作就是抛出 - 从析构函数中抛出是邪​​恶的、糟糕的和错误的。这意味着您不能在这种特殊情况下使用 RAII。

如果进行检查真的很重要，那么您拥有的代码将执行它。如果它只是“一个好主意”，那么我认为这两个调用(之前和之后)可能比 lambda 稍微干净一些。 (我不会理会 std::move(sin) - 使用 move 在这里并没有真正添加任何东西。)