<%
If Request.Querystring("lank") = "ut" Then
SQL = "SELECT klick, hemsida FROM lank WHERE ID =" &Request.Querystring("ID")
Set RecSet = connect.execute(SQL)
hemsida = RecSet("hemsida")
SQL = "UPDATE lank SET klick = klick + 1 WHERE ID = "& Request.Querystring("ID")
connect.execute sql,,128
Response.Redirect(hemsida)
End If
%>
Jag tog bort lite felaktiga ' och " och lade till ett executeanrop.
------------------ essentitia preter non sans multiplicandum
<%
If Request.Querystring("lank") = "ut" Then
SQL = "SELECT hemsida FROM lank WHERE ID =" &Request.Querystring("ID")
Set RecSet = connect.execute(SQL)
hemsida = RecSet("hemsida")
SQL = "UPDATE lank SET klick = klick + 1 & " WHERE ID = "& Request.Querystring('ID')
connect.execute SQL
Response.Redirect(hemsida)
End If
%>
jag tror jag tar och citerar LarsG, så slipper han göra det själv.
128 innebär att man inte skapar något recordset överhuvudtaget. Om man inte anger det skapar ado ett tomt och stängt recordset när man gör execute på en fråga som inte returnerar några poster. Det blir en liten prestandafördel på det sättet, vet inte exakt hur mycket det betyder, förmodlugen ganska lite.