Question

当我使用多线程应用程序并具有单例资源时。在此单例中，当另一个线程进入时不显示状态变化。

public class WhoConnect
{
    private ReaderWriterLockSlim userLock = new ReaderWriterLockSlim();
    private const int timeout = 2000;
    private List<User> users = new List<User>();

    private WhoConnect() { }

    public static WhoConnect Instance => instance ?? instance = new WhoConnect();

    public List<User> Users 
    {
        get
        {
            userLock.TryEnterReadLock(timeout);
            try { return users; }
            finally { userLock.ExitReadLock(); }
        }
    } 
    public void AddUser(string sessionId, string characterName, int characterID, int terminalId, string ip, string language)
    {
        userLock.TryEnterWriteLock(timeout);
        try
        {
            users.Add(new User(sessionId, characterName, characterID, terminalId, ip, language));
            Thread.Sleep(50000);
        }
        finally { userLock.ExitWriteLock(); }
    }
}

我第一次使用的地方：

WhoConnect.Instance.AddUser(socketId, "haha", 44, 3, "xx", "pl");

并在另一个线程中：

var k = WhoConnect.Instance.Users;

比获取用户时，我的writebloc为false，而不是true。

Answer 1

一个问题是您正在使用TryEnterWriteLock()和TryEnterReadLock()而不检查该锁是否真正被获取。要解决此问题，只需使用EnterWriteLock()和EnterReadLock()即可。

    public List<User> Users 
    {
        get
        {
            userLock.EnterReadLock()
            try { return users; }  // DON'T DO THIS!
            finally { userLock.ExitReadLock(); }
        }
    }

如果您确实需要超时，则必须检查是否确实获得了锁定，但是随后您必须处理无法及时获得锁定的情况……因此，我怀疑这是否是一个好方法。您的情况的解决方案。例如：

    public List<User> Users 
    {
        get
        {
            if(userLock.TryEnterReadLock(timeout))
            {
                try { return users; } // DON'T DO THIS!
                finally { userLock.ExitReadLock(); }
            }
            else throw new Exception("Could not acquire lock");
        }
    }

我看到的另一个关键问题是，仅在锁定状态下返回用户列表时，该锁定就没有用了，因为用户列表上的活动是在调用Users属性的方法中完成的，但随后进行了锁定已经退出...而且，该列表可用于写入而不是读取，因此readlock可能不正确。一种解决方案是从列表中返回元素的副本：

    public User[] Users 
    {
        get
        {
            userLock.EnterReadLock()
            try { return users.ToArray(); }
            finally { userLock.ExitReadLock(); }
        }
    }

最后，汉斯·帕桑特（Hans Passant）在他的评论中提到了一个有效点，即创建实例不是线程安全的。像建议的那样使用Lazy<T>或使用锁。

单例，异步，ReaderWriterLock

1 个答案: