<% if isNumeric(Request.Form("pris")&"") AND request.form("update") <> "" then
strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret "
else
if request.form("update") <> "" then Alert = "Du skal angive pris som et tal" end if
end if
strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) %> Kommentar: l_otto 08/06-2006 22:28:11 og så skal jeg have et svar ... sådan jubii og 1000000 tak Accepteret svar: madeindk 08/06-2006 22:28:46 Point: 30,00 Det var godt, vi fik det til at virke :-)
Men her er svaret. Kommentar: l_otto 08/06-2006 22:40:07 ja så skal jeg lige have denne chekker ind som gør at man ikke kan rettet til en pris der allerede er oprettet
strSQL = "Select pris From pris Where pris = '" & replace(Request.Form("pris"),"'","''") & "';"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then
Alert = "Du har allerede oprettet den pris, prøv igen"
<% SQL = "Select pris From pris Where pris = '" & replace(Request.Form("pris"),"'","''") & "';" set rst = DataConn.Execute (strSQL)
do until rst.eof
if Cstr(rst("id")) = Cstr(request.querystring("id")) then Alert = "Du har allerede oprettet den pris, prøv igen" response.redirect("default.asp") end If
<% if isNumeric(Request.Form("pris")&"") AND request.form("update") <> "" then
strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret "
else
if request.form("update") <> "" then Alert = "Du skal angive pris som et tal" end if
end if
strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) %>
Prøv evt. med følgende - kan ikke garantere det virker, da det ikke er testet.
<% if isNumeric(Request.Form("pris")&"") AND request.form("update") <> "" then
' Tjekker om den allerede eksisterer SQL = "Select pris From pris Where pris = '" & replace(Request.Form("pris"),"'","''") & "';" set rst = DataConn.Execute (strSQL)
do until rst.eof
if Cstr(rst("id")) = Cstr(request.querystring("id")) then response.redirect("default.asp") end If
rst.MoveNext Loop
strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret "
else
if request.form("update") <> "" then Alert = "Du skal angive pris som et tal" end if
end if
strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) %>
Undskyld jeg kommenterer lidt på din kode, men synes den er lidt klodset bygget op. Jeg laver lige en mindre omstrukturering her:
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %>
Men prøv det her... det burde virke. den laver et tjek om den pris er på den aktuelle vare i forvejen.
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id") & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %>
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %>
linie 82 = <input type="text" name="pris" value="<%=rs("pris")%>" class="Formindhold" size="40">
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %>
ja min fejl :-) der manglede lige en slut parentes...
prøv den her sql i steden for den du viser... strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id")) & ";"
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") Set rs = DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %> <form method="post" action="retpris.asp?id=<%=request.querystring("id")%>">
skal den vise formen hver gang, eller kun første gang...?
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
else strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) end if %> <form method="post" action="retpris.asp?id=<%=request.querystring("id")%>">
<% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id")) & ";"
set rst = DataConn.Execute (strSQL)
if rst.bof and rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & replace(Request.Form("pris"),"'","''") & "' where id = " & request.querystring("id") DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
end if strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) %> <form method="post" action="retpris.asp?id=<%=request.querystring("id")%>">
hmm.. det er vidst fordi det virker den anden vej af hvad jeg lige havde forstået... <% if request.form("update") <> "" then if isNumeric(Request.Form("pris")&"") then strSQL = "Select pris From pris Where pris = " & CInt(Request.Form("pris")) & " AND id = " & CInt(request.form("id")) & ";"
set rst = DataConn.Execute (strSQL)
if not rst.eof then alert ="Du har allerede oprettet den pris, prøv igen." else strSQL = "update pris set pris='" & CInt(Request.Form("pris")) & "' where id = " & request.querystring("id") DataConn.Execute(strSQL)
Alert = "Prisen er nu opdateret " end if else Alert = "Du skal angive pris som et tal" end if
end if strSQL = "Select * From pris Where Id = " & Request.Querystring("id") Set rs = DataConn.Execute(strSQL) %> <form method="post" action="retpris.asp?id=<%=request.querystring("id")%>">
men kan sige du har fået det til at virke.. 23:32:58 siger den.. at prisen er allerede oprettet og man kan ikke taste bogstaver.(men så kan den ikke update den pris man er ved at rette. 23:31:01 nu kan man god update men den skriver ikke at prisen allerede er oprettet hvis man prøver at rette prs 250 til 255 (og 255 allerede eksisterer)
så du har fået det til at virke bare ikke i samme kode
Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.