Ngoại lệ khi thêm mục nhập từ điển

Question

Chúng tôi thấy ngoại lệ này xảy ra trong khối mã sau trong ngữ cảnh ASP.NET đang chạy trên máy chủ IIS 7.

1) Exception Information 
********************************************* 
Exception Type: System.Exception 
Message: Exception Caught in Application_Error event 
Error in: InitializationStatus.aspx 
Error Message:An item with the same key has already been added. 
Stack Trace: at 
System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add) 
at CredentialsSession.GetXmlSerializer(Type serializerType) 

Đây là mã mà các ngoại lệ được xảy ra trong:

[Serializable()] 
public class CredentialsSession 
{ 
    private static Dictionary<string, System.Xml.Serialization.XmlSerializer> localSerializers = new Dictionary<string, XmlSerializer>(); 

    private System.Xml.Serialization.XmlSerializer GetXmlSerializer(Type serializerType) 
    { 
     string sessionObjectName = serializerType.ToString() + ".Serializer"; 

     if (Monitor.TryEnter(this)) 
     { 
      try 
      { 
       if (!localSerializers.ContainsKey(sessionObjectName)) 
       { 
        localSerializers.Add(sessionObjectName, CreateSerializer(serializerType)); 
       } 
      } 
      finally 
      { 
       Monitor.Exit(this); 
      } 
     } 
     return localSerializers[sessionObjectName]; 
    } 

    private System.Xml.Serialization.XmlSerializer CreateSerializer(Type serializerType) 
    { 
     XmlAttributes xmlAttributes = GetXmlOverrides(); 

     XmlAttributeOverrides xmlOverrides = new XmlAttributeOverrides(); 
     xmlOverrides.Add(typeof(ElementBase), "Elements", xmlAttributes); 

     System.Xml.Serialization.XmlSerializer serializer = 
      new System.Xml.Serialization.XmlSerializer(serializerType, xmlOverrides); 

     return serializer; 
    } 
} 

Các Monitor.TryEnter nên ngăn ngừa nhiều chủ đề từ khi vào khối cùng một lúc, và mã được kiểm tra từ điển để xác minh rằng nó không chứa khóa đang được thêm vào.

Bất kỳ ý tưởng nào về điều này có thể xảy ra như thế nào?

Answer 1

Mã của bạn không an toàn chỉ.

1. Bạn đang khóa trên this, một trường hợp CredentialsSession, nhưng việc tiếp cận một cuốn từ điển tĩnh mà có thể được chia sẻ bởi nhiều CredentialsSession trường. Điều này giải thích tại sao bạn nhận được lỗi - hai trường hợp khác nhau CredentialsSession đang cố ghi vào từ điển đồng thời.

2. Ngay cả khi bạn thay đổi điều này thành khóa trên trường tĩnh như được đề xuất trong câu trả lời của @ sll, bạn không an toàn chỉ vì bạn không khóa khi đọc từ điển. Bạn cần một số ReaderWriterLock hoặc ReaderWriterLockSlim để cho phép nhiều người đọc và một người viết duy nhất có hiệu quả.

   Vì vậy, có lẽ bạn nên sử dụng từ điển an toàn cho chủ đề. ConcurrentDictionary như những người khác đã nói nếu bạn đang sử dụng .NET 4.0. Nếu không, bạn nên thực hiện của riêng bạn, hoặc sử dụng một thực hiện hiện có như http://devplanet.com/blogs/brianr/archive/2008/09/26/thread-safe-dictionary-in-net.aspx.

Nhận xét của bạn đề nghị bạn muốn tránh gọi CreateSerializer cho cùng một loại nhiều lần. Tôi không biết tại sao, bởi vì lợi ích hiệu suất có thể là không đáng kể, vì tranh chấp có thể là hiếm và không thể vượt quá một lần cho mỗi loại trong suốt thời gian tồn tại của ứng dụng.

Nhưng nếu bạn thực sự muốn điều này, bạn có thể làm điều đó như sau:

var value; 
if (!dictionary.TryGetValue(key, out value)) 
{ 
    lock(dictionary) 
    { 
     if(!dictionary.TryGetValue(key, out value)) 
     { 
      value = CreateSerializer(...); 
      dictionary[key] = value; 
     } 
    } 
} 

Từ bình luận:

nếu tôi thực hiện điều này với ConcurrentDictionary và chỉ cần gọi TryAdd (sessionObjectName, CreateSerializer (serializerType)) mỗi lần.

Câu trả lời là không gọi TryAdd mỗi lần - trước tiên hãy kiểm tra xem nó có trong từ điển hay không, sau đó thêm nếu không. Cách thay thế tốt hơn có thể là sử dụng the GetOrAdd overload có tham số Func.

Answer 2

Thử khóa trên localSerializers thay vì this. BTW, tại sao bạn đang sử dụng Màn hình một cách rõ ràng? Chỉ có một lý do tôi thấy là để cung cấp lock timeout mà rõ ràng là bạn không sử dụng, vì vậy sử dụng chỉ đơn giản lock() statement thay vì điều này sẽ tạo ra try/cuối cùng cũng như:

lock (localSerializers) 
{ 
    if (!localSerializers.ContainsKey(sessionObjectName))     
    {      
     localSerializers.Add(
      sessionObjectName, 
      CreateSerializer(serializerType));     
    } 
} 

EDIT: Vì bạn đã không được chỉ rõ trong các thẻ mà bạn đang sử dụng .NET 4 tôi sẽ đề nghị sử dụng ConcurrentDictionary<TKey, TValue>

---

Monitor.Enter() Method:

Sử dụng C# try… finally block (Thử… Cuối cùng trong Visual Basic) để đảm bảo mà bạn nhả màn hình hoặc sử dụng lệnh C# lock (SyncLock statement trong Visual Basic), bao gồm các phương thức Enter và Exit trong một thử ... cuối cùng chặn

Answer 3

Nếu bạn đang ở trên .NET Framework 4 hoặc mới hơn, tôi sẽ đề nghị bạn nên sử dụng một ConcurrentDictionary để thay thế. Phương pháp TryAdd giữ cho bạn an toàn từ loại kịch bản, mà không cần phải xả rác mã của bạn với ổ khóa:

localSerializers.TryAdd(sessionObjectName, CreateSerializer(serializerType)) 

Nếu bạn đang lo lắng về CreateSerializer được gọi khi nó không cần thiết, thay vào đó bạn nên sử dụng AddOrUpdate:

localSerializers.AddOrUpdate(
    sessionObjectName, 
    key => CreateSerialzer(serializerType), 
    (key, value) => value); 

Điều này sẽ đảm bảo rằng phương thức này chỉ được gọi khi bạn cần tạo một giá trị mới (khi cần phải thêm vào từ điển). Nếu nó đã có mặt, mục nhập sẽ được "cập nhật" với giá trị đã tồn tại.