CA1001 Ostrzeżenie Visual Studio 2012 Analiza kodu. Co to znaczy?

Question

To nie jest tak ważne, ale próbuję dowiedzieć się, co mi mówi i czy jest to uzasadnione ostrzeżenie? Czy ktoś może mi wyjaśnić ten błąd w prostych słowach?

CA1001 Rodzaje jednorazowych, które są właścicielami pól powinny być jednorazowe

Wdrożenie IDisposable na 'MemVoteManager', ponieważ tworzy członków o IDisposable następujących typów: 'CongressDBEntities. Jeśli wcześniej dostarczono "MemVoteManager" , dodawanie nowych członków, które implementują IDisposable do tego typu, jest uważane za przełomową zmianę w stosunku do istniejących konsumentów .

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

Ty mógłby wdrożyć IDisposable więc kontekst będą usuwane kiedy konsumenci skończą z twoją klasą, ale możesz być lepiej NIE mając kontekstu być członkiem klasy. Wystarczy utworzyć go, gdy jest to potrzebne i rozporządzania nim, kiedy skończysz:

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

Konteksty są lekkie, więc nie jest to ogromne kary za tworzenie ich za każdym razem. Poza tym nie musisz się martwić o to, że konsumenci powiadamiają Cię, że musisz pozbyć się kontekstu, i nie masz zbyt wielu nagromadzonych zmian w pamięci, jeśli jedna instancja klasy jest używana wiele razy.

Answer 2

To powiadomieniem tego pola context zawiera członków jednorazowych. Oznacza to, że ci członkowie muszą mieć Dispose(), aby wywołać Garbage Collection. Dlatego chce on zaimplementować interfejs IDisposable na MemVoteManager, aby można było wywoływać Dispose() w kontekście i/lub jego elementach, które są jednorazowe.

więc zmodyfikować twój kod jako takie:

public class MemVoteManager : AbstractDataManager, IMemVoteManager, IDisposable 

a następnie wdrożyć członków interfejsu IDisposable tak:

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