Eviter les doublons dans la bdd

jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   -  
jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   -
Bonjour à tous !

Je viens de créer un système d'amis entre membres
Je peux ajouter quelqu'un comme ami quand je veux, pas besoin que cet ami confirme.

J'ai créer une table amis dans la bdd qui comporte 3 colonnes :

id : ben c'est l'id tout simplement ^^

pseudo : le pseudo du membre qui clique sur "Ajouter comme ami"

pseudoAmi : le pseudo de l'ami que l'on vient d'ajouter

Cependant, si la personne rajoute le même ami, cela créé une nouvelle ligne dans la bdd.
Je ne sais pas comment faire pour éviter cela.

Il aurait été sympa qu'on message apparaisse pour signaler que la personne est déjà ami avec ce membre.

Voici le code qui permet d'ajouter un ami (et qui fonctionne) :

if(isset($_GET['pseudo']))
    {
        $pseudoAmi = htmlspecialchars($_GET['pseudo']);
        $reqpseudo = $bdd->prepare("SELECT * FROM membres WHERE pseudo = ?");
        $reqpseudo->execute(array($pseudo));
        $pseudoexist = $reqpseudo->fetch();
        if($pseudoexist == false)
        {
            $insertAmi = $bdd->prepare("INSERT INTO amis(pseudo, pseudoAmi) VALUES(?, ?)");
            $insertAmi->execute(array($user['pseudo'], $pseudoAmi));
            $_SESSION['error'] = "".$pseudoAmi." a été ajouté dans votre liste d'amis !";
            header('Location: amis.php');
        }
        else
        {
            header('Location:amis.php');
            $_SESSION['error'] = "Le pseudo n'existe pas";
            exit;
        }
    }


Comment faire ?

Merci et bonne soirée :)
A voir également:

2 réponses

jordane45 Messages postés 38486 Date d'inscription   Statut Modérateur Dernière intervention   4 752
 
Bonsoir,

A la place d'un INSERT tu peux utiliser un REPLACE.

Penses également à définir le couple "pseudo, pseudoAmi" comme clé unique.

ALTER TABLE amis ADD UNIQUE `unique_index`(pseudo, pseudoAmi);



Au passage... voici ton code corrigé et quelque peu amélioré.
// Récupéaration Propre des variables AVANT de les utiliser
$pseudo = !empty($_GET['pseudo'])?$_GET['pseudo'] : NULL;
if($pseudo) {

  $pseudoAmi = htmlspecialchars($pseudo);
  
  $sql = "SELECT * FROM membres WHERE pseudo = :pseudo" ;
  $params = array(":pseudo"=>$pseudo);
  
  try{
    $reqpseudo = $bdd->prepare($sql);
    $reqpseudo->execute($params);
    $resultat = $reqpseudo->fetchall();
  }catch(Exception $e) {
   echo "<pre><br>".$e->getMessage()."<br>";
   echo "</pre>";
   exit;
  }

  if(count($resultat)>0){
    //Ce pseudo existe  dans la BDD
    $sql = "REPLACE INTO amis(pseudo, pseudoAmi) VALUES(:userPseudo, :pseudoAmi)";
	$params = array(":userPseudo"=>$user['pseudo'], ":pseudoAmi"=>$pseudoAmi);
    
	try{
      $insertAmi = $bdd->prepare($sql);
      $insertAmi->execute($params);
	}catch(Exception $e) {
      echo "<pre><br>".$e->getMessage()."<br>";
      echo "</pre>";
	  exit;
    }
    $_SESSION['error'] = $pseudoAmi." a été ajouté dans votre liste d'amis !";
    header('Location: amis.php');
    exit;
  }else{
    //Le pseudo n'existe pas
    $_SESSION['error'] = "Le pseudo n'existe pas";
    header('Location:amis.php');
    exit;
  }
  
} // Fin  if($pseudo)


0
jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   10
 
Merci beaucoup, tout fonctionne !

Je tenais juste à vous féliciter car la plupart du temps, c'est vous qui répondez aux problèmes php et de manière brillante :)

Enfin, serait-il possible de rajouter un peu de code qui signale si on a déjà cette personne comme ami ?

