Un if qui passe à else sans raison

**SnoT-** · 28/01/2008, 23h48

Bonjour, je ne comprend pas pourquoi lorsque je fais mon test pour vérifier si un pseudo est déjà présent dans la BDD, celui-ci renvoit toujours TRUE alors que qu'en je m'inscris, je choisi un pseudo qui n'existe pas dans ma BDD.

Voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
function verifInscription()
  {
if(!empty($this->nom) && !empty($this->prenom) && !empty($this->email) && !empty($this->pseudo) && !empty($this->mdp) && !empty($this->mdp2))
      {
 
      $sql = "SELECT user_pseudo,user_email from tab_user";
      $resultat = mysql_query($sql);
 
      while($verif = mysql_fetch_object($resultat))
      {
        $pseudo = $verif->user_pseudo;
        $email = $verif->user_email;
      }
 
        if(!$pseudo == $this->pseudo)
        {
          $sql = "INSERT into tab_user (user_verifmdp,user_siteweb,user_email,user_nom,user_prenom,user_pseudo,user_mdp,user_age) VALUES ('$this->mdp2','$this->siteweb','$this->email','$this->nom','$this->prenom','$this->pseudo','$this->mdp','$this->age')";
 
            if(mysql_query($sql))
            {
              echo "Merci de vous être enregistrés";
            }else{
              echo "Enregistrement impossible";
            }
        }else{
          echo "Ce pseudo ou cette addresse e-mail est déjà dans la base de donnée";
        }
 
      }else{
          echo "Veuillez remplir tous les champs obligatoires";
      }
  }

Merci pour votre future aide

**DaRiaN** · 29/01/2008, 00h29

Bonsoir,
votre code m'a donné mal à la tête.

Pour vérifier qu'un pseudo existe déjà, on ne lit pas tous les pseudos de la table "tab_user", ça serait trop long et gourmand en ressources système.

La meilleur chose à faire, c'est de compter le nombre d'enregistrement où pseudo = $pseudo :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

SELECT count(*) AS `total` FROM `tab_user` WHERE `user_pseudo` = '$pseudo';

Par contre, c'est vrai que je suis fatigué mais j'ai une incompréhension quasi-totale de votre script... je crois avoir vu plein de choses étranges.

Cordialement,
DaRiaN.

**SnoT-** · 29/01/2008, 09h12

Merci bien pour votre réponse j'avais pas vu ca comme ca

Pour la compréhension de mon code je sais pas. Il est pas entier et fait appel a des class peut-être c'est ca ou alors c'est parce que je commence à coder et du coup je suis pas encore à l'aise ^^

En tout cas merci.

**papyphp** · 29/01/2008, 10h30

Bonjour,

Un clou de plus à votre cercueil

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(!$pseudo == $this->pseudo)

le not a un effet sur $pseudo qui devient la valeur "true"
et c'est cette valeur que vous comparez à $this=>pseudo
il est donc logique que vous alliez toujours dans le else.

propositions :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part