3
abstract class Foo
{
    private List<Object> container;
    private bool update;

    Foo Foo()
    {
        container = new List<object>();
        update = false;
    }

    public abstract Bar CreateBar();

    public void BeginUpdate()
    {
        if (!update)
        {
            Thread update_thread = new Thread(new ThreadStart(Update));
            update_thread.Start();
        }
    }

    private void Update()
    {
        update = true;
        while (update)
        {
            lock (container)
            {
                if (...)
                    container.Add(this.CreateBar());
                else
                    container.Remove(...);
            }

            Thread.Sleep(1337);
        }
    }

    public void EndUpdate()
    {
        update = false;
    }

    public List<Object> Objects
    {
        get
        {
            lock (container)
            {
                return this.container;
            }
        }
    }
}

当 Foo 之外的东西调用 Foo 的 Object 访问器时,例如,

List<Objects> objects = foo_instance.Objects;
foreach (Object o in objects)
{
    Thread.Sleep(31173);
}

锁定将如何发生?运行 Update() 的线程是否必须等到上述 foreach 完成处理对象列表?我希望这两个可以同时工作,是制作对象深层副本的唯一解决方案吗?

4

4 回答 4

3

这段代码有几个问题:

  1. 你不启动线程
  2. 您可以有一个竞争条件(可能不适用于您的程序),其中多个线程调用 BeginUpdate,两者都认为更新为假,并且都启动一个线程,现在您有两个线程正在运行,可能相互干扰,因为您有一个共享数据的字段成员
  3. 锁定房产有什么意义?你不需要它。请注意,由于内部更新,您从属性返回的容器有时会处于不断变化的状态。这意味着调用该属性的外部代码必须在访问其内容之前锁定容器本身。不过,您不需要锁来返回容器引用。
  4. 如果您紧接着调用 EndUpdate 和 BeginUpdate ,则旧线程可能还没有机会退出

我想说的是,在您开始担心要锁定什么以及何时锁定之前,您应该修复代码,以便它不会有很多其他与线程相关的问题。

于 2008-11-28T18:00:21.887 回答
2

您的代码没有按照您的想法执行。这种方法

public List<Object> Objects
{
    get
    {
        lock (container)
        {
            return this.container;
        }
    }
}

返回值后不持有锁。所以你的循环没有被锁定。

您不能从类中返回容器实例

于 2008-11-28T17:53:54.507 回答
1

你可以把锁作用域想象成函数作用域。在访问器方法中加锁只会在大括号之间加锁,一旦返回值,锁就会被释放。您需要在类之外(在调用者中)进行锁定以获得所需的效果。

所以循环和线程都可以同时访问对象。这很糟糕,因为如果您的线程在循环期间更改对象,则循环将抛出异常,因为集合已被修改。

于 2008-11-28T17:53:26.410 回答
0

krosenvald 是正确的,只要属性返回指向容器对象的指针,就会释放对象访问器上的锁定...

在你的代码中

List<Objects> objects = foo_instance.Objects;
foreach (Object o in objects)
{    
    Thread.Sleep(31173);
}

锁只持续到第一行......正在填充引用变量“objects”......实际上没有必要为该行锁定任何东西,因为没有通过获取指针来修改内存......

于 2008-11-28T18:01:28.343 回答