正确处理跨线程共享类的锁定方式

Question

我已经清理并缩短了多用户游戏的代码片段，以显示我想要完成的操作。所以在这里，它是：

public class Subject { 
     public List<IObject> Objects = new List<IObject>(); 
    } 

    public interface IOpenable 
    { 
     void Open(Subject by, params string[] p); 
     void Close(Subject by, params string[] p); 
     bool IsOpen { get; } 
    } 

    public interface IObject { 
     void Pickup(Subject by, params string[] p); 
     void Drop(Subject by, params string[] p); 
    } 

    public class Box : IObject, IOpenable 
    { 
     bool opened = false; 
     Subject owner = null; 

     public void Pickup(Subject subject, params string[] p) { 
      subject.Objects.Add(this); 
      this.owner = subject; 
     } 

     public void Drop(Subject subject, params string[] p) 
     { 
      subject.Objects.Remove(this); 
      this.owner = null; 
     } 

     public void Open(Subject by, params string[] p) 
     { 
      this.opened = true; 
     } 

     public void Close(Subject by, params string[] p) 
     { 
      this.opened = false; 
     } 

     public bool IsOpen { get { return opened; } } 
    } 

我想知道的是： 如何防止某些用户（从另一个线程执行代码）来打开当前正在由其他一些用户拿起一个盒子。 我已经想到了一些方法，但我认为这里的人经常想出聪明的想法，这可以让我避免一些愚蠢的设计问题。

编辑：正如在回答提出，要使用锁关键字在打开的方法：这是不是我想要什么，我会尝试用解释是允许什么，什么不是：

网络我们得到的请求以某种方式异步并且如果发布得很快就会出错。

• （1）用户1次发出指令PICKUP BOX
• （2）用户1次发出指令OPEN BOX
• （3）用户1所发出命令关闭BOX
• （4）用户2次发出指令OPEN BOX
• （5）用户2次发出指令LOOK BOX
• （6）用户1次发出指令打开框中

我们得到这个顺序：

2,3,1,5,4,6

2 - allow 
3 - allow 
1 - allow [remains in execution and has not set the owner] 
5(comes in between 1) - allow 
4(comes in between 1) - disallow (not because already open but because 1 is in execution) 
6(comes in between 1) - allow since it is from user 1, and he is currently picking it up 

谢谢！

Answer 1

我的这个解决方案是：

Monitor.TryEnter(obj) 

类似锁，但不阻塞。

所以我们锁定的对象，当用户开始拿起它，

，并打开时，盒子做了检查，说：

if (this.subject == subject || Monitor.TryEnter(obj)) 

Answer 2

您可以使用lock statement来防止两个线程访问打开和关闭。为防止race condition检查框是否已打开，我们可以将Open()更改为TryOpen()，如果该框已打开，则返回false。有办法做到这一点，以便我们不必返回布尔值，但这可能是最简单的。

如果一个线程到达锁定语句，而另一个线程已经在锁定语句中，则第二个线程将等待，直到第一个线程退出锁定语句，然后再继续。

private object locker=new object(); 
public bool TryOpen(Subject by, params string[] p) 
{ 
    lock(locker) 
    { 
     if(this.opened) 
      return false; 
     this.opened = true; 
     return true; 
    } 
} 

public void Close(Subject by, params string[] p) 
{ 
    lock(locker) 
    { 
     this.opened = false; 
    } 
}