CA1001 Visual Studio 2012コード分析の警告。どういう意味ですか？

Question

それは重要ではありませんが、私はそれが私に何を伝えているのか把握しようとしており、正当な警告ですか？誰かが私にこのエラーを簡単に説明できますか？ 'CongressDBEntities'：使い捨てのフィールドを所有し

CA1001タイプは、それが 次IDisposableインタータイプのメンバーを作成するため、

は 'MemVoteManager' にIDisposableを実装を使い捨てにする必要があります。 「MemVoteManager」 が以前に出荷された場合、このタイプにIDisposable を実装する新しいメンバを追加することは、既存の コンシューマへの急変と考えられます。

public class MemVoteManager : AbstractDataManager, IMemVoteManager 
{ 
    private CongressDBEntities context = new CongressDBEntities(); 

    public int AddMemVote(tMemVoteScore mvs) 
    { 
     //Insert Model 
     context.tMemVoteScores.Add(mvs); 
     context.SaveChanges(); 

     int newPK = mvs.MemVoteScoresID; 

     //Update funky column ID with PK as well 
     var memVoteItem = (from m in context.tMemVoteScores 
          where m.MemVoteScoresID == newPK 
          select m).SingleOrDefault(); 

     memVoteItem.ID = memVoteItem.MemVoteScoresID; 
     context.SaveChanges(); 
     return newPK; 
    } 

Answer 1

だから、のようなあなたのコードを変更コンシューマーがあなたのクラスで終わったときに、あなたがクラスのメンバーであるというコンテキストを持たないほうが良いかもしれません。ちょうどあなたがそれを必要なときにそれを作成し、完了したら、それを処分：

public int AddMemVote(tMemVoteScore mvs) 
{ 
    //Insert Model 
    using(CongressDBEntities context = new CongressDBEntities()) 
    { 
     context.tMemVoteScores.Add(mvs); 
     context.SaveChanges(); 

     int newPK = mvs.MemVoteScoresID; 

     //Update funky column ID with PK as well 
     var memVoteItem = (from m in context.tMemVoteScores 
          where m.MemVoteScoresID == newPK 
          select m).SingleOrDefault(); 

     memVoteItem.ID = memVoteItem.MemVoteScoresID; 
     context.SaveChanges(); 
    } 
    return newPK; 
} 

コンテキストは、軽量であるので、それらを毎回作成するための巨大なペナルティはありません。さらに、消費者がコンテキストを破棄するように通知する心配はありません。クラスの1つのインスタンスが何度も使用されていれば、メモリ内に多くのビルドアップ変更がありません。

Answer 2

あなたがフィールドcontextは使い捨てのメンバーが含まれていることを知らせています。つまり、それらのメンバーは、Dispose()を呼び出す必要があるため、ガベージコレクションが実行されます。したがって、コンテキストをMemVoteManagerに実装して、コンテキスト上でDispose()を呼び出したり、使い捨てのメンバーを呼び出すことができます。このようIDisposableインターフェイスのメンバーを実装し、その後

public class MemVoteManager : AbstractDataManager, IMemVoteManager, IDisposable 

をして：あなたはので、コンテキストが処分されるIDisposableを実装することができ

public void Dispose() 
{ 
    // call dispose on the context and any of its members here 
}