c++ - 以下是有效的 C++ 代码吗？

Question

如果是，它应该做什么？

typedef struct Foo_struct{
  Dog d;
  Cat* c;
  struct Foo_struct(Dog dog, Cat* cat){ this->d = dog; this->c = cat;}
} Foo;

(背景故事:将用 Visual C++(在 Windows 上)编写的程序移植到 g++(在 MacOSX 上)；不知道这段代码应该做什么。

谢谢!

Answer 1

我不这么认为。 (Comeau 同意我的看法。)您不能像这样定义构造函数。

在 C++ 中，结构名称是一等公民。无需使用 C 中的旧 typedef 技巧。此外，d 和 c 应在成员初始化列表中进行初始化。这将是有效的(并且更好(C++):

struct Foo {
  Dog d;
  Cat* c;
  Foo(Dog dog, Cat* cat) : d(dog), c(cat) {}
};

代码定义了一个结构(在 C++ 中，与类相同，除了它的成员默认是公共(public)的)和一个构造函数来在创建时初始化它的成员。

编辑:正如 Travis 在他的评论中所说，您可能需要考虑将 dog 作为 const 引用而不是复制它:

Foo(const Dog& dog, Cat* cat) : d(dog), c(cat) {}

如果 Dog(我们还没有看到)是一个具有多个内置成员的类，这可能比按拷贝传递它要便宜得多。