Discussion :

[mroudy]

Bonjour, j'ai cherché peu partout sur le site et ailleurs et je n'arrive pas à mettre en place l'obligation pour un utilisateur d'entrer un mdp de minimum 8 caractères.

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
 
	public function addUser($nom, $prenom, $email, $pseudo, $passworda, $passwordb){
		try{		
 
			if($passworda != $passwordb){
				echo "Les mots de passe doivent être identiques";
 
				$passworda = $_POST['passworda'];  /* c'est ici où je ne suis certain de rien, le reste fonctionne normalement*/
				$longueur = strlen($passworda);				
				if ($longueur < 8) {
					echo "Mot de passe trop court !";
					return false;
				}
			} else {
 
				$passworda = sha1($passworda);
 
				$req=$this->db->prepare("select * from site where pseudo=?");
				$req->execute(array($pseudo));
				if($req->rowCount() !=0){
					echo "Le pseudo est déjà pris ! ";
					return false;
 
 
				}else {
					$passworda = sha1($passworda);
					$reponse = $this->db->prepare("INSERT INTO site VALUES(Default,?,?,?,?,?)");
					$reponse->execute(array($nom,$prenom,$email,$pseudo,$passworda));
					return true;
				}
			}
		}catch (PDOException $e){
			echo $e->getMessage();
		}
 
	}
 
}
 
?>

[sabotage]

Ce code ne fonctionne pas ?

Au passage, cette ligne ne sert à rien, $passworda est déjà défini par la fonction.

Code :

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

$passworda = $_POST['passworda'];

[mroudy]

Je viens d'enlever la ligne comme tu me l'a dis. Je pense que mon erreur vient en fait de plus haut dans le code, sur une autre page j'ai :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
 if(isset($_POST['inscription'])){
	extract($_POST);
	$rep=  new Connection('localhost', 'site', 'utf8', 'root', '');	
	$answer=$rep->dbconnect();
	$user = new Authentification($answer);
	$retour = $user->addUser($nom, $prenom, $email, $pseudo, $passworda, $passwordb);
	echo $retour;
	if($retour){
		echo "Inscription réalisée avec succès";
 
 
 }

Je pense que c'est ici mon problème, je créé deux fois la base, non ? même si il n'y a qu'une seule entrée dans cette dernière. Devrais-je mettre le premier code en remplacement de celui-ci ?

[demenvil]

Je viens d'enlever la ligne comme tu me l'a dis. Je pense que mon erreur vient en fait de plus haut dans le code, sur une autre page j'ai :

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
 
 
 if(isset($_POST['inscription'])){
	extract($_POST);
	$rep=  new Connection('localhost', 'site', 'utf8', 'root', '');	
	$answer=$rep->dbconnect();
	$user = new Authentification($answer);
	$retour = $user->addUser($nom, $prenom, $email, $pseudo, $passworda, $passwordb);
	echo $retour;
	if($retour){
		echo "Inscription réalisée avec succès";
 
 
 }
 
Edit: Code non testé

Je pense que c'est ici mon problème, je créé deux fois la base, non ? même si il n'y a qu'une seule entrée dans cette dernière. Devrais-je mettre le premier code en remplacement de celui-ci ?

Salut,
Il te manque aussi une accolade à la fin

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
 if(isset($_POST['inscription'])){
	extract($_POST);
	$rep=  new Connection('localhost', 'site', 'utf8', 'root', '');	
	$answer=$rep->dbconnect();
	$user = new Authentification($answer);
	$retour = $user->addUser($nom, $prenom, $email, $pseudo, $passworda, $passwordb);
	echo $retour;
	if($retour){
		echo "Inscription réalisée avec succès";
 
       }
 }

Ensuite oublie les ==> extract($_POST);
Sécurise ton code autrement


Pour ta question ta méthode addUser suffit à faire le taf oui.
Pense juste à bien établir la connexion avec la bdd.

Essaye ce 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
 
<?php
public function addUser($nom, $prenom, $email, $pseudo, $passworda, $passwordb){
		try{		
			if($passworda != $passwordb){
				echo "Les mots de passe doivent être identiques";
					return false;
				} else {
				  $longueur = strlen($passworda);				
				  if ($longueur < 8) {
					  echo "Mot de passe trop court !";
					  return false;
				  } else {
						$passworda = sha1($passworda);
						$req=$this->db->prepare("select * from site where pseudo=?");
						$req->execute(array($pseudo));
						if($req->rowCount() !=0){
							echo "Le pseudo est déjà pris ! ";
							return false;
						} else {
							$passworda = sha1($passworda);
							$reponse = $this->db->prepare("INSERT INTO site VALUES(Default,?,?,?,?,?)");
							$reponse->execute(array($nom,$prenom,$email,$pseudo,$passworda));
							return true;
						}
					}
			  }
		}catch (PDOException $e){
			echo $e->getMessage();
		}
	}