Syntax error (missing operator) in query expression

strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Request("matchnavn") & "'"

Hvis jeg vil have den sidste tilmeldte øvers på listen, hvor skal den placeres

strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Request("matchnavn") & "' ORDER BY ID DESC " :-)

Til foregående indlæg vil jeg lige tilføje: Uden de sidste tre tegn  :-)

Med du er vel klar over problemer omkring sql-injection?

Hehe. Hvad tænker du på med problemer omkring SQL-injection?

Det var nu et spørgsmål til /cdc  - lyt med, så får du jo i hvert fald en forklaring

jo da og det fungerer, skal nok give points, og forhøjer den lidt hvis i også kan svare mig på hvordan jeg så laver denne:

response.redirect ("default_players.asp") og den så skal vises som dette f.eks:
default_players.asp?matchnavn=Trøst-/vinderturnering

Denne: response.redirect ("default_players.asp") ligge i filen hvor man gemmer sin tilmelding til matchen.

Den del af scriptet ser således ud:

<%
If Request("mode") = "ny" then

Set Conn = Server.CreateObject("ADODB.Connection")
Conn.Open "DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=" & ("*******\matchmaker.mdb")

SQL= "INSERT INTO spiller (matchnavn, navn, medlemsnr, hcp, email, matchinfo)"
SQL = SQL & " VALUES ('" & request.form("matchnavn") & "','" & request.form("navn") & "','" & request.form("medlemsnr") & "','" & request.form("hcp") & "','" & request.form("email") & "','" & request.form("matchinfo") & "')"

Conn.Execute (SQL)

conn.Close
Set conn = nothing
Set SQL = nothing

'response.write "Spilleren er nu oprettet."
response.redirect ("default_players.asp")

hvad mener du med sql-injection? ikke noget jeg har hørt om. Roder ikke så meget med webdesign mere, så jeg følger ikke med så meget

hvis det har noget med at man bare kan skifte ?id=5 ud med ?id=6 så gør det ikke noget her, for det er kun match tilmldinger og inetet hemmelig

Nej, det er indskrivning af '-tegn i inputfelter, querystrings, cookies etc. af en ondsindet bruger, der kan få dine sql-sætninger til at afsløre for meget, eller ødelægge dine data. Du er formentlig sikret med

strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Replace(Request("matchnavn"),"'","''") & "'"

Og det er egentlig basal viden, hvis man laver webapplikationer, der skal bruges ud over ens egen maskine uden nettilslutning. Det svarer lidt til at man ikke kan få kørekort til automobil uden at kende diverse færdselsregler. Men alle kan sætte sig ned og lave en "webapplication" - og i bedste fald blive til grin.  ;)

OK tak for det...kigger på det

har du en løsning på det andet

Vil det så sige den skal se således ud:

strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Replace(Request("matchnavn"),"'","''") & "'" ORDER BY ID DESC "

Argh, på den måde erikjacobsen - SQL injections på den måde. Det kender jeg godt til ;-) Selvfølgelig da - troede du mente på en anden måde. Men Erik - man skal da ikke lave SQL injection, hvis man bruger ID=" & request.querystring fx. vel?

Men ja /cdc den skal se sådan ud :-) strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Replace(Request("matchnavn"),"'","''") & "'" ORDER BY ID DESC "

Eller nej...

strSQL = "SELECT * FROM spiller WHERE matchnavn = '" & Replace(Request("matchnavn"),"'","''") & "' ORDER BY ID DESC "

Hvad er problemet med:

<%
If Request("mode") = "ny" then

Set Conn = Server.CreateObject("ADODB.Connection")
Conn.Open "DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=" & ("*******\matchmaker.mdb")

SQL= "INSERT INTO spiller (matchnavn, navn, medlemsnr, hcp, email, matchinfo)"
SQL = SQL & " VALUES ('" & request.form("matchnavn") & "','" & request.form("navn") & "','" & request.form("medlemsnr") & "','" & request.form("hcp") & "','" & request.form("email") & "','" & request.form("matchinfo") & "')"

Conn.Execute (SQL)

conn.Close
Set conn = nothing
Set SQL = nothing

'response.write "Spilleren er nu oprettet."
response.redirect ("default_players.asp")

End if
%>

Problemet ligger i min redirect default_players.asp den fil lister ingen da der skal stå matchnavn bag: default_players.asp?matchnavn=Trøst-/vinderturnering f.eks. så den henter dem fra DB som har tilmeldt sig denne match.

Alle sopillere ligger i en tabel som hedder spiller, og de bliver "sorteret" efter matchnavn

hvordan kan jeg redirecte til given match rigtig

response.redirect("default_players.asp?matchnavn=" & request.form("matchInfo")

nu skriver du matchinfo du mener vel matchnavn? :o)

Expected ')'

GemSpiller.asp, line 18

response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn")
-----------------------------------------------------------------------------^

Ja, matchnavn - selvfølgelig.

Enten, så gør sådan her:
response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn") & ")
Eller...
response.redirect "default_players.asp?matchnavn="&request.form("matcnavn")

Resultat af denne: response.redirect "default_players.asp?matchnavn="&request.form("matcnavn"):

Unterminated string constant

GemSpiller.asp, line 18

response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn") & ")
----------------------------------------------------------------------------------^

Resultat ef denne:

response.redirect "default_players.asp?matchnavn="&request.form("matcnavn")

GemSpiller.asp?mode=ny i adresse feltet og blank side

en lille skrive fejl retter lige den

så blev fejlen således:

Unterminated string constant

GemSpiller.asp, line 18

response.redirect("default_players.asp?matchnavn=" & request.form("matchnavn") & ")
-----------------------------------------------------------------------------------^

Der skal også kun stå: response.redirect "default_players.asp?matchnavn=" & request.form("matcnavn") - ikke ":" efter det!

response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn") & "")

Tak for alle gode inputs, smid et svar begge to, så fordeler jeg lige points

Her er mit svar :-)

så venter jeg lige på erikjacobsen...hvis det tager lidt tid, er jeg først tilbage mandag

Ja, selvfølgelig. ericjakobsen kom lige med god info der.

Men ha' en rigtig god weekend :-)

Tak, men jeg samler slet ikke på point.

Det er lidt tumpet at skrive
    response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn") & "")
når man kan nøjes med
    response.redirect("default_players.asp?matchnavn=" & request.form("matcnavn"))

tak for hjælpen

/cdc