Membre existe déjà

Résolu
papyclic -  
Bonjour
Je souhaiterai faire une dernière condition à savoir éviter d'insérer un membre déjà présent dans ma BDD.
S'ils vous plait pouvez jeter un oeil sur mon script pour me dire ce qui ne va pas en vous remerciant.
En fait je passe pas la deuxième condition "$erreur = 'Ce membre existe déjà !';"
même si le membre n'existe pas encore, je ne sais vraiment pas ou je me suis trompé.
Merci beaucoup 
	public function verif() {

		$requete = $this->bdd->prepare("SELECT * FROM  tb_adherent WHERE pseudo = '$this->pseudo'");
		$requete->execute();
		$result = $requete->fetchAll();
		
		$requete2 = $this->bdd->prepare("SELECT COUNT(*) FROM  tb_adherent WHERE nom = '$this->nom' AND prenom = '$this->prenom'");
		$requete2->execute();
		$result2 = $requete2->fetchAll();

		if(empty($this->nom) || empty($this->prenom) || empty($this->pseudo) || empty($this->password) || empty($this->debut)) {
			$erreur = 'Veuillez remplir tous les champs';
			return $erreur;
		} elseif (count($result2) > 0 ) {
			$erreur = 'Ce membre existe déjà !';
			return $erreur;	
		} elseif (count($result) > 0 ) {
			$erreur = 'Ce Login existe déjà, veuillez en choisir un autre !';
			return $erreur;			
		} else {
			return 'success';
		}
	}

6 réponses

  1.  jordane45 Modérateur
    Bonjour,

    Essayes ça : 

    <?php

public function verif() {
$resultat = "";
  if(empty($this->nom) || empty($this->prenom) || empty($this->pseudo) || empty($this->password) || empty($this->debut)) {
   $resultat = 'Veuillez remplir tous les champs';
  } elseif (count(checkIfuserExistByPseudo()) > 0 ) {
      $resultat = 'Ce Login existe déjà, veuillez en choisir un autre !'; 
  } elseif (checkIfuserExistByNomPrenom() > 0 ) {   
      $resultat = 'Ce membre existe déjà !';      
  } else {
   $resultat = 'success';
  }
  return $resultat;
 }
  
  public function checkIfuserExistByPseudo(){
  $sql="SELECT * 
        FROM  tb_adherent 
        WHERE pseudo = '$this->pseudo'";
   $requete = $this->bdd->prepare($sql);
  $requete->execute();
  $result = $requete->fetchAll();
    return count($result);
  }

  public function checkIfuserExistByNomPrenom(){
  $sql ="SELECT COUNT(nom) as NB 
          FROM  tb_adherent 
          WHERE nom = '$this->nom' 
          AND prenom = '$this->prenom'  
          GROUP BY nom";
    $requete2 = $this->bdd->prepare($sql);
  $requete2->execute();
  $result2 = $requete2->fetchAll();
    return $result2[0]['NB'];
  }
?>


    Avant de poser une question, merci de lire la charte du site.
    Cordialement, Jordane
    1. papyclic Auteur
      bonjouir,
      Cela me renvoi une erreur la fonction n'est pas définie à la ligne 7 de ton post en fait
      Merci 
       Fatal error: Call to undefined function checkIfuserExistByPseudo() in C
      1. jordane45 Modérateur
        Oui désolé.
        Il faut mettre un $this-> devant... 

        public function verif() {
$resultat = "";
  if(empty($this->nom) || empty($this->prenom) || empty($this->pseudo) || empty($this->password) || empty($this->debut)) {
   $resultat = 'Veuillez remplir tous les champs';
  } elseif (count($this->checkIfuserExistByPseudo()) > 0 ) {
      $resultat = 'Ce Login existe déjà, veuillez en choisir un autre !'; 
  } elseif ($this->checkIfuserExistByNomPrenom() > 0 ) {   
      $resultat = 'Ce membre existe déjà !';      
  } else {
   $resultat = 'success';
  }
  return $resultat;
 }
      2. jordane45 Modérateur
        Si la question est résolue..
        Merci de ne pas oublier de clôturer le sujet
        (en cliquant sur le lien "Marquer comme résolu" qui se trouve sous le titre de la question)

        Cordialement,
        Jordane
    2. papyclic Auteur
      Bonjour
      Encore une toute petite notice s'il te plait
      à la ligne 24 de ton premier post au niveau du return
      Merci 
       return count($result);


      Notice: Undefined variable: result in C:
      1. papyclic Auteur
        Bonjour
        Je vais encore regarder demain je pense que ça devrait le faire,
        Je te redis
        A demain
        Merci
        2. papyclic Auteur
          Bonjour
          A prioro tyout fonctionne il y a juste une petite erreur.

          Notice: Undefined offset: 0 in C:

          au niveau du return

          return $result2[0]['NB'];

          Merci si tu peux me dire pourquoi.
          Merci beaucoup
          1. jordane45 Modérateur
            Bonjour,

            Essayes comme ça : 
              public function checkIfuserExistByNomPrenom(){


  $sql ="SELECT COUNT(nom) as NB 
          FROM  tb_adherent 
          WHERE nom = '$this->nom' 
          AND prenom = '$this->prenom'  
          GROUP BY nom";
    $requete2 = $this->bdd->prepare($sql);
  $requete2->execute();
  $result2 = $requete2->fetchAll();
    return isset($result2[0]['NB'])?$result2[0]['NB']:0;
  }
        3. papyclic Auteur
          Bonjour

          Un très grand Merci pour tout.
          Toutes les conditions fonctionnent très bien.
          Bonne fête de fin d'année.
          Merci

          Discussions similaires

          ymail.fr ça existe?
          asb

          8 réponses
          ymail.FR, ça existe?
          asb

          3 réponses
          Le bon coin, comment retrouver un pseudonyme ?
          HYPERSENSIBLE

          8 réponses
          obtenir une adresse ymail
          neophyte

          5 réponses
          un périphérique qui n'existe pas a été spécifié
          KyzeK

          9 réponses