Perdu dans l'indexation

**Omnes** · 01/12/2015, 13h28

Euh voilà, je crois que je me suis perdu dans l'indexation ...

Je vous explique, j’étais en train de faire les test de mon formulaire, et je m'amuse à mettre le même pseduo qui est déjà présent dans la bdd, pour qu'il test celui-ci et me retourne l'erreur, seulement il m'as retourné l'erreur : "Les adresses mails ne correspondent pas" hors elles correspondent, je vous montre 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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
<?php
 
$bdd = new PDO('mysql:host=127.0.0.1; dbname=my_covergratuit', 'covergratuit', '');
 
if(isset($_POST['forminscription']))
 
{
	$pseudo = htmlspecialchars($_POST['pseudo']);
	$mail = htmlspecialchars($_POST['mail']);
	$mailconfirm = htmlspecialchars($_POST['mailconfirm']);
	$mdp = sha1($_POST['mdp']);	
	$mdpconfirm = sha1($_POST['mdpconfirm']);
	$deckstart = ($_POST['deckstart']);
 
	if(!empty($_POST['pseudo']) AND !empty($_POST['mail']) AND !empty($_POST['mailconfirm']) AND !empty($_POST['mdp']) AND !empty($_POST['mdpconfirm']) AND !empty($_POST['deckstart']) )
	{
		$pseudolenght = strlen($pseudo);
		if($pseudolenght <= 255)
		{
			$reqpseudo = $bdd->prepare("SELECT * FROM membres WHERE pseudo = ?");
					$reqpseudo ->execute(array($pseudo));
					$pseudoexist = $reqpseudo->rowCount();
					if($pseudoexist == 0)
					{	
			if($mail == $mailconfirm)
				{
					if(filter_var($mail, FILTER_VALIDATE_EMAIL))
					{
						$reqmail = $bdd->prepare("SELECT * FROM membres WHERE mail = ?");
						$reqmail ->execute(array($mail));
						$mailexist = $reqmail->rowCount();
						if($mailexist == 0)
						{
							if($mdp == $mdpconfirm)
							{
								$insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, mdp, deck) VALUES (?, ?, ?, ?)");
								$insertmbr->execute(array($pseudo, $mail, $mdp, $deckstart));
								$erreur = "Votre compte à bien était créé.";
							}
							else
							{
								$erreur = "Vos mots de passe ne correspondent pas.";
							}	
						}	
							else
							{
							$erreur = "Adresse mail déjà inscrite.";
							}
						}
							else
							{
								$erreur = "Pseudo déjà inscrit";
							}	
 
				}
				else
				{
					$erreur = "Votre adresse mail n'est pas valide.";
				}
			}	
				else
			{
					$erreur = "Vos adresses mail ne correspondent pas.";
			}
		}
		else
		{
			$erreur = "Votre pseudo ne doit pas dépasser 255 caractères.";
		}
	}
	else
	{
		$erreur = "Tous les champs doivent êtres complétés !";
	}
} 
 
?>
<html>
	<head>
		<title>HVSP</title>
		<meta charset="utf-8">
	</head>
	<body>
		<div align="center">
			<h2>Inscription</h2>
			<br /><br />
			<form method="POST" action="
			">
				<table>
					<tr>
						<td align="right">
							<label for="pseudo">Pseudo :</label>
						</td>
						<td>
							<input type="text" placeholder="Pseudo" id="pseudo" name="pseudo" value="<?php if(isset($pseudo)) {echo $pseudo;} ?>" />
						</td>
					</tr>
					<tr>
						<td align="right">
							<label for="mail">Mail :</label>
						</td>
						<td>
							<input type="email" placeholder="Mail" id="mail" name="mail" value="<?php if(isset($mail)) {echo $mail;} ?>" />
						</td>
					</tr>
					<tr>
						<td align="right">
							<label for="mailconfirm">Confirmation du Mail :</label>
						</td>
						<td>
							<input type="email" placeholder="Confirmation Mail" id="mailconfirm" name="mailconfirm" value="<?php if(isset($pseudo)) {echo $pseudo;} ?>"/>
						</td>
					</tr>
					<tr>
						<td align="right">
							<label for="mdp">Mot de Passe :</label>
						</td>
						<td>
							<input type="password" placeholder="Mot de Passe" id="mdp" name="mdp" />
						</td>
					</tr>
					<tr>
						<td align="right">
							<label for="mdpconfirm">Confirmation Mot de Passe :</label>
						</td>
						<td>
							<input type="password" placeholder="Confirmation Mot de Passe" id="mdpconfirm" name="mdpconfirm" />
						</td>
					</tr>
					<tr>
						<td></td>
						<td align="center">
							<label for="feu">Feu :</label>
 
							<input type="radio" id="deckstart" name="deckstart" value="feu" />
 
							<label for="eau">Eau :</label>
 
							<input type="radio" id="deckstart" name="deckstart" value="eau" />
 
							<label for="feu">Air :</label>
 
							<input type="radio" id="deckstart" name="deckstart" value="air" />
						</td>
					</tr>
					<tr>
							<td></td>
							<td align="center">
								<br />
								<input type="submit" name="forminscription" value="Inscription" />
							</td>
				</table>
			</form>
			<?php 
if(isset($erreur))
{
	echo '<font color="red">'.$erreur."</font>";
}
			?>
	</body>		
</html>

Merci d'avance :/

**Celira** · 01/12/2015, 13h44

