3

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

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

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

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

这是简化的类：

 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(); 
            };
        }            
    }
4

4 回答 4

9

首先，让属性访问器改变状态通常是个坏主意。它应该做的最多的是懒惰地初始化一些东西——或者可能给出一个不稳定的值（就像DateTime.Now做的那样）。

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

于 2010-01-12T12:22:37.860 回答
2 
Urls.Pop();

想成为

return Urls.Pop();

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

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

于 2010-01-12T12:21:56.480 回答
1

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

于 2010-01-12T12:23:29.227 回答
1

IMO，这里的问题是具有不明显副作用的属性；这应该是一种方法：

    public string GetNextUrl() { /* */ }

否则到处都会发生坏事（调试器、数据绑定等）。不要假设某人只读取一次属性。

属性中副作用的唯一合理用途是延迟加载、延迟初始化等。在没有任何其他明显的变异调用的情况下顺序调用时，它仍然应该报告相同的值。

于 2010-01-12T12:23:47.870 回答