login  Naam:   Wachtwoord: 
Registreer je!
 Forum

If AND else positie

Offline Duikboot - 05/09/2004 11:45 (laatste wijziging 05/09/2004 11:46)
Avatar van DuikbootGrafische gevorderde Hallo,
Ik heb een if en else gemaakt, maar ik krijg nu een error met unexpected T_ELSE op line 126 erin. Ik denk dat er een else (of een if) op een verkeerde plek staat, maar zou alleen niet weten waar ik hem anders neer moet zetten. Het is alleen best lang stukje code, heb er daarom even bijgezet waar regel 126 is.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2.  
  4. require("config.php");
  5. include "includes/header.php";
  6. include "config.php"; 
  7. //als de persoon op Submit heeft gedrukt...
  8.  $sql = mysql_query("SELECT auteur FROM reviews"); 
  9.     while($select = mysql_fetch_assoc($sql));
  10. if (auteur == $gebruikersnaam){
  11.  
  12.     if (isset($_POST["Submit"])){
  13.  
  14.  
  15. $query="UPDATE reviews SET  
  16.         auteur = '". $_POST['auteur'] ."',
  17.         email = '". $_POST['email'] . "', 
  18.         titel = '". $_POST['titel'] . "', 
  19.         genre = '". $_POST['genre'] . "', 
  20.         jaartal = '". $_POST['jaartal'] . "', 
  21.         rating = '". $_POST['rating'] . "', 
  22.         verhaal = '". $_POST['verhaal'] . "', 
  23.         review = '". $_POST['review'] . "', 
  24.         mening = '". $_POST['mening'] . "'     
  25.         WHERE id='" .$_POST['id'] ."'"; 
  26.        $result = mysql_query($query) or die ("FOUT: " . mysql_error()); 
  27.     //als er iets fout is gebeurt dan... 
  28.     if ($result){    
  29.         echo ("De review is aangepast.<br>\n"); 
  30.         echo ("<a href=\"review.php?id=id\">Bekijk de review</a>"); //nu gaat hij naar index.php 
  31.     } 
  32. }else{ 
  33. ?> 
  34.  
  35.  
  36.  
  37. <html> 
  38. <head> 
  39. <title>Bewerk review</title> 
  40. </head> 
  41. <body> 
  42. <font size="1" face="Verdana, Arial, Helvetica, sans-serif">Review wijzigen</font> 
  43. <? 
  44. $sql = mysql_query("SELECT * FROM reviews where id=$id"); 
  45. $result= mysql_fetch_array($sql); 
  46. ?> 
  47. <!-- Hier staat het formulier waarmee je de review aanpast --> 
  48. <form action="<?php echo($_SERVER["PHP_SELF"]);?>" method="post"> <font size="1" face="Verdana, Arial, Helvetica, sans-serif"> 
  49.   <input type="hidden" name="bevestiging" value="1"> 
  50.   <input type="hidden" name="id" value="<?php echo($_GET["id"]);?>"> 
  51.   Auteur: <br> 
  52.   <input type="text" name="auteur" value="<?php echo $result[auteur];?>" size="30"> 
  53.   <br> 
  54.   E-mail adres v/d auteur: <br> 
  55.   <input type="text" name="email" value="<?php echo $result[email];?>" size="30"> 
  56.   <br> 
  57.   Titel v/d film: <br> 
  58.   <input type="text" name="titel" value="<?php echo $result[titel];?>" size="30"> 
  59.   <br> 
  60.   Genre:<br> <Select name="genre" value="<?php echo $result[genre];?>"> 
  61. <option value='Aktie/Avontuur'>Aktie/Avontuur</option> 
  62. <option value='Animatie'>Animatie</option> 
  63. <option value='Comedie'>Comedie</option> 
  64. <option value='Drama'>Drama</option> 
  65. <option value='Horror/Thriller'>Horror/Thriller</option> 
  66. <option value='Science Fiction'>Science Fiction</option> 
  67. <option value='Western'>Western</option> 
  68. <option value='Overig'>Overig</option></select><br> 
  69.   <br> 
  70. Jaartal:<br> <Select name="jaartal" value="<?php echo $result[jaartal];?>"> 
  71. <option value='onbekend'>onbekend</option> 
  72. <option value='2004'>2004</option> 
  73. <option value='2003'>2003</option> 
  74. <option value='2002'>2002</option> 
  75. <option value='2001'>2001</option> 
  76. <option value='2000'>2000</option> 
  77. <option value='1999'>1999</option> 
  78. <option value='1998'>1998</option> 
  79. <option value='1997'>1997</option> 
  80. <option value='1996'>1996</option> 
  81. <option value='1995'>1995</option> 
  82. <option value='1994'>1994</option> 
  83. <option value='1993'>1993</option> 
  84. <option value='1992'>1992</option> 
  85. <option value='1991'>1991</option> 
  86. <option value='1990'>1990</option> 
  87. <option value='1989'>1989</option> 
  88. <option value='1988'>1988</option> 
  89. <option value='1987'>1987</option> 
  90. <option value='1986'>1986</option> 
  91. <option value='1985'>1985</option> 
  92. <option value='1984'>1984</option> 
  93. <option value='1983'>1983</option> 
  94. <option value='1982'>1982</option> 
  95. <option value='1981'>1981</option> 
  96. <option value='1980'>1980</option> 
  97. <option value='1979'>1979</option> 
  98. <option value='1978'>1978</option> 
  99. <option value='1977'>1977</option> 
  100. <option value='1976'>1976</option> 
  101. <option value='1975'>1975</option> 
  102. <option value='1974'>1974</option> 
  103. <option value='1973'>1973</option> 
  104. <option value='1972'>1972</option> 
  105. <option value='1971'>1971</option> 
  106. <option value='1970'>1970</option> 
  107. <option value='Prehistorisch'>Prehistorisch</option></select><br> 
  108. <br> 
  109. Rating:<br> <input type="text" name="rating" value="<?php echo $result[rating];?>"><br> 
  110. <br> 
  111. Verhaal:<br> <textarea name="verhaal" value=" rows="7" cols="50"><?php echo $result[verhaal];?></textarea><br> 
  112. <br> 
  113. Review:<br> <textarea name="review" value=" rows="7" cols="50"><?php echo $result[review];?></textarea><br> 
  114. <br> 
  115. Eigen mening:<br> <textarea name="mening" value=" rows="7" cols="50"><?php echo $result[mening];?></textarea><br> 
  116. <br> 
  117. </font> 
  118.  
  119.   <font size="2" face="Verdana, Arial, Helvetica, sans-serif"> 
  120.   <input type="Submit" name="Submit" value="Sla de bewerkte review op"> 
  121.   <input type="Button" value="Vorige pagina" onclick="javascript:history.back();"> 
  122.   </font> 
  123. </form> 
  124.  
  125. <?php // line 126 is hieronder, die else{
  126. else{ 
  127. 	echo "Je kunt je eigen review alleen aanpassen als je ingelogt bent. Ga Terug.";
  128. }	
  129. } 
  130. include "includes/footer.php";
  131. ?>

16 antwoorden

Gesponsorde links
Offline Button - 05/09/2004 11:57 (laatste wijziging 05/09/2004 12:02)
Avatar van Button PHP ver gevorderde volgens mij moet die else er niet.
Offline Thomas - 05/09/2004 12:00 (laatste wijziging 05/09/2004 12:00)
Avatar van Thomas Moderator Als je puur kijkt naar de for's, while's en if's krijg je dit:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. while($select = mysql_fetch_assoc($sql));
  2. if(auteur == $gebruikersnaam){
  3. 	if(isset($_POST["Submit"])){
  4. 		if($result){    
  5. 		}
  6. 	} else {
  7. 		else{ // foute else... typisch ?
  8. 		}    
  9. 	}


Die eerste while is niet nodig. De foute else hoort daar niet en mist een } en de buitenste if mist een } aan het einde.

