Résolu Problème avec une condition en PHP

Gueckoz SEC · 15 Février 2016

Bonsoir à vous,

Je sais que je trouverais surement pas ma réponse directement à l'heure actuelle mais c'est pas grave.

Je suis entrain d'apprendre les bases de PHP et je viens justement les mettre en pratique pour pouvoir m'améliorer par la suite.

Je commence par créer déjà un espace membre mais je me heurte à un soucis.

Qui est ceci :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

On peut voir dans le screen que le 1e rectangle numéroté 1, je prépare une requête ainsi qu'une condition pour vérifier que l'adresse email existe déjà.

Ensuite, dans le 2eme rectangle numéroté 2, j'affiche l'erreur si le mail existe.

Hors le soucis c'est que ça me fait une error syntaxe de ce genre :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

mon problème doit venir de ma condition que j'ai rajouté qui indique que l'email existe déjà.

Mais ai-je l'ai mal placé ou j'ai oublié un truc ? Car pourtant je pense l'avoir bien mis au bon endroit...

J'avait réussi à ne pas avoir l'error syntaxe en le mettant la :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

mais la condition ne fonctionnait pas malgré le fait d'avoir mis un email déjà valide...

Alors désolé si le code a l'air d'être un peu mal organisé, j'arrangerai tout ça par la suite car je viens d'apprendre les bases de PHP et j'essaye de les mettre en pratique. ^^

Merci d'avance à ceux qui m'aideront à résoudre ce problème qui a l'air tout con d'ailleurs !

HaXGamiG · 15 Février 2016

J'ai du mal a comprend ton else viens de quel if, tu pourrait m'envoyer le code en mp pour y voir plus claire ?

Gueckoz SEC · 15 Février 2016

Le Else vient de ce if la :

"if($mailexist == 0)" mais il se pourrait que j'ai fait une erreur là-dedans aussi. ^^

HaXGamiG · 15 Février 2016

Gueckoz SEC a dit:
Le Else vient de ce if la :

"if($mailexist == 0)" mais il se pourrait que j'ai fait une erreur là-dedans aussi. ^^

Je pense avoirs trouver, send ton code a partir du if qui pose problème jusqu'a la ligne 50 environ je t'envois la correction ^^

HaXGamiG · 15 Février 2016

Gueckoz SEC a dit:
Le Else vient de ce if la :

"if($mailexist == 0)" mais il se pourrait que j'ai fait une erreur là-dedans aussi. ^^

C'est une petite erreur avec les {}

Gueckoz SEC · 15 Février 2016

Voici le code :

Code:

    else
                    {
                        $erreur ="Adresse mail déjà utilisé !";
                    }
                  }
                  else
                  {
                      $erreur = "Votre adresse mail n'est pas valide !";
                  }

C'est de la ligne 44 (la ou y'a mon problème) jusqu'à la ligne 52 (la ou y'a eu la fermeture d'une {}. ^^

HaXGamiG · 15 Février 2016

Gueckoz SEC a dit:
Voici le code :

Code:

else { $erreur ="Adresse mail déjà utilisé !"; } } else { $erreur = "Votre adresse mail n'est pas valide !"; }

C'est de la ligne 44 (la ou y'a mon problème) jusqu'à la ligne 52 (la ou y'a eu la fermeture d'une {}. ^^

Tu a pas fermer ton if avant le else ^^

Code:

    }else
                    {
                        $erreur ="Adresse mail déjà utilisé !";
                    }
                  }
                  else
                  {
                      $erreur = "Votre adresse mail n'est pas valide !";
                  }

Essaye comme ça sa devrais fonctionnez ^^

Gueckoz SEC · 15 Février 2016

Tu penses ?

Car regarde bien :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Du coup ça me provoque une aute erreur mais à la ligne 65 :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

HaXGamiG · 15 Février 2016

Gueckoz SEC a dit:
Tu penses ?

Car regarde bien :
Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Du coup ça me provoque une aute erreur mais à la ligne 65 :
Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Je vois que ça mais je t'avoue que je suis pas 100% sur j'ai du mal a comprendre ton code avec des screens ^^

Gueckoz SEC · 15 Février 2016

HaX GamiG RS a dit:
Je vois que ça mais je t'avoue que je suis pas 100% sur j'ai du mal a comprendre ton code avec des screens ^^

J'te comprends, avec tous ces else et les {} on s'y perd vite

Veux-tu que je t'envoie le fichier source comme ça tu pourras vérifier les différents problèmes ?

