我在.net 3.5工作。 我有一个类“A”,它有一个堆栈和一个getter属性,当被调用时,它会删除堆栈中的第一个项目并检索下一个项目。
初始化类之后,我看到getter在没有被调用的情况下工作,并删除了堆栈中的顶层项,从而给了我不好的结果。吸气剂中的断点并未显示任何人通过它。
当我将属性更改为函数时,堆栈将返回ok。
如果有人可以解释原因,我会很高兴。
这是简化的课程:
public class A
{
private Stack<string> Urls;
public A(string title, string[] array)
{
Urls = new Stack<string>();
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();
};
}
}
答案 0 :(得分:9)
首先,使属性访问器改变状态通常是个坏主意。它应该做的最多是懒惰地初始化某些东西 - 或者可能给出一个不稳定的值(如DateTime.Now
那样)。
其次,如果您在调试器下运行,您可能会看到这一点 - 它会在您逐步执行代码时访问属性。这可能解释了为什么断点也没有被击中。
答案 1 :(得分:2)
Urls.Pop();
希望成为
return Urls.Pop();
因为它返回值并且同时将其从列表中删除
实际上重新阅读了您的问题,看起来是因为调试器评估了属性。如果你在没有调试器的情况下运行应用程序,你会遇到同样的问题吗?
答案 2 :(得分:1)
我认为这是糟糕的设计。 get访问器不应该以在后续调用中导致不同结果的方式改变对象。
答案 3 :(得分:1)
IMO,这里的问题是拥有一种非明显副作用的财产;这应该是一种方法:
public string GetNextUrl() { /* */ }
否则到处都会发生不好的事情(调试器,数据绑定等)。不要以为有人只读过一次房产。
在属性中唯一合理使用副作用的是延迟加载,延迟初始化等等。它在顺序调用时仍然应该报告相同的值而没有任何其他明显的变异调用。