没有使用智能或原始指针调用析构函数[关闭]

Question

从previous question继续：

在这段代码中，Apple 和 Fruit 的析构函数根本不会被调用。我在两者中都有std::cerr 语句，并且Apple 中有一些清理代码无法运行。我认为调用删除就足够了？我正确地做 RAII 吗？我还用 std::unique_ptr 替换了原始指针，结果相同。

int32_t Fruit::frutificate(const Settings& settings) {
  Fruit *listener;
  if (settings.has_domain_socket()) {
    listener = new Apple(settings);
  } else {
    listener = new Orange(settings);
  }
  int r = uv_run(listener->loop, UV_RUN_DEFAULT);
  delete listener;
  return r;
}

更新： 所有类都有虚拟析构函数。

Answer 1

首先，您的直接问题几乎可以肯定是~Fruit() 不是virtual。添加（virtual ~Fruit() = default 或 virtual ~Fruit() {} 到 class Fruit），您的代码（如发布的）将神奇地开始工作。

但这不是您的代码应有的样子。只是工作，好吧，还不够好。

您可以对代码进行许多改进。作为第一个改进，我们将使用 unique_ptr:（如上面 cmets 中提到的 @Deduplicator）

int32_t Fruit::frutificate(const Settings& settings) {
  std::unique_ptr<Fruit> listener;
  if (settings.has_domain_socket()) {
    listener.reset( new Apple(settings) );
  } else {
    listener.reset( new Orange(settings) );
  }
  int r = uv_run(listener->loop, UV_RUN_DEFAULT);
  return r;
}

它使用 RAII 来确保监听器的生命周期是有限的。好多了，不再需要手动 delete（可能会因意外或异常而遗漏）。

在 C++14 中，.reset(new Blah(whatever)) 可以替换为= std::make_unique<Blah>(whatever);，现在您的代码不会显式调用new 和delete，这是一个很好的习惯。但是您的代码标记为 C++11，所以我将 C++11 版本保留在上面。

虽然这样更好，但我们可以做到最好。完全不需要使用免费的存储（堆）。

避免使用免费商店的一个简单方法是：（从上面 cmets 中的@Jarod 窃取）

int32_t Fruit::frutificate(const Settings& settings) {
  if (settings.has_domain_socket()) {
    return uv_run(Apple(settings).loop, UV_RUN_DEFAULT);
  } else {
    return uv_run(Orange(settings).loop, UV_RUN_DEFAULT);
  }
}

它的缺点是重复uv_run 代码（因此会滋生错误）。我们可以用 lambda 解决这个问题：

int32_t Fruit::frutificate(const Settings& settings) {
  auto fruit_the_uv = [&](Fruit&& fruit) {
    return uv_run(fruit.loop, UV_RUN_DEFAULT);
  };
  if (settings.has_domain_socket()) {
    return fruit_the_uv( Apple(settings) );
  } else {
    return fruit_the_uv( Orange(settings) );
  }
}

我们将公共代码分解为 lambda，然后在两个分支上调用它。我在传递临时水果时使用了右值引用。

另外，fruit_the_uv 让我想起 90s rap song 每当我读到它。这是一个加分项。