For mange context-switches?

Det betyder ikke noget som helst...

Får du en fejlmeddelse eller...

har du deklaret dine RS ??

Der kommer ikke en massiv load på serverens ASP-engine?

ingen fejlmeddelelser eller noget, jeg vil bare optimere for hastighed....

Det giver en lille performance sænkning når du skifter så ofte som du gør i dit eksempel, men hvis der ikke kommer mange brugere på samme tid gør det ikke den store forskel ...

Det er vel også ren og skær dovenskab at du ikke gider lave det hele som en stor response.write :-)

har du en kode med noget i stil...

RS = "SELECT * from .............

:), ja, det er det............hvilket bringer mig til det næste spørgsmål:
Er det bedre at samle "tags og kode" i en stor Response.Write eller at bruge f.eks. 20 linier der begynder med Response.Write?

SQLtemp="SELECT * FROM Artikler WHERE artikler_id= " & artikler_id & " ORDER BY opdateret_d DESC"
Set RS = Con.Execute( SQLtemp )

<%= RS("artikler_id") %> {en forkortelse for <% Response.Write RS("artikler_id") %> } brug den... spare plads...

Mundi, faktisk ser koden sådan her ud

<%
'ASP-kode
%>
<a href="deleteartikel.asp?artikler_id=<%=artikler_id%>" onclick='return slet(this)'>Slet</a> <a href="editartikelform.asp?artikler_id=<%=RS("artikler_id")%>">Rediger</a>
<table border="0" width="702">
  <tr>
    <td width="92" bgcolor="#EAEAEC">Artikel nr.</td>
    <td width="199" bgcolor="#EAEAEC">Kategori</td>
    <td width="199" bgcolor="#EAEAEC">Oprettet af</td>
    <td width="298" bgcolor="#EAEAEC">Oprettet d.</td>
  </tr>
  <tr>
    <td width="92"><% Response.Write RS("artikler_id") %></td>
    <td width="199"><% Response.Write RS("kategori") %></td>
    <td width="199"><% Response.Write RS("oprettet_af") %></td>
    <td width="298"><% Response.Write RS("oprettet_d") %></td>
  </tr>
  <tr>
    <td width="299" bgcolor="#EAEAEC">Hardware ID</td>
    <td width="299" bgcolor="#EAEAEC"></td>
    <td width="292" bgcolor="#EAEAEC">Opdateret af</td>
    <td width="299" bgcolor="#EAEAEC">Opdateret d.</td>
    <td width="198"></td>
  </tr>
  <tr>
  <td><a href="hardwareinfo.asp?pc_nr=<% Response.Write RS("hardware_id") %>" onClick="window.open('','popup','toolbar=0,scrollbars=0,location=0,statusbar=0,menubar=0,resizable=0,height=220,width=300,left = 330,top =250')"target="popup"><%= RS("hardware_id") %></a>
    <td width="299"></td>
    <td width="192"><% Response.Write RS("opdateret_af") %></td>
    <td width="199"><% Response.Write RS("opdateret_d") %></td>
    <td width="198"></td>
  </tr>
</tabel>
<table border="0" width="630">
<tr>
    <td width="500" bgcolor="#EAEAEC">Titel</td>
  </tr>
  <tr>
    <td width="500"><% Response.Write RS("titel") %></td>
  </tr>
<tr>
    <td width="500" bgcolor="#EAEAEC">Link</td>
  </tr>
  <tr>
    <td width="500"><a href="<% If (InStr(RS("link"), "\")) Then Response.Write RS("link") Else Response.Write "http://" & RS("link") End If %>" target="_blank"><% Response.Write RS("link") %></a></td>
  </tr>
  <tr>
    <td width="500" bgcolor="#EAEAEC">Resume</td>
  </tr>
  <tr>
    <td width="500"><% Response.Write RS("resume") %></td>
  </tr>
  <tr>
    <td width="500" bgcolor="#EAEAEC">Beskrivelse</td>
  </tr>
  <tr>
    <td width="500">
<%
'ASP-kode
%>

Set RS = conn.Execute(SQLtemp)

ikke helt med på hvad du mener, aspprogram (10:27:54)

Ok hvis det ikke er mere end det, kan det være det samme, men man burde faktisk bestræbe sig på at lave så få skift som muligt

du mangler et n i conn.execute, og prøv at fjerne mellemrummet i ()

kan du ikke smide din aspkode op... svært at finde asp fejl i html delen.....

aspprogram-->
Det kommer jo an på hvad han har kaldt connection variablen - den kunne jo hede con...

aspprogram, der ER ikke fejl i koden.........problemet bestod i om der VAR et problem i at skifte mellem HTML og ASP så ofte som jeg gør det...

hov - hedde ikke hede...

lenn har ret, den hedder Con

er det kun denne del, som er galt?

<td width="92"><% Response.Write RS("artikler_id") %></td>
<td width="199"><% Response.Write RS("kategori") %></td>
<td width="199"><% Response.Write RS("oprettet_af") %></td>
<td width="298"><% Response.Write RS("oprettet_d") %></td>

aspprogram -->

Der er jo faktisk ikke noget "galt" med hans kode - han vil, som jeg forstår det, bare gerne vide om man kan optimere koden lidt...

lenn har ret.

så skal han jo bare bruge: <%= RS("artikler_id") %>

Det spare ham for en masse plads... optimmeret en pæn del....

Du har da ganske ret i at det vil gøre koden en hel del pænere at se på....

Det er faktisk vigtigere at have response.buffer=true. Så
kan du skifte ud og ind af <%...%> som du vil og lave
<%= og response.write uden at kunne måle en forskel

erik, jeg kigger lige på Response.Buffer=True.....

skal ligge som det første i documentet... gør bare at siden bliver afviklet hurtigere....

Hmmm....i følge www.learnasp.com, ER buffer=true som standard på windows 2000 systemer:

NT4 and Win 2000 differences

NT4 <%buffer=false%> by default which hurts overall server speed (http://www.learnasp.com/advice/whybuffer.asp). Win 2000 <%buffer=true%> by default. A clever administrator can change NT4 registry so buffer=true for all scripts and will see major server performance improvements (see http://www.learnasp.com/learn/speedserver.asp for other ones).

eksempel på response.buffer:

<%
Response.Buffer = True
%>
<html>
....
....
....

det er en stor fordel at bruge OPTION EXPLICIT i din kode.. det er noget der godt kan have indflydelse på hastigheden

og når du bruger request skal du også fortælle hvor den skal request fra, f.eks. request.form("form") eller request.querystring("id") ellers skal request kigge det hele igennem..

jeg vil stadig sige du ikke skal skifte mellem HTML og asp.. i dit tilfælde kan det garanteret ikke måles at du skifter mellem HTML og asp, men det ville da være dumt at lade brugere vente mere end nødvendigt, så det er bedre at vænne sig til at bruge response.write på hele lortet på en gang..

-- xs