login  Naam:   Wachtwoord: 
Registreer je!
 Forum

SQL-fout

Offline BigBug - 28/05/2006 14:04
Avatar van BigBugPHP expert
  1. <?php
  2. if($qQuery = mysql_query("SELECT * FROM pm WHERE naar='".$_SESSION['username']."' WHERE gelezen='0'"))
  3. {
  4. $iCount = mysql_num_rows($qQuery);
  5. if($iCount !== 0)
  6. {
  7. echo "Je hebt ".$iCount." nieuwe berichten.";
  8. }
  9. }
  10. else
  11. {
  12. echo "Er ging iets fout bij de PM-controle:".mysql_error()."";
  13. }
  14. ?>

Er ging iets fout bij de PM-controle: 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 'WHERE gelezen='0'' at line 1

Probleem spreekt voor zich lijkt me 

Bvd, Jeroen

8 antwoorden

Gesponsorde links
Offline nielsvdwal - 28/05/2006 14:06
Avatar van nielsvdwal PHP gevorderde geen '' om de 0 heen?
Offline ThAlmighty - 28/05/2006 14:09
Avatar van ThAlmighty HTML beginner ja gebruikt 2x where. Volgens mij gaat dat niet. Porbeer dit is:
  1. "SELECT * FROM pm WHERE naar='".$_SESSION['username']."' AND gelezen='0'"
Offline Simon - 28/05/2006 14:11
Avatar van Simon PHP expert @niels: rond integers horen GEEN quotes
je hebt 2 keer WHERE
Offline webstab - 28/05/2006 14:35 (laatste wijziging 28/05/2006 14:41)
Avatar van webstab PHP ver gevorderde
  1. <?php
  2. if($qQuery = mysql_query("SELECT COUNT(*) FROM pm WHERE naar='".$_SESSION['username']."' AND gelezen=0"))
  3. {
  4. $iCount = mysql_result($qQuery, 0);
  5. if($iCount != 0)
  6. {
  7. echo "Je hebt ".$iCount." nieuwe berichten.";
  8. }
  9. }
  10. else
  11. {
  12. echo "Er ging iets fout bij de PM-controle:".mysql_error()."";
  13. }
  14. ?>

Zoiets? Ik raad je aan om count te gebruiken als je enkel het aantal wilt weten. Dat gaat meestal wat sneller

@ Simon: Nu dat je het zegt
Offline Simon - 28/05/2006 14:37 (laatste wijziging 28/05/2006 15:20)
Avatar van Simon PHP expert volgens mij heb je ook een = teveel ...
if($iCount !== 0)
dus die moet weg

@hieronder: verkeerd begrepen
Offline nielsvdwal - 28/05/2006 14:55
Avatar van nielsvdwal PHP gevorderde @simon.. ik zeg ook niet dat die erom moete.. maar die heeft hij eromheen staan.. ik bedoelde: die '' om de 0 weghalen?
Offline nemesiskoen - 28/05/2006 15:42
Avatar van nemesiskoen Gouden medaille

PHP expert
webstab:
"count(1) ipv count(*)" werkt volgens mij nog sneller, maar bij jou is er inderdaad al een enorme verbetering.
Offline Maarten - 28/05/2006 15:47
Avatar van Maarten Erelid Die quotes moeten er wel om als het om enum gaat.
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.336s