Блог

Я очистил и сократил фрагмент моего кода для многопользовательской игры, чтобы показать, чего я хотел бы достичь. Итак, вот оно:

     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; } }
    }
  

Что я хотел бы знать, так это:
Как запретить какому-либо пользователю (выполняющему код из другого потока) открывать окно, которое в данный момент получает какой-либо другой пользователь.
Я думал о нескольких способах, но я думаю, что люди здесь часто выдвигают умные идеи, которые могли бы помочь мне избежать какой-нибудь глупой проблемы с дизайном.

РЕДАКТИРОВАТЬ: Как предложено в ответах, использовать ключевое слово lock в методе open: это не совсем то, что я хочу, я попытаюсь объяснить, что разрешено, а что нет:

Сетевые запросы, которые мы получаем в качестве входных данных, каким-то образом асинхронны и выходят из строя, если выполняются быстро.

• (1) Пользователь 1 выдает ОКНО ВВОДА команды
• (2) Пользователь 1 выдает команду ОТКРЫТЬ ОКНО
• (3) Пользователь 1 выдает команду ЗАКРЫТЬ ОКНО
• (4) Пользователь 2 выдает команду ОТКРЫТЬ ОКНО
• (5) Пользователь 2 выдает ОКНО ПОИСКА команд
• (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
  

Спасибо!

Вы можете использовать инструкцию lock, чтобы запретить двум потокам доступ к Open и Close. Чтобы предотвратить состояние гонки при проверке, открыто ли уже окно, мы можем изменить Open() на TryOpen() и вернуть false, если окно уже открыто. Есть способы сделать это так, чтобы нам не приходилось возвращать логическое значение, но это, вероятно, самый простой.

Если один поток достигает инструкции lock, в то время как другой поток уже находится внутри инструкции lock, второй поток будет ждать, пока первый поток не завершит выполнение инструкции lock, прежде чем продолжить.

 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;
    }
}
  

Мое решение для этого:

 Monitor.TryEnter(obj)
  

Похоже на блокировку, но не блокирует.

Таким образом, мы блокируем объект, когда пользователь начинает его использовать,

и при открытии окна сделайте проверку, которая гласит:

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