[php]MySQLquery werkt niet in php, rechtstreeks in mysql wel

[Erik]

Voor m'n werk ben ik bezig met een systeempje om klanten in te registreren.
Ik heb daarvoor de volgende code in php geschreven:

PHP:

$insertquery="INSERT INTO Employees(EmployeeID, EmployeeFirstName, EmployeeLastName, EmployeePhone, CompanyID, ActiveDate, PackageID, HasToken, HasBlackberry) VALUES";

while (!empty($_POST["EmployeeFirstName$i"]) && $error==0)  {

  $insertquery.="(NULL,  '$EmployeeFirstName', '$EmployeeLastName', $EmployeePhone, $CompanyID, '$ActiveDate', $PackageID, $HasToken, $HasBlackberry),";

$i++;
}

Bij het uitvoeren van de query, verderop in de code krijg ik de standaard mysql error
Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 2

Alle waarden die niet in single quotes staan zijn numeriek. Input is gecheckt

Nu komt het: Ik heb géén idee wat er fout gaat! Ik heb de output ge-echo't, en die in een terminal met mysql geplakt, en het resultaat is prima! De gegevens worden direct in de database geplaatst.
Als ik echo, krijg ik deze query:

code:

INSERT INTO Employees(EmployeeID, EmployeeFirstName, EmployeeLastName, EmployeePhone, CompanyID, ActiveDate, PackageID, HasToken, HasBlackberry) VALUES
('NULL', 'erik', 'lkjasfd', 12, 18, '2010-12-12', 2, 0, 1),
('NULL', 'erik', 'lkjsaflkjlkj', 12, 18, '2010-12-12', 1, 1, 1),

