[ASP] Syntax error in INSERT INTO statement

[Verwijderd]

Goedenacht,

Ik kom er even niet meer uit... Wat gaat er mis?

Foutmelding:

Microsoft JET Database Engine error '80040e14'

Syntax error in INSERT INTO statement.

cp/actie.asp, line 87

Line 87 is de op één na onderste regel.

ASP:

Set rsaddprod = Server.CreateObject("ADODB.Recordset")
strSql = "INSERT INTO PRODUCTEN (" &_
"NUMMER," &_
"MERK," &_
"CATEGORIE," &_
"NAAM," &_
"OMSCHRIJVING," &_
"MAAT," &_
"KLEUR) VALUES (" &_
"" & fixQuotes( nummer ) & ", " &_
"" & fixQuotes( merk ) & ", " &_
"" & fixQuotes( categorie ) & ", " &_
"'" & fixQuotes( naam ) & "', " &_
"'" & fixQuotes( omschrijving ) & "', " &_
"'" & fixQuotes( maat ) & "', " &_
"'" & fixQuotes( kleur ) & "')"
set rsaddprod = My_conn.Execute (strSql)
Set rsaddprod = nothing

[ Voor 59% gewijzigd door Verwijderd op 02-02-2005 00:17 ]

[NMe]

Druk de string in strSql maar eens af naar de browser in plaats van de query uit te voeren. Waarschijnlijk zie je dan wel waar het misgaat. Debuggen is ook een kunst.
Je kan moeilijk verwachten dat je hier je code dumpt en wij het zonder ook maar een klein beetje extra info voor je oplossen. Eigen inzet doet geen pijn.

[Suaver]

Vervang "set rsaddprod = My_conn.Execute (strSql)" door "rsaddprod.Execute strSql"
Dat moet zeker werken, maar als je nu een foutmelding krijgt ligt het zeker aan het SQL query.

edit:
Ik heb nog ff je INSERT veranderd naar hoe ik het altijd schrijf, zelf vind ik dit overzichtelijker, maar dat moet je zelf maar even kijken:

strSql = "INSERT INTO PRODUCTEN(NUMMER, MERK, CATEGORIE, NAAM, OMSCHRIJVING, MAAT, KLEUR) VALUES (" & fixQuotes(nummer) & ", " & fixQuotes(merk) & ", " & fixQuotes(categorie) & ", '" & fixQuotes(naam) & "', '" & fixQuotes(omschrijving) & "', '" & fixQuotes(maat) & "', '" & fixQuotes(kleur) & "')"

Weet je trouwens zeker dat alle namen kloppen ? of dat bij "fixQuotes(nummer)" bijvoorbeeld geen quotes bent vergeten ? Probeer dat eens eerst.

[ Voor 65% gewijzigd door Suaver op 02-02-2005 00:38 ]

[NMe]

Suaver schreef op woensdag 02 februari 2005 @ 00:31:
Vervang "set rsaddprod = My_conn.Execute (strSql)" door "rsaddprod.Execute strSql"
Dat moet zeker werken, maar als je nu een foutmelding krijgt ligt het zeker aan het SQL query.

Haakjes boeien niet en die aanpassing zou ook niet consistent zijn met zijn foutmelding. Volgens de melding klopt de query niet, de ASP syntax is in orde.

edit:
Ik heb nog ff je INSERT veranderd naar hoe ik het altijd schrijf, zelf vind ik dit overzichtelijker, maar dat moet je zelf maar even kijken:

strSql = "INSERT INTO PRODUCTEN(NUMMER, MERK, CATEGORIE, NAAM, OMSCHRIJVING, MAAT, KLEUR) VALUES (" & fixQuotes(nummer) & ", " & fixQuotes(merk) & ", " & fixQuotes(categorie) & ", '" & fixQuotes(naam) & "', '" & fixQuotes(omschrijving) & "', '" & fixQuotes(maat) & "', '" & fixQuotes(kleur) & "')"

En dat vind jij overzichtelijker? Zet de zooi liever verdeeld over een aantal regels, dan is de query ook leesbaar. Topicstarter doet dat prima. Verder is het niet handig om voor veldnamen hoofdletters te gebruiken, aangezien die meestal worden gebruikt voor SQL-taalelementen. Het is erg vervelend om die door elkaar te lezen als je het niet op een standaard manier doet.

Weet je trouwens zeker dat alle namen kloppen ?

Als dat niet zo was, dan kreeg hij waarschijnlijk wel een melding dat er een onbekend veld is opgenomen in zijn query.

of dat bij "fixQuotes(nummer)" bijvoorbeeld geen quotes bent vergeten ? Probeer dat eens eerst.

Het lijkt me sterk dat er quotes om een nummer (integer) heen moeten. Volgens mij zijn de velden zonder quotes integervelden. zo niet, dan heb je inderdaad gelijk.

[ Voor 9% gewijzigd door NMe op 02-02-2005 01:32 ]

[OZ-Gump]

Eerste opmerking: gebruik parametrized queries!

Wat voor veldtypes is elk veld en horen daar wel (of geen) quotes omheen? Zoals eerder al gepost: druk je querystring eens af en kijk wat je probeert uit te voeren. Iets meer info over de opbouw van je tabel zou geen overbodige luxe zijn.

[ Voor 13% gewijzigd door OZ-Gump op 02-02-2005 09:36 ]

[wboevink]

-NMe- schreef op woensdag 02 februari 2005 @ 01:30:
[...]
Haakjes boeien niet en die aanpassing zou ook niet consistent zijn met zijn foutmelding. Volgens de melding klopt de query niet, de ASP syntax is in orde.
[...]

Vervang "set rsaddprod = My_conn.Execute (strSql)" door "rsaddprod.Execute strSql"

Het ging ook niet om de haakjes maar meer om de 'set rsaddprod = ' een INSERT INTO zal waarschijnlijk geen recordset returnen. Dan kan de bovenste regel ook weg 'Set rsaddprod = Server.CreateObject("ADODB.Recordset")'

Kun je eens een response.write strSql boven de execute zetten en deze info hier posten?

[ Voor 23% gewijzigd door wboevink op 02-02-2005 09:44 ]

[gorgi_19]

Zie dit over hoe te debuggen in ASP en wat je moet doen met SQL Statements.

Dan kom je er al wel achter dat strings in een SQL Statement omgeven moeten worden door single quotes; deze mis je in jouw SQL Statement.