[PHP] T_ELSE error

[Bellerophon]

Ik heb nu al een tijd lang een php error waar ik mij totaal geen raad mee weet.

Pagina.php

code:

<?
require("config.php");

// Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";

if(!$mid)
{
  $query = mysql_query("SELECT * FROM members");
  $result = mysql_num_rows($query);

  if(isset($_POST['delete']) && $_SERVER['REQUEST_METHOD']= 'post')
  {
    $aantal_verwijderen = count($_POST['deletepic']);

    if(empty($_POST['deleteuser']))
    {
      echo"Er zijn geen gebruikers geselecteerd.";
    }

    for ($i = 1; $i <= $aantal_verwijderen; $i++)
    {
      $sql   = "DELETE FROM members WHERE memberid = '".$_POST['id'][$i -1]."' LIMIT 1";
      /* Verwijder de gebruiker. De variabele $i telt steeds eentje op,
      dus gaat het verwijderen net zolang door tot alle geselecteerde gebruikers verwijderd zijn. */
      $query = mysql_query($sql) or die(mysql_error());
    }
  }
}
else 
{
  if(!$result)
  {
    echo "We hebben nog geen members...";
  }
  else
  {
    ?>
      <form method="post" action="">
      <?
        while($object = mysql_fetch_object($query))
        {
          echo "> <a href=\"?mid=$object->memberid\">$object->naam</a> ( ID: $object->memberid )
          <input type=\"checkbox\" name=\"deleteuser[]\" value=\"".$object->memberid."\">\n<br>";
        }
      ?>
      <input type="submit" name="delete" value="Verwijder geselecteerde gebruikers">
      </form>
    <?
  }
  else ### <--- Klopt niet...
  {
    $query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
    $result = mysql_num_rows($query);

    if(!$result)
    {
      echo "Deze member bestaat niet...";
    }
    else
    {
      $query = mysql_query('UPDATE members SET bekeken = bekeken + 1 WHERE memberid = "'.intval($mid).'"');
      
      while($object = mysql_fetch_object($query))
      {
        ?>
          Je bekijkt nu het profiel van <b><?=htmlspecialchars($object->gebruikersnaam);?></b>
          ( <?=(intval($object->bekeken));?> keer bekeken )
          <hr />
          
          <b>Naam:</b><br/>
          <?=htmlspecialchars($object->naam);?><br/><br/>
          <b>Email:</b>
          <br/>
          <a href="mailto:<?=htmlspecialchars($object->email);?>">
            <?=htmlspecialchars($object->email);?>
          </a>
          <br/><br/>
          
          <b>Homepage:</b><br/>
          <a href="htmlspecialchars($object->homepage)" target="_blank">
            <?=htmlspecialchars($object->homepage);?>
          </a>
          <br/><br/>
          
          <b>Woonplaats:</b><br/>
          <?=htmlspecialchars($object->woonplaats);?>
          <br/><br/>
          
          <b>Geslacht:</b><br/>
          [img]"img/<?=($object->geslacht=='Man'[/img].gif"
               alt="<?=htmlspecialchars($object->geslacht);?>" />
          <br/><br/>
          
          <b>Relatie:</b><br/>
          <?php
          switch($object->relatie)
          {
            case '0': echo 'Nee';break;
            case '1': echo 'Ja'; break;
            default : echo 'Binnenkort';break;
          }
          ?>
          <br/><br/>
          
          <b>Hobby's:</b><br/>
          <?=htmlspecialchars($object->hobbys);?>
          <br/><br/>

          <b>Favo drankje(s):</b><br/>
          <?=htmlspecialchars($object->favodrankje);?>
          <br/><br/>

          <b>Favo artiest(en):</b><br/>
          <?=htmlspecialchars($object->favoartiest);?>
          <br/><br/>

          <b>Favo film(s):</b><br/>
          <?=htmlspecialchars($object->favofilm);?>

          <?php
          
          if($object->pm)
          {
            echo '<hr /><a href="priveberichten.php?new=1&naar='.intval($object->memberid).'">Stuur privé- bericht</a>';
          }
      } # End while
    }
  }
?>

Er staat in aangegeven waar de fout zit, ik heb er al dit geprobeerd van te maken

Aanpassing:

code:

if(!$result){
  echo "We hebben nog geen members...";
  }
  else ### <--- Klopt niet...
  {
    $query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
    $result = mysql_num_rows($query);

Maar dan bleef de error nog steeds staan, te weten onderstaande error

Parse error: parse error, unexpected T_ELSE in /home/martinne/public_html/bob/test3/profiel.php on line 47

Maar als ik die else statement weghaal dan krijg ik dit:

Parse error: parse error, unexpected $ in /home/martinne/public_html/bob/test3/profiel.php on line 134

En dat is dit stuk code

code:

?>

Ik weet dus echt niet wat er fout gaat, weet alleen dat die else statement een error geeft.

[mulder]

Je bruikt meer dan 1 keer else, dus zonder voorwaarde:

PHP:

if
{
...
}
else <-- elseif 
{
...
}
else
{
...
}

[T-MOB]

Je gebruikt twee else's bij een if statement, dat kan natuurlijk niet.


Spuit 11

[ Voor 25% gewijzigd door T-MOB op 05-07-2005 11:18 ]

[Bellerophon]

Dus dit stuk is corrupt?

code:

{
  if(!$result)
  {
    echo "We hebben nog geen members...";
  }
  else
  {
    ?>
      <form method="post" action="">
      <?
        while($object = mysql_fetch_object($query))
        {
          echo "> <a href=\"?mid=$object->memberid\">$object->naam</a> ( ID: $object->memberid )
          <input type=\"checkbox\" name=\"deleteuser[]\" value=\"".$object->memberid."\">\n<br>";
        }
      ?>
      <input type="submit" name="delete" value="Verwijder geselecteerde gebruikers">
      </form>
    <?
  }
    
 
 
  else ### <--- Klopt niet...
  {
    $query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
    $result = mysql_num_rows($query);

Maar het typische is, hij heeft het zo wel gedaan

[whoami]

Wij is 'hij' ?

Een if kan slechts één else hebben.

indien ...
anders
anders

slaat nergens op he...

[mulder]

Nee dat heeft ie niet:
Als het stoplicht op rood staat doe dit,
anders doe dit,
anders doe dit.
Moet zijn:
Als het stoplicht op rood staat doe dit,
anders als het op groen staat doe dit,
anders doe dit.

[Bellerophon]

whoami schreef op dinsdag 05 juli 2005 @ 11:28:
Wij is 'hij' ?

Een if kan slechts één else hebben.

indien ...
anders
anders

slaat nergens op he...

Deze pagina had ik erg standaard gemaakt voor de weergave van gebruikers in een database, na een tijd ging ik aan een verwijder optie werken en daarna kreeg ik de fout, daarvoor stond die dubbele else er ook al in en toen werd de pagina nog wel weergegeven

Maar die else weg halen is ook geen optie want dan geeft hij ook een foutmelding

[sjroorda]

Een fout op de laatste regel (?>) duidt meestal op een niet of verkeerd afgesloten haakje, accolade, aanhalingsteken, etc... Zoek die er eens uit?

[mulder]

Maar die else weg halen is ook geen optie want dan geeft hij ook een foutmelding

Anders lees je even bovenstaande replies.
http://nl3.php.net/manual...php#control-structures.if

[Bellerophon]

Don Facundo schreef op dinsdag 05 juli 2005 @ 11:38:
[...]

Anders lees je even bovenstaande replies.
http://nl3.php.net/manual...php#control-structures.if

Die heb ik ook gelezen en ik besef ook heel goed wat er fout gaat, het vervelende is dat ik dus niet weet hoe ik de fout eruit moet halen.

De pagina zag er in het begin zo uit (de php code dan):

Pagina:

code:

<?
require("config.php");

// Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";

if(!$mid){
$query = mysql_query("SELECT * FROM members");
$result = mysql_num_rows($query);

if(!$result){
echo "We hebben nog geen members...";
}
else{
while($object = mysql_fetch_object($query)){
echo "> <a href=\"?mid=$object->memberid\">$object->naam</a> ( ID: $object->memberid )\n<br>";
}
}
}
else{
$query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
$result = mysql_num_rows($query);

if(!$result){
echo "Deze member bestaat niet...";
}
else{
mysql_query("UPDATE members SET bekeken = bekeken + 1 WHERE memberid = '$mid'");
while($object = mysql_fetch_object($query)){
?>

[BreeeZe]

Bellerophon schreef op dinsdag 05 juli 2005 @ 11:42:
[...]


Die heb ik ook gelezen en ik besef ook heel goed wat er fout gaat, het vervelende is dat ik dus niet weet hoe ik de fout eruit moet halen.

De pagina zag er in het begin zo uit (de php code dan):

Pagina:

PHP:

<?

if(!$mid){
$query = mysql_query("SELECT * FROM members");
$result = mysql_num_rows($query);

     if(!$result){
     echo "We hebben nog geen members...";
     }else{
     while($object = mysql_fetch_object($query)){
     echo "> <a href=\"?mid=$object->memberid\">$object->naam</a> ( ID: $object->memberid  )\n<br>";
    }
// } <-- is die niet te veel ?
}else{
$query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
$result = mysql_num_rows($query);

     if(!$result){
     echo "Deze member bestaat niet...";
     }else{
     mysql_query("UPDATE members SET bekeken = bekeken + 1 WHERE memberid = '$mid'");
     while($object = mysql_fetch_object($query)){
?>

Heb je niet een } te veel (zie bovenstaande aanpassing)
die is van de while...

[ Voor 43% gewijzigd door BreeeZe op 05-07-2005 12:12 ]

[Bosmonster]

Ik zie aan je bovenstaande code uit de openingspost aan het einde al een } missen...

En je hebt ze nog wel netjes ge-indent..

(even uitgaande dat je die dubbele else fixed natuurlijk..)

[ Voor 29% gewijzigd door Bosmonster op 05-07-2005 11:50 ]

[Sybr_E-N]

Bellerophon schreef op dinsdag 05 juli 2005 @ 11:42:

PHP:

<?
require("config.php");

// Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";

if(!$mid) {
  $query = mysql_query("SELECT * FROM members");
  $result = mysql_num_rows($query);

  if(!$result) {
    echo "We hebben nog geen members...";
  } else {
    while($object = mysql_fetch_object($query)) {
      echo "> <a href=\"?mid=$object->memberid\">$object->naam</a> ( ID: $object->memberid          )\n<br>";
    }
  }
} else {
  $query = mysql_query("SELECT * FROM members WHERE memberid = '$mid'");
  $result = mysql_num_rows($query);

  if(!$result) {
    echo "Deze member bestaat niet...";
  } else {
    mysql_query("UPDATE members SET bekeken = bekeken + 1 WHERE memberid = '$mid'");
    while($object = mysql_fetch_object($query)) {
    }
  }
}
?>

Voortaan even goed je source copy-pasten (er misten wat } aan het eind) en indenten, dat maakt het geheel een stuk leesbaarder ook icm php kleurtjes. Maar wat is er nu mis met dit stuk source?

[Bellerophon]

Sybr_E-N schreef op dinsdag 05 juli 2005 @ 11:52:
[...]

Voortaan even goed je source copy-pasten (er misten wat } aan het eind) en indenten, dat maakt het geheel een stuk leesbaarder ook icm php kleurtjes. Maar wat is er nu mis met dit stuk source?

Met die niks, dat was de originele code, die werkte nog wel.
Maar na een paar aanpassingen gaf hij errors?

Daarom probeer ik die fout dus te zoeken.

[WormLord]

Die originele code werkt, omdat elke else daar een bijbehorende if heeft.
Blijkbaar heb je bij je aanpassing een if en/of een accolade teveel verwijderd.

Trouwens, als ik de code van de TS zo even doorneem, dan lijkt het er op dat je niet echt weet waar je mee bezig bent. Zo zit de code voor het weergeven van de member-lijst in het blok dat word uitgevoerd als er een member geselecteerd is. En op regel 32 test je een variabele die helemaal niet gezet is.
Misschien is het goed om eerst vast te leggen wat je wilt doen, in psuedo code of zo, en dan pas gaat de code gaat maken.

[ Voor 59% gewijzigd door WormLord op 05-07-2005 12:49 ]

[RedRose]

Gebruik daarom ook goede indentation om het gebruik van je openings en sluitings accolades overzichtelijk te houden.

[Guldan]

/me Guldan heeft ook nog een goed truckje
Je code lezen er iedere keer bij een { +1 optellen en bij een } -1 eraf halen. Aan het eind moet je dan op 0 uitkomen. Anders klopt er iets niet.
Ook kun je in ultraedit gewoon op accolades zoeken en het aantal open accolades tellen en dit vergelijken met het aantal gesloten. Verder snap ik het probleem niet meer... En een stukje commentaar zou ook wel makkelijk zijn. Wat je met die code wil doen etc. Het gaat dus fout bij die 2e else maar het is niet echt moeilijk op te lossen lijkt mij zo.

[Bellerophon]

WormLord schreef op dinsdag 05 juli 2005 @ 12:36:
Die originele code werkt, omdat elke else daar een bijbehorende if heeft.
Blijkbaar heb je bij je aanpassing een if en/of een accolade teveel verwijderd.

Trouwens, als ik de code van de TS zo even doorneem, dan lijkt het er op dat je niet echt weet waar je mee bezig bent. Zo zit de code voor het weergeven van de member-lijst in het blok dat word uitgevoerd als er een member geselecteerd is. En op regel 32 test je een variabele die helemaal niet gezet is.
Misschien is het goed om eerst vast te leggen wat je wilt doen, in psuedo code of zo, en dan pas gaat de code gaat maken.

Daar ben ik mij van bewust dat de code nog totaal niet netjes enzo is, ben ook nog maar een week ongeveer goed met php bezig.

En ik weet ook dat er een if te weinig is of een else te veel is.
Zou ik kunnen proberen om bepaalde statements te verwijderen?

[NMe]

Sorry hoor, maar dit gaat echt de verkeerde kant op. Parse errors zijn sowieso altijd door simpel debugwerk op te lossen: een haakje teveel of te weinig, een puntkomma teveel op te weinig, een else teveel (wat hier nu het geval is)... Zorg gewoon dat elke else ook één if heeft, elk openingshaakje een sluithaakje, enz. Als dat eenmaal het geval is, dan heb je in elk geval geen parse error meer.

Omdat het verder niet de bedoeling is dat we je hier aan het handje gaan nemen, doe ik dit topic maar op slot. Met de gegeven tips en een goed boek/tutorial moet je er wel uit kunnen komen.