CA2000と依存性注入

Question

私はかなり単純なDIパターンを使用して自分のコントローラクラスにデータリポジトリを注入していますが、CA2000コード分析警告（スコープを失う前にオブジェクトを廃棄する）が表示されています。私は警告が起こっている理由を知っており、通常はそれを修正する方法を見つけ出すことができますが、この場合には、私は、オブジェクトの作成と返すメソッドの間にスローされた例外のいずれかの可能性があるどのように

1. を把握することはできません、または
2. 私はtry/finallyブロックを入れてエラーを取り除くことができます。

警告メッセージを全面的に放棄して表示しないようにする前に、潜在的な不在オブジェクトにならない同じ効果を達成するより良い方法がありますか？

public class AccountController : Controller 
{ 
    public AccountController() 
     : this(new SqlDataRepository()) 
    { 
    } 

    public AccountController (IDataRepository db) 
    { 
     this.db = db ?? new SqlDataRepository(); 

     // Lots of other initialization code here that I'd really like 
     // to avoid duplicating in the other constructor. 
    } 

    protected override void Dispose(bool disposing) 
    { 
     if (disposing && (this.db != null)) 
     { 
      IDisposable temp = this.db as IDisposable; 
      if (temp != null) 
      { 
       temp.Dispose(); 
      } 
     } 
    } 
} 

Answer 1

ASP.Net MVCを使用している場合は、コントローラーにIDisposableを実装させることができます。パイプラインによって処理が行われます。 ASP MVC: When is IController Dispose() called?を参照してください。

Answer 2

あなたのリポジトリはIDisposableを実装しています。あなたのコントローラにもIDisposableを実装させ、disposeメソッドでリポジトリをクリーンアップします。