为了线程安全,这是要锁定的正确对象吗?

下面的代码将从网站调用,因此在非静态类中具有静态字典对象需要是线程安全的。基本上,代码的目的是封装逻辑并维护存储在反容器实例中的PerfMon计数器的生命周期。调用构造函数传入instancename。构造函数需要检查该instancename的反容器是否已定义并存储在字典中。如果是这样,它可以(并且必须)使用该实例。如果没有,它将创建反容器的实例,并将其存储在字典中,然后使用该实例。要使用的反容器的实例存储在非静态成员中,因此从该点开始线程是安全的。

因为静态字典的唯一使用位置是在构造函数中,所以我觉得在访问字典的过程中锁定它是安全的。这是否会导致以后出现任何无法预见的问题,比如阻塞/死锁?我什么也看不见,但在过去,我真的没有必要过多地考虑这类事情。

我也考虑过 锁定(此): 但我认为这不会起作用,因为它只会锁定正在创建的PerformanceCounter的实例,而不会锁定底层静态字典(线程安全也是如此)。

namespace ToolKit
{
    using System;
    using System.Diagnostics;
    using System.Collections.Generic;

    public class PerformanceCounters : IPerformanceCounters
    {
        private static Dictionary<string, CounterContainer> _containers = new Dictionary<string, CounterContainer>();
        private CounterContainer _instanceContainer;

        public PerformanceCounters(string instanceName)
        {
            if (instanceName == null) throw new ArgumentNullException("instanceName");
            if (string.IsNullOrWhiteSpace(instanceName)) throw new ArgumentException("instanceName");

            // Is this the best item to lock on?
            lock (_containers)
            {

                if (_containers.ContainsKey(instanceName))
                {
                    _instanceContainer = _containers[instanceName];
                    return;
                }

                _instanceContainer = new CounterContainer(instanceName);
                _containers.Add(instanceName, _instanceContainer);
            }
        }
        public void Start()
        {
            _instanceContainer.AvgSearchDuration.Start();
        }

        public void FinishAndLog()
        {
            _instanceContainer.SearchesExecuted.Increment();
            _instanceContainer.SearchesPerSecond.Increment();
            _instanceContainer.AvgSearchDuration.Increment();
        }
    }
}