Script identification utilisateur/mot de passe

Version imprimable

Bonjour à toutes et à tous,

Pouvez-vous me dire ce qui cloche dans mon code car si je mets le else dans la boucle while l'identification ne s'effectue pas.

Code:

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
 
<pre>
<?php 
	session_start();
	require_once("includes/inc_connexion.php"); // on se connecte à la Base de données	
	$conn=opendb_conn();
 
	if(isset($_POST['submit'])) {
		// bouton submit pressé, je traite le formulaire
		$login = (isset($_POST['pseudo']))?$_POST['pseudo']:'';
		$password = (isset($_POST['pass']))?$_POST['pass']:'';
		extract($_POST); // importation des variables dans la table des symboles
		// constitution de la requete
		$req="select login, password from membres";	
		// on prépare la requete a executer sur le serveur
		$stm=$conn->query($req);
		// on exécute la requete
		//$stm->execute();	
		// parcourir l'ensemble des tuples de la table des membres
		while($result=$stm->fetch(PDO::FETCH_ASSOC)) {
			// tester si le login et le password sont en correspondance
			if(($result['login']==$login) && ($result['password']==$password)) {
				// on debute alors la session
				//session_start();
				// on assigne les variables de session
				$_SESSION['pseudo'] = $_POST['pseudo'];
				$_SESSION['pass'] = $_POST['pass'];
				// on affiche le message de connexion
				echo '<p style="color:green">authentifi&eacute;'.' ['.$_SESSION['pseudo'].']'.'</p>';			 
			} //if			
			else {
				//echo "ko";
				header('Location: http://localhost/index.php');
			}			
		} //while											
	} //if
	$stm->closeCursor();
?>
</pre>

Merci d'avance.
Transact

12/09/2012, 09h47
Rony Rauzduel

Il semblerait que la condition else ne soit jamais pris en compte dans la boucle while.
12/09/2012, 09h53
FirePrawn

Bonjour,

Peut-être parce que tu ne rentres pas dans ton while ?
L'exécution de ta requête est commentée, donc à mon avis le while ne renvoie jamais true...
12/09/2012, 10h08
Rony Rauzduel

J'ai décommenté le execute() mais cela ne change rien, et je rentre bien dans le while.
12/09/2012, 10h13
FirePrawn

Alors ça vient peut être de ta requête.

Ta requête renvoie TOUS les utilisateurs avec leur mot de passe.
Il ne serait pas plus simple de faire un Select login, pass from membres where login = $login plutôt ?
Comme ça s'il y a un résultat c'est bon, sinon c'est pas bon.
12/09/2012, 10h28
Rony Rauzduel

Si j'écris cela :

Code:

$req="select login, password from membres where login =".$login;

J'ai le message d'erreur suivant :

Citation:

Fatal error: Call to a member function fetch() on a non-object in C:\Users\sogia\Desktop\www\wineBusiness\connexion.php on line 19
12/09/2012, 11h11
FirePrawn

Il faut adapter la syntaxe...

J'ai donné cette requête pour te faire comprendre l'idée. Tu utilises PDO, sers t'en.
12/09/2012, 11h18
Rony Rauzduel

Je ne vois pas comment faire avec PDO pour adapter la syntaxe!

Voici un code qui fonctionne à la perfection

Code:

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
 
/** script d'identification du couple login/password prÃ©sent dans la table des membres */
require_once("includes/inc_connexion.php");
$conn = opendb_conn(); // on se connecte Ã* la Base de donnÃ©es
 
if (isset($_POST['submit'])) {
    // bouton submit presse, je traite le formulaire
    $pseudo = (isset($_POST['pseudo'])) ? $_POST['pseudo'] : '';
    $pseudo = htmlentities(trim($pseudo));
    $password = (isset($_POST['password'])) ? $_POST['password'] : '';
    $password = sha1($password);
    /* constitution de la requete avec la clause where RECUPERANT LES PARAMETRES login ET password
      Cette requete a pour avantage de cibler exactement le couple login/password
      ceci evite de faire plus bas un test if() ... else
      Dans la liste de sÃ©lection nous rÃ©cupÃ©rons les champs id et actif de la table producteurs
     */
    $req = "select membre_id,pseudo, password,actif from membres where pseudo='" . $pseudo . "' and password='" . $password . "'";
    // on prÃ©pare la requete a executer sur le serveur
    $stm = $conn->prepare($req);
    // on exÃ©cute la requete
    $stm->execute();
    // PDO::FETCH_ASSOC: retourne un tableau indexÃ© par le nom de la colonne comme retournÃ© dans le jeu de rÃ©sultats
    $result = $stm->fetch(PDO::FETCH_ASSOC);
    /* // pour les tests
      print '<pre>';
      print_r($result);
      print '</pre>';
     */
    $actif = $result['actif']; //rÃ©cupÃ©ration de la valeur du champ 'actif'
    $id = $result['membre_id']; //rÃ©cupÃ©ration de la valeur de l'id producteur en base
    $pseudo_db = $result['pseudo'];
    $password_db = $result['password'];
    // si le compte a Ã©tÃ© activÃ© via le lien du mail d'activation et que le couple pseudo/password correspond
    if (($actif == '1') && ($pseudo == $pseudo_db) && ($password == $password_db)) {
        // on assigne les variables de session ... mais on n'ouvre pas de session pour l'instant
        $_SESSION['pseudo'] = $pseudo;
        $_SESSION['id'] = $id; // cette variable sera utilisÃ©e dans ajout_publierAnnonce.php
        $stm->closeCursor(); // fermeture du curseur
    } else {
        // cependant si le compte est inactif ou que le couple login/password ne correspond pas
        echo "Votre login ou votre mot de passe est incorrect !";
        //header('Location: http://localhost/clothesBusiness/index.php'); // on reste sur la page index.php
    } // fin if(($actif == '1') && ($pseudo == $pseudo_db) && ($password == $password_db))
} //fin if(isset($_POST['submit']))
$conn = null;