Question

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

这有什么意义？为什么不仅仅将那些私有数据成员公开？它们是const，因此访问按定义是只读的，人们可以通过theClass.theMember而不是theClass.getTheMember()来访问它们，这更容易，而且您不必在第一个方法中创建所有这些公共方法。地点。每个成员变量一个，这意味着O（n）低效率！

Answer 1

除非访问器方法除了提供引用以外实际上做了其他操作，否则它们确实是多余的。

例如，这可以是

设置器：检查给定的值实际上是有效的。
字母：从其他信息中即时获取请求的值。
设置器：确保给定值与其他数据成员一致。
设置者：通知某些数据使用者该更改
...

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

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设置为他们喜欢的任何值，而对象不会注意到它。

所以，我的建议是：

默认情况下将数据成员声明为私有。
如果您需要只读访问权限，但数据成员是可变的，请提供一个吸气剂。
如果数据成员是const并且可读，请使用public: const
如果要允许用户随时提供任何值，请继续并使其成为成员public。实话实说：类本身无法控制成员的价值。

请注意，这种情况也很少发生。在大多数情况下，您希望您的班级提供一些抽象，这意味着它不仅仅是一群公共数据成员。编写良好的类应该实现一些必要的行为，而不仅仅是数据的集合。另外，如果该类中只有一堆公共数据成员，那么您将其声明为struct，不是吗？
如果在访问公共变量时发现需要该类来执行某事，请将其转换为私有变量，提供相关的访问器，并让编译器将您指向代码中所有需要更新的地方。

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

Answer 2

我认为为班级成员提供吸气剂（在适当情况下提供吸气剂）通常被认为是一种好习惯。这样做的理由是，以后可以更轻松地更改类的表示形式，因为您可以这样做，而不会破坏任何客户端代码。

Answer 3

嗯，这只是样式。对于您的具体示例，将它们声明为公共可能有意义。

但是，这是一种提供get和set方法的实践，这些方法可能会或可能不会在字段中做其他事情。
考虑需求是否有所变化。

另外，对于被引入代码的程序员来说，这可能更方便。
他们中的大多数人都具有通过getter访问的概念，因此他们希望代码像这样。

Answer 4

@PetarVelev指出这是风格。对于许多具有二传手和消气剂的人来说，这是一个样板代码，但通常他们忘记了为什么要发明它。

这是为了简化代码维护而发明的。想象一些具有某些属性的类。此属性在整个项目中使用。现在需求已更改，并且每次访问属性时都必须执行某些操作。例如，当值更改时，应发送通知（应更新设置者）。或者您已经注意到计算某些属性的成本很高，因此必须进行惰性评估（应该更正getter）。

编写此属性（类字段）时，您不知道将要解决此类问题。如果将其公开显示为公共字段，则要解决此问题，您必须花费数小时才能找到对此值的每次使用并进行更正。另一方面，如果通过使用setter和getter完成此操作，则将花费5-10分钟来修复它，包括添加测试。

因此，使用setter和getter是一种好习惯，在这种情况下，您将花费10秒的时间将它们编写为样板代码，但是如果需要进行一些重大更改，则可以节省大量时间。 Qt的框架就是一个很好的例子。

因此，如果您确定要求不会改变，则没有必要使用getter setter，但是如果您有疑问，最好使用它们。

现在，在常量字段被getter公开的情况下，有些奇怪。我怀疑这是函数式编程的一些示例，但是没有看到实际的代码很难分辨。

C ++：const公共成员怎么了？

4 个答案: