---

Hej, jeg har dette script:

<%
if session.contents("login") <> 1 then
response.redirect "admin.asp?login=nej"
end if

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

strSQL = "Update news set "
strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "',"
strSQL = strSQL & " Where id = " & Request.Form("id")


Conn.Execute(strSQL)   

Conn.Close
Set Conn = Nothing
%>
Indlægget er rettet!



Men den kommer med fejlen:
Der er en syntaksfejl i UPDATE-sætningen.
/nyhedssystem/retresponse.asp, line 18


Har læst:
Error in update statement, plz hjælp

Men fik ikke så meget ud af det....

Håber i vil give mig den rigtige kode...

MaRk!
- www.markjensen.dk


--
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

---

MaRk! skrev:

> strSQL = "Update news set "
> strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "',"
> strSQL = strSQL & " Where id = " & Request.Form("id")

Fjern kommaet før Where.

> Har læst:
> Error in update statement, plz hjælp

Det var sjovt - for det er præcis den samme fejl der var årsagen.
Har du læst <news:Xns9238E901D889Dspamstocholmdk@192.38.208.81>?

Eller hvad med <news:Xns9238727491075jcdmfdk@193.88.15.213>

Når du får en SQL-fejl er det _altid_ en god ide at lave et print
af den samlede SQL-sætning - dvs. Response.Write strSQL inden du
fyrer den af. Det er lettere både for dig selv og os andre at finde
fejlen når man har en konkret (samlet) sql-sætning at forholde sig
til.


--
Jens Gyldenkærne Clausen
MF (medlem af FIDUSO - www.fiduso.dk)
I ovenstående tekst benyttes nyt komma.

---

Jens Gyldenkærne Clausen wrote in
dk.edb.internet.webdesign.serverside.asp:
> MaRk! skrev:
>
> > strSQL = "Update news set "
> > strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> > strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> > strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "',"
> > strSQL = strSQL & " Where id = " & Request.Form("id")
>
> Fjern kommaet før Where.
>
> > Har læst:
> > Error in update statement, plz hjælp
>
> Det var sjovt - for det er præcis den samme fejl der var årsagen.
> Har du læst <news:Xns9238E901D889Dspamstocholmdk@192.38.208.81>?
>
> Eller hvad med <news:Xns9238727491075jcdmfdk@193.88.15.213>
>
> Når du får en SQL-fejl er det _altid_ en god ide at lave et print
> af den samlede SQL-sætning - dvs. Response.Write strSQL inden du
> fyrer den af. Det er lettere både for dig selv og os andre at finde
> fejlen når man har en konkret (samlet) sql-sætning at forholde sig
> til.
>
>
> --
> Jens Gyldenkærne Clausen
> MF (medlem af FIDUSO - www.fiduso.dk)
> I ovenstående tekst benyttes nyt komma.


--
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

---

Jens Gyldenkærne Clausen wrote in
dk.edb.internet.webdesign.serverside.asp:
> MaRk! skrev:
>
> > strSQL = "Update news set "
> > strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> > strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> > strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "',"
> > strSQL = strSQL & " Where id = " & Request.Form("id")
>
> Fjern kommaet før Where.
>
> > Har læst:
> > Error in update statement, plz hjælp
>
> Det var sjovt - for det er præcis den samme fejl der var årsagen.
> Har du læst <news:Xns9238E901D889Dspamstocholmdk@192.38.208.81>?
>
> Eller hvad med <news:Xns9238727491075jcdmfdk@193.88.15.213>
>
> Når du får en SQL-fejl er det _altid_ en god ide at lave et print
^ Fatter ik hva du mener
> af den samlede SQL-sætning - dvs. Response.Write strSQL inden du
> fyrer den af. Det er lettere både for dig selv og os andre at finde
> fejlen når man har en konkret (samlet) sql-sætning at forholde sig
> til.
>
>
> --
> Jens Gyldenkærne Clausen
> MF (medlem af FIDUSO - www.fiduso.dk)
> I ovenstående tekst benyttes nyt komma.





strSQL = "Update news set "
strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "'"
strSQL = strSQL & " Where id = " & Request.Form("id")


Der er en syntaksfejl, fordi der mangler en operator. i
forespørgselsudtrykket "id =".
/nyhedssystem/retresponse.asp, line 19

--
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

---