Encore merci :)
0
jordane45 Messages postés 38486 Date d'inscription   Statut Modérateur Dernière intervention   4 752
 
Oui.
Il te suffit de faire un SELECT sur ta table avant l'ajout .... et si la requête retourne un résultat.. afficher qq chose.

// Récupéaration Propre des variables AVANT de les utiliser
$pseudo = !empty($_GET['pseudo'])?$_GET['pseudo'] : NULL;
if($pseudo) {

  $pseudoAmi = htmlspecialchars($pseudo);
  
  $sql = "SELECT * FROM membres WHERE pseudo = :pseudo" ;
  $params = array(":pseudo"=>$pseudo);
  
  try{
      $reqpseudo = $bdd->prepare($sql);
      $reqpseudo->execute($params);
      $resultat = $reqpseudo->fetchall();
   }catch(Exception $e) {
     echo "<pre><br>".$e->getMessage()."<br>";
     echo "</pre>";
    exit;
   }

if(count($resultat)>0){
// Le pseudo du user existe en BDD ...

// on regarde si ils sont déjà amis :
  $sql = "SELECT * 
         FROM  amis
         WHERE pseudo = userPseudo
         AND  pseudoAmi = :pseudoAmi";
   $params = array(":userPseudo"=>$user['pseudo'], ":pseudoAmi"=>$pseudoAmi);
    
 try{
       $reqpseudo = $bdd->prepare($sql);
       $reqpseudo->execute($params);
       $resultat2 = $reqpseudo->fetchall();
   }catch(Exception $e) {
      echo "<pre><br>".$e->getMessage()."<br>";
      echo "</pre>";
      exit;
    }

   if(count($resultat2)>0){
       // Il y a déjà une entrée existante dans la BDD
        $_SESSION['error'] = "Vous êtes déjà amis....";
        header('Location:amis.php');
     }else{
         // ici tu mets le code d'insertion en BDD
        $sql = "REPLACE INTO amis(pseudo, pseudoAmi) VALUES(:userPseudo, :pseudoAmi)";
 $params = array(":userPseudo"=>$user['pseudo'], ":pseudoAmi"=>$pseudoAmi);
    
       try{
            $insertAmi = $bdd->prepare($sql);
            $insertAmi->execute($params);
        }catch(Exception $e) {
             echo "<pre><br>".$e->getMessage()."<br>";
              echo "</pre>";
            exit;
         }
        $_SESSION['error'] = $pseudoAmi." a été ajouté dans votre liste d'amis !";
        header('Location: amis.php');
       exit;

   }
}else{
    //Le pseudo n'existe pas
    $_SESSION['error'] = "Le pseudo n'existe pas";
    header('Location:amis.php');
    exit;
  }
  
} // Fin  if($pseudo)

0
jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   10
 
ça ne fonctionne pas :/

C'est toujours écrit "$pseudoAmi a été ajouté dans votre liste d'amis !"
0
jordane45 Messages postés 38486 Date d'inscription   Statut Modérateur Dernière intervention   4 752
 
Il manque un ":" dans la requête SELECT devant la variable userPseudo
0
jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   10
 
Parfait ça fonctionne !

Merci beaucoup ! :D
0
jisisv Messages postés 3645 Date d'inscription   Statut Modérateur Dernière intervention   934
 
Au nivaeu de ta base de données, elle deviendra rapidement ingérable (quid du changement de pseudo par exemple).
Je te conseille de créer au minimum deux tables:
  • la première membres(id, pseudo, e-mail...)
  • la seconde amis (id_membre, id_ami...) contiendra les relations d'amitié


Lors de l'ajout d'un lien d'amitié, c'est cette seconde table qui contiendra un nouveau t-uple. La première table ne contiendra que les informations personnelles des membres
0
jobouille Messages postés 286 Date d'inscription   Statut Membre Dernière intervention   10
 
Il est effectivement plus logique de faire ainsi. Cependant il est impossible aux membres du site de changer leur pseudo.
Pour le moment je laisse comme ça, mais je garde en tête votre solution, merci :)
0