Discussion :

[okoweb]

Bonjour,

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
 
//Retour d'erreur
$error = '';
if(!empty($_POST['action'])){
    if($_POST['action']=='register'){
        if(!empty($_POST['email']) AND !empty($_POST['pswd1']) AND !empty($_POST['pswd2']) AND $_POST['pswd1']==$_POST['pswd2'] AND preg_match('/^([^@\s<&>]+)@(?:([-a-z0-9]+)\.)+([a-z]{2,})$/i', $_POST['email'])){
      				if(checkEmail($_POST['email'])==0){
            				   if( !insertClient( $_POST['email'], md5($_POST['pswd1']) ) ){ 
												 $error = 'Compte non créé';
											 }
            					 else{
											   $error = 'Compte créé avec succès.<br />Vous pouvez à présent vous identifiez.';
											 }
      				}
							else{
							 $error = 'Cet email existe déjà, compte non créé.';
							}
        }
				else{
				  $error = 'Veillez à bien remplir vos informations pour la création de votre compte.';
				}
    }

Ce bout de permet une inscription au site. Il marche bien mais le retour $error lorsqu'un compte a été créé devrait me renvoyer :

Code :

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

1
2
 
$error = 'Compte créé avec succès.<br />Vous pouvez à présent vous identifiez.';

bizar j'ai ceci :

Code :

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

1
2
 
$error = 'Compte non créé';

Ca bug où?

[Kwizatz]

Bonsoir Okoweb,

Visiblement, comme le dis l'adage, ta question est bien posée et la réponse est à moitié dedans
Il semble que ça bug au niveau de ta fonction insertClient non ?
Tu pourrais nous donner le corps de cette fonction pour éventuellement voir ce qui cloche à ce niveau là ?

[gene69]

utilise filter_var(, FILTER_VALIDATE_EMAIL) pour valider les mails. c'est mieux !

on peut pas répondre ya trop de choses qui ne sont pas connues.

[Madfrix]

utilise filter_var(, FILTER_VALIDATE_EMAIL) pour valider les mails. c'est mieux !

Pourquoi ? un argument ?

[gene69]

yes
1) plus on l'utilise moins on trouve des expressions régulieres hasardeuses comme celle-ci.
2) c'est plus lisible
3) j'ai aucune idée si c'est plus performant, mais n'oublions pas qu'on compare à une expression réguliere...

[Madfrix]

yes
1) plus on l'utilise moins on trouve des expressions régulieres hasardeuses comme celle-ci.
2) c'est plus lisible
3) j'ai aucune idée si c'est plus performant, mais n'oublions pas qu'on compare à une expression réguliere...

je m'attendais plus à un argument de rapidité d'exécution

1) les regex faut toujours les employer pour bien se les approprier y'en a partout
2) mouais...
3) A mon avis moins performant, filter_var utilise probablement les regex pour valider (ce n'est que mon avis)

Perso j'ai jamais trouvé une utilité à cette fonction, je suis peut être un peu vieux jeu mais je préfère tout faire avec les regex, au moins je sais sur qui taper si ca marche pô

[okoweb]

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
 
//Comptes clients
function insertClient($email, $pswd){
	global $db;
	$sql = 'INSERT INTO clients (email, password) VALUES (:email, :pswd)';
	$statement = $db->prepare($sql);
	$statement->execute(array(':email' => $email, ':pswd'=> $pswd ));
}
 
function checkEmail($email){
	global $db;
	$sql = 'SELECT count(*) as nbre FROM clients WHERE email=:email';
	$statement = $db->prepare($sql);
	$statement->execute(array(':email' => $email));
	$tabl_result = $statement->fetch();
	return $tabl_result['nbre'];
}
 
function checkClient($email, $pswd){
	global $db;
	$sql = 'SELECT count(*) as nbre FROM clients WHERE email=:email AND password=:pswd';
	$statement = $db->prepare($sql);
	$statement->execute(array(':email' => $email, ':pswd' => $pswd));
	$tabl_result = $statement->fetch();
	return $tabl_result['nbre'];
}
 
function infoClient($email){
	global $db;
	$sql = 'SELECT * FROM clients WHERE email=:email';
	$statement = $db->prepare($sql);
	$statement->execute(array(':email' => $email));
	$tabl_result = $statement->fetch();
	return $_ENV['client'] = $tabl_result['id'];
}

[julp]

Il manque une valeur de retour (return) à insertClient, en ces circonstances PHP renvoie NULL (valeur par défaut). Ainsi, le test !insertClient(...) est toujours vrai.