Perdita di memoria SharePoint

Question

Questo contiene una perdita di memoria? Ho cercato di capire meglio le perdite di memoria, ma non posso dire se ho corretto questo? In caso contrario, in che modo si eliminano correttamente l'oggetto SPweb e l'oggetto SPSite?

using (SPWeb oWebsite = new SPSite(webUrl).OpenWeb()) //Open SP Web 
{ 
    SPListCollection collList = oWebsite.Lists; //Open Lists 

    foreach (SPList oList in collList) 
    //For Each List Execute this 
    { 
     if (!oList.Hidden) 
     //If the list is hidden do this else nothing 
     { 
      ListSitesDropDownBox.Items.Add(new ListItem(SPEncode.HtmlEncode(oList.Title), SPEncode.HtmlEncode(oList.Title))); 
      ViewState["Item" + counter] = SPEncode.HtmlEncode(oList.Title); 
      counter++; 
     } 
    } 
} 

Answer 1

Sì, lo fa. Disponi di SPWeb ma dimentica lo smaltimento di SPSite.
Il modo giusto per fare questo:

using (var site = new SPSite(webUrl)) 
using (var web = site.OpenWeb()) { 
    // ... 
} 

noti che questo è equivalente a:

using (var site = new SPSite(webUrl)) { 
    using (var web = site.OpenWeb()) { 
     // ... 
    } 
} 

ma omesso bretelle per esterno using per ridurre codice nesting. Le regole di controvento sono le stesse di if.

Qualche osservazione stilistiche:

• prega, don't use Systems Hungarian in C# code. Solo list.

• I commenti che riformulano il codice sopra non hanno alcun senso. Mantenerli costruttivi, cioè spiegando lo scopo del codice. qualcosa invece di quelle quattro osservazioni, si dovrebbe aver scritto così che in principio:

  // Populate drop-down list with list names and save them in ViewState 
  

Answer 2

Non di gettare via oggetti SPSite nel vostro usando, solo SPWeb.

Answer 3

Altri hanno risposto alla tua domanda, ma io aggiungo che si dovrebbe essere in esecuzione il tuo costruisce attraverso SPDisposeCheck: http://archive.msdn.microsoft.com/SPDisposeCheck
Non è perfetto, ma si punta voi fuori presto per un sacco di problemi.