c++ - const public 成员有什么问题？

Question

我正在阅读一些代码，其中一个类具有私有(private) const 数据成员，然后是一大堆提供对这些数据成员的只读访问的公共(public)方法。

这有什么意义呢？为什么不直接公开这些私有(private)数据成员呢？它们是常量，因此根据定义访问是只读的，人们可以通过 theClass.theMember 而不是 theClass.getTheMember() 访问它们，后者更容易，另外您可以避免首先创建所有这些公共(public)方法。每个成员变量一个，这意味着 O(n) 效率低下!

Answer 1

除非访问器方法除了提供引用之外实际上还做了一些事情，否则它们确实是多余的。

这个东西可以是，例如:

• Setter:检查给定值是否实际有效。

• Getter:从其他事物中动态获取请求的值。

• Setter:确保给定的值与其他数据成员一致。

• Setter:将更改通知一些数据消费者

• ...

但是，如果你正在写类似的东西

private:
    const int _foo;

public:
    const int& foo() const { return _foo; }

没有比写的优势

public:
    const int foo;

相反。前者做不到的事，后者做不了。

同样，代码

private:
    int _foo;

public:
    int& foo() { return _foo; }
    const int& foo() const { return _foo; }

没有任何好处

public:
    int foo;

因为任何用户都可以将 _foo 设置为他们喜欢的任何值，而对象甚至不会注意到它。

---

所以，我的建议是:

• 默认声明数据成员私有(private)。

• 如果您需要只读访问权限但数据成员是可变的，请提供一个 getter。

• 如果数据成员是const并且可读，使用public: const

• 如果允许用户随时提供任何值，请继续将成员设为public。老实说:类(class)本身无法控制成员的值(value)。

  请注意，这种情况也极少发生。在绝大多数情况下，您希望您的类提供一些抽象，这意味着它不仅仅是一堆公共(public)数据成员。一个编写良好的类应该实现一些所需的行为，而不仅仅是数据集合。另外，如果类中只有一堆公共(public)数据成员，您会将其声明为 struct，不是吗？

• 如果您发现在访问公共(public)变量时您需要该类做某事，请将其转换为私有(private)变量，提供相关的访问器，并让您的编译器指向您代码中所有需要更新的地方。

  这也很少发生，但它会迫使您在更改其访问行为之前实际查看数据成员的所有用途。这可能是一件好事，因为它让您有机会在更改的行为出现在测试和/或生产中之前发现问题。如果您已经为变量提供了普通访问器，您将不会被迫重新访问调用站点。