Het móet iets kleins zijn, maar ik ben al úúúren aan het zoeken en googlen geweest.
Wat ik heb gedaan:
- alle backquotes weggehaald om de tabelnamen (resultaat zie je in de code hierboven
- alles mét backquotes om de tabelnamen
- alle variabelen in single quotes gezet
- een andere query opgesteld, waarbij ook het eerste deel van de insertquery in de while loop zat.
- gegoogled of php misschien iets doet met de quotes
- andere mogelijke oorzaken gegoogled.

Ik ben radeloos! Wie geeft me een zet in de juiste richting?

[418O2]

Eh, je probeert een string met de waarde NULL in het ID veld te zetten?

en eindigt je query met een , (komma)?

[Janoz]

Haal je de laatste komma wel weg?

(en wat doen die quotes rond NULL daar eigenlijk?)

[Y0ur1]

spatie na VALUES in je php? (boeit niet)
Die laatst komma idd ja

[ Voor 45% gewijzigd door Y0ur1 op 28-07-2010 23:10 ]

[Erik]

418O2 schreef op woensdag 28 juli 2010 @ 23:02:
Eh, je probeert een string met de waarde NULL in het ID veld te zetten?

en eindigt je query met een , (komma)?

die waarde 0 is een integer met auto increment.
Die komma zal ik er even uit-toveren, alhoewel dat in mysql niet uitmaakte. Dit is echter idd niet netjes, bedankt. Ik laat hiervan het resultaat weten

---

Janoz schreef op woensdag 28 juli 2010 @ 23:03:
Haal je de laatste komma wel weg?

(en wat doen die quotes rond NULL daar eigenlijk?)

die quotes was ook een poging om te kijken of het daar aan lag.

---

Oh Mijn God.

Het lag gewoon aan de komma.

Ik heb een if-statement toegevoegd die controleert of er wel of geen komma moet komen, waardoor de komma nu niet meer op het einde van de query komt.

Nu werkt het.
Wel een beetje kansloos dit.

Iedereen die mij gewezen heeft op de komma: zéér vriendelijk bedankt.

Ik ga maar gauw in een hoekje staan, me schamen

edit: verdorie, dit heeft me gewoon een dikke 4 uur gekost, en dan blijkt dit het te zijn.

[ Voor 47% gewijzigd door MueR op 28-07-2010 23:41 . Reden: berichten samengevoegd ]

[Cartman!]

En wat heb je dus geleerd? Gewoon simpel debuggen had je al die moeite kunnen besparen.

Doe je verder nog iets tegen SQL-injection?

offtopic:
Het is de bedoeling om je bericht te editten ipv. steeds nieuwe posts te doen als je eigen post de laatste is, die melding onder je post staat er niet voor jan l*l natuurlijk...

* Cartman! schopt MueR

[ Voor 4% gewijzigd door Cartman! op 28-07-2010 23:43 ]

[MueR]

Ga je je dan ook even schamen voor het driedubbelposten? Wil je volgende keer de knop gebruiken alsjeblieft?

Cartman!

[ Voor 3% gewijzigd door MueR op 28-07-2010 23:43 ]

[418O2]

Als je álle velden gaat inserten hoef je ze niet te benoemen.

Als je een autoincrement Id hebt, dan kan je dat veld gewoon weglaten uit je fieldlist

Erik schreef op woensdag 28 juli 2010 @ 23:04:
[...]

die waarde 0 is een integer met auto increment.
Die komma zal ik er even uit-toveren, alhoewel dat in mysql niet uitmaakte. Dit is echter idd niet netjes, bedankt. Ik laat hiervan het resultaat weten

NULL is met apostroffen eromheen gewoon een string, losstaand is het NULL zoals jij het bedoelt.

[ Voor 58% gewijzigd door 418O2 op 28-07-2010 23:55 ]

[Erik]

418O2 schreef op woensdag 28 juli 2010 @ 23:54:
Als je álle velden gaat inserten hoef je ze niet te benoemen.

dit zijn niet alle velden

Als je een autoincrement Id hebt, dan kan je dat veld gewoon weglaten uit je fieldlist

Hmm. Ok!

NULL is met apostroffen eromheen gewoon een string, losstaand is het NULL zoals jij het bedoelt.

Yep. De apostroffen had ik er even neergezet om te testen, aangezien de query wel werkte in mysql, maar niet via php was mijn gedachtegang dat er wellicht iets niet goed ging bij het opstellen van de query. Dit is in mijn codevoorbeeld abusievelijk blijven staan.

Cartman! schreef op woensdag 28 juli 2010 @ 23:42:
En wat heb je dus geleerd? Gewoon simpel debuggen had je al die moeite kunnen besparen.

Hmm. Hoe kan ik sql queries debuggen? ik heb in mijn zoektocht op google ook gekeken naar een syntaxchecker voor sql statements

Doe je verder nog iets tegen SQL-injection?

Ja, ik gebruik mysql_real_escape_string voor elke variabele die gepost kan worden. Overigens draait het intern op ons lan, niet op het internet. (Maar met toekomstig gebruik in het achterhoofd heb ik toch maar voor alle posts mysql_real_escape_string gebruikt)

Verder zeer bedankt voor jullie hulp. Opmerkingen worden gewaardeerd!

MueR schreef op woensdag 28 juli 2010 @ 23:42:
Ga je je dan ook even schamen voor het driedubbelposten?

Helemaal niet gemerkt? Mijn excuses.

[ Voor 36% gewijzigd door Erik op 29-07-2010 00:10 ]

[RobIII]

Erik schreef op donderdag 29 juli 2010 @ 00:00:
Hmm. Hoe kan ik sql queries debuggen? ik heb in mijn zoektocht op google ook gekeken naar een syntaxchecker voor sql statements

Debuggen: Hoe doe ik dat?

[GlowMouse]

418O2 schreef op woensdag 28 juli 2010 @ 23:54:
[...]

NULL is met apostroffen eromheen gewoon een string, losstaand is het NULL zoals jij het bedoelt.

Ja? Zou hij NULL in een primary_key auto_increment-veld willen opslaan? Zo te zien weet hij niet eens het verschil tussen 0, "0" en NULL.

Hmm. Hoe kan ik sql queries debuggen? ik heb in mijn zoektocht op google ook gekeken naar een syntaxchecker voor sql statements

Echo'en vlak voor de mysql-query aanroep en dan zelf uitvoeren.

[ Voor 23% gewijzigd door GlowMouse op 29-07-2010 02:10 ]

[418O2]

GlowMouse schreef op donderdag 29 juli 2010 @ 02:09:
[...]

Ja? Zou hij NULL in een primary_key auto_increment-veld willen opslaan? Zo te zien weet hij niet eens het verschil tussen 0, "0" en NULL.

Ik schrijf zelf al een tijdje geen insert query's meer (Leve NHibernate), maar als je NULL in je ID veld zet bij een insert dan genereert hij toch alsnog zelf een id?

MySQL:

INSERT INTO gebruiker (gebruiker_id, voornaam) VALUES (NULL, 'Max')

Waarbij gebruiker_id een auto increment is, maar dan vult hij wel zelf een id in. Het is niet netjes, maar het werkt wel.

[cariolive23]

Veruit de meeste databases zullen de NULL weigeren op een primary key, dit is een vaag stukje MySQL waardoor het per ongeluk goed gaat. Je wilt hier de default waarde aanroepen, de volgende waarde van de auto_increment. Gebruik dan DEFAULT of laat de kolommen en waarde gewoon weg, dan zal de database ook de default waarde gebruiken.

Maar een NULL opgeven en eigenlijk de DEFAULT bedoelen, dat is heel vaag. Of bestel jij ook een biertje wanneer je eigenlijk een bak koffie wilt hebben?

[MueR]

Dat is helemaal geen vaag stukje MySQL, dat is by design. Het word een beetje flauw om je bij elk topic over MySQL te zien bashen. Reageer dan niet, voer die oorlog tegen MySQL maar ergens anders.

[Orion84]

GlowMouse schreef op donderdag 29 juli 2010 @ 02:09:
[...]
Echo'en vlak voor de mysql-query aanroep en dan zelf uitvoeren.

Dat heeft ie toch ook gedaan? Alleen blijkbaar geeft die komma wel een probleem als het vanuit PHP komt, maar niet bij handmatig uitvoeren van de query.

[Kalentum]

Orion84 schreef op donderdag 29 juli 2010 @ 10:22:
[...]

Dat heeft ie toch ook gedaan? Alleen blijkbaar geeft die komma wel een probleem als het vanuit PHP komt, maar niet bij handmatig uitvoeren van de query.

een komma aan het eind van een statement levert altijd een parse error op, welke client je ook gebruikt. Hij zal die komma wel zijn vergeten ofzo.

[Janoz]

418O2 schreef op woensdag 28 juli 2010 @ 23:54:
Als je álle velden gaat inserten hoef je ze niet te benoemen.

Doe toch maar wel. Het is net als met 'Select *'. Het lijkt wel even lekker snel bij het tikken voor de eerste keer, maar gaat een hel worden bij het onderhoud. Je hebt eigenlijk geen enkele garantie dat de kolommen ook daadwerkelijk in die volgorde blijven en daarnaast moet ik de eerste applicatie nog tegen komen waar nooit meer iets veranderd is aan het database schema na het eerste create table statement.

[Erik]

GlowMouse schreef op donderdag 29 juli 2010 @ 02:09:
[...]

Ja? Zou hij NULL in een primary_key auto_increment-veld willen opslaan? Zo te zien weet hij niet eens het verschil tussen 0, "0" en NULL.

0 is het getal 0, "0" is een string, NULL is geen waarde.

Echo'en vlak voor de mysql-query aanroep en dan zelf uitvoeren.

Heb ik gedaan!

Orion84 schreef op donderdag 29 juli 2010 @ 10:22:
[...]

Dat heeft ie toch ook gedaan? Alleen blijkbaar geeft die komma wel een probleem als het vanuit PHP komt, maar niet bij handmatig uitvoeren van de query.

Juist. Overigens krijg ik wel een parse error als ik het nu probeer te reproduceren

[ Voor 30% gewijzigd door Erik op 29-07-2010 10:52 ]