Min sida är uppbyggd med If satser och sedan "Include filer" i startsidan (default.asp). Exempel...

<%If Request.Querystring("sida")="forum" Then%>
<!--#include file="forum.asp"-->

Är det bra att göra så här? Default blir ju ganska full med "kod" efter ett tag om man har många sidor. Finns det bättre sätt?

Varför inte såhär?
<code>
<% If Len(Request.Querystring("sida")) <> 0 Then
Respone.Execute Request.Querystring("sida")
Else
Response.Write "Ingen sida eftersöktes"
End If %>
</code>

Molles variant rekommenderas mer, men att bara anropa forum.asp rakt av är det bästa...

Man bör validera värdet som klienten skickar. Annars vet man inte vad en hacker kan skicka för information.

Hur gör man det Andreas?

Vad är det förresten för skillnad mellan

<% Server.Execute("fil.asp") %>

och

<!--#include file="fil.asp"-->

?

Genom att med ifsats eller på annat sätt begränsa vilka sidor som kan exetueras.
<code>
<%

Select Case LCase(Request.Querystring("sida"))
Case "forum"
Server.Execute "forum.asp"
Case "sida2"
Server.Execute "sida.asp"
Case Else
Server.Execute "start.asp"
End Select
%>
</code>
Kanske inte lika bekvämt. Men kommer nog ge en mer säker lösning.

Nackdelen med Server.Execute tycker jag är att man inte kan skicka med värden när man ska hämta en sida.

Tex att skicka en sträng som "forum.asp?id=8" till Server.Execute går ej.

Går ej: Server.Execute("forum.asp?id=8")
Går: Server.Execute("forum.asp")

MVH/Marcus