这种同步结构正确吗？

Question

我有一个多线程应用程序。线程使用 ABC.Connector。我希望一次只有一个线程可以访问连接器属性。

class ABC
{
    /// <summary>
    /// Synchronization object.
    /// </summary>
    static object _syncObject = new object();

    static DataAccess _connector;
    /// <summary>
    /// Global object for data access.
    /// </summary>
    public static DataAccess Connector
    {
        get
        {
            lock (_syncObject)
            {
                return _connector.CreateCopy(); // get a copy of original _connector
            }
        }
        set
        {
            lock (_syncObject)
            {
                _connector = value;
            }
        }
    }
}

正确吗？

Answer 1

好吧，这肯定会使获取和设置 Connector 属性成为线程安全的（尽管我会将 _syncObject 设为只读）。但是，它并不使 DataAccess 线程安全...互斥体仅在线程获取和设置属性时应用。。

换句话说，如果两个线程都执行以下操作：

ABC.DataAccess.DoSomeLongRunningOperation();

则两个线程仍将同时运行 DoSomeLongRunningOperation()。如果该操作不是线程安全的，它仍然会是一个问题。

如果您一次只希望一个线程使用DataAccess，那么您可以这样写：

public static void UseConnector(Action<DataAccess> action)
{
    lock (_syncObject)
    {
        action(_connector);
    }
}

那么如果两个线程都这样做：

ABC.UseConnector(access => access.DoLongRunningOperation());

那么DoLongRunningOperation()一次只会在一个线程中运行。您仍然遇到行为不端的客户可能会写的问题：

DataAccess naughty = null;
ABC.UseConnector(access => naughty = access);
// Haha! I've circumvented your thread safety!
naughty.DoLongRunningOperation();

...但希望这对您来说不是问题。

Answer 2

是的，这通常是正确的。但请注意，在 Connector get 返回 _connector 引用后，对其的访问是不同步的。