A mon avis j'ai peut-être mal géré l’ouverture/fermeture des {}. ^^

HaXGamiG · 15 Février 2016

Gueckoz SEC a dit:
J'te comprends, avec tous ces else et les {} on s'y perd vite

Veux-tu que je t'envoie le fichier source comme ça tu pourras vérifier les différents problèmes ?

A mon avis j'ai peut-être mal géré l’ouverture/fermeture des {}. ^^

Envois simplement le php dans un mp et je vérif tout ça et je te renvois le code en essayant de t'expliquer les erreur

Gueckoz SEC · 15 Février 2016

HaX GamiG RS a dit:
Envois simplement le php dans un mp et je vérif tout ça et je te renvois le code en essayant de t'expliquer les erreur

Je te l'ai envoyé en MP. ^^

Walky 🇫🇷 · 15 Février 2016

Ligne 38-39 tu fermes deux fois le if($mdp == $mdp2) donc cela va fermer le if ($mailexist)

Gueckoz SEC · 15 Février 2016

Walky SEC a dit:
Ligne 38-39 tu fermes deux fois le if($mdp == $mdp2) donc cela va fermer le if ($mailexist)

Ah merci, cependant je me retrouve avec un autre hic, je n'ai plus l'erreur syntaxe mais déjà quand je rentre sur le site de l'inscription, j'ai directement en bas "il y'a des champs qui sont vides..." qui apparaît alors qu'avant non. :/

Ensuite, quand je remplis le formulaire avec le mail qui existe déjà pour vérifier que ma condition fonctionne bien, il me met que les mots de passe ne correspondent pas alors qu'il sont identiques et pas le truc du mail existant.

Voila le code que @HaX GamiG RS m'a fait en MP.

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Le code est un peu un casse-tête j'avoue mais c'est bizarre que ce problème me le fasse alors que j'ai bien suivi l'ouverture/fermeture de {} sur une vidéo d'un mec.

Whyouuu · 15 Février 2016

Gueckoz SEC a dit:
Ah merci, cependant je me retrouve avec un autre hic, je n'ai plus l'erreur syntaxe mais déjà quand je rentre sur le site de l'inscription, j'ai directement en bas "il y'a des champs qui sont vides..." qui apparaît alors qu'avant non. :/

Ensuite, quand je remplis le formulaire avec le mail qui existe déjà pour vérifier que ma condition fonctionne bien, il me met que les mots de passe ne correspondent pas alors qu'il sont identiques et pas le truc du mail existant.

Voila le code que @HaX GamiG RS m'a fait en MP.

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Le code est un peu un casse-tête j'avoue mais c'est bizarre que ce problème me le fasse alors que j'ai bien suivi l'ouverture/fermeture de {} sur une vidéo d'un mec.

Envoie le code ici ça sera plus facile pour t'aider non ? ^^

Gueckoz SEC · 15 Février 2016

Whyouuu a dit:
Envoie le code ici ça sera plus facile pour t'aider non ? ^^

Oui désolé, tiens :

PHP:

<?php
   
$bdd = new pdo('mysql:host=localhost;dbname=espace_membre', 'root','');

if(isset($_POST['forminscription'])){
    $pseudo = htmlspecialchars($_POST['pseudo']);
    $mail = htmlspecialchars($_POST['mail']);
    $mail2 = htmlspecialchars($_POST['mail2']);
    $mdp = sha1($_POST['mdp']);
    $mdp2 = sha1($_POST['mdp2']);
    if(!empty($_POST["pseudo"]) AND !empty($_POST["mail"]) AND !empty($_POST["mail2"]) AND !empty($_POST["mdp"]) AND !empty($_POST["mdp2"])){
        $pseudolength = strlen($pseudo);
        if ($pseudolength <= 12){
            if ($mail == $mail2){
                if(filter_var($mail, FILTER_VALIDATE_EMAIL)){
                    /* Nous allons préparer la requête pour inscrire notre membre dans la Base de Données */
                    $reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                    $reqmail->execute(array($mail));
                    $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas !
                    if($mailexist == 0){
                        if ($mdp ==$mdp2){
                            /* Si tous les critères sont remplis, nous créeons une session et l'utilisateur est inscrit dans la Base de Données */
                            $insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, motdepasse) VALUES(?, ?, ?)");
                            $insertmbr->execute (array($pseudo, $mail, $mdp));
                            $_SESSION['comptecree'] = "Votre compte a bien été créé. Bienvenue sur le site !";
                            header('location: portail.php');
                        }
                    }else{
                        $erreur = "Vos mot de passes ne se correspondent pas ! Veuillez recommencez.";
                    }
                }else{
                    $erreur ="Adresse mail déjà utilisé !";
                }
            }else{
                $erreur = "Votre adresse mail n'est pas valide !";
            }
        }else{
            $erreur = "Vos mails ne se correspondent pas !";
        }
    }else {
        $erreur = "Votre pseudo dépasse les 12 caractères !";
    }
    }else{
        $erreur = "Il y'a des champs qui sont vides, veuillez tous les compléter ! ";
   
}

