login  Naam:   Wachtwoord: 
Registreer je!
 Forum

[Opgelost]syntax error, unexpected ';'

Offline crasher - 20/08/2006 12:35 (laatste wijziging 20/08/2006 13:19)
Avatar van crasherLid ik krijg een fout die ik maar niet vind wat het probleem is waarschijnlijk is het iets doms maar vind het niet

Parse error: syntax error, unexpected ';' in /customers/the-falcons.be/the-falcons.be/httpd.www/admin/members.php on line 195

http://www.plaatscode.be/2142/

hopelijk ziet iemand het

8 antwoorden

Gesponsorde links
Offline Stefan14 - 20/08/2006 12:44 (laatste wijziging 20/08/2006 12:50)
Avatar van Stefan14 PHP gevorderde De fout zegt letterlijk het volgende, er staat hier ergens een puntkomma, maar die verwacht ik hier niet, en is dus een fout. Je zal dus moeten zoeken naar een puntkommma die niet op zijn plaats staat.

ik zie zogauw nog geen fout, maar dat komt misschien ook omdat je niet echt overzichtelijk programmeert.
neem nou dit
  1. <?php
  2. mysql_query("UPDATE members SET username = '".$username."', pass = '".sha1(md5($paswoord)."', lvl = '".$lvl."', naam = '".$naam."', adres = '".$adres."', mail = '".$mail."', tel = '".$tel."', fax = '".$fax."', gsm = '".$gsm."', foto = '".$foto."', functie = '".$functie."', leden = '".$leden."', contact = '".$contact."', bestuur = '".$bestuur."', speelt = '".$speelt."', speler = '".$speler."', actief = '".$actief."' WHERE naam = '".$name."'") or die(mysql_error());
  3. ?>


ik kan daar niet echt makkelijk in zoeken.
Dit is al een stuk overzichtelijker.

  1. <?php
  2. mysql_query("UPDATE members
  3. SET username = '".$username."',
  4. pass = '".sha1(md5($paswoord)."',
  5. lvl = '".$lvl."',
  6. naam = '".$naam."',
  7. adres = '".$adres."',
  8. mail = '".$mail."',
  9. tel = '".$tel."',
  10. fax = '".$fax."',
  11. gsm = '".$gsm."',
  12. foto = '".$foto."',
  13. functie = '".$functie."',
  14. leden = '".$leden."',
  15. contact = '".$contact."',
  16. bestuur = '".$bestuur."',
  17. speelt = '".$speelt."',
  18. speler = '".$speler."',
  19. actief = '".$actief."'
  20. WHERE naam = '".$name."'
  21. ")


en dit stukje is ook niet echt overzichtelijk. Beter is het zo.

  1. <?php
  2. if ( $leden == "on" )
  3. {
  4. $leden = 1;
  5. }
  6. else
  7. {
  8. $leden = 0;
  9. }
  10.  
  11. if ( $contact == "on" )
  12. {
  13. $contact = 1;
  14. }
  15. else
  16. {
  17. $contact = 0;
  18. }
  19.  
  20. if ( $bestuur == "on" )
  21. {
  22. $bestuur = 1;
  23. }
  24. else
  25. {
  26. $bestuur = 0;
  27. }
  28.  
  29. if ( $speelt == "on" )
  30. {
  31. $speelt = 1;
  32. }
  33. else
  34. {
  35. $speelt = 0;
  36. }
  37.  
  38. if ( $actief == "on" )
  39. {
  40. $actief = 1;
  41. }
  42. else
  43. {
  44. $actief = 0;
  45. }
  46. ?>


Leer jezelf alsjeblieft dit soort dingen aan omdat het het foutzoeken zoveel vergemakkelijkt, en ook als je later nog eens terug moet kijken op de code is het veel makkelijker te begrijpen.
Offline crasher - 20/08/2006 12:48
Avatar van crasher Lid ik had het eerst zo staan maar wanneer ik dien sha1(md5()) weg haal heb ik dat probleem niet meer en had het eerst op een andere lijn dus had het in query zlef gezet nu terug op oude plaats en het werkt

zeer raar maar toch bedankt nu ga ik alles op zen normale manier zetten
Offline Stefan14 - 20/08/2006 12:51
Avatar van Stefan14 PHP gevorderde nu je het zegt inderdaad, daar zit in ieder geval al een fout, je hebt namelijk een haakje te weinig.
Offline crasher - 20/08/2006 13:15
Avatar van crasher Lid nu zie ik het ook thx dat was de fout waarschijnlijk die ik dan altijd kreeg

zie zo'n dinge overzie ik altijd 
Offline Simon - 20/08/2006 13:19
Avatar van Simon PHP expert
  1. <?php
  2. if ( $leden == "on" )
  3. {
  4. $leden = 1;
  5. }
  6. else
  7. {
  8. $leden = 0;
  9. }
  10.  
  11. if ( $contact == "on" )
  12. {
  13. $contact = 1;
  14. }
  15. else
  16. {
  17. $contact = 0;
  18. }
  19.  
  20. if ( $bestuur == "on" )
  21. {
  22. $bestuur = 1;
  23. }
  24. else
  25. {
  26. $bestuur = 0;
  27. }
  28.  
  29. if ( $speelt == "on" )
  30. {
  31. $speelt = 1;
  32. }
  33. else
  34. {
  35. $speelt = 0;
  36. }
  37.  
  38. if ( $actief == "on" )
  39. {
  40. $actief = 1;
  41. }
  42. else
  43. {
  44. $actief = 0;
  45. }
  46. ?>

dit kan uiteraard veel korter...
  1. <?php
  2. $leden = ( $leden == "on" ) ? 1 : 0;
  3. $contact = ( $contact == "on" ) ? 1 : 0;
  4. $bestuur = ( $bestuur == "on" ) ? 1 : 0;
  5. $speelt = ( $speelt == "on" ) ? 1 : 0;
  6. $actief = ( $actief == "on" ) ? 1 : 0;
  7. ?>
Offline Stefan14 - 20/08/2006 13:20
Avatar van Stefan14 PHP gevorderde @Simon, die notatie kende ik ook nog niet, die moet ik eens proberen te onthouden, kan nog wel eens handig zijn.
Offline Simon - 20/08/2006 13:25
Avatar van Simon PHP expert het is heel simpel hoor:
$var = (voorwaarde) ? voorwaarde_is_true : voorwaarde_is_false;

5 regels code ipv 44, het mijne lijkt me toch iets handiger 
Offline crasher - 20/08/2006 13:33
Avatar van crasher Lid @simon: thx voor de tip zal ik zeker gebruiken en weer wat bij geleerd 
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.343s