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:

  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.  
  22. INSERT INTO
  23. `biedingen`
  24. (`van`, `naar`, `speler`, `prijs`)
  25. VALUES
  26. ('$memberid', '$object->team', '".$_GET['id']."', '".$_POST['bod']."')
  27. ") or die (mysql_error());
  28.  
  29.  
  30. $query11 = mysql_query("
  31. SELECT * FROM
  32. members
  33. WHERE
  34. memberid = '$object->team'
  35. ");


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
  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

  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.  
  22. INSERT INTO
  23. `biedingen`
  24. (`van`, `naar`, `speler`, `prijs`)
  25. VALUES
  26. ('.$memberid.', '.$object->team.', '".$_GET['id']."', '".$_POST['bod']."')
  27. ") or die (mysql_error());
  28.  
  29.  
  30. $query11 = mysql_query("
  31. SELECT * FROM
  32. members
  33. WHERE
  34. memberid = '.$object->team.'
  35. ");
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
  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.  
  22. INSERT INTO
  23. `biedingen`
  24. (`van`, `naar`, `speler`, `prijs`)
  25. VALUES
  26. (".$memberid.", ".$object->team.", '".$_GET['id']."', '".$_POST['bod']."')
  27. ") or die (mysql_error());
  28.  
  29.  
  30. $query11 = mysql_query("
  31. SELECT * FROM
  32. members
  33. WHERE
  34. memberid = ".$object->team."
  35. ");
  36. }
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..

  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.264s