将新对象作为默认参数传递给构造函数

Question

我有一个看起来像的构造函数：

RandomClass(const int16_t var1, const SecondClass& var2);

我需要将默认参数传递给第二个参数，因此目前我正在执行以下操作：

RandomClass(const int16_t var1, const SecondClass& var2 = *(new SecondClass(*(new std::unordered_map<int16_t, double>())));

真是尴尬 请注意，我不想使用重载的构造函数，也不想将第二个参数从引用更改为指针。

传递默认参数的一种优雅方式是什么？

Answer 1

您可以将const引用绑定到临时目录：

RandomClass(const int16_t var1, const SecondClass& var2 = SecondClass(std::unordered_map<int16_t, double>()));

当然，这假定您将var2复制到成员中，或者在构造函数退出后不使用它。 如果不是这种情况，则说明您的设计有问题。 也许使用原始指针或shared/weak_ptr更合适。

Answer 2

很好地做到这一点的方法是为构造函数提供两个重载：

RandomClass(const int16_t var1, const SecondClass& var2)
{
  // ...
}

RandomClass(const int16_t var1)
{
  const SecondClass var2 = *(new SecondClass(*(new std::unordered_map<int16_t, double>()));
}

但是，您遇到的问题要比看起来的糟糕得多。 第二次过载中的内容令人难以置信，因为这会导致内存泄漏-您需要delete所有new但现在您丢失了所有new痕迹。 您只需执行以下操作：

RandomClass(const int16_t var1)
{
  const SecondClass var2{std::unordered_map<int16_t, double>()};
}

不要使用new偏好自动存储时间-除非你要。