login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Error

Offline tomk - 18/04/2009 12:37
Avatar van tomkASP interesse Hej mensen ik krijg een error maar ik weet niet wat er fout is.

Error: Parse error: syntax error, unexpected T_ELSE


Script:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. }
  2.  
  3. }
  4.  
  5. if($_GET['bied'] == "ja"){
  6.  
  7. if($_POST['bod'] == "0" || $_POST['bod'] < 0 || $_POST['bod'] == "" ){
  8.  
  9. 	echo "Je hebt geen bod gedaan ";
  10.  
  11. }
  12. elseif(htmlentities($_POST['bod'], ENT_QUOTES)) {
  13. { 
  14.  
  15.     	echo "Je kan enkel getallen invoeren"; 
  16.  
  17. }                     // hier zit de fout
  18. else {              // Hier zit de fout
  19.  
  20. echo "Bod gedaan";
  21.  
  23. 		INSERT INTO 
  24. 		`biedingen` 
  25. 		(`van`, `naar`, `speler`, `prijs`) 
  26. 		VALUES 
  27. 		('$memberid', '$object->team', '".$_GET['id']."', '".$_POST['bod']."')
  28. 	") or die (mysql_error());
  29.  
  30.  
  31. 	$query11 = mysql_query("
  32. 		SELECT * FROM 
  33. 		members 
  34. 		WHERE 
  35. 		memberid = '$object->team'
  36. 	");


Alvast bedankt

8 antwoorden

Gesponsorde links
Offline Ibrahim - 18/04/2009 12:39
Avatar van Ibrahim PHP expert Ten eerste, verzin een betere titel. Ten tweede, je bent een accolade ergens vergeten.
Offline ArieMedia - 18/04/2009 13:33
Avatar van ArieMedia Gouden medaille

PHP ver gevorderde
php code - Bekijk de code zonder highlighting - Klap code in 
  1. elseif(htmlentities($_POST['bod'], ENT_QUOTES)) {
  2. {


Je opent 2x de elseif.
Offline bas1995 - 18/04/2009 16:19 (laatste wijziging 18/04/2009 16:20)
Avatar van bas1995 Onbekend zoals al werd gezegt je opent 2x de else if en je was een paar punten vergeten bij de vars in de query

php code - Bekijk de code zonder highlighting - Klap code in 
  1. }
  2.  
  3. }
  4.  
  5. if($_GET['bied'] == "ja"){
  6.  
  7. if($_POST['bod'] == "0" || $_POST['bod'] < 0 || $_POST['bod'] == "" ){
  8.  
  9.     echo "Je hebt geen bod gedaan ";
  10.  
  11. }
  12. elseif(htmlentities($_POST['bod'], ENT_QUOTES))
  13. { 
  14.  
  15.         echo "Je kan enkel getallen invoeren"; 
  16.  
  17. }                     // hier zit de fout
  18. else {              // Hier zit de fout
  19.  
  20. echo "Bod gedaan";
  21.  
  23.         INSERT INTO 
  24.         `biedingen` 
  25.         (`van`, `naar`, `speler`, `prijs`) 
  26.         VALUES 
  27.         ('.$memberid.', '.$object->team.', '".$_GET['id']."', '".$_POST['bod']."')
  28.     ") or die (mysql_error());
  29.  
  30.  
  31.     $query11 = mysql_query("
  32.         SELECT * FROM 
  33.         members 
  34.         WHERE 
  35.         memberid = '.$object->team.'
  36.     ");
Offline Arnold - 18/04/2009 21:09
Avatar van Arnold Nieuw lid Sluit ook ff de else af op regel 37 ;)
Offline Joost - 18/04/2009 21:12
Avatar van Joost PHP expert
bas1995 schreef:
zoals al werd gezegt je opent 2x de else if en je was een paar punten vergeten bij de vars in de query

[..code..]
Kijk je verbeterde code nog eens na, nu klopt de substitutie van de variabelen in de query al helemaal niet meer..
Offline bas1995 - 18/04/2009 21:23 (laatste wijziging 18/04/2009 21:26)
Avatar van bas1995 Onbekend @ arnold sorry die had ik niet gezien

@medieval wat is er mis met de query

@ mezelf de eerste if moest ook nog afgesloten worden
php code - Bekijk de code zonder highlighting - Klap code in 
  1. }
  2.  
  3. }
  4.  
  5. if($_GET['bied'] == "ja"){
  6.  
  7. if($_POST['bod'] == "0" || $_POST['bod'] < 0 || $_POST['bod'] == "" ){
  8.  
  9.     echo "Je hebt geen bod gedaan ";
  10.  
  11. }
  12. elseif(htmlentities($_POST['bod'], ENT_QUOTES))
  13. { 
  14.  
  15.         echo "Je kan enkel getallen invoeren"; 
  16.  
  17. }}                    // hier zit de fout
  18. else {              // Hier zit de fout
  19.  
  20. echo "Bod gedaan";
  21.  
  23.         INSERT INTO 
  24.         `biedingen` 
  25.         (`van`, `naar`, `speler`, `prijs`) 
  26.         VALUES 
  27.         (".$memberid.", ".$object->team.", '".$_GET['id']."', '".$_POST['bod']."')
  28.     ") or die (mysql_error());
  29.  
  30.  
  31.     $query11 = mysql_query("
  32.         SELECT * FROM 
  33.         members 
  34.         WHERE 
  35.         memberid = ".$object->team."
  36.     "); 
  37. }
Offline ArieMedia - 19/04/2009 02:04 (laatste wijziging 19/04/2009 02:04)
Avatar van ArieMedia Gouden medaille

PHP ver gevorderde		 Laat ik de code maar even iniedergeval beter voor je maken want je doet overbodige controles..

php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($_GET['bied'] == 'ja' && is_numeric($_GET['id'])) {
  2.   if($_POST['bod'] < 0) {
  3.     $txt = 'Je hebt geen bod gedaan';
  4.   } elseif(!is_numeric($_POST['bod']) {
  5.     $txt = 'Je kan enkel getallen invoeren';
  6.   } else {
  7.      $sql = 'INSERT INTO biedingen
  8.               (van, naar, speler, prijs)
  9.               VALUES ('.$memberid.', '.$object->team', '.$_GET['id'].', '.$_POST['bod'].')';
  10.     if(mysql_query($sql)) { $txt = 'Bod gedaan'; } 
  11. }
Offline Wave6 - 19/04/2009 10:54
Avatar van Wave6 PHP beginner vooral met dit soort dingen kun je beter een SWITCH/ CASE statement gebruiken.

Werkt veel overzichtelijker vind ik....
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.259s