Gérer le css du formulaire en fonction des erreurs

**lifala** · 23/11/2018, 19h21

Salut a tous.

je travail sur un formulaire qui fonctionne. J'arrive a ajouter les utilisateurs dans la bases de données, hey déjà pour moi c'est fonctionnel :p

Mon soucis c'est pour le traitement des erreurs du formulaire.

En lisant le code vous trouverez un teste sur le pseudo, le mot de passe et l'email. Si des erreurs sont identifié on informe l'utilisateur qu'il doit modifier les valeurs avec un message.

Par exemple si le mot de passe est déjà utilisé, on lui indique en texte "pseudo déjà utilisé" et en css on rend la zone de saisis rouge.

Si je peux dire clairement ... le problème c'est si l'utilisateur fait plusieurs erreurs.

par exemple il rentre un pseudo trop court et un email déjà utilisé

nous avons,

deux zone de saisis rouge avec toutes les deux un message d'erreur "pseudo trop court" et "mail déjà utilisé"

L'utilisateur tente donc de faire une correction.

Il entre un mail valide mais un pseudo déjà utilisé

Ben ça ne modifie pas le formulaire.

Nous avons toujours le message d'erreur pour le mail, et toujours la zone de saisi en rouge.

Il doit surement manquer un petit quelque chose ou une façon d’écrire le code qui permet de parrer ce bug. Et j'en viens donc a vos lumières.

Voici le code html du formulaire

Code html :

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
        <form id="register_form" onsubmit="return false;" style="margin:10px;">
          <div class="form-group">
            <label for="nom">Pseudo</label>
            <input type="text" placeholder="Entrez votre pseudo" id="pseudo" name="pseudo" maxlength="16" class="form-control" required/>
            <small id="output_checkuser"></small>
          </div>
          <div class="form-group">
            <label for="pass1">Mot de passe</label>
            <input type="password" placeholder="6 caractères minimum" id="pass1" name="pass1" class="form-control" required/>
            <small id="output_pass1"></small>
          </div>
          <div class="form-group">
            <label for="pass2">Confirmez votre mot de passe</label>
            <input type="password" id="pass2" name="pass2" class="form-control" required/>
            <small id="output_pass2"></small>
          </div>
          <div class="form-group">
            <label for="email">Votre adresse e-mail</label>
            <input type="email" placeholder="pseudo@devosi.org" id="email" name="email" class="form-control" required/>
            <small id="output_email"></small>
          </div>
          <div id="status" class="alert alert-primary" role="alert">
            Merci de remplir tous les champs.
          </div>
          <input type="submit" id="bRegister" class="btn btn-primary" value="Inscription" />
        </form>

