c++ - 返回指向对象的参数指针

Question

在 C++ for Windows 中，我有一些对象工厂，它应该通过将指向对象的指针传递给 Create 函数并返回创建的对象来创建一系列 Info 对象。

void CreateInfoObject(AbstractInfo** info);  // The creation function 

AbstractInfo 是一个基类，我们从中派生出许多类型的 Info 对象。

我想我现在可以创建一个 Info 对象，如下所示:

MyInfoObject* InfoObj = NULL;  // derived from AbstractInfo object
InfoFactory fc;

fc.CreateInfoObject(&InfoObj); // Now I want to get my initialized pointer back

但是它说它不能做转换...有什么问题吗？

错误: 无法从 MyInfoObject**_W64 转换为 AbstractInfo**

编辑:第一个答案提到界面很糟糕，看不到谁在分配等等……我该如何改进？

Answer 1

让我们考虑一下 CreateInfoObject 的可能实现方式:

void InfoFactory::CreateInfoObject(AbstractInfo** info)
{
  *info = new SuperInfo;
}

现在，SuperInfo 和 MyInfoObject 没有任何共同点吧？

这就是为什么一般来说，以下内容是被禁止的:

struct Base {};
struct D1: Base {};
struct D2: Base {};

int main(int argc, char* argv[])
{
  Base** base = nullptr;
  D1* d = nullptr;
  base = d;
}

因为它允许 D1 指向不相关的东西。

有几种解决方法:

// 1. Simple
AbstractInfo* info = nullptr;
fc.CreateInfoObject(info);

// 2. Better interface
std::unique_ptr<AbstractInfo> info = fc.CreateInfoObject();

然后，如果您确定您确实拥有一个 MyInfoObject，您可以使用:

MyInfoObject* myInfo = static_cast<MyInfoObject*>(info);

或者如果您不确定:

MyInfoObject* myInfo = dynamic_cast<MyInfoObject*>(info);

如果 info 没有指向 MyInfoObject 的实例，它将设置 myInfo 为 nullptr(或派生)。

不过请记住，您的界面真的很糟糕。它非常 C-ish，不清楚是否实际分配了内存......如果是，谁负责处理它。

编辑:

在 good C++ 风格中，我们使用 RAII 来表示所有权并确保清理。 RAII 是众所周知的，但不是很明确，我自己更喜欢新的 SBRM(范围限制资源管理)。

我们的想法是，与其使用裸指针，它不指示任何所有权(即您是否必须对其调用 delete ？)，不如使用智能指针，例如unique_ptr.

您还可以利用该方法的返回参数，以避免有两步初始化过程(首先创建指针，然后使其指向一个对象)。这是一个简洁的例子:

typedef std::unique_ptr<AbstractInfo> AbstractInfoPtr;

// Note: if you know it returns a MyInfoObject
// you might as well return std::unique_ptr<MyInfoObject>
AbstractInfoPtr InfoFactory::CreateInfoObject()
{
  return AbstractInfoPtr(new MyInfoObject());
}

// Usage:
int main(int argc, char* argv[])
{
  InfoFactory factory;
  AbstractInfoPtr info = factory.CreateInfoObject();

  // do something

} // info goes out of scope, calling `delete` on its pointee

这里，关于所有权没有歧义。

此外，请注意您如何在此处更好地理解您的问题:

  std::unique_ptr<MyInfoObject> info = factory.CreateInfoObject();

不会编译，因为如果不使用 static_cast 或 dynamic_cast 就无法将 AbstractInfo* 转换为 MyInfoObject* .