Als je fatsoenlijk in de code had ingesprongen, had je dit probleem waarschijnlijk niet gehad.
Offline Dennisvb - 05/09/2004 12:01
Avatar van Dennisvb MySQL ver gevorderde @buttonman, ik zie geen inhoud in je post


probeer voor else { eens } te zetten. Die moet je niet vergeten, want een andere else staat nog open en die moet hier eerst gesloten zijn.

Het valt me op dat je 3 ifstatements opent en 2 else. Moeten dit er ook gee 3 zijn?
Offline Duikboot - 05/09/2004 12:02
Avatar van Duikboot Grafische gevorderde Aha,nu zie ik het. Dus die moet weg? Maar volgens mij werkt de if (auteur == $gebruikersnaam) dan niet meer of wel? Want dat is juist de bedoeling, anders kunnen members de reviews van elkaar aanpassen, en da's niet de bedoeling 
Offline Button - 05/09/2004 12:02
Avatar van Button PHP ver gevorderde @Dennisvb: ik weet, foutje
Offline Duikboot - 05/09/2004 12:08
Avatar van Duikboot Grafische gevorderde Ik heb nu voor de onderste else een } gezet zodat die daarboven afgesloten is, maar nogsteeds de unexpected else 
Offline Dennisvb - 05/09/2004 12:22
Avatar van Dennisvb MySQL ver gevorderde Ik vind het trouwens wel een beetje een vreemd script. Boven aan staat:
//als de persoon op Submit heeft gedrukt