et le js

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
$(document).ready(function(){
 
  //$('#general').html().fadeOut();
 
    // On fait disparaitre la div #status quand on clique dans un champ du formuaire
    $("#register_form input").focus(function(){
  		$("#status").fadeOut(800);
  	});
 
  //Fonction qui permet de tester le pseudo entre
  function check_pseudo(){
    $.ajax({
      type:	'post',
      url:	'index.php',
      data: {
        'pseudo_check' : $("#pseudo").val()
      },
      success: function(data){
        if(data == "success"){
          $("#pseudo").addClass("is-valid");
          return true;
        } else {
          $("#output_checkuser").css("color", "red").html(data);
          $("#pseudo").addClass("is-invalid");
        }
      }
    });
  }
 
  function check_password(){
    $.ajax({
      type: "post",
      url: "index.php",
      data: {
        'pass1_check' : $("#pass1").val(),
        'pass2_check' : $("#pass2").val()
      },
      success: function(data){
        if(data == "success"){
          $("#pass2").addClass("is-valid");
          $("#pass1").addClass("is-valid");
        } else {
          $("#output_pass2").css("color", "red").html(data).fadeIn(400);
          $("#pass1").addClass("is-invalid");
          $("#pass2").addClass("is-invalid");
        }
      }
    });
  }
 
  function check_email(){
    $.ajax({
      type: "post",
      url: "index.php",
      data: {
        'email_check' : $("#email").val()
      },
      success: function(data){
        if(data == "success"){
          $("#email").addClass("is-valid");
        } else {
          $("#email").addClass("is-invalid");
          $("#output_email").css("color", "red").html(data);
        }
      }
    });
  }
 
  // Traitement du formulaire d'inscription
  $("#register_form").submit(function(){
    //on verifie si le pseudo est libre ou deja utilisateur
    check_pseudo();
 
    check_password();
 
    check_email();
 
    var status = $("#status");
    var pseudo = $("#pseudo").val();
    var pass1 = $("#pass1").val();
    var pass2 = $("#pass2").val();
    var email = $("#email").val();
 
    if(pseudo == "" || pass1 =="" || pass2 == "" || email == ""){
      status.addClass("alert-danger");
      status.html("Le formulaire contient des erreurs").fadeIn(400);
    } else if (pass1 != pass2) {
      status.addClass("alert-danger");
      status.html("Le formulaire contient des erreurs").fadein(400);
    } else {
      $.ajax({
        type: "post",
        url: "index.php",
        data: {
          'pseudo' : pseudo,
          'pass1' : pass1,
          'pass2' : pass2,
          'email' : email,
        },
        beforeSend: function(){
          $("#bRegister").attr("value", "Traitement en cours ...");
        },
        success: function(data){
          if(data != "register_success"){
            status.addClass("alert-danger");
            status.html("Le formulaire contient des erreurs").fadeIn(400);
            $("#bRegister").attr("value", "Inscription");
            $("#bRegister").addClass("btn-primary").css("color", "white");
          } else {
            //status.toggleClass("alert-success");
            //$("#general").addClass("alert-primary").fadeIn(800);
            //$("#general").html("<strong>Juste une derniere etape " + pseudo + " !</strong><br/>Un lien d'activation vient de vous etre envoyé à l'adresse mail indiqué.<br/>Veuillez cliquer sur ce lien pour terminer votre inscripton et ainsi pourvoir vous connectez avec vos identifiants").addClass("alert-primary").fadeIn(400);
          }
        }
      });
    }
  });
});

il manque evidement le code .php pour les testes sur la base de données, et la base de données.

je ne vois pas comment vous permettre de faire les testes en live.

en esperant que vous compreniez le probleme que je rencontre.

merci d'avance.

**lifala** · 24/11/2018, 00h10

j'ai tenté plusieurs manip avec removeClass() mais l'erreur reste malgré un pseudo disponible par exemple .

pour la fonction check_pseudo par exemple si je fais ceci j'ai un bon fonctionnement mais je voudrais modifier le css et non ajouter une image pour dire que c'est bon. Mais je ne comprend pas pourquoi je n'ai pas un bon résultat.

je tourne en rond autout de ce addClass() :/

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
function check_pseudo(){
		$.ajax({
			type:	"post",
			url:	"register.php",
			data: {
				'pseudo_check' : $("#pseudo").val()
			},
			success: function(data){
				if(data == "success"){
					$("#output_checkuser").html('<img src="img/check.png" alt="valide" />');
					return true;
				} else {
					$("#output_checkuser").css("color", "red").html(data);
				}
			}
		});
	}

**Toufik83** · 24/11/2018, 11h19

montre nous le code php, j'ai constaté qu'il y a trop de requêtes ajax dans ton code, une seule est suffisante...

**lifala** · 24/11/2018, 11h52

merci pour cette réponse.

voici le code php

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
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
<?php
// Verification du pseudo
if (!empty($_POST['pseudo_check'])){
  $pseudo = $_POST['pseudo_check'];
  $pseudo = preg_replace('#[^a-z0-9]#i', '', $pseudo);
  if (strlen($pseudo) < 3 || strlen($pseudo) > 16){
    echo 'Entre 3 et 16 caractères SVP.';
    exit();
  }
 
  if (is_numeric($pseudo[0])){
    echo 'Le pseudo doit commencer par une lettre.';
    exit();
  }
 
  //Connection a la base de donnee
  require "includes/connectdb.php";
 
  $q = $db->prepare('SELECT id FROM baseName WHERE pseudo = ?');
  $q->execute(array($pseudo));
 
  $numRows = $q->rowCount();
  if ($numRows > 0){
    echo 'Pseudo déjà utilisé !';
    exit();
  } else {
    echo 'success';
    exit();
  }
}
 