?>

Kévin 🚀 · 15 Février 2016

Gueckoz SEC a dit:

Oui désolé, tiens :

PHP:

<?php
  
$bdd = new pdo('mysql:host=localhost;dbname=espace_membre', 'root','');

if(isset($_POST['forminscription'])){
    $pseudo = htmlspecialchars($_POST['pseudo']);
    $mail = htmlspecialchars($_POST['mail']);
    $mail2 = htmlspecialchars($_POST['mail2']);
    $mdp = sha1($_POST['mdp']);
    $mdp2 = sha1($_POST['mdp2']);
    if(!empty($_POST["pseudo"]) AND !empty($_POST["mail"]) AND !empty($_POST["mail2"]) AND !empty($_POST["mdp"]) AND !empty($_POST["mdp2"])){
        $pseudolength = strlen($pseudo);
        if ($pseudolength <= 12){
            if ($mail == $mail2){
                if(filter_var($mail, FILTER_VALIDATE_EMAIL)){
                    /* Nous allons préparer la requête pour inscrire notre membre dans la Base de Données */
                    $reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                    $reqmail->execute(array($mail));
                    $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas !
                    if($mailexist == 0){
                        if ($mdp ==$mdp2){
                            /* Si tous les critères sont remplis, nous créeons une session et l'utilisateur est inscrit dans la Base de Données */
                            $insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, motdepasse) VALUES(?, ?, ?)");
                            $insertmbr->execute (array($pseudo, $mail, $mdp));
                            $_SESSION['comptecree'] = "Votre compte a bien été créé. Bienvenue sur le site !";
                            header('location: portail.php');
                        }
                    }else{
                        $erreur = "Vos mot de passes ne se correspondent pas ! Veuillez recommencez.";
                    }
                }else{
                    $erreur ="Adresse mail déjà utilisé !";
                }
            }else{
                $erreur = "Votre adresse mail n'est pas valide !";
            }
        }else{
            $erreur = "Vos mails ne se correspondent pas !";
        }
    }else {
        $erreur = "Votre pseudo dépasse les 12 caractères !";
    }
    }else{
        $erreur = "Il y'a des champs qui sont vides, veuillez tous les compléter ! ";
  
}

?>

Hey

PHP:

if(isset($_POST['forminscription'])){

    //Variables
    $pseudo = htmlspecialchars($_POST['pseudo']);
    $mail   = htmlspecialchars($_POST['mail']);
    $mail2  = htmlspecialchars($_POST['mail2']);
    $mdp    = sha1($_POST['mdp']);
    $mdp2   = sha1($_POST['mdp2']);

    if(!empty($_POST["pseudo"]) AND !empty($_POST["mail"]) AND !empty($_POST["mail2"]) AND !empty($_POST["mdp"]) AND !empty($_POST["mdp2"])){

        $pseudolength = strlen($pseudo);

        if ($pseudolength <= 12){

            if ($mail == $mail2){

                if(filter_var($mail, FILTER_VALIDATE_EMAIL)){

                    /* Nous allons préparer la requête pour inscrire notre membre dans la Base de Données */
                    $reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                    $reqmail->execute(array($mail));
                    $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas !
                    if($mailexist == 0){

                        if ($mdp == $mdp2){
                            /* Si tous les critères sont remplis, nous créeons une session et l'utilisateur est inscrit dans la Base de Données */
                            $insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, motdepasse) VALUES(?, ?, ?)");
                            $insertmbr->execute (array($pseudo, $mail, $mdp));
                            $_SESSION['comptecree'] = "Votre compte a bien été créé. Bienvenue sur le site !";
                            header('location: portail.php');
                        }else{
                            $erreur = "Vos mot de passes ne se correspondent pas ! Veuillez recommencez.";
                        }

                    }else{
                        $erreur ="Adresse mail déjà utilisé !";
                    }
                }else{
                    $erreur = "Votre adresse mail n'est pas valide !";
                }
            }else{
                $erreur = "Vos mails ne se correspondent pas !";
            }
        }else{
            $erreur = "Votre pseudo dépasse les 12 caractères !";
        }
    }else{
           $erreur = "Il y'a des champs qui sont vides, veuillez tous les compléter ! ";
    }
}