A priori, tu t'es emmelé entre les conditions des if et le texte des else correspondants.

Ton code réindenté, sans modifications, donne ceci :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
if (isset($_POST['forminscription'])) {
    $pseudo = htmlspecialchars($_POST['pseudo']);
    $mail = htmlspecialchars($_POST['mail']);
    $mailconfirm = htmlspecialchars($_POST['mailconfirm']);
    $mdp = sha1($_POST['mdp']);
    $mdpconfirm = sha1($_POST['mdpconfirm']);
    $deckstart = ($_POST['deckstart']);
    if (!empty($_POST['pseudo']) AND !empty($_POST['mail']) AND !empty($_POST['mailconfirm']) AND !empty($_POST['mdp']) AND !empty($_POST['mdpconfirm']) AND !empty($_POST['deckstart'])) {
        $pseudolenght = strlen($pseudo);
        if ($pseudolenght <= 255) {
            $reqpseudo = $bdd->prepare("SELECT * FROM membres WHERE pseudo = ?");
            $reqpseudo->execute(array(
                $pseudo
            ));
            $pseudoexist = $reqpseudo->rowCount();
            if ($pseudoexist == 0) {
                if ($mail == $mailconfirm) {
                    if (filter_var($mail, FILTER_VALIDATE_EMAIL)) {
                        $reqmail = $bdd->prepare("SELECT * FROM membres WHERE mail = ?");
                        $reqmail->execute(array(
                            $mail
                        ));
                        $mailexist = $reqmail->rowCount();
                        if ($mailexist == 0) {
                            if ($mdp == $mdpconfirm) {
                                $insertmbr = $bdd->prepare("INSERT INTO membres(pseudo, mail, mdp, deck) VALUES (?, ?, ?, ?)");
                                $insertmbr->execute(array(
                                    $pseudo,
                                    $mail,
                                    $mdp,
                                    $deckstart
                                ));
                                $erreur = "Votre compte à bien était créé.";
                            }
                            else {
                                $erreur = "Vos mots de passe ne correspondent pas.";
                            }
                        }
                        else {
                            $erreur = "Adresse mail déjà inscrite.";
                        }
                    }
                    else {
                        $erreur = "Pseudo déjà inscrit";
                    }
                }
                else {
                    $erreur = "Votre adresse mail n'est pas valide.";
                }
            }
            else {
                $erreur = "Vos adresses mail ne correspondent pas.";
            }
        }
        else {
            $erreur = "Votre pseudo ne doit pas dépasser 255 caractères.";
        }
    }
    else {
        $erreur = "Tous les champs doivent êtres complétés !";
    }
}

A priori, le message "vos adresses mail ne correspondent pas" correspond à la condition if ($pseudoexist == 0).

**Omnes** · 01/12/2015, 13h47

Merci mais c'est bien ce qu'il me semblait seulement sa m'as complètement perturbé sa, je sais plus quoi faire maintenant

**Celira** · 01/12/2015, 14h27

Une seule solution : passer en revue toutes les conditions des if et changer les messages d'erreur des else en conséquence.
Ce serait peut-être plus simple d'effacer tous les else et de les réécrire un par un.

**Omnes** · 01/12/2015, 14h44

Envoyé par Celira

Ce serait peut-être plus simple d'effacer tous les else et de les réécrire un par un.

Je suis débutant tu te rend pas compte ça m'as prit bien une heure à écrire tout sa

**crozet.magenta** · 01/12/2015, 17h38

bonjour,

en général, il faut éviter au maximum les trop grandes imbrications de if/else. le code devient beaucoup moins lisible puisque le else se retrouve à des dizaines de lignes du if et du coup, ça peut être source d'erreurs.

quand j'ai des nombreux tests à faire dans ce genre, j'utilise cette méthode qui est je trouve plus lisible :

Code php :

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
if (empty($_POST['pseudo']) || empty($_POST['mail']) || empty($_POST['mailconfirm']) || empty($_POST['mdp']) || empty($_POST['mdpconfirm']) || empty($_POST['deckstart'])) {    $message = "tous les champs doivent être remplis";
} else {
    $errors = false; //au début, pas d'erreur rencontrée
    $message = ''; //au début pas de message
    if (strlen($_POST['pseudo']) >= 255) {
        $errors = true;
        $message .= "pseudo trop long\n";
    }
    if ($_POST['mail'] != $_POST['mailconfirm']) {
        $errors = true;
        $message .= "mails différents\n";
    }
    /**
     * la suite des tests
     */
    if (!$errors) { //si on a pas d'erreur
        /**
         * code pour sauver en BDD
         */
        $message = 'Enregistré';
    }
}

En premier, tu teste si tous les champs sont remplis et tu ne vas pas plus loin si ce n'est pas le cas. Ensuite, chaque if est indépendant : il teste sa partie et ajoute son message sans se soucier des autres tests.
La variable $errors permet de vérifier si tout s'est bien passé : s'il n'y a pas eu d'erreurs, elle reste à false et donc on peut enregistrer, sinon, l'enregistrement ne se fait pas.

Avec cette méthode, tu as plusieurs avantages : le code est plus lisible donc le debug facilité, il est très simple d'ajouter des tests supplémentaires et enfin, tu peux afficher toutes les erreurs à l'utilisateur (avec ton code, seule la première erreur rencontrée était affichée)

Perdu dans l'indexation [MySQL]

PHP & Base de données

Vue hybride

Discussions similaires

Partager

Partager