Response.Redirect

hvad præcis er "tom"? en tom string eller null?

...eller fordi der ingen rækker er med det anførte "id" ?

så måtte der næsten komme en eof-fejl (og så er der ikke engang nævnt noget om SQL-injection eller manglende lukning af conn) :)

Det er en tom string. Og det jeg har vist er kun en lille del af siden.

Prøv med:

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then ....

Mange tak, det var lige det :-)

Må jeg bede om din hjælp en gang mere:
Jeg er nemlig en amatør til det her så :-)

Denne streng:

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "href="showmapudenbooking.asp?id=<%=id%>""

Ender med denne fejl:

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "href="showmapudenbooking.asp?id=<%=id
------------------------------------------------------------------------------------------------------^

Hvad gør jeg galt nu?

Nils

Du kan ikke have en " inden i en "..." streng på den måde. Du kan have en " i en "..." streng ved at skrive den 2 gange.

Men hvad laver den "href"-dims der? Du skal jo bare

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=<%=id%>"

Jeg får stadig denne fejl:

Microsoft VBScript compilation  error '800a0409'

Unterminated string constant

/showmap.asp, line 127

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=<%=id
-------------------------------------------------------------------------------------------------------------------------------^

Men er problemet ikke nu noget omkring: ?id=<%=id%>"

Nils

Som erikjacobsen skriver så har href= ikke noget at gøre i din kode.

Brug:

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=" & id


Du må ikke bruge <%= %> rundt om din id variabel, da du allerede er inde i en <% %> blok.

Og så bør du omskrive din (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) blok til en funktion, så skal du ikke skrive så meget når du skal teste andre variabler og din kode bliver nemmere at læse, når du skal lave et eller andet om om nogle måneder/år.

f.eks. som her:

Public Function IsEmptyIsNull(S)
    if (S = "") OR (IsEmpty(S) = True) OR (IsNull(S) = True) then
        IsEmptyIsNull = True
    else
        IsEmptyIsNull = False
    end if
End Function

Ved at bruge funktionen her, kan du skrive følgende:

if IsEmptyIsNull(rs("bookid")) Then Response.Redirect("showmapudenbooking.asp?id=" & id)

På den måde sparer du at skrive rs("bookid")) 3 gange med mulighed for skrivefejl.

Må jeg bede om lidt mere hjælp, det ser ikke ud til at viderestillingen sender "ID" med over jeg får nemlig denne fejl nu:

Microsoft OLE DB Provider for ODBC Drivers error '80040e14'

[Microsoft][ODBC Microsoft Access Driver] Syntax error (missing operator) in query expression 'id ='.

/showmapudenbooking.asp, line 125

Nils

Det er meget svært at hjælpe dig, når man ikke har adgang til den kode som du bruger til at kalde siden med dataopslaget.

Og så skal du nok bruge lidt tid på at fejlhåndtere. I dette tilfælde skal du INDEN du begynder at opbygge dit query tjekke om id er et gyldigt tal. Og hvis ikke, så skal du sørge for at der vises en fejlmeddelelse og en mulighed for at komme tilbage.

Kom med noget kode...

Ok jeg prøver,

id = er autonummering i en accessdb som i mit tilfælde er unikt.

Jeg kalder nedenstående side via et link far en anden side på denne måde: <a target="_blank" href="showmap.asp?id=<%=id%>">


<%
Set Conn = Server.CreateObject("ADODB.Connection")
Conn.Open "DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=" & server.MapPath("/fpdb/hid.mdb")

strSQL = "Select * from hotelopl Where id =" & Request.Querystring("id")
Set rs=Conn.Execute(strSQL)

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=" & id

%>

<%
DO WHILE NOT rs.EOF

    id = rs("id")
 

%>

Gør det det lidt mere klart?

I linjen her:

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=" & id

er ID ikke tildelt en værdi og er sikkert heller ikke erklæret som en variabel med DIM, så den indeholder en tilfældi værdi.

I linjerne:

strSQL = "Select * from hotelopl Where id =" & Request.Querystring("id")
Set rs=Conn.Execute(strSQL)

henter du ID med Request.QueryString("id")

Du bør gøre følgende:

<%
Dim id
id = Request.QueryString("id") ' hent id fra querystring ind i variablen id

Set Conn = Server.CreateObject("ADODB.Connection")
Conn.Open "DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=" & server.MapPath("/fpdb/hid.mdb")

strSQL = "Select * from hotelopl Where id =" & id
Set rs=Conn.Execute(strSQL)

if (rs("bookid") = "") OR IsNull(rs("bookid")) OR IsEmpty(rs("bookid")) then Response.Redirect "showmapudenbooking.asp?id=" & id

%>

<%
DO WHILE NOT rs.EOF

    id = rs("id")
%>

Jeg ved ikke hvorfor du henter id fra recorden, den kender du jo allerede, og da du udvælger en record med en unik id, så vil der max, være 1 record. ;-)

Håber det får dig lidt videre.

Mange tak der er perfekt nu.

Som den amatør jeg er, er det her hvad jeg ville opnå:

Jeg har en side hvor man kan vælge at se et kort, men hvis man vælger en record hvor der er en værdi i "bookid" skal man til en side og hvis værdien er tom skal man til en anden side, og dette var hvad jeg kunne finde ud af.

Men endnu engang mange tak for hjælpen.

Nils

Nils, alle er jo startet som amatør og den eneste måde at lære på er ved at forsøge, så bare klø på.

- Kim.