c++ - 在 header 中声明和初始化 vector (还有其他 STL 容器)

Question

现在我正在头文件中创建一个 vector 。请注意，我不只是宣布它。

std::vector<int> attacked_nodes = std::vector<int> ();

它工作正常，但它让我有些怀疑。这不在适当类的上下文中，它只是许多函数使用的变量。

我想知道这是不是

• 正确(会适得其反吗？)
• 可接受的做法(可能不是好)

我搜索了一下，是的。

注意:这可能是一种糟糕的编码习惯，我明白了。我想知道它是否真的不正确？

我不希望使用 new 来创建指针，也许我可以使用 new 并保存引用？这是still inelegant , 但可能有效。

谢谢。

Answer 1

它是有效的 C++，但不是好的设计。您正在头文件中定义一个全局对象。这有两个问题:“全局”位和“在头文件中”位。

让我们从头文件问题开始。 C++ 有一个称为单一定义规则的规则，其中一部分要求您在整个程序中不能对一个对象有多个定义。如果您将此头文件包含在多个实现文件中(请记住，这就像将头文件的内容复制并粘贴到您的源文件中)，您将有多个定义，因此打破了一个定义规则。

现在全局状态的问题在于它隐藏了依赖关系。您提到您有许多使用此对象的函数，但从这些函数的调用站点来看它们使用该对象并不清楚。即使您使用完全相同的参数调用它们，您的函数也可能返回不同的结果，这仅仅是因为这个全局状态可能已经改变。这是糟糕的界面设计，使功能更难测试(和信任)。