多次调用初始化函数时的最佳实践？

Question

这可能是一个主观问题，但我或多或少是在问这个问题，并希望人们分享他们的经验。 （因为这是我在 C++ 中缺少的最大的东西）

无论如何，假设我有一个初始化函数，它从堆中初始化数据结构：

void initialize() {
    initialized = true;
    pointer = new T;
}

现在当我调用初始化函数两次时，就会发生内存泄漏（正确的？）。所以我可以通过多种方式来防止这种情况：

• 忽略调用（只需检查我是否已初始化，如果我不执行任何操作）
• 抛出错误，
• 自动“清理”代码，然后重新初始化该事物。

现在，通常什么是“最佳”方法，有助于保持我的代码将来易于管理？

编辑：感谢您迄今为止的回答。不过我想知道人们如何处理这个更通用的方式。 - 人们如何处理可以被忽略的“简单”错误。 （比如，调用同一个函数两次，但只有 1 次才有意义）。

Answer 1

您是唯一能够真正回答这个问题的人：您是否认为 initialize 函数最终可能被调用两次，或者这是否意味着您的程序遵循了意外的操作？执行流程 ?

• 如果initialize函数可以被多次调用：只需通过测试分配是否已经发生来忽略该调用。
• 如果 initialize 函数没有合理的理由被多次调用：我相信这将是一个很好的例外候选者。

需要明确的是，我不认为清理和重新生成是一个可行的选择（或者您应该认真考虑重命名该函数以反映这种行为）。

Answer 2

对于可能并不总是需要的昂贵数据结构的按需或延迟初始化来说，这种模式并不罕见。 Singleton 就是一个例子，或者是满足这些条件的类数据成员的例子。

如果结构已经就位，我要做的就是跳过初始化代码。

void initialize() {
    if (!initialized)
    {
      initialized = true;
      pointer = new T;
    }
}

如果您的程序有多个线程，则必须包含锁定以使其线程安全。

Answer 3

我会考虑使用 boost 或 STL 智能指针。

Answer 4

我认为答案完全取决于 T （以及此类的其他成员）。如果它们是轻量级的并且不存在重新创建新的副作用，那么一定要清理并重新创建（但使用智能指针）。另一方面，如果它们很重（例如网络连接或类似的东西），那么如果设置了布尔值，您应该简单地绕过...

您还应该调查 boost::Optional，这样您就可以不需要整体标志，并且对于应该存在的每个对象，您可以检查是否已实例化，然后根据需要进行实例化...（在第一遍中说，有些构造正常，但有些失败......）

Answer 5

在构造函数之后设置数据成员的想法很常见，所以不用担心，您绝对不是第一个遇到此问题的人。

有两种典型的用例：

• 按需/延迟实例化：如果您不确定是否会使用它并且创建成本高昂，那么最好不要在构造函数中初始化它
• 缓存数据：缓存可能昂贵的操作的结果这样后续调用就不需要再次计算它

您处于“惰性”类别，在这种情况下，更简单的方法是使用标志或可为空值：

• 标志+值组合：重用现有类而不进行堆分配，但是这需要默认构造
• 智能指针：这绕过了默认构造问题，但以堆分配为代价。检查您需要的复制语义...
• boost::Optional：类似于指针，但具有深层复制语义并且没有堆分配。但需要完全定义类型，因此依赖性更重。

我强烈推荐 boost::Optional 习惯用法，或者如果您希望提供依赖隔离，您可能会退回到像 std::unique_ptr 这样的智能指针。 code> （或 boost::scoped_ptr 如果您无权访问 C++0x 编译器）。

Answer 6

我认为这可能是可以应用 Singleton 模式的场景。