Gör mitt script efektivare och lättare

Ja du... vad är det egentligen du ska uppnå? Vad innehåller Re? Dagar? Datum?
Spontant sett förefaller det som om du använder lite många fält i din db.

Det är såhär..
Att jag har andvändare i medlem. och så ska dom kunne boka in sig till en tunering som jag kan lägga till från min A access i samma inloggning. typ...

Altså.

Jag -> kunne lägga in en ny tunering genom knapptryck, då jag anger datum

Andvändare -> Kunna regga sig till tunering(om han inte redan är reggad) -> Han blir reggad, han blir t.ex nummer 2 -> visa vilken bords plasering han får genom något script att slumpa fram olika nummer upp till 30(som är max antal gräns).

Vid 30 reggade så ska det inte gå o anmäla sig längre.

om någon förstår är jag tacksam :P

Oj, många IF-satser. Du måste använda databasen flitigare tror jag. En kotroll per värde ska räcka. Tänk om det skulle vara en tunering för 20.000st användare :q

if not user_registered then register
else
if tourlimit <30 then add user
else
sorry, the limit is already reach :(
end if
end if

Det skall inte så många rader till för att uppnå din funktion.

OT: badbet? ;)

Du öppnar och stänger ASP-tolken 164 gånger på en sida... Det räcker med att göra det en gång speciellt på en sån där kod...

Och så ska du nog ta och kolla på If-funktionen lite.

If nått = nått Then
'nått
Else If nåttannat = nåtannat Then
'nåttannat
End If
End If

Det är två If-satser, behövs bara en och det uppnår man med att använda ElseIf

If nått = nått Then
'nått
ElseIf nåttannat = nåtannat Then
'nåttannat
End If

Jo det e klart, varför skrev jag två stycken If-satser :q :OO

Poffe skrev:

Du öppnar och stänger ASP-tolken 164 gånger på en sida... Det räcker med att göra det en gång speciellt på en sån där kod...

Kan ju hålla med om att typ %><% är ganska onödigt. Men att inte göra några context-switchar är bara dumt. Ger sämre prestanda och är mer oläsligt.

Skulle man ta den här koden...

<A Href="badmin.asp?b=2tun&ID=<% =Re ("ID") %>">Anmäl dig till tuneringen den <% =Re ("Dat") %></A><BR>

Och skriva den som Response.Write's så blir den ganska jobbig att läsa och tar upp många rader.

Response.Write "<A Href=""badmin.asp?b=2tun&ID="
Response.Write Re ("ID")
Response.Write """>Anmäl dig till tuneringen den "
Response.Write Re ("Dat")
Response.Write "</A><BR>"

Skriver man den med konkateneringar så blir den jobbig att läsa samtidigt som den blir långsammare p.g.a. konkateneringarna i VB är prestandakrävande.

Response.Write "<A Href=""badmin.asp?b=2tun&ID=" & Re ("ID") & """>Anmäl dig till tuneringen den " & Re ("Dat") & "</A><BR>"

Kanske är en smaksak, jag tycker dock inte det är så svårtläst.

Response.Write "<A Href='badmin.asp?b=2tun&ID=" &_
	Re ("ID") &_
 	"'>Anmäl dig till tuneringen den " &_
	Re ("Dat") &_
	"</A><BR>"

Okej =)...
har tänkt om lite och kör på denna.. men då har jag ett problem... Nästa tävling.. då måste jag göra en helt ny tabell... :/

Sico: Mer svårläst än context-switcharna anser jag. Och konkateneringarna gör scriptet långsammare.

Sen så tycker jag allmänt att man ska undvika apostroftecken. Vill man senare köra efter XHTML-standarden så får man problem eftersom att man måste använda citattecken där. Och använder man apostroftecken så får man problem när man har textsträngar som ska in i attribut. Då får man göra extra-escapningar. Ett exempel:

'när man använder citattecken
Response.Write "<img src=""image.jpg"" alt=""" & Server.HTMLEncode(str) & """>"

'när man använder apostrofer
Response.Write "<img src='image.jpg' alt='" & Replace(Server.HTMLEncode(str), "'", "&amp;#39;") & "'>"

Har du ett administrationsystem över tuneringarna? Genom det lägger till en ny tunering på ett nytt ID så är det bara att plocka fram värdena där ifrån så att anmälan's information skickas in på just den touren/ID man valt att registera sig på.

Funktionen blir lite som ett forum, att informationen postas just till den tråd man valt att skriva i. Hoppas du förstår.

jo.. har börjat tänka så nu oxå.. har gjort två olika tabeller.. en som har hand om dom olika tuneringarna och en som har hand om dom som reggar sig i dom olika tuneringarna...