是否可以（避免死锁、饥饿等）尝试在具有睡眠的循环中获取锁？

Question

我正在使用 ReaderWriterLock 类来锁定作为 SortedDictionary 的 Quotes 集合。我正在考虑使用 while 循环，直到线程可以获取读取器锁，以防它被临时锁定以进行写入。第一个问题，我的测试运行良好，但是这种方法有缺点吗？第二个问题，这样做的最佳/最佳实践方式是什么？

        public void RequestQuote(string symbol, QuoteRequestCallback qrc)
        {
            // add the call back on a list and take care of it when the quote is available
            while (!AcquireReaderLock(100)) Thread.Sleep(150);
            if (Quotes.ContainsKey(symbol))
            {
                qrc(Quotes[symbol]);
                rwl.ReleaseReaderLock();
            }
            else
            {
                rwl.ReleaseReaderLock();
                lock (requestCallbacks)
                    requestCallbacks.Add(new KeyValuePair<string, QuoteRequestCallback>(symbol, qrc));
                // request symbol to be added
                AddSymbol(symbol);
            }
        }


        private bool AquireReaderLock(int ms)
        {
            try
            {
                rwl.AcquireReaderLock(ms);
                return true;
            }
            catch (TimeoutException)
            {
                return false;
            }
        }

        private bool AquireWriterLock(int ms)
        {
            try
            {
                rwl.AcquireWriterLock(ms);
                return true;
            }
            catch (TimeoutException)
            {
                return false;
            }
        }

Answer 1

在使用AcquireReaderLock() 方法锁定之前，您是否有第二段密码锁定requestCallbacks？如果是这样，它可能会因此而死锁。

添加循环和睡眠延迟以避免死锁在一般情况下不起作用。严格的hierarchy of lock acquisition 将在一般情况下工作。

Answer 2

我不太明白你想达到什么目的。 ReaderWriterLockSlim（我会使用它）已经毫不费力地表现得那样 - 不需要额外的旋转，你只是在浪费资源。 a) 如果符号未找到或更好，则将 R-Lock 提升为 W-Lock，将 R 和 W 访问分离到不同的方法中。 b) 不要提供从受保护部分内部调用未知代码的可能性

public void RequestQuote(string pS, QuoteRequestCallback pQrc) {
    Quote tQ;
    // acquire/release ReadLock inside TryGet
    if (TryGetQuote(pS, out tQ)) {
        pQrc(tQ);
    } else {
        // acquire/release WriteLock inside AddQuote
        // remark: I left the other collection
        // out since it seems unrelated to the actual problem
        AddQuote(new KeyValuePair(...)); // as above
    }
}