1

我们有一个简单的 WCF 服务，它被标记为 InstanceContextMode = Single 和 ConcurrencyMode = Multiple。对于返回列表的方法，我们无需在调用时访问我们的数据库并填充列表，而是简单地返回列表的本地副本。该列表由一个后台线程维护，该线程每 24 小时访问我们的数据库并重新填充该列表。我们通过在两个地方锁定对象来处理并发问题：在返回集合的方法内部和填充集合的方法内部。

我的问题是，我们如何才能提高效率。目前，我们在客户端将调用的服务方法“GetCustomers”中存在瓶颈：

public List<ZGpCustomer> GetCustomers()
{
    List<ZGpCustomer> customerListCopy = new List<ZGpCustomer>();
    lock (_customerLock)
    {
        customerListCopy = new List<ZGpCustomer>(_customers);
    }

    return customerListCopy;
}

由于“_customers”仅每 24 小时填充一次，看来我们应该只在修改集合时才需要锁定 GetCustomers 方法。按照目前的设置方式，如果同时有 1000 个请求进入，我们实际上是在对 1000 个请求进行排队，因为一次只有 1 个线程可以访问该方法。这使得服务的多线程方面有些无用，不是吗？

这种模式有最佳实践吗？我应该使用更合适的数据收集来存储我的对象吗？“BlockingCollection”会更合适吗？

4

1 回答 1

2

肯定有一种更有效的方法来做到这一点。诀窍是保持对集合的主引用不可变。这样，您就不必在阅读器端同步访问。只有事物的作者方需要锁。读者端什么都不需要，这意味着读者保持高度并发。您唯一需要做的就是将_customers引用标记为 volatile。

// Variable declaration
object lockobj = new object();
volatile List<ZGpCustomer> _customers = new List<ZGpCustomer>();

// Writer
lock (lockobj)
{
  // Create a temporary copy.
  var copy = new List<ZGpCustomer>(_customers);

  // Modify the copy here.
  copy.Add(whatever);
  copy.Remove(whatever);

  // Now swap out the references.
  _customers = copy;
}

// Reader
public List<ZGpCustomer> GetCustomers()
{
    return new List<ZGpCustomer>(_customers);
}

如果您愿意GetCustomers稍微更改签名，则可以利用的不变性_customers并返回只读包装器。

// Reader
public IList<ZGpCustomer> GetCustomers()
{
    return _customers.AsReadOnly();
}
于 2012-04-05T14:04:21.780 回答