---

Her er kilden:

<%
' SQL sætning opbygges
strSQL = "Insert into annoncer ("

strSQL = strSQL & "amt, "
strSQL = strSQL & "kategori, "
strSQL = strSQL & "navn, "
strSQL = strSQL & "adresse, "
strSQL = strSQL & "postnr, "
strSQL = strSQL & "by, "
strSQL = strSQL & "tlf, "
strSQL = strSQL & "fax, "
strSQL = strSQL & "email, "
strSQL = strSQL & "web, "
strSQL = strSQL & "beskrivelse)"

strSQL = strSQL & " values("

strSQL = strSQL & Request.Form("geovalg") & ", "
strSQL = strSQL & Request.Form("anntypvalg") & ", "
strSQL = strSQL & "'" & Request.Form("navn") & "',"
strSQL = strSQL & "'" & Request.Form("adresse") & "', "
strSQL = strSQL & "'" & Request.Form("postnr") & "', "
strSQL = strSQL & "'" & Request.Form("by") & "', "
strSQL = strSQL & "'" & Request.Form("tlf") & "', "
strSQL = strSQL & "'" & Request.Form("fax") & "', "
strSQL = strSQL & "'" & Request.Form("email") & "', "
strSQL = strSQL & "'" & Request.Form("web") & "', "
strSQL = strSQL & "'" & Request.Form("beskrivelse") & "')"

' SQL sætning eksekveres
'Conn.Execute(strSQL)

Response.Write strSQL

' Luk databaseforbindelse
Conn.Close
Set Conn = Nothing
%>

Jeg kører Win2000...

--
Vil du lære at kode HTML, XHTML, CSS, SSI eller ASP ???
- Pædagogiske tutorials på dansk
- Kom godt i gang med koderne
KLIK HER! => http://www.html.dk/tutorials

---

"Mercury" <petersen_simon@yahoo.dk> wrote in message
news:anug0n$5vo$1@sunsite.dk...
> Her er kilden:
>
> <%
> ' SQL sætning opbygges
> strSQL = "Insert into annoncer ("
>
> strSQL = strSQL & "amt, "
> strSQL = strSQL & "kategori, "
> strSQL = strSQL & "navn, "
> strSQL = strSQL & "adresse, "
> strSQL = strSQL & "postnr, "
> strSQL = strSQL & "by, "
> strSQL = strSQL & "tlf, "
> strSQL = strSQL & "fax, "
> strSQL = strSQL & "email, "
> strSQL = strSQL & "web, "
> strSQL = strSQL & "beskrivelse)"
>
> strSQL = strSQL & " values("
>
> strSQL = strSQL & Request.Form("geovalg") & ", "
> strSQL = strSQL & Request.Form("anntypvalg") & ", "
> strSQL = strSQL & "'" & Request.Form("navn") & "',"
> strSQL = strSQL & "'" & Request.Form("adresse") & "', "
> strSQL = strSQL & "'" & Request.Form("postnr") & "', "
> strSQL = strSQL & "'" & Request.Form("by") & "', "
> strSQL = strSQL & "'" & Request.Form("tlf") & "', "
> strSQL = strSQL & "'" & Request.Form("fax") & "', "
> strSQL = strSQL & "'" & Request.Form("email") & "', "
> strSQL = strSQL & "'" & Request.Form("web") & "', "
> strSQL = strSQL & "'" & Request.Form("beskrivelse") & "')"
>
> ' SQL sætning eksekveres
> 'Conn.Execute(strSQL)
>
> Response.Write strSQL
>
> ' Luk databaseforbindelse
> Conn.Close
> Set Conn = Nothing
> %>
>
> Jeg kører Win2000...
>
> --
> Vil du lære at kode HTML, XHTML, CSS, SSI eller ASP ???
> - Pædagogiske tutorials på dansk
> - Kom godt i gang med koderne
> KLIK HER! => http://www.html.dk/tutorials



Hvordan fejler den ?
Har du husket method=post i <form> taggen ?

Jan

---

On Tue, 8 Oct 2002 11:36:55 +0000 (UTC), Mercury
<petersen_simon@yahoo.dk> wrote:

>Her er kilden:
<snip>
>strSQL = strSQL & "by, "

"by" er et reserveret ord og bør normalt ikke bruges som feltnavn.
Hvis du alligevel gør det, så sæt firkant-parantes om den i
forespørgsler o.l.:
strSQL = strSQL & "[by], "

Desuden:
>strSQL = strSQL & "adresse, "
>strSQL = strSQL & "postnr, "
>strSQL = strSQL & "by, "
>strSQL = strSQL & "tlf, "

Denne kode er ikke særligt effektiv, når du laver *så* mange
streng-konkateneringer (som det vist hedder) - altså lægger så stykker
streng til efterhånden.
I dette tilfælde er det ikke så galt, det ville være langt værre, hvis
det foregik i en loop, hvor metoden evt. gentages mange gange.

Jeg ville foretrække:
strSQL = "Insert into annoncer (amt, kategori, navn, adresse, ...
altså hele strengen på én linie - eller måske blot dele

strSQL = "Insert into annoncer ( <felter> )" & _
" values( <værdier> )

Blot min mening

Good luck!

--
Jørn Andersen,
Brønshøj

---

On Wed, 09 Oct 2002 04:59:05 +0200, Jørn Andersen <jorn@jorna.dk>
wrote:

>Denne kode er ikke særligt effektiv, når du laver *så* mange
>streng-konkateneringer (som det vist hedder) - altså lægger så stykker
>streng til efterhånden.

Der skulle have stået: altså lægger så mange små stykker streng til
efterhånden.

Sorry!

--
Jørn Andersen,
Brønshøj