MaRk! wrote in dk.edb.internet.webdesign.serverside.asp:
> Jens Gyldenkærne Clausen wrote in
> dk.edb.internet.webdesign.serverside.asp:
> > MaRk! skrev:
> >
> > > strSQL = "Update news set "
> > > strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> > > strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> > > strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "',"
> > > strSQL = strSQL & " Where id = " & Request.Form("id")
> >
> > Fjern kommaet før Where.
> >
> > > Har læst:
> > > Error in update statement, plz hjælp
> >
> > Det var sjovt - for det er præcis den samme fejl der var årsagen.
> > Har du læst <news:Xns9238E901D889Dspamstocholmdk@192.38.208.81>?
> >
> > Eller hvad med <news:Xns9238727491075jcdmfdk@193.88.15.213>
> >
> > Når du får en SQL-fejl er det _altid_ en god ide at lave et print
> ^ Fatter ik hva du mener
> > af den samlede SQL-sætning - dvs. Response.Write strSQL inden du
> > fyrer den af. Det er lettere både for dig selv og os andre at finde
> > fejlen når man har en konkret (samlet) sql-sætning at forholde sig
> > til.
> >
> >
> > --
> > Jens Gyldenkærne Clausen
> > MF (medlem af FIDUSO - www.fiduso.dk)
> > I ovenstående tekst benyttes nyt komma.
>
>
>
>
>
> strSQL = "Update news set "
> strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "'"
> strSQL = strSQL & " Where id = " & Request.Form("id")
>
>
> Der er en syntaksfejl, fordi der mangler en operator. i
> forespørgselsudtrykket "id =".
> /nyhedssystem/retresponse.asp, line 19
>
> --
> 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





Hmmm... Nu virker det! Næsten...

strSQL = "Update news set "
strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
strSQL = strSQL & "skrevetaf= '" & Request.Form("3")
strSQL = strSQL & " Where id = '" & Request.Form("id")

Nu gemmer den bare : det_man_skrev_i_formen Where id =

Og det er jo ik så godt :P


--
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

---

MaRk! skrev:

>> strSQL = "Update news set "
>> strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
>> strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
>> strSQL = strSQL & "skrevetaf= '" & Request.Form("3") & "'"
>> strSQL = strSQL & " Where id = " & Request.Form("id")

Du har stadig ikke lavet et debug-print.

>> Der er en syntaksfejl, fordi der mangler en operator. i
>> forespørgselsudtrykket "id =".
>> /nyhedssystem/retresponse.asp, line 19

Noget tyder på at du ikke får hentet noget med Request.Form("id").
Prøv at lave debug-print på den også (hvis ikke fejlen er
indlysende - f.eks. at dit id-felt hedder noget andet end "id").

> strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> strSQL = strSQL & "skrevetaf= '" & Request.Form("3")
> strSQL = strSQL & " Where id = '" & Request.Form("id")

Det ser ud som om du mangler et afsluttende anførselstegn - eller
også har du et for meget.


> Nu gemmer den bare : det_man_skrev_i_formen Where id =

Ikke forstået. Kom med et debug-print eller få en anden til at
hjælpe.

PS: Kan du lokkes til at læse min signatur.

--
Jens Gyldenkærne Clausen
Svar venligst under det du citerer, og citer kun det der er
nødvendigt for at forstå dit svar i sammenhængen. Se hvorfor og
hvordan på http://usenet.dk/netikette/citatteknik.html

---

Er det et screenshot du vil ha? www.markjensendk/screeenshot.jpg

Det gemmer den.. med koden:

strSQL = "Update news set "
strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
strSQL = strSQL & "skrevetaf= '" & Request.Form("3")
strSQL = strSQL & " Where id = '" & Request.Form("id")


Formen ser sådan ud:

Response.write "<form method=post action=retresponse.asp>Overskrift:
<textarea name=1>" & rs("overskrift") & "</textarea><br>"
Response.write "Nyhed: <textarea name=2>" & rs("nyhed") &
"</textarea><br>"
Response.write "Skrevet af: <textarea name=3>" & rs("skrevetaf") &
"</textarea><br>"
Response.write "ID: <input type=text disabled name=id value=" &
rs("id") & "><br>"
Response.write "<input type=submit value=Ret></form>"

--
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

---

MaRk! skrev:

> Er det et screenshot du vil ha?

Nej - jeg vil have dig til at udskrive værdien af strSQL
umiddelbart før din .execute-linje. Skift conn.execute(strSQL) ud
med Response.Write(strSQL) - eller sæt r.w.-linjen ind umiddelbart
før den anden.

> www.markjensendk/screeenshot.jpg

Du skriver vist lidt hurtigt. Efter at have indsat et punktum og
slettet et e nåede jeg frem til:
http://www.markjensen.dk/screenshot.jpg

Men det hjælper faktisk lidt. Du har glemt en afsluttende pling i
"skrevetaf"-linjen, men til gengæld en for meget i "where id"-
linjen. Din strSQL kommer til at se nogenlunde sådan ud:

