Har ett bokningssystem där jag vill sätta ewtt fält i databasen till green om plasten bilr avbokad. Detta görs geniopm en Admin sida. Jag har en sida med formulärfält som skickas till en sida med denna kod. Vad är det för fel??
<%
katid = request.form("katid")
plats = request.form("plats")
status = request.form("status")
%>
<body bgcolor="#FFFFFF">
<%
set conn = Server.CreateObject("ADODB.Connection")
conn.open "boka"
SQL = "UPDATE biljetter SET"
SQL = SQL & "status = '" & Request.Form("status") & "' Where katid = " & Request.Form("katid") and plats =" & Request.Form("plats")
Conn.Execute(SQL)
%>
Ser ut som att du glömmer mellanslag när du "lägger ihop" SQL-strängarna.
Använd
SQL = "UPDATE biljetter SET "
...så borde det fungera bättre.
Sedan så är det ju onödigt att hämta formulärdata två gånger. Antingen använder du de redan definerade variablerna i SQL-frågan eller så tar du bort de tre första raderna (katid =... osv). Beror lite på om du skall använda katid, plats och status fler gånger.
Du växlar mellan stor och liten bokstav på "conn". Var konsekvent för säkerhets skull. I och för sig så har väl detta ingen betydelse då du kör VBScript, men skadar aldrig att göra det "rätt" ändå.
Du stänger "conn" två gånger på slutet. Ta bort en av dessa.
[Redigerat av cf den 13 aug 2000]
261 ms totalt · 4 externa anrop · v20260731065814-full.1dc6f849
