System.Linq.Dynamic .Select(“new ...”) 似乎不是线程安全的

Question

我从这里获取了 System.Linq.Dynamic.DynamicQueryable ： http://weblogs.asp.net/scottgu/archive/2008/01/07/dynamic-linq-part-1-using-the-linq-dynamic-query-library.aspx

我遇到的问题是在如下所示的代码中：

var results = dataContext.GetTable<MyClass>.Select("new (MyClassID, Name, Description)").Take(5);

看来，如果该行代码由多个线程几乎同时执行，则 Microsoft 的动态 Linq 代码会在其 ClassFactory.GetDynamicClass() 中崩溃方法，如下所示：

    public Type GetDynamicClass(IEnumerable<DynamicProperty> properties)
    {
        rwLock.AcquireReaderLock(Timeout.Infinite);
        try
        {
            Signature signature = new Signature(properties);
            Type type;
            if (!classes.TryGetValue(signature, out type))
            {
                type = CreateDynamicClass(signature.properties);
                classes.Add(signature, type);  // <-- crashes over here!
            }
            return type;
        }
        finally
        {
            rwLock.ReleaseReaderLock();
        }
    }

崩溃是一个简单的字典错误：“已添加具有相同键的项目。”

在Ms代码中，rwLock变量是一个ReadWriterLock类，但它没有做任何事情来阻止多个线程进入类。TryGetValue() if语句，所以很明显，Add将失败。

我可以在创建两个或多个尝试执行 Select("new") 语句的线程的任何代码中很容易地复制此错误。

无论如何，我想知道是否有其他人遇到过这个问题，以及是否有我可以实施的修复或解决方法。

谢谢。

Answer 1

我执行了以下操作（需要 .NET 4 或更高版本才能使用 System.Collections.Concurrent）：

• 将 classes 字段更改为 ConcurrentDictionary< /code> ，
• 删除了所有 ReaderWriterLock rwLock 字段以及引用它的所有代码，

• 将 GetDynamicClass 更新为：

  public Type GetDynamicClass(IEnumerable 属性) {
      var 签名 = 新签名(属性);
      返回classes.GetOrAdd(signature, sig => CreateDynamicClass(sig.properties));
  }
  

• 删除了classCount字段并更新了CreateDynamicClass以使用类。改为计数：

  类型 CreateDynamicClass(DynamicProperty[] 属性) {
      字符串类型名称 = "DynamicClass" + Guid.NewGuid().ToString("N");
  ...