Avviso di analisi del codice di Visual Studio 2012 CA1001. Cosa significa?

Question

Non è importante ma sto cercando di capire cosa mi sta dicendo ed è un avvertimento legittimo? Qualcuno può spiegare questo errore in termini semplici per me?

CA1001 tipi che possiedono campi monouso devono essere usa e getta

Implementare IDisposable su 'MemVoteManager' perché crea i membri dei seguenti tipi IDisposable: 'CongressDBEntities. Se "MemVoteManager" è stato precedentemente spedito, l'aggiunta di nuovi membri che implementano IDisposable a questo tipo è considerata una modifica irrisoria ai consumatori esistenti di .

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

Si potrebbe implementare IDisposable in modo che il contesto sarà smaltito quando i consumatori hanno finito con la tua classe, ma potresti stare meglio NON avere il contesto come membro della classe. Basta creare quando ne avete bisogno e smaltirlo quando hai finito:

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

I contesti sono leggeri, quindi non c'è un enorme pena per la creazione di ogni volta. Inoltre, non devi preoccuparti dei consumatori che notificano di disporre del contesto e non hai molte modifiche incorporate in memoria se una istanza della classe viene utilizzata più volte.

Answer 2

E 'ti permette di sapere che il campo context contiene componenti usa e getta. Ciò significa che questi membri devono avere Dispose() chiamato in modo che si possa verificare la Garbage Collection. Pertanto, desidera implementare l'interfaccia IDisposable su MemVoteManager in modo da poter chiamare Dispose() sul contesto e/o sui suoi membri che sono monouso.

Quindi modificare il vostro codice ed in quanto tale:

public class MemVoteManager : AbstractDataManager, IMemVoteManager, IDisposable 

e quindi implementare i membri dell'interfaccia IDisposable in questo modo:

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