T_DEFAULT

[gamichan]

hello I have a big problem with my code to my forum I do not understand too much I try everything I had nothing to do I give you the code and hope you can help me...

<?php
session_start();
$titre="Profil";
include("includes/identifiants.php");
include("includes/debut.php");
include("includes/menu.php");
//On récupère la valeur de nos variables passées par URL
$action = isset($_GET['action'])?htmlspecialchars($_GET['action']):'consulter';
$membre = isset($_GET['m'])?(int) $_GET['m']:'';
?>

<?php
//On regarde la valeur de la variable $action
switch($action)
{
    //Si c'est "consulter"
    case "consulter":
       //On récupère les infos du membre
       $query=$db->prepare('SELECT membre_pseudo, membre_avatar,
       membre_email, membre_msn, membre_signature, membre_siteweb, membre_post,
       membre_inscrit, membre_localisation
       FROM forum_membres WHERE membre_id=:membre');
       $query->bindValue(':membre',$membre, PDO::PARAM_INT);
       $query->execute();
       $data=$query->fetch();

   //On affiche les infos sur le membre
   echo '<p><i>Vous êtes ici</i> : <a href="./index.php">Index du forum</a> --> 
   profil de '.stripslashes(htmlspecialchars($data['membre_pseudo']));
   echo'<h1>Profil de '.stripslashes(htmlspecialchars($data['membre_pseudo'])).'</h1>';

   echo'<img src="./images/avatars/'.$data['membre_avatar'].'"
   alt="Ce membre n a pas d avatar" />';

   echo'<p><strong>Adresse E-Mail : </strong>
   <a href="mailto:'.stripslashes($data['membre_email']).'">
   '.stripslashes(htmlspecialchars($data['membre_email'])).'</a><br />';

   echo'<strong>MSN Messenger : </strong>'.stripslashes(htmlspecialchars($data['membre_msn'])).'<br />';

   echo'<strong>Site Web : </strong>
   <a href="'.stripslashes($data['membre_siteweb']).'">'.stripslashes(htmlspecialchars($data['membre_siteweb'])).'</a>
   <br /><br />';

   echo'Ce membre est inscrit depuis le
   <strong>'.date('d/m/Y',$data['membre_inscrit']).'</strong>
   et a posté <strong>'.$data['membre_post'].'</strong> messages
   <br /><br />';
   echo'<strong>Localisation : </strong>'.stripslashes(htmlspecialchars($data['membre_localisation'])).'
   </p>';
   $query->CloseCursor();
   break;
?>
<?php
    //Si on choisit de modifier son profil
    case "modifier":
    if (empty($_POST['sent'])) // Si on la variable est vide, on peut considérer qu'on est sur la page de formulaire
    {
        //On commence par s'assurer que le membre est connecté
        if ($id==0) erreur(ERR_IS_NOT_CO);

    //On prend les infos du membre
    $query=$db->prepare('SELECT membre_pseudo, membre_email,
    membre_siteweb, membre_signature, membre_msn, membre_localisation,
    membre_avatar
    FROM forum_membres WHERE membre_id=:id');
    $query->bindValue(':id',$id,PDO::PARAM_INT);
    $query->execute();
    $data=$query->fetch();
    echo '<p><i>Vous êtes ici</i> : <a href="./index.php">Index du forum</a> --> Modification du profil';
    echo '<h1>Modifier son profil</h1>';

    echo '<form method="post" action="voirprofil.php?action=modifier" enctype="multipart/form-data">


    <fieldset><legend>Identifiants</legend>
    Pseudo : <strong>'.stripslashes(htmlspecialchars($data['membre_pseudo'])).'</strong><br />       
    <label for="password">Nouveau mot de Passe :</label>
    <input type="password" name="password" id="password" /><br />
    <label for="confirm">Confirmer le mot de passe :</label>
    <input type="password" name="confirm" id="confirm"  />
    </fieldset>

    <fieldset><legend>Contacts</legend>
    <label for="email">Votre adresse E_Mail :</label>
    <input type="text" name="email" id="email"
    value="'.stripslashes($data['membre_email']).'" /><br />

    <label for="msn">Votre adresse MSN :</label>
    <input type="text" name="msn" id="msn"
    value="'.stripslashes($data['membre_msn']).'" /><br />

    <label for="website">Votre site web :</label>
    <input type="text" name="website" id="website"
    value="'.stripslashes($data['membre_siteweb']).'" /><br />
    </fieldset>

    <fieldset><legend>Informations supplémentaires</legend>
    <label for="localisation">Localisation :</label>
    <input type="text" name="localisation" id="localisation"
    value="'.stripslashes($data['membre_localisation']).'" /><br />
    </fieldset>

    <fieldset><legend>Profil sur le forum</legend>
    <label for="avatar">Changer votre avatar :</label>
    <input type="file" name="avatar" id="avatar" />
    (Taille max : 10 ko)<br /><br />
    <label><input type="checkbox" name="delete" value="Delete" />
    Supprimer l avatar</label>
    Avatar actuel :
    <img src="./images/avatars/'.$data['membre_avatar'].'"
    alt="pas d avatar" />

    <br /><br />
    <label for="signature">Signature :</label>
    <textarea cols="40" rows="4" name="signature" id="signature">
    '.stripslashes($data['membre_signature']).'</textarea>


    </fieldset>
    <p>
    <input type="submit" value="Modifier son profil" />
    <input type="hidden" id="sent" name="sent" value="1" />
    </p></form>';
    $query->CloseCursor();   
}   
else //Sinon on est dans la page de traitement
{
    //Traitement (voir plus bas)
    $mdp_erreur = NULL;
$email_erreur1 = NULL;
$email_erreur2 = NULL;
$msn_erreur = NULL;
$signature_erreur = NULL;
$avatar_erreur = NULL;
$avatar_erreur1 = NULL;
$avatar_erreur2 = NULL;
$avatar_erreur3 = NULL;

//Encore et toujours notre belle variable $i :p
$i = 0;
$temps = time(); 
$signature = $_POST['signature'];
$email = $_POST['email'];
$msn = $_POST['msn'];
$website = $_POST['website'];
$localisation = $_POST['localisation'];
$pass = md5($_POST['password']);
$confirm = md5($_POST['confirm']);


//Vérification du mdp
if ($pass != $confirm || empty($confirm) || empty($pass))
{
     $mdp_erreur = "Votre mot de passe et votre confirmation diffèrent ou sont vides";
     $i++;
}

//Vérification de l'adresse email
//Il faut que l'adresse email n'ait jamais été utilisée (sauf si elle n'a pas été modifiée)

//On commence donc par récupérer le mail
$query=$db->prepare('SELECT membre_email FROM forum_membres WHERE membre_id =:id'); 
$query->bindValue(':id',$id,PDO::PARAM_INT);
$query->execute();
$data=$query->fetch();
if (strtolower($data['membre_email']) != strtolower($email))
{
    //Il faut que l'adresse email n'ait jamais été utilisée
    $query=$db->prepare('SELECT COUNT(*) AS nbr FROM forum_membres WHERE membre_email =:mail');
    $query->bindValue(':mail',$email,PDO::PARAM_STR);
    $query->execute();
    $mail_free=($query->fetchColumn()==0)?1:0;
    $query->CloseCursor();
    if(!$mail_free)
    {
        $email_erreur1 = "Votre adresse email est déjà utilisé par un membre";
        $i++;
    }

    //On vérifie la forme maintenant
    if (!preg_match("#^[a-z0-9A-Z._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$#", $email) || empty($email))
    {
        $email_erreur2 = "Votre nouvelle adresse E-Mail n'a pas un format valide";
        $i++;
    }
}
//Vérification de l’adresse MSN
if (!preg_match("#^[a-z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$#", $msn) && !empty($msn))
{
    $msn_erreur = "Votre nouvelle adresse MSN n'a pas un format valide";
    $i++;
}

//Vérification de la signature
if (strlen($signature) > 200)
{
    $signature_erreur = "Votre nouvelle signature est trop longue";
    $i++;
}


//Vérification de l'avatar

if (!empty($_FILES['avatar']['size']))
{
    //On définit les variables :
    $maxsize = 30072; //Poid de l'image
    $maxwidth = 100; //Largeur de l'image
    $maxheight = 100; //Longueur de l'image
    //Liste des extensions valides
    $extensions_valides = array( 'jpg' , 'jpeg' , 'gif' , 'png', 'bmp' );

    if ($_FILES['avatar']['error'] > 0)
    {
    $avatar_erreur = "Erreur lors du tranfsert de l'avatar : ";
    }
    if ($_FILES['avatar']['size'] > $maxsize)
    {
    $i++;
    $avatar_erreur1 = "Le fichier est trop gros :
    (<strong>".$_FILES['avatar']['size']." Octets</strong>
    contre <strong>".$maxsize." Octets</strong>)";
    }

    $image_sizes = getimagesize($_FILES['avatar']['tmp_name']);
    if ($image_sizes[0] > $maxwidth OR $image_sizes[1] > $maxheight)
    {
    $i++;
    $avatar_erreur2 = "Image trop large ou trop longue :
    (<strong>".$image_sizes[0]."x".$image_sizes[1]."</strong> contre
    <strong>".$maxwidth."x".$maxheight."</strong>)";
    }

    $extension_upload = strtolower(substr(  strrchr($_FILES['avatar']['name'], '.')  ,1));
    if (!in_array($extension_upload,$extensions_valides) )
    {
            $i++;
            $avatar_erreur3 = "Extension de l'avatar incorrecte";
    }
}
break;

default; //Si jamais c'est aucun de ceux-là c'est qu'il y a eu un problème :o
echo'<p>Cette action est impossible</p>';

} //Fin du switch
?>
</div>
</body>
</html>

Parse error: syntax error, unexpected T_DEFAULT in C:\xampp\htdocs\voirprofil.php on line 244

[dalecosp]

You've obviously not finished some case statement prior to the default case properly, so PHP is expecting something else before DEFAULT: and doesn't see it.

It might help to comment out half the code and test, and then the other half, and gradually break it down into smaller sections. You've got some parse error: a missing semicolon, an extra brace (or too few braces), etc.

The trick, of course, is knowing WHERE in something as long as that.

[bradgrafelman]

Actually, I don't even see "default:" in the code at all. Note, however, that I do see "default;" (which is most certainly not the same as the former).

I also see other parsing problems, such as this:

} //Fin du switch 

on line #247 having an incorrect code comment, since it's actually the closing brace for this:

    else //Sinon on est dans la page de traitement 
    { 

brace on line #128. The switch() block opened by the brace on line #15 is never closed.

[johanafm]

dalecosp;10991076 wrote:

The trick, of course, is knowing WHERE in something as long as that.

One good way of getting a better ovewview is to break out the code in the respective cases and place it inside functions, or create a class to handle these things, so that you can

case "consulter":
	$md = new MemberData($member);
	# Note that createOutput only creates output. It doen't echo it directly like
	# you did in your code example. This way, you can decide wether to just
	# collect or echo the information.
	echo $md->createOutput();
	break;
case "...":