Tràn ngăn xếp khi loại bỏ tài nguyên được quản lý

Question

Tôi có một trình bao bọc đơn giản cho bộ chứa Unity IoC (sử dụng tạm thời Dịch vụ định vị [chống] Mô hình giới thiệu DI cho một mã số cũ) và từ IUnityContainer trong Unity thực hiện IDisposable I muốn phơi bày điều đó thông qua trình bao bọc.

Các wrapper đơn giản là đủ:

public class IoCContainer : IIoCContainer 
{ 
    private IUnityContainer _container; 

    public IoCContainer(IUnityContainer container) 
    { 
     _container = container; 
    } 

    public T Resolve<T>() 
    { 
     return _container.Resolve<T>(); 
    } 

    public void Dispose() 
    { 
     Dispose(true); 
     GC.SuppressFinalize(this); 
    } 

    ~IoCContainer() 
    { 
     Dispose(false); 
    } 

    protected virtual void Dispose(bool disposing) 
    { 
     if (disposing) 
      if (_container != null) 
      { 
       _container.Dispose(); 
       _container = null; 
      } 
    } 
} 

IIoCContainer là giao diện miền mà không có gì nhưng T Resolve<T>() vào nó, và tất nhiên IDisposable. Vì vậy, mọi thứ bên dưới rằng một phương pháp đơn giản là việc triển khai IDisposableas I found it on MSDN.

Tuy nhiên, khi .Dispose() được gọi trên đối tượng này (chẳng hạn như khi thoát khỏi khối using), hãy tăng số StackOverflowException. Gỡ lỗi, có vẻ như các cuộc gọi stack lặp đi lặp lại giữa:

• Dispose() được gọi vào lớp này
• nào gọi Dispose(true) trên lớp này
• nào gọi Dispose() trên IUnityContainer
• nào gọi Dispose() trên lớp này

tôi có thể giải quyết này trong trường hợp này bằng cách đặt một lá cờ bool trên lớp, đặt nó trên dòng đầu tiên của Dispose(), và kiểm tra nó trong Dispose(bool), vì vậy đệ quy kết thúc vào lần lặp thứ hai của mình. Nhưng tại sao điều này lại xảy ra ngay từ đầu? Tôi chỉ có thể giả định rằng tôi đã bỏ lỡ điều gì đó hiển nhiên hoặc hiểu nhầm điều gì đó về việc xử lý tài nguyên. Nhưng cái gì?

Answer 1

Đây là thực hiện IDisposable trong UnityContainer. Rõ ràng là bạn không thể vứt bỏ thùng chứa mẹ của bạn. Nó sẽ lặp qua tất cả các đăng ký và xử lý chúng nếu chúng cũng là IDisposable.Hãy xem:

protected virtual void Dispose(bool disposing) 
{ 
    if (disposing) 
    { 
     if (lifetimeContainer != null) 
     { 
      lifetimeContainer.Dispose(); 
      lifetimeContainer = null; 

      if (parent != null && parent.lifetimeContainer != null) 
      { 
       parent.lifetimeContainer.Remove(this); 
      } 
     } 

     extensions.OfType<IDisposable>().ForEach(ex => ex.Dispose()); 
     extensions.Clear(); 
    } 
} 

Answer 2

Tôi tin rằng vấn đề ở đây là bạn không phải xử lý trường hợp vùng chứa triển khai IUnityContainer trong lớp học của bạn.

Ví dụ này đang được chuyển thành tài nguyên cho lớp của bạn, nhưng nó được tạo ra bên ngoài, vì vậy bất kỳ mã nào tạo ra cá thể đó là trường hợp phải chịu trách nhiệm xử lý đúng.

IoCContainer của bạn chỉ nên quan tâm đến việc xử lý tài nguyên mà nó tạo ra trong nội bộ.

Answer 3

Đây không phải là câu trả lời tốt nhất có thể, nhưng nếu bạn không có controll qua việc thực hiện các IUnityContainer và nó có để xử lý IoCContainer bạn luôn có thể phá vỡ các chuỗi recurrection trong ít nhất hai, cách hợp lệ:

protected virtual void Dispose(bool disposing) 
{ 
    if (disposing) 
     if (_container != null) 
     { 
      var tempContainer = _container; 
      _container = null; 
      tempContainer.Dispose(); 
     } 
} 

hoặc

private bool isDisposed = false; 
protected virtual void Dispose(bool disposing) 
{ 
    if(isDisposed) return; 
    isDisposed = true; 

    if (disposing) 
     if (_container != null) 
     { 
      _container.Dispose(); 
      _container = null; 
     } 
}