Getter属性在没有人调用的情况下运行

我在.net 3.5工作。 我有一个类“A”，它有一个堆栈和一个getter属性，当被调用时，它会删除堆栈中的第一个项目并检索下一个项目。

在初始化类之后，我看到getter在没有被调用的情况下工作，并且删除了堆栈中的顶部项，从而给了我不好的结果。 吸气剂中的断点并未显示任何人通过它。

当我将属性更改为函数时，堆栈返回ok。

如果有人可以解释为什么会这样，我会很高兴。

这是简化的类：

public class A { private Stack Urls; public A(string title, string[] array) { Urls = new Stack(); foreach (string s in array) { Urls.Push(s); } } public string Url { get { return Urls.Peek(); } } public string NextUrl { get{ if (Urls.Count > 1) { Urls.Pop(); } return Urls.Peek(); }; } } 

Answer 1

首先，使属性访问器改变状态通常是一个坏主意。 它应该做的最多是懒惰地初始化一些东西 – 或者可能给出一个易变的值（比如DateTime.Now ）。

其次，如果您在调试器下运行，您可能会看到这一点 – 它会在您逐步执行代码时访问属性。 这可能解释了为什么断点也没有受到打击。

Answer 2

 Urls.Pop(); 

想成为

 return Urls.Pop(); 

因为它返回值并同时从列表中删除它

---

实际上重新阅读了你的问题，看起来是因为调试器评估了属性。 如果你在没有调试器的情况下运行应用程序，你会遇到同样的问题吗？

Answer 3

我认为这是糟糕的设计。 get访问器不应该以在后续调用中导致不同结果的方式改变对象。

Answer 4

国际海事组织，这里的问题是拥有一种具有非明显副作用的财产; 这应该是一种方法：

  public string GetNextUrl() { /* */ } 

否则，到处都会发生坏事（调试器，数据绑定等）。 不要以为有人只读一次财产。

在属性中唯一合理使用副作用的是延迟加载，延迟初始化等等。它在顺序调用时仍应报告相同的值，而不进行任何其他明显的变异调用。