UPDATE news....,
skrevetaf = '<indhold af form("3")> Where id = '
<indhold af Form("id")>


Det sidste anførselstegn (pling) skal flyttes hen efter <indhold af
form("3")>.

Du bør i øvrigt escape indholdet af dine strengkonstanter for
plinger. Din UPDATE-sætning vil fejle hvis jeg f.eks. prøver at
sætte Mc'Brien ind i [skrevetaf]. Lav en aspfunktion der erstatter
alle plinger med dobbelte plinger (ikke dobbelte anførselstegn, men
['] => ['][']).

Når du benytter <textarea> skal du i øvrigt også være opmærksom på
at den resulterende sqlsætning kan blive længere end databasen kan
håndtere. Det kan du komme uden om ved at benytte en ADO-cursor i
stedet for en updatesætning.

--
Jens Gyldenkærne Clausen
Svar venligst under det du citerer, og citer kun det der er
nødvendigt for at forstå dit svar i sammenhængen. Se hvorfor og
hvordan på http://usenet.dk/netikette/citatteknik.html

---

MaRk! wrote in dk.edb.internet.webdesign.serverside.asp:
> Er det et screenshot du vil ha? www.markjensendk/screeenshot.jpg

jeg tror at det Jens gerne vil have er at du efter du har opbygget din
SQL-streng skriver

> strSQL = "Update news set "
> strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> strSQL = strSQL & "skrevetaf= '" & Request.Form("3")
> strSQL = strSQL & " Where id = '" & Request.Form("id")

response.write strSQL
response.end

og som Jens også skrev ... så ser det ud til, at du manger en
afsluttende glyph (') i din SQL-sætning.

Prøv at udskifte linien
strSQL = strSQL & " Where id = '" & Request.Form("id")

med
strSQL = strSQL & " Where id = '" & Request.Form("id") & "'"

og se om det ikke virker.

> Response.write "<form method=post action=retresponse.asp>Overskrift:
> <textarea name=1>" & rs("overskrift") & "</ textarea><br>"

det er en god idé at huske citations-tegn ved attribut-værdierne i dine
elementer. Dvs at dit textarea bør skrives som:

<textarea name="1"><textarea>

og _ikke_ <textarea name=1><textarea>

:)

/Jesper Stocholm - http://stocholm.dk

--
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

---

Jesper Stocholm wrote in dk.edb.internet.webdesign.serverside.asp:
> MaRk! wrote in dk.edb.internet.webdesign.serverside.asp:
> > Er det et screenshot du vil ha? www.markjensendk/screeenshot.jpg
>
> jeg tror at det Jens gerne vil have er at du efter du har opbygget din
> SQL-streng skriver
>
> > strSQL = "Update news set "
> > strSQL = strSQL & "overskrift= '" & Request.Form("1") & "',"
> > strSQL = strSQL & "nyhed= '" & Request.Form("2") & "',"
> > strSQL = strSQL & "skrevetaf= '" & Request.Form("3")
> > strSQL = strSQL & " Where id = '" & Request.Form("id")
>
> response.write strSQL
> response.end
Gjort!

>
> og som Jens også skrev ... så ser det ud til, at du manger en
> afsluttende glyph (') i din SQL-sætning.
>
> Prøv at udskifte linien
> strSQL = strSQL & " Where id = '" & Request.Form("id")
>
> med
> strSQL = strSQL & " Where id = '" & Request.Form("id") & "'"
gjort!
>
> og se om det ikke virker.
>
> > Response.write "<form method=post action=retresponse.asp>Overskrift:
> > <textarea name=1>" & rs("overskrift") & "</ textarea><br>"
>
> det er en god idé at huske citations-tegn ved attribut-værdierne i dine
> elementer. Dvs at dit textarea bør skrives som:
>
> <textarea name="1"><textarea>
har brugt ' og ' nu
>
> og _ikke_ <textarea name=1><textarea>
>
> :)
>
> /Jesper Stocholm - http://stocholm.dk
>


Men... Nu gemmer den ikke mere!

Den tekst jeg får når jeg skriver det der response.write strSQL er:
Update news set overskrift= 'test',nyhed= 'test',skrevetaf= 'test' Where
id = '35'

Skal jeg fjerne det?
Har prøvet... men så kommer serveren med endnu en fejl:
Datatyperne stemmer ikke overens i kriterieudtrykket.
/nyhedssystem/retresponse.asp, line 19

Linje 19:
Conn.Execute(strSQL)

--
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

---

MaRk! skrev:

> Men... Nu gemmer den ikke mere!

Det kan godt være den ikke vil gemme når du har skrevet strSQL ud.
Men du skal naturligvis også kun skrive ud når du leder efter fejl.
Bagefter kan linjen udkommenteres eller slettes.

> Den tekst jeg får når jeg skriver det der response.write
> strSQL er: Update news set overskrift= 'test',nyhed=
> 'test',skrevetaf= 'test' Where id = '35'

id er sandsynligvis et talfelt. Slet anførselstegnene på begge
sider af værdien (her 35).

Det ville i øvrigt være lettere at læse dine svar hvis du klippede
lidt mere i citaterne.

--
Jens Gyldenkærne Clausen
Svar venligst under det du citerer, og citer kun det der er
nødvendigt for at forstå dit svar i sammenhængen. Se hvorfor og
hvordan på http://usenet.dk/netikette/citatteknik.html

---

Jens Gyldenkærne Clausen wrote in
dk.edb.internet.webdesign.serverside.asp:


> id er sandsynligvis et talfelt. Slet anførselstegnene på begge
> sider af værdien (her 35).
Prøver jeg :)


> Det ville i øvrigt være lettere at læse dine svar hvis du klippede
> lidt mere i citaterne.

Hmmm... ok... Har jeg gjort her... :) Er ik så god til det her, ved ik
om det er fordi det ser anderledes ud når man læser det på HTML.dk og
ikke et usenetprogram...

