[SQL & ASP] SQL syntax fout door variabelnaam

[Verwijderd]

Hoe moet je een variablenaam (het is of een integer of een string) in een SQL query krijgen? Ik weet niet zeker of dat het probleem is, maar ik zal het hieronder verder uitleggen:

Aan de hand van welk forum er in de querystring staat worden de posts uit de table gehaald. Daarbij worden alleen de bovenste posts eruit gehaald, dus sneller. Daarnaast worden ze negatief gerangschikt op postId:

Dim QuerystringForumId
QuerystringForumId = Request.Querystring("forum")

objAcc.Execute("SELECT * FROM Posts WHERE ForumId = QuerystringForumId AND PostPlace = 0 ORDER BY TopicId DESC")

Ik krijg bij de bovenstaande query devolgende foutmelding:

Microsoft OLE DB Provider for ODBC Drivers fout '80040e10'

[Microsoft][ODBC Microsoft Access-stuurprogramma] Too few parameters. Expected 1.

/webscripts/forum/subs.asp, regel 16

Op microsoft.com hebben ze het over niet bestaande records, maar echt alles bestaat, dus wat is er mis? doei

tmmk

[whoami]

Verwijderd schreef op 19 September 2002 @ 19:58:
Hoe moet je een variablenaam (het is of een integer of een string) in een SQL query krijgen? Ik weet niet zeker of dat het probleem is, maar ik zal het hieronder verder uitleggen:

Aan de hand van welk forum er in de querystring staat worden de posts uit de table gehaald. Daarbij worden alleen de bovenste posts eruit gehaald, dus sneller. Daarnaast worden ze negatief gerangschikt op postId:

Dim QuerystringForumId
QuerystringForumId = Request.Querystring("forum")

objAcc.Execute("SELECT * FROM Posts WHERE ForumId = QuerystringForumId AND PostPlace = 0 ORDER BY TopicId DESC")

Ik krijg bij de bovenstaande query devolgende foutmelding:

Microsoft OLE DB Provider for ODBC Drivers fout '80040e10'

[Microsoft][ODBC Microsoft Access-stuurprogramma] Too few parameters. Expected 1.

/webscripts/forum/subs.asp, regel 16

Op microsoft.com hebben ze het over niet bestaande records, maar echt alles bestaat, dus wat is er mis? doei

tmmk

Too few parameters wil zeggen dat er een syntaxfout in je query zit.
In jouw geval is dat, dat je de variablenamen gebruikt in je query terwijl je eigenlijk de inhoud van die variablen moet gebruiken.
Dit is eigenlijk redelijk basic programming practice:

code:

sqlstr = "SELECT * FROM tabel WHERE id = " & ForumId & " 
             AND Name = '& blaat & "'";

[Verwijderd]

Thnx, ik wist niet dat dat in SQL queries nodig was. doei

tmmk

[robjanssen]

Probeer het eens zo:

ASP.NET Visual Basic:

Dim strSQL
Dim rsTmp

strSQL = "SELECT * " _
           & "FROM Posts " _
           & "WHERE ForumId = " + CStr(Request.Querystring("forum")) + " " _
           & "AND PostPlace = 0 " _
           & "ORDER BY TopicId DESC"
Set rsTmp = Server.CreateObject("ADODB.Recordset")
rsTmp.Ooen strSQL, objAcc, 3, 1
------code-------
rsTmp.Close
Set rsTmp = Nothing

[whoami]

me dunkt dat jij eens dringend een cursus/tutorial SQL en ASP bij de hand moet nemen...

[robjanssen]

Wie ikke?

[gorgi_19]

robjanssen schreef op 20 september 2002 @ 00:11:
Wie ikke?

Ik gok tmmk

Het enige minder handige in je code wat ik kan ontdekken is:

code:

CStr(Request.Querystring("forum"))

Ik weet ook niet zeker of + altijd goed gaat; persoonlijk gebruik ik &

[farlane]

gorgi_19 schreef op 20 september 2002 @ 00:36:
[/code]

Ik weet ook niet zeker of + altijd goed gaat; persoonlijk gebruik ik &

Dat is gebruikelijk in VB code. Het kan gebeuren dat je denkt dat je twee strings aan het plakken bent, terwijl VB vind dat ie twee integers moet optellen en dat naar een string moet omzetten.

[Crazy D]

Anders gezegt, met een + kijkt VB(script) of de 2 strings getallen zijn, en zo ja, gaat ie rekenen. & is daarom _altijd_ beter omdat dat per definitie strings-aan-elkaar-plakken () is.

[whoami]

robjanssen schreef op 20 september 2002 @ 00:11:
Wie ikke?

Neen, tmmk.

SQL Tutorial

[robjanssen]

Als je CStr() gebruikt mag je ook een + gebruiken.

edit:

& moest + zijn

[Crazy D]

Da's waar. Maar als je nou gewoon aanleerd dat + tellen is en & strings-plakken dan kunnen dat soort stomme foutjes (1 + 1 = 2 ipv 1 & 1 = 11) iig niet voor

[edit]las verkeerd. & mag altijd aangezien vbscript zelf wel ff uitzoekt welk datatype het over gaat. Bij + zal ie in 1e instantie kijken of het om numerieke waardes gaat, bij & behandelt ie ze sowieso als string. Al is Cstr() gebruiken wel netjes.