Getters 和 Setters 应该做什么和不应该做什么

Question

可能的重复：
约定问题：何时你使用 Getter/Setter 函数而不是使用 Property

我最近遇到了很多关于 Getters 和 Setters 的不同意见，所以我想我应该把它变成它自己的问题。

我的我之前的问题立即收到了评论（后来删除）指出 setter 不应该有任何副作用，并且 SetProperty 方法将是更好的选择。

事实上，这似乎是微软的观点 也是如此。但是，它们的属性通常会引发事件，例如设置表单的 Width 或 Height 属性时的 Resized 事件。 OwenP 还指出“你不应该让属性抛出异常，属性不应该有副作用，顺序不重要，并且属性应该相对较快地返回。”

然而 Michael Stum 指出应该抛出异常在设置器中验证数据。如果您的设置器不抛出异常，那么您如何有效地验证数据，正如

当您需要像几乎所有 Microsoft Control 一样发起事件时该怎么办？那么您不是受到订阅您活动的人的摆布吗？如果他们的处理程序执行大量信息，或者本身抛出错误，那么您的设置程序会发生什么？

最后，getter 中的延迟加载怎么样？这也可能违反之前的指导方针。

什么可以放在 getter 或 setter 中，什么应该只保留在访问器方法中？

编辑：

来自另一个文章：

get 和 set 方法通常与其他方法没有什么不同。它们可以执行任何程序逻辑、引发异常、被覆盖以及使用编程语言允许的任何修饰符进行声明。但请注意，属性也可以是静态的。如果属性是静态的，则 get 和 set 方法的功能会受到限制。有关详细信息，请参阅您的编程语言参考。

Answer 1

我的观点：

1. 如果 setter 或 getter 预计会很昂贵，请不要将其设为属性，而应将其设为方法。

2. 如果设置属性会因更改而触发事件，这很好。您还能如何让听众收到更改通知？但是，您可能希望提供 BeginInit/EndInit 对来抑制事件，直到完成所有更改。通常，事件处理程序有责任及时返回，但如果您确实不能相信它会这样做，那么您可能希望在另一个线程中向该事件发出信号。

3. 如果设置属性会引发无效值异常，也没关系。当值完全错误时，这是​​发出问题信号的合理方式。在其他情况下，您设置一堆属性，然后调用使用它们执行某些操作（例如建立连接）的方法。这将允许推迟验证和错误处理，直到使用属性，因此属性不需要抛出任何内容。

4. 访问属性可能会产生副作用，只要这些副作用不是意外且无关紧要。这意味着 getter 中的 JIT 实例化是可以的。同样，每当进行更改时为实例设置脏标志就可以了，因为它设置了相关属性，例如相同值的不同格式。

5. 如果它做某事而不仅仅是访问一个值，那么它应该是一个方法。方法是动词，因此创建连接将由 OpenConnection() 方法完成，而不是由 Connection 属性完成。 Connection 属性将用于检索正在使用的连接，或将实例绑定到另一个连接。

编辑 - 添加 5 个，更改 2 和 3

Answer 2

我同意 getter/settings 不应该有副作用的想法，但我想说它们不应该有不明显的副作用。

就抛出异常而言，如果您将属性设置为无效值（在非常基本的意义上），那么验证异常就可以了。但是，如果设置器正在运行一系列复杂的业务规则验证，或者尝试去更新其他对象，或者任何其他可能导致异常的事情，那么那就不好了。但这个问题实际上并不是异常本身的问题，而是 setter 正在关闭并秘密执行调用者不会（或不应该）期望的许多功能。

事件也是如此。如果 setter 抛出一个事件说“此属性已更改”，那么没关系，因为这是一个明显的副作用。但是，如果它触发了其他一些自定义事件，从而导致一些隐藏的代码块在系统的另一部分中执行，那就很糟糕了。

这与我避免在吸气剂中延迟加载的原因相同。事实上，它们在很多时候可以让事情变得更容易，但有时它们也可以让事情变得更加混乱，因为在您想要加载子对象的确切时间周围总是会出现令人费解的逻辑。当您填充父对象时，通常只需多一行代码即可显式加载子对象，并且可以避免对对象状态的很多混淆。但这方面可能会变得非常主观，很大程度上取决于具体情况。

Answer 3

无论如何，在使用 C# 工作时，我始终发现保守方法是最好的。因为属性在语法上与字段相同，所以它们应该像字段一样工作：没有例外，没有验证，没有有趣的事情。 （事实上​​，我的大多数属性都是从简单字段开始的，除非绝对必要，否则不会成为属性。）这个想法是，如果您看到看起来像是获取或设置字段集的东西，那么它就像获取或设置设置字段，包括功能（不会抛出异常）、整体效率（例如，设置变量不会触发级联委托调用）以及对程序状态的影响（设置变量会设置该变量，并且不会不要召集很多几乎可以做任何事情的代表）。

属性集要做的明智的事情包括设置一个标志来指示发生了更改：

set {
    if(this.value!=value) {
        this.changed=true;
        this.value=value;
    }
}

也许实际上在另一个对象上设置了一个值，例如：

set { this.otherObject.value=value; }

也许稍微解开输入，以简化类的内部代码：（

set {
    this.isValid=(value&Flags.IsValid)!=0;
    this.setting=value&Flags.SettingMask;
}

当然，在后两种情况下， get 函数很可能会做相反的事情。）

如果需要发生任何更复杂的事情，特别是调用委托、执行验证或抛出异常，那么我的观点是函数更好。 （很多时候，我的字段通过 get 和 set 变成属性，然后最终成为 get 属性和 set 函数。） 对于 getter 也是如此；如果你要返回对某些东西的引用，那没问题，但如果你要创建一个全新的大对象并在每次读取属性时填充它——那就不太热了。