[vb.net] Caching cached ook selected items in dropdown

[jp]

Ik wil de performance van mijn webapplicatie verhogen, dus ik dacht bepaalde waardes uit de database te cachen.

Het idee is als volgt: Een medewerker kan zowel een assistent als een chef hebben en beide waardes komen uit dezelfde tabel. (Empoyee). Dus dacht ik om die lijst te cachen.

Nu cached hij echter ook, om voor mij onduidelijke redenen de huidige selected value.

Visual Basic .NET:

iactShared.DropDown.GetEmployeeList(Me.drpBoss, True)
iactShared.DropDown.GetEmployeeList(Me.drpAssistent, True)
' ...
Me.drpAssistent.SelectedValue = insEmployee._AssistentId
Me.drpBoss.SelectedValue = insEmployee._SuperiorId

En de functie die het geheel vult:

Visual Basic .NET:

Public Shared Sub GetEmployeeList(ByRef DropDown As System.Web.UI.WebControls.DropDownList, Optional ByVal InsertEmpty As Boolean = False, Optional ByVal DepartmentId As Integer = 0, Optional ByVal Defaulttext As String = "")
   IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList", "Begin")
   IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList:DepartmentId", DepartmentId)

   Dim drpItems As ArrayList
   Dim strCacheName As String = "GetEmployeeList" & "|" & DepartmentId
   Dim intCacheDuration As Integer = Settings.Caching.Dropdown.CacheDuration

   DropDown.Items.Clear()
   If InsertEmpty Then iactShared.DropDown.InsertEmpty(DropDown, Defaulttext)

      drpItems = HttpContext.Current.Cache(strCacheName)
         If drpItems Is Nothing Or True Then 'cache temporary turned off
            ' items not found in cache
            IACTLibrary.TraceWarn.WithoutLevel("GetEmployeeList", "Not found in cache, retrieving from database")

            drpItems = New ArrayList ' place to store the items in

            Dim EmployeeAdapter As New dsWylanceTableAdapters.dsEmployeeTableAdapter
            Dim EmployeeData As dsWylance.dsEmployeeDataTable
            Dim strFullname As String = ""

            If Not DepartmentId > 0 Then
               IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList:DepartmentId=0", "Department is 0 or < 0, getting all employees")
               EmployeeData = EmployeeAdapter.GetEmployeeList()
            Else
               IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList:DepartmentId>0", "Getting all employees for this company")
               EmployeeData = EmployeeAdapter.GetEmployeeByDepartment(DepartmentId)
            End If

            IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList:EmployeeData", EmployeeData.Count)

            For Each row As dsWylance.dsEmployeeRow In EmployeeData
               strFullname = row.Surname & ", " & row.Firstname
               Dim s As New ListItem(strFullname, row.Id)
               DropDown.Items.Add(s)
               drpItems.Add(s)
            Next

            HttpContext.Current.Cache.Insert(strCacheName, drpItems, Nothing, DateTime.Now.AddSeconds(intCacheDuration), System.Web.Caching.Cache.NoSlidingExpiration)
         Else
            ' item was found in cache
            IACTLibrary.TraceWrite.WithoutLevel("GetEmployeeList", "Retrieving values from cache")
            For Each s As ListItem In drpItems
               DropDown.Items.Add(s)
            Next
         End If

         IACTLibrary.TraceWrite.WithoutLevel("iactshared.DropDown.GetEmployeeList", "End")
End Sub

Volgens mij doe ik het cachen "by the book" , maar hij cached dus ook de selected value van de dropdown... en ik niet dnap....

[ Voor 72% gewijzigd door jp op 05-12-2006 14:16 ]

[gorgi_19]

Waarom gebruik je geen outputcache?

[jp]

Bij een output cache bind ik (voor zover ik weer) het cachen aan 1 specifieke control (dropdownbox). En ik wil dat hij de gecachte waardes ook kan delen tussen twee controls.

De hele output wil ik niet cachen, want we hebben het over editpagina's, en die waardes wil ik dus niet cachen...

[jp]

***schop***

[PhysicsRules]

Je cacht de ListItems. Eén van de properties van een ListItem is Selected. Je cacht dat gewoon mee. Daarom kun je beter de onderliggende datatable cachen.

Daarnaast, waarom gebruik je eigenlijk geen databind?

[jp]

Ik cache de Listitems, maar voorzocer ik weet cache ik de listitems VOORDAT er wordt ingesteld dat een van die listitems selected zou zijn.

Geen databind... goeie vraag, maar een tikkie laat om nu nog aan te passen, tenzij het enorm veel beter is/zou zijn

[PhysicsRules]

jp schreef op donderdag 07 december 2006 @ 13:29:
Ik cache de Listitems, maar voorzocer ik weet cache ik de listitems VOORDAT er wordt ingesteld dat een van die listitems selected zou zijn.

Klopt. Maar je DropDown list en je cache verwijzen naar dezelfde objecten. Dus als je een item in je dropdownlist selecteert, is dat dezelfde item die in de cache staat.

Geen databind... goeie vraag, maar een tikkie laat om nu nog aan te passen, tenzij het enorm veel beter is/zou zijn

En daarom is het dus beter om een databind te gebruiken. Je slaat dan je data op de de cache, niet de listitems die allemaal extra zaken opslaan.

In beide gevallen moet je trouwens voorzichtig zijn om dezelfde listitems of dataset voor meerdere lijsten te gebruiken. Ook in je huidige geval zul je lijkt me tegen het probleem aanlopen dat de twee lijsten altijd dezelfde selected item hebben. (hebben ze altijde beide de waarde van SuperiorID?).

De oplossing is om een dataset te maken van je gegevens en voor beide lijsten een DataView hierop te definiëren, en die dataview aan je dropdownlist te binden.

Succes

[Janoz]

ALs je ze in je cache opslaat maakt hij er geen kopie van lijkt me. Je hebt de objecten dus gewoon twee referenties naar hetzelfde object. Als je dan vervolgens een object selected maakt dan zal ook de versie in de cache selected worden.