Tu avais décalé tes else d'un rang donc les messages n'étaient pas dans les bonnes conditions

Gueckoz SEC · 15 Février 2016

Fara' SEC a dit:

Hey

PHP:

if(isset($_POST['forminscription'])){

    //Variables
    $pseudo = htmlspecialchars($_POST['pseudo']);
    $mail   = htmlspecialchars($_POST['mail']);
    $mail2  = htmlspecialchars($_POST['mail2']);
    $mdp    = sha1($_POST['mdp']);
    $mdp2   = sha1($_POST['mdp2']);

    if(!empty($_POST["pseudo"]) AND !empty($_POST["mail"]) AND !empty($_POST["mail2"]) AND !empty($_POST["mdp"]) AND !empty($_POST["mdp2"])){

        $pseudolength = strlen($pseudo);

        if ($pseudolength <= 12){

            if ($mail == $mail2){

                if(filter_var($mail, FILTER_VALIDATE_EMAIL)){

                    /* Nous allons préparer la requête pour inscrire notre membre dans la Base de Données */
                    $reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                    $reqmail->execute(array($mail));
                    $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas !
                    if($mailexist == 0){

                        if ($mdp == $mdp2){
                            /* Si tous les critères sont remplis, nous créeons une session et l'utilisateur est inscrit dans la Base de Données */
                            $insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, motdepasse) VALUES(?, ?, ?)");
                            $insertmbr->execute (array($pseudo, $mail, $mdp));
                            $_SESSION['comptecree'] = "Votre compte a bien été créé. Bienvenue sur le site !";
                            header('location: portail.php');
                        }else{
                            $erreur = "Vos mot de passes ne se correspondent pas ! Veuillez recommencez.";
                        }

                    }else{
                        $erreur ="Adresse mail déjà utilisé !";
                    }
                }else{
                    $erreur = "Votre adresse mail n'est pas valide !";
                }
            }else{
                $erreur = "Vos mails ne se correspondent pas !";
            }
        }else{
            $erreur = "Votre pseudo dépasse les 12 caractères !";
        }
    }else{
           $erreur = "Il y'a des champs qui sont vides, veuillez tous les compléter ! ";
    }
}

Tu avais décalé tes else d'un rang donc les messages n'étaient pas dans les bonnes conditions

Merci, ça fonctionne cependant j'ai une autre erreur maintenant (encore oui...)

Ca me fait ça :

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Et voila le code à partir de la ligne 22 :

PHP:

 $reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                    $reqmail->execute(array($mail));
                    $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas !
                    if($mailexist == 0){

Kévin 🚀 · 15 Février 2016

Gueckoz SEC a dit:
Merci, ça fonctionne cependant j'ai une autre erreur maintenant (encore oui...)

Ca me fait ça :
Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.

Et voila le code à partir de la ligne 22 :

PHP:

$reqmail=$bdd->prepare("SELECT * FROM membres WHERE mail = ?"); $reqmail->execute(array($mail)); $mailexist = $reqmail->rowCount(); //Nous allons savoir si le mail du membre existe déjà ou pas ! if($mailexist == 0){

Tu as pas supprimé la ligne qui se connecte à la bdd ?

Gueckoz SEC · 15 Février 2016

Fara' SEC a dit:
Tu as pas supprimé la ligne qui se connecte à la bdd ?

Je suis devenu un boulet car oui je l'avais supprimé. :rofl:

Du coup c'est résolu ! Merci !

Résolu Problème avec une condition en PHP

Gueckoz SEC

HaXGamiG

Développeurs

Gueckoz SEC

HaXGamiG

Développeurs

HaXGamiG

Développeurs

Gueckoz SEC

HaXGamiG

Développeurs

Gueckoz SEC

HaXGamiG

Développeurs

Gueckoz SEC

HaXGamiG

Développeurs

Gueckoz SEC

Walky 🇫🇷

Cryptoboy

Gueckoz SEC

Whyouuu

Gueckoz SEC

Kévin 🚀

Administrateur

Gueckoz SEC

Kévin 🚀

Administrateur

Gueckoz SEC