21 svar · 542 visningar · startad av pRob3 · sida 2 av 2
Någon som har lust o kolla igenom min inloggnings kod o optimera den på det allra bästa sättet? Använder en MySQL databas. <%@ LANGUAGE="VBSCRIPT" %> <% Response.Expires=-1 %> <% Response.Buffer = True %> <% Session.LCID = 1053 %> <!-- #include file="adovbs.inc" --> <!-- #INCLUDE VIRTUAL="/include/koppling.asp" --> <% Dim vem vem=lcase(Request.Form("username")) Set RecSet = Connect.Execute(
Läs frågan i sin helhet →Använder du din inkluderade fil med kopplingen i andra filer kan det kanske vara en ida att tex infoga funktionen secure(txt) i den.
Annars tycker jag detta ser lite lattjo ut:
Session("sex") = ""& sex &""& newage &""
Varför de tomma strängarna? (Finns på flera ställen i din kod)
Session("sex") = sex & newage
Eller missar jag något?
Community. Jippie. ;)
Personligen är jag lite emot alla de där sessionsvariablarna. Den där informationen kan du ju komma åt när du vill ändå. Det enda (med undantag för vissa saker) du behöver i sessions är besökarens användar-ID och sedan kan du anropa en (sub)funktion för att hämta den där datan. För den behövs väl knappast på varje sida?
Vidare kan du kolla ifall användaren är inloggad eller inte genom att se ifall session("userId") är 0 eller ingenting.
sub setUserProps(lngUserId)
Set RecSet = Connect.Execute("SELECT * FROM members where id="& lngUserId &"")
if not RecSet.eof then
strSex = recset("sex") & recset("newage")
strUsername = RecSet("username")
strStad = RecSet("stad")
lngMinOnline = recset("loginTime")
end if
end sub
Kanske blir lite krångligt att införa detta i din kod, men det är ett förslag. :)