If-sats med databas

6 svar · 311 visningar · startad av scoxi 14 okt. 2004

scoxiMedlem sedan jan. 200460 inlägg

Frågan14 okt. 2004 12:21#1

Alltså jag e säkert helt ute o cyklar nu inte rört asp på 6 månader nu så jag behöver lite hjälp o komma igång igen..


<%
username=Request.Querystring("namn")
password1=Request.Querystring("pass1")
password2=Request.Querystring("pass2")

If username="" OR password1="" OR password2="" then
Response.Write "Alla fält måste fyllas i"

Else

Set Connect = Server.CreateObject("ADODB.Connection")
Connect.Open "driver={Microsoft Access Driver (*.mdb)};dbq=c:\inetpub\wwwroot\test\anvandare.mdb"

sql = "INSERT INTO anvandare (username, password) VALUES ( '" & username & "', '" & password1 & "' )"

connect.execute(sql)
Connect.Close
End if
Response.Write "va kul att du e medlem nu då.."
%>

Felet blir att det står på sidan sen "Alla fält måste fyllas iva kul att du e medlem nu då..

ni fattar nog vad jag vill göra fixa till detta nu :P

@ndersMedlem sedan juni 200032 967 inlägg

14 okt. 2004 12:23#2

Eftersom du har utskriften av "va kul..." allra sist, utanför if-satser och dylikt kommer det alltid att skrivas ut. Lägg den inne i if-satsen.

scoxiMedlem sedan jan. 200460 inlägg

14 okt. 2004 12:24#3

mm upptäckte det men det blir ändå fel...
den skriver att man måste fylla i alla fälten trots att dom e ifyllda ;(

@ndersMedlem sedan juni 200032 967 inlägg

14 okt. 2004 12:27#4

Men använder du verkligen metoden GET i ditt formulär då?

scoxiMedlem sedan jan. 200460 inlägg

14 okt. 2004 12:44#5

oj, hade glömt o skriva in det..
tack för hjälpen

@ndersMedlem sedan juni 200032 967 inlägg

14 okt. 2004 12:48#6

Du bör dock:

använda metoden POST, och Request.Form.
Se till att skydda dig mot SQL-injections genom att byta apostrofer mot två dylika

mvh

scoxiMedlem sedan jan. 200460 inlägg

14 okt. 2004 12:49#7

juste.. tack asså jag e helt lost nu ska testa nåra dar nu så jag kommer tillbaka :)