dan gaat hij kijken of auteur == gebruikersnaam.
dan wordt er gekeken of de persoon op Submit heeft gedrukt.

Kan je niet beter eerst doet
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if(isset($_POST['submit])) {
?

maar je hebt het volgende:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. //als er iets fout is gebeurt dan... 
  2.     if ($result){     
  3.         echo ("De review is aangepast.<br>\n"); 
  4.         echo ("<a href=\"review.php?id=id\">Bekijk de review</a>"); //nu gaat hij naar index.php 
  5.     }

maak daar eens dit van:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. //als er iets fout is gebeurt dan... 
  2.     if ($result){     
  3.         echo ("De review is aangepast.<br>\n"); 
  4.         echo ("<a href=\"review.php?id=id\">Bekijk de review</a>"); //nu gaat hij naar index.php 
  5.     } elseif(!$result) {
  6. echo "Foutje";
  7. }


En die haakjes zijn niet nodig bij echoën. Je kan gewoon echo "bla"; doen
Offline Thomas - 05/09/2004 12:45 (laatste wijziging 05/09/2004 12:46)
Avatar van Thomas Moderator Je kan geen twee else's na elkaar gebruiken, daarom is ie unexpected...

Is toch ook logisch: een else wil zeggen "voer het volgende blok uit als de stelling in het if-statement niet waar is". Een tweede else zou dus beweren dat het if-statement wel waar was, maar het bijbehorende blok werd toch niet uitgevoerd... Wat is dan het nut van een if-statement ?
Offline Duikboot - 05/09/2004 13:27
Avatar van Duikboot Grafische gevorderde Nu begin ik het een beetje te vatten. Maar hoe zou ik dat dan kunnen doen?
Offline Dennisvb - 05/09/2004 13:32
Avatar van Dennisvb MySQL ver gevorderde zorgen dat je je if statements goed afsluit met een else ?
Offline Duikboot - 05/09/2004 13:37
Avatar van Duikboot Grafische gevorderde Dus dan zo: if {.....}else {......}if{.....}else{...}

want nu heb ik d8 ik zo: if{.....}if{.....}else{.....}else{......}

Pff... net zow ingewikkeld dit (doe ook maar havo )
Offline Thomas - 05/09/2004 13:45 (laatste wijziging 05/09/2004 13:46)
Avatar van Thomas Moderator Er staat commentaar toegevoegd (met een * erbij).
Los de rest zelf op:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  3. require("config.php");
  4. include("includes/header.php");
  5. include("config.php");
  6.  
  7. //als de persoon op Submit heeft gedrukt...
  8. $sql = mysql_query("SELECT auteur FROM reviews");
  9.  
  10. // * controle op resultaten ?
  11. $row = mysql_fetch_assoc($sql));
  12. if($row['auteur'] == $gebruikersnaam) { // * waar komt $gebruikersnaam vandaan ?
  13.   if(isset($_POST["Submit"])) {
  14.     $query="UPDATE reviews SET
  15.         auteur = '". $_POST['auteur'] ."',
  16.         email = '". $_POST['email'] . "',
  17.         titel = '". $_POST['titel'] . "',
  18.         genre = '". $_POST['genre'] . "',
  19.         jaartal = '". $_POST['jaartal'] . "',
  20.         rating = '". $_POST['rating'] . "',
  21.         verhaal = '". $_POST['verhaal'] . "',
  22.         review = '". $_POST['review'] . "',
  23.         mening = '". $_POST['mening'] . "'
  24.         WHERE id='" .$_POST['id'] ."'";
  25.         $result = mysql_query($query) or die ("FOUT: " . mysql_error());
  26.     //als er iets fout is gebeurt dan...
  27.     if($result) {
  28.       // * text voor de <html> tag ?
  29.       echo ("De review is aangepast.<br>\n");
  30.       echo ("<a href=\"review.php?id=id\">Bekijk de review</a>"); //nu gaat hij naar index.php
  31.     }
  32.   } else {
  33.     // * er is geen formulier gesubmit
  34. ?>
  35. <html>
  36. <head>
  37. <title>Bewerk review</title>
  38. </head>
  39. <body>
  40. <font size="1" face="Verdana, Arial, Helvetica, sans-serif">Review wijzigen</font>
  41. <?php
  42.     // * misschien een idee je php ook hier in te springen
  43.     // * als een constante herinnering in welk blok je zit ?
  44.     $sql = mysql_query("SELECT * FROM reviews where id=$id"); // * waar komt $id vandaan ?
  45.     $result = mysql_fetch_array($sql); // * fetch array is dubbel op - gebruik fetch_assoc of fetch_row
  46.     // * gebruik in ieder geval QUOTES om associatieve namen, het zijn immers geen CONSTANTEN
  47. ?>
  48. <!-- Hier staat het formulier waarmee je de review aanpast -->
  49. <form action="<?= $_SERVER['PHP_SELF'] ?>" method="post">
  50. <font size="1" face="Verdana, Arial, Helvetica, sans-serif">
  51. <input type="hidden" name="bevestiging" value="1">
  52. <input type="hidden" name="id" value="<?= $_GET['id'] ?>">
  53. Auteur: <br>
  54. <input type="text" name="auteur" value="<?= $result['auteur'] ?>" size="30">
  55. <br>
  56. E-mail adres v/d auteur: <br>
  57. <input type="text" name="email" value="<?= $result['email'] ?>" size="30">
  58. <br>
  59. Titel v/d film: <br>
  60. <input type="text" name="titel" value="<?= $result['titel'] ?>" size="30">
  61. <br>
  62. Genre:<br>
  63. <!-- * SELECT VELDEN HEBBEN GEEN !!! VALUE !!! -->
  64. <Select name="genre" value="<?= $result['genre'] ?>">
  65. <option value='Aktie/Avontuur'>Aktie/Avontuur</option>
  66. <option value='Animatie'>Animatie</option>
  67. <option value='Comedie'>Comedie</option>
  68. <option value='Drama'>Drama</option>
  69. <option value='Horror/Thriller'>Horror/Thriller</option>
  70. <option value='Science Fiction'>Science Fiction</option>
  71. <option value='Western'>Western</option>
  72. <option value='Overig'>Overig</option>
  73. </select><br>
  74. <br>
  75. Jaartal:<br>
  76. <!-- * SELECT VELDEN HEBBEN GEEN !!! VALUE !!! -->
  77. <Select name="jaartal" value="<?= $result['jaartal'] ?>">
  78. <option value='onbekend'>onbekend</option>
  79. <option value='2004'>2004</option>
  80. <option value='2003'>2003</option>
  81. <option value='2002'>2002</option>
  82. <option value='2001'>2001</option>
  83. <option value='2000'>2000</option>
  84. <option value='1999'>1999</option>
  85. <option value='1998'>1998</option>
  86. <option value='1997'>1997</option>
  87. <option value='1996'>1996</option>
  88. <option value='1995'>1995</option>
  89. <option value='1994'>1994</option>
  90. <option value='1993'>1993</option>
  91. <option value='1992'>1992</option>
  92. <option value='1991'>1991</option>
  93. <option value='1990'>1990</option>
  94. <option value='1989'>1989</option>
  95. <option value='1988'>1988</option>
  96. <option value='1987'>1987</option>
  97. <option value='1986'>1986</option>
  98. <option value='1985'>1985</option>
  99. <option value='1984'>1984</option>
  100. <option value='1983'>1983</option>
  101. <option value='1982'>1982</option>
  102. <option value='1981'>1981</option>
  103. <option value='1980'>1980</option>
  104. <option value='1979'>1979</option>
  105. <option value='1978'>1978</option>
  106. <option value='1977'>1977</option>
  107. <option value='1976'>1976</option>
  108. <option value='1975'>1975</option>
  109. <option value='1974'>1974</option>
  110. <option value='1973'>1973</option>
  111. <option value='1972'>1972</option>
  112. <option value='1971'>1971</option>
  113. <option value='1970'>1970</option>
  114. <option value='Prehistorisch'>Prehistorisch</option>
  115. </select><br>
  116. <br>
  117. Rating:<br> <input type="text" name="rating" value="<?= $result['rating'] ?>"><br>
  118. <br>
  119. Verhaal:<br> <textarea name="verhaal" value=" rows="7" cols="50"><?= $result['verhaal'] ?></textarea><br>
  120. <br>
  121. Review:<br> <textarea name="review" value=" rows="7" cols="50"><?= $result['review'] ?></textarea><br>
  122. <br>
  123. Eigen mening:<br> <textarea name="mening" value=" rows="7" cols="50"><?= $result['mening'] ?></textarea><br>
  124. <br>
  125. </font>
  126.  
  127.   <font size="2" face="Verdana, Arial, Helvetica, sans-serif">
  128.   <input type="Submit" name="Submit" value="Sla de bewerkte review op">
  129.   <input type="Button" value="Vorige pagina" onclick="javascript:history.back();">
  130.   </font>
  131. </form>
  132.  
  133. <?php // line 126 is hieronder, die else
  134.   } // * EINDE geen formulier gesubmit
  135. } else {
  136.   echo "Je kunt je eigen review alleen aanpassen als je ingelogt bent. Ga Terug.";
  137. } // * EINDE gebruikersnaam check
  138. include("includes/footer.php");
  139. ?>
Offline Dennisvb - 05/09/2004 13:47 (laatste wijziging 05/09/2004 13:49)
Avatar van Dennisvb MySQL ver gevorderde Hij heeft volgens mij gewoon een oneven aantal if en else statement.
Offline Duikboot - 05/09/2004 13:49 (laatste wijziging 05/09/2004 13:49)
Avatar van Duikboot Grafische gevorderde Kijk, dat ziet er goed uit, als het me nu niet meer lukt weet ik het ook niet meer. Thx al lot :D 

Ook jij natuurlijk dennisvb 
Offline Dennisvb - 05/09/2004 13:50 (laatste wijziging 05/09/2004 13:51)
Avatar van Dennisvb MySQL ver gevorderde ^^  FangorN is eenmaal veel beter in php
Offline Duikboot - 05/09/2004 14:08
Avatar van Duikboot Grafische gevorderde Ik heb hem aan de praat. Nu weet ik hoe het moet, kan ik de code ook voor andere onderwerpen gebruiken. Iedereen die gereageerd heeft thx a lot 
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.435s