双重检查锁定模式可能不正确的实现

本文关键字：不正确 实现 模式 检查 锁定 | 更新日期: 2023-09-27 18:08:46

private readonly Object _syncRoot = new Object();
public IdGenerator ClientIdGenerator
{
    get
    {
        if ( clientIdGenerator != null )
            return clientIdGenerator;
        lock ( _syncRoot )
        {
            if ( clientIdGenerator != null )
                return clientIdGenerator;
            return clientIdGenerator = ClientIdPrefix != null ? new IdGenerator( ClientIdPrefix ) : new IdGenerator();
        }
    }
}

r#显示一个警告" Double-Check Locking模式可能不正确的实现"。

在创建IdGenerator实例的行中对选中字段"的读访问权。

r#在我将代码更改为以下代码后不显示警告:

public IdGenerator ClientIdGenerator
{
    get
    {
        if ( clientIdGenerator == null )
            lock ( _syncRoot )
            {
                if ( clientIdGenerator != null )
                    return clientIdGenerator;
                clientIdGenerator = ClientIdPrefix != null ? new IdGenerator( ClientIdPrefix ) : new IdGenerator();
            }
        return clientIdGenerator;
    }
}

第一个例子有什么问题，或者r#显示"错误"警告吗?

双重检查锁定模式可能不正确的实现

您的第一个版本应该可以很好地工作，但是为了避免部分代码分析引擎或同事的误解，您可以使用标准模式:

private readonly Object _syncRoot = new Object();
public IdGenerator ClientIdGenerator
{
    get
    {
        if (clientIdGenerator == null)
        {
            lock (_syncRoot)
            {
                if (clientIdGenerator == null)
                {
                    clientIdGenerator = ClientIdPrefix != null ? new IdGenerator(ClientIdPrefix) : new IdGenerator();
                }
            }
        }
        return clientIdGenerator;
    }
}
相关文章：