MaRk!
- www.markjensen.dk

--
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

---

DET VIRKER :D:D:D

Systemet bliver brugt på www.hg.markjensen.dk

Skal til at lave designet i newssystemet om (såen at det passer til
HTML-Gear)

--
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

---

MaRk! skrev:

>> Det ville i øvrigt være lettere at læse dine svar hvis du
>> klippede lidt mere i citaterne.
>
> Hmmm... ok... Har jeg gjort her... :)

Det er meget, meget bedre - tak.


> Er ik så god til det her, ved ik om det er fordi det ser
> anderledes ud når man læser det på HTML.dk og ikke et
> usenetprogram...

Det er bestemt ikke utænkeligt (jeg har ikke set hvordan det ser ud
på html.dk). Men teknikken med at klippe i citater og svare
nedenunder er afprøvet gennem mange år på usenet - og den er
særdeles praktisk når man følger med i mange diskussioner.
Sådan som du har svaret i herover er helt fint. Det er altid en god
idé at have en tom linje mellem citatet og ens eget svar - så er
det lettere at adskille visuelt.

Du kan evt. kigge på www.usenet.dk hvis du vil vide mere om
nyhedsgrupperne i dk.*-hierarkiet.

--
Jens Gyldenkærne Clausen
Svar venligst under det du citerer, og citer kun det der er
nødvendigt for at forstå dit svar i sammenhængen. Se hvorfor og
hvordan på http://usenet.dk/netikette/citatteknik.html

---

Jens Gyldenkærne Clausen wrote in
dk.edb.internet.webdesign.serverside.asp:

> Du kan evt. kigge på www.usenet.dk hvis du vil vide mere om
> nyhedsgrupperne i dk.*-hierarkiet.
ok

> Jens Gyldenkærne Clausen
tak for hjælpen :D, begge to

--
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

---

On Wed, 26 Jun 2002 12:30:35 +0200, "Jens Gyldenkærne Clausen"
<jens@gyros.invalid> wrote:

<SNIP>
>Fjern kommaet før Where.
<SNIP>

>--
>Jens Gyldenkærne Clausen
>MF (medlem af FIDUSO - www.fiduso.dk)
>I ovenstående tekst benyttes nyt komma.

Nååh, det er det "nyt komma" går ud på ...

- Undskyld, ku' ikke la være :)


Mvh.

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

---

Jørn Andersen <jorn@jorna.dk> skrev:

>>Fjern kommaet før Where.

>>I ovenstående tekst benyttes nyt komma.
>
> Nååh, det er det "nyt komma" går ud på ...

Ikke helt. Kommaer i programmeringssprog skal bare sættes så
syntaks og semantik er korrekt - dér er ethvert sprogligt
kommasystem irrelevant.
Men du kan da læse om det ny komma her:
<http://www.dsn.dk/folder.htm>

> - Undskyld, ku' ikke la være :)

Alt forladt

FUT langt væk fra denne gruppe (dk.snak.off-topic)
(svar på dette indlæg henvises automatisk til nævnte gruppe - jeg
følger _ikke_ med)
--
Jens Gyldenkærne Clausen
MF (medlem af FIDUSO - www.fiduso.dk)
I ovenstående tekst benyttes nyt komma.