我需要定义一些仅由一个类使用的常量字符串。看起来我有三个选择:
将字符串直接嵌入到使用它们的位置。
将它们定义为类的私有静态常量成员:
//A.h
class A {
private:
static const std::string f1;
static const std::string f2;
static const std::string f3;
};
//A.cpp
const std::string f1 = "filename1";
const std::string f2 = "filename2";
const std::string f3 = "filename3";
//strings are used in this file
在cpp文件中的匿名命名空间中定义它们:
//A.cpp
namespace {
const std::string f1 = "filename1";
const std::string f2 = "filename2";
const std::string f3 = "filename3";
}
//strings are used in this file
鉴于这些选项,您会推荐哪一个?为什么?感谢。
答案 0 :(得分:21)
我将它们放在CPP文件中的匿名命名空间中。它使它们对实现是私有的,同时使它对作为实现一部分的非成员函数可见(例如operator<<
)。
答案 1 :(得分:5)
如果它们仅在单个文件中使用,则不需要将它们包含在头文件中,从而将它们暴露给外界。
如果它们被使用并且总是只在一个地方使用,那么就没有理由不将它们写成需要使用它们的文字。
如果它们在cpp的多个位置使用,我会选择匿名命名空间。
你没有提到的另一个选择是将它们定义为cpp中的静态变量。这有点等同于匿名命名空间选项,而且比C ++更像C语言。
答案 2 :(得分:3)
班级的静态成员。
如果它们在一个类的多个地方使用,通常更容易保持组织 - 以及稍后找到你定义的所有内容 - 如果你在使用它们的类中定义它们。在原地定义它们使得它们难以定位并且稍后修改。我会选择匿名命名空间上的特定类来进行更清晰的类定义和使用。
答案 3 :(得分:2)
如果要让类的用户看到字符串,请将它们放入类中。否则,将它们隐藏在实现文件的未命名命名空间中。
答案 4 :(得分:2)
如果只在类的.cpp文件中使用,则不需要使用任何类型的命名空间,只需说:
const std::string f1 = "filename1";
const std::string f2 = "filename2";
const std::string f3 = "filename3";
过度使用命名空间似乎是新事物 - 我个人无法看到这种吸引力。
答案 5 :(得分:0)
在这三个选项中,唯一应该避免的是#1。不要在代码中使用魔术cookie。通过将常量放在命名空间或类中,可以在将来更轻松地扩展和维护代码。
如果你的常数本质上是全局性的,那么在2到3之间,这很重要。重要的是你选择一个并坚持下去。但是如果你有适用于特定类的常量,那么它们应该是该类的一部分。
就个人而言,我会为大多数事情使用命名空间。
答案 6 :(得分:0)
我认为真正的问题是:字符串实际上是仅在实现类时在内部使用,还是在其他地方使用。
要真正挑剔,我会尽量保持班级的界面尽可能干净,所以如果文件名字符串不应该对“外部”世界感兴趣。我只会在.cpp文件中内部隐藏它们。在这种情况下,我不认为我会打扰命名空间,只是保持“静态”(即.cpp文件的内部)。
答案 7 :(得分:0)
但是你要这样做,有一点需要注意:我不建议使用静态std :: string对象,而是使用静态char *。造成这种情况的原因是初始化顺序可能存在问题。假设您有一个类的静态实例,其构造函数引用字符串A::f1
。无法保证A::f1
已经构建完毕,您将遇到崩溃,或者更糟糕的是,没有崩溃但是数据不准确。
跟踪初始化错误的顺序可能非常糟糕,并且在一个项目中一切都可能看起来很好,但随后您可能使用相同的库构建另一个项目,并且链接顺序中的细微差别将导致此错误神秘地出现。
答案 8 :(得分:0)
只需将const字符串放在实现文件的文件范围内,匿名命名空间就不必限制它们仅限于该类。
C++ 003 Standard C.1.2第3条:基本概念
Change: A name of file scope that is explicitly declared const, and not explicitly declared extern, has internal linkage, while in C it would have external linkage
注意:匿名命名空间确实有助于减少命名冲突。