CA1001 Visual Studio 2012 Code Analyse Warnung. Was heißt das?

Question

Es ist nicht so wichtig, aber ich versuche herauszufinden, was es mir sagt, und ist es eine legitime Warnung? Kann mir jemand diesen Fehler in einfachen Worten erklären?

CA1001 Typen, die Wegwerf-Felder besitzen sollte

implementieren IDisposable auf 'MemVoteManager' Einweg sein, weil sie Mitglieder der folgenden IDisposable Typen erstellt: 'CongressDBEntities'. Wenn 'MemVoteManager' zuvor versandt wurde, wird das Hinzufügen neuer Mitglieder, die IDisposable zu diesem Typ implementieren, als eine Änderung der bestehenden Verbraucher angesehen.

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

Sie implementieren könnte IDisposable so der Kontext entsorgt werden von wenn die Konsumenten mit Ihrer Klasse fertig sind, aber Sie könnten besser dran sein, wenn der Kontext kein Mitglied der Klasse ist. schaffen es nur, wenn Sie es brauchen und entsorgen Sie es, wenn Sie fertig sind:

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; 
} 

Contexts leicht sind so sie jedes Mal für die Erstellung nicht eine große Strafe gibt. Außerdem müssen Sie sich keine Sorgen darüber machen, dass Verbraucher Sie darüber informieren, den Kontext zu entfernen, und Sie haben nicht viel aufgebaute Änderungen im Speicher, wenn eine Instanz der Klasse mehrmals verwendet wird.

Answer 2

Es lassen Sie dieses Feld wissen context Einweg Elemente enthält. Das bedeutet, dass diese Mitglieder Dispose() aufgerufen haben müssen, damit Garbage Collection stattfinden kann. Daher möchte er, dass Sie die Schnittstelle IDisposable unter MemVoteManager implementieren, so dass Sie Dispose() auf den Kontext und/oder seine Mitglieder, die Disposable sind, aufrufen können.

So Sie Code als solche ändern:

public class MemVoteManager : AbstractDataManager, IMemVoteManager, IDisposable 

und implementieren dann die Mitglieder der IDisposable Schnittstelle wie folgt aus:

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