PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
W

Rare SQL syntax error!

Door Wouter op 07-10-2009 21:45
1.065 views
Hallo, PHPhulp

Ik dus deze simpele functie:


<?
function AddMatch($team1, $punten1, $kd1, $team2, $punten2, $kd2) {
	$sql = "UPDATE poule SET punten=" . $punten1 . ", kd=" . $kd1 . " WHERE team='" . $team1 . "'";
	$query = mysql_query($sql) or die("Error at AddMatch(Team 1)(): " . mysql_error());
	
	$sql2 = "UPDATE poule SET punten=" . $punten2 . ", kd=" . $kd2 . " WHERE team='" . $team2 . "'";
	$query2 = mysql_query($sql2) or die("Error at AddMatch(Team 2)(): " . mysql_error());
}
?>


Maar ik krijg deze foutmelding:

Error at AddMatch(Team 2)(): 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 ' kd=6 WHERE team='Joep & Berend'' at line 1

De 2 queries zijn gewoon identiek. En de inhoud heeft dezelfde structuur. Enige verklaring? Alvast bedankt!
- SanThe -
07-10-2009 21:51
Echo de query eens.
W
Wouter
07-10-2009 21:53
Hoe bedoel je?
- SanThe -
07-10-2009 21:56
Jelmer schreef op 07.10.2009 21:54
echo "$query";

Vanwaar al die quotes?

En het gaat om $sql2.
Dus echo $sql2; zou logischer zijn.
P
Pieter ten Boomen
07-10-2009 21:58
had nog niet zn hele post gelezen dus wist niet welke hij wilde, en zo heb ik t geleerd... word hier ondertussen helemaal gek van die """""
W
Wouter
07-10-2009 21:59
UPDATE poule SET punten=, kd=6 WHERE team='Joep & Berend'
Ik zie de fout al, bedankt! ;-)

EDIT: Had in de form een fout gemaakt. Er was 2 keer punten1
Jesper Diovo
07-10-2009 22:08
@Jelmer: is het je fout aangeleerd dan. Om een variabele horen geen quotes. Je hoort variabelen juist buiten quotes te halen.

<?php
echo "$query"; // is dus fout
echo $query; // is dus goed

echo "Hoi ik ben $naam, $leeftijd jaar oud. $groet"; // fout
echo "Hoi ik ben ".$naam.", ".$leeftijd." jaar oud. ".$groet; // goed

md5("$string"); // fout
md5($string); // goed
?>
P
Pieter ten Boomen
07-10-2009 22:50
Oke, dan weet ik dat, maar heeft dat ook een reden ofzo?
Ariën
07-10-2009 23:01
Jelmer schreef op 07.10.2009 22:50
Oke, dan weet ik dat, maar heeft dat ook een reden ofzo?

Logica en snelheid en leesbaarheid.

Reageren

Inloggen om te reageren