/* Verification des mots de passe */
if (!empty($_POST['pass1_check']) && !empty($_POST['pass2_check'])){
  if(strlen($_POST['pass1_check']) < 6 || strlen($_POST['pass2_check']) < 6){
    echo '<br/>6 caractères minimum pour le mot de passe !';
    exit();
  } else if ($_POST['pass1_check'] == $_POST['pass2_check']){
    echo 'success';
    exit();
  } else {
    echo '<br/>Les deux mots de passe sont differents';
    exit();
  }
}
 
/* Verification de l'adresse email */
if(!empty($_POST['email_check'])){
  $email = $_POST['email_check'];
 
  // Verifier l'adresse mail
  if(!filter_var($email, FILTER_VALIDATE_EMAIL)){
    echo '<br/>Adresse invalide !';
    exit();
  }
  //Connection a la base de donnees
  require "includes/connectdb.php";
 
  $q = $db->prepare('SELECT id FROM baseName WHERE email = ?');
  $q->execute(array($email));
 
  $numRows = $q->rowCount();
  if($numRows > 0){
    echo '<br/>Cette adresse e-mail est déjà utilisé !';
    exit();
  } else {
    echo 'success';
    exit();
  }
}
 
//Traitement de l'insription
if(isset($_POST['pseudo'])){
	require "includes/connectdb.php";
	extract($_POST);
	$pseudo = preg_replace('#[^a-z0-9]#i', '', $pseudo);
	$q = $db->prepare('SELECT id FROM baseName WHERE pseudo = ?');
	$q->execute(array($pseudo));
	$pseudo_check = $q->rowCount();
 
	$q = $db->prepare('SELECT id FROM baseName WHERE email = ?');
	$q->execute(array($email));
	$email_check = $q->rowCount();
 
	if(empty($pseudo) || empty($pass1) || empty($pass2) || empty($email)){
			echo "Tous les champs doivent être remplis.";
	} else if ($pseudo_check > 0) {
			echo "Pseudo deja utilisé";
	} else if ($email_check > 0){
			echo "Cette adresse mail est deja utilisée";
	} else if (strlen($pseudo) < 3 || strlen($pseudo) > 16) {
			echo "pseudo eronne !";
	} else if (is_numeric($pseudo[0])) {
			echo "le pseudo doit commencer par une lettre";
	} else if ($pass1 != $pass2) {
			echo "Les mots de passe sont different";
	} else {
	  $hash_pass = sha1($pass1);
	  $q = $db->prepare('INSERT INTO baseName(pseudo, email, password, created)
				VALUES(:pseudo, :email, :password, now())');
	  $q->execute(array(
	    'pseudo' => $pseudo,
	    'email' => $email,
	    'password' => $hash_pass
	  ));
 
	  $user_id = $db->lastInsertId();
 
	  //if(!file_exists("members/$user_id")){
	    //mkdir("members/$user_id", 0755);
	  //}
 
	  $to = $email;
	  $from = "contact@domaine.org";
	  $subject = "DOMAINE.ORG - ACTIVATION de votre compte";
	  $message = "<!DOCTYPE html>
			<html>
			  <head>
			    <meta charset=\"UTF-8\" />
			  </head>
			  <body>
			    Bonjour $pseudo,<br/><br/>

			    <h2> Completez cette derniere étape pour activer votre compte <strong>domaine.org</strong>
			    <p>Pour ce faire il vous suffit de cliquer sur le lien suivant:<br/>

			    http://localhost:41062/www/domaine/formulaireConnectionInscription/activation.php?id=$user_id&amp;u=$pseudo&amp;e=$email&amp;ssl=$hash_pass<br/>
			    Si l'URL n'apparait pas comme un lien actif, veuillez SVP copier/coller ce dernier dans
			    la barre d'adresse de votre navigateur.</p>

			    <h2>Identifiant de connection:</h2>
			    <p>
			    email: $email<br/>
			    mot de pass : $pass1<br/>
			    </p>
			    <p>Rendez vous sur le site <a href=\"http://domaine\">DOMAINE.ORG</a></p>
			  </body>
			</html>";
	  $headers = "From: $from\n";
	  $headers .= "MIME-Version: 1.0\n";
	  $headers .= "Content-type: text/html; charset=iso-8859-1\n";
	  if(mail($to, $subject, $message, $headers)){
	    echo 'register_success';
	    exit();
	  } else {
	    //On supprime le compte de la bdd vue qu'il ne sert a plus rien
	    //$q = $db->prepare("DELETE FROM users WHERE id = ?");
	    //$q->execute(array($user_id));
	    //echo "Erreur lors de l'envoi du mail";
      echo 'register_success';
	    exit();
    }
	}
  }
?>

**Toufik83** · 24/11/2018, 14h23

bonjour,

c'est faisable avec une seule requête ajax

formulaire.php

Code html :

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
<!DOCTYPE html>
<html lang="fr">
	<head>
		<meta charset="utf-8">
		<title>Inscription</title>
		<link rel="stylesheet" href='https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css' />
		<style>
                        .is-valid{background-color:white}
                        .is-invalid{background-color:red !important;}
                </style>
		<script
		  src="https://code.jquery.com/jquery-3.3.1.js"
		  integrity="sha256-2Kok7MbOyxpgUVvAk/HJ2jigOSYS2auK4Pfzbm7uH60="
		  crossorigin="anonymous">
		</script>
		<script src="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/js/bootstrap.min.js"></script>
		<script>
                        $(document).ready(function(){
 
                          //$('#general').html().fadeOut();
                         
                        // On fait disparaitre la div #status quand on clique dans un champ du formuaire
                        $("#register_form input").focus(function(){
                                $("#status").fadeOut(800);
                                $(this).removeClass('is-invalid');
                        });
                         // Traitement du formulaire d'inscription
                          $("#register_form").submit(function(e){
                                  e.preventDefault();
                                var status = $("#status");
                                var pseudo = $("#pseudo").val();
                                var pass1 = $("#pass1").val();
                                var pass2 = $("#pass2").val();
                                var email = $("#email").val();
                                 $('#pseudo,#pass1,#pass2,#email').removeClass('is-invalid');
                                if(pseudo == "" || pass1 =="" || pass2 == "" || email == ""){
                                  status.addClass("alert-danger");
                                  status.html("Le formulaire contient des erreurs").fadeIn(400);
                                } else if (pass1 != pass2) {
                                  status.addClass("alert-danger");
                                  status.html("Le formulaire contient des erreurs").fadeIn(400);
                                } else {
                                        $.ajax({
                                                method: "post",
                                                url: "index.php",
                                                dataType:'json',
                                                data: {
                                                  'pseudo' : pseudo,
                                                  'pass1' : pass1,
                                                  'pass2' : pass2,
                                                  'email' : email,
                                                },
                                                beforeSend: function(){
                                                  $("#bRegister").attr("value", "Traitement en cours ...");
                                                },
                                                success: function(data){
                                                        console.log("success :"+JSON.stringify(data));
                                                        $("#bRegister").attr("value", "Inscription");
                                                  if(data.erreur){
                                                        status.removeClass('alert-success').addClass("alert-danger")
                                                        .html("Le formulaire contient des erreurs").fadeIn(400);
                                                        $("#bRegister").attr("value", "Inscription")
                                                        .addClass("btn-primary").css("color", "white");
                                                        if(data.erreur.pseudo||data.erreur.pseudo_numerique){
                                                                $('#pseudo').addClass('is-invalid');
                                                        }
                                                        if(data.erreur.passeword || data.erreur.password_length){
                                                                $('#pass1,#pass2').addClass('is-invalid');
                                                        }
                                                        if(data.erreur.email){
                                                                $('#email').addClass('is-invalid');
                                                        }
                                                        $.each(data.erreur,function(v,k){
                                                                status.append('<p><b>'+k+'</b></p>');
                                                        });
                                                  } else {
                                                        status
                                                        .removeClass('alert-danger')
                                                        .addClass("alert-success")
                                                        .html('<strong>Juste une derniere etape ' + pseudo + ' !</strong><br/>Un lien d\'activation vient de vous etre envoyé à l\'adresse mail indiqué.<br/>Veuillez cliquer sur <a href="#">ce lien</a> pour terminer votre inscripton et ainsi pourvoir vous connectez avec vos identifiants')
                                                        .fadeIn(400);
                                                  }
                                                },
                                                error:function(err){alert(err.responseText);}
                                          });
                                }
                          });
                        });
                </script>
	<head>
	<body>
		<form id="register_form" method="post" style="margin:10px;">
          <div class="form-group">
            <label for="nom">Pseudo</label>
            <input type="text" placeholder="Entrez votre pseudo" id="pseudo" name="pseudo" maxlength="16" class="form-control" required/>
            <small id="output_checkuser"></small>
          </div>
          <div class="form-group">
            <label for="pass1">Mot de passe</label>
            <input type="password" placeholder="6 caractères minimum" id="pass1" name="pass1" class="form-control" required/>
            <small id="output_pass1"></small>
          </div>
          <div class="form-group">
            <label for="pass2">Confirmez votre mot de passe</label>
            <input type="password" id="pass2" name="pass2" class="form-control" required/>
            <small id="output_pass2"></small>
          </div>
          <div class="form-group">
            <label for="email">Votre adresse e-mail</label>
            <input type="email" placeholder="pseudo@devosi.org" id="email" name="email" class="form-control" required/>
            <small id="output_email"></small>
          </div>
          <div id="status" class="alert alert-primary" role="alert">
            Merci de remplir tous les champs.
          </div>
          <input type="submit" id="bRegister" class="btn btn-primary" value="Inscription" />
        </form>
	</body>
</html>

index.php (fichier php de traitement)

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
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
<?php 
$data=[];
try{
/*ne pas oublier de mettre le bon nom de ta base de donnée et le bon mot de passe si t'en a un */
$db=new PDO('mysql:host=localhost;dbname=gestion_menage', 'root', '', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
}catch(PDOException $e){
	die("erreur :".$e->getMessage());
}
$pseudo=$_POST['pseudo'];
$email=$_POST['email'];
$passe1=$_POST['pass1'];
$passe2=$_POST['pass2'];
 
 
if($passe1!==$passe2){
	$data['erreur']['passeword']="les mots de passes ne sont pas identiques!";
}
if(strlen($passe1)<3){
	$data['erreur']['password_length']="le mot de passe est trop court!";
}
if(strlen($passe1)>16){
       $data['erreur']['password_length']="le mot de passe est trop long";
}
if(!filter_var($email, FILTER_VALIDATE_EMAIL)){
	$data['erreur']['email']=utf8_encode("cette adresse émail n'est pas correcte!");
}
$c= $db->prepare("SELECT id from baseName where pseudo=:pseudo");
$c->execute([
  ':pseudo' =>$pseudo
]);
 
if($c->rowCount()>0){
	$data['erreur']['pseudo']=utf8_encode("Pseudo déjà utilisé, veuillez choisir un autre.");
}
if(is_numeric($pseudo[0])){
	$data['erreur']['pseudo_numerique']="Un pseudo doit commencer avec une lettre!";
}
$c= $db->prepare("SELECT id from baseName where email=:email"); 
$c->execute([
  ':email' =>$email
]);
if($c->rowCount()>0){
	isset($data['erreur']['email'])
		?$data['erreur']['email'].=utf8_encode("émail déjà utilisé, veuillez choisir un autre.")
		:$data['erreur']['email']=utf8_encode("émail déjà utilisé, veuillez choisir un autre.");
}
if(!isset($data['erreur'])){//si tout va bien
	/*code d'insertion ici... (sans mettre echo), 
	si tu veux stocker une valeur pour la récupérer dans ajax, stocke là dans $data[]
	, genre $data['quelque_chose']='quelque_chose'; */
}
echo json_encode($data);
?>

**lifala** · 24/11/2018, 18h55

hey ben je te remercie grandement

pour le moment je joue avec le code que tu viens de me refaire et je suis satisfait du résultat.

j'arrive a inscrire des utilisateurs.
j'ai pu modifier les class comme je souhaitait

mais mes lacunes de débutant doivent me pousser à faire des recherches comme sur ce genre de chose que je ne comprend pas :

Code :

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

console.log("success :"+JSON.stringify(data));

Petite question au passage, si tu peux m'orienter, m'informer ... sur la sécurité des formulaires, merci.

je bûche sur mon envois de mail maintenant.

contant de pouvoir avancer maintenant.

Gérer le css du formulaire en fonction des erreurs

jQuery

Vue hybride

Discussions similaires

Partager

Partager