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

Question

我正在使用 ReaderWriterLock 类来锁定 Quotes 集合，它是一个 SortedDictionary。我正在考虑使用 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 ？如果是这样，它可能会陷入僵局。

添加循环和睡眠延迟以避免死锁在一般情况下不起作用。严格的锁获取层次结构将在一般情况下发挥作用。

Answer 2

我不太明白你想达到什么目的。 ReaderWriterLockSlim（我使用的）已经以这种方式运行，无需再多费吹灰之力 - 不需要额外的旋转，您只是在浪费资源。 a) 如果未找到符号，则将 R 锁升级为 W 锁，或者更好地将 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
    }
}