Formulaire de connexion

**oOBaalberithOo** · 01/10/2013, 15h27

Bonjour à tous,

Voici j'ai un petit souci sur mon code que je n'arrive pas à m'expliquer.
Il s'agit d'un formulaire de connexion avec vérification des entrées de l'utilisateur avant l'envoi.

Et j'ai un souci au niveau des vérification.

Je check tout d'abord si il n'y a pas de champ vide (ça, ça marche), ensuite si ma requête obtient une réponse ou pas dans la base avec le login/password et c'est là qu'il y a un souci car si un mauvais login ou mot de passe est entrée je tombe sur une page blanche au lieu du message d'erreur, en revanche tout fonctionne si le login/password est bon.

Voici 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
<?php
mysql_connect("localhost", "mysql_user", "mysql_password") or die("Impossible de se connecter : " . mysql_error());
 
if(isset($_POST[''connexion]) && $_POST['connexion'] == 'Connexion'){
	if(!empty($_POST['login']) &&  !empty($_POST['login'])){
		//si une entrée de la base contient login / pass
		$mysql_result = mysql_query("SELECT login, groupe, nom, prenom, password FROM users WHERE login='".mysql_real_escape_string($_POST['login'])."' AND password='".mysql_real_escape_string($_POST['password']."'");
 
		$nbUsers = mysql_num_rows($mysql_result) or die(mysql_error());
		$ligne = mysql_fetch_array($mysql_result) or die(mysql_error());
 
		mysql_close;
 
		// si oin obtient une réponse
		if($nbUsers > 0){
			session_start();
			$_SESSION['login'] = $ligne[0];
			$_SESSION['groupe'] = $ligne[1];
			$_SESSION['nom'] = $ligne[2];
			$_SESSION['prenom'] = $ligne[3];
			header('Location: index.php?page=Accueil');
			exit();
		}
		else{
			$errreur = 'Login ou mot de passe incorrect.';
		}
	}
	else{
		$erreur = 'Au moins un de ces champs est vide.';
	}
}
?>
 
<form action="index.php?page=log" method="POST" name="log">
	<table>
		<tr>
			<td>Login</td>
			<td><input type="text" name="login" /></td>
		</tr>
		<tr>
			<td>Mot de passe</td>
			<td><input type="password" name="password" /></td>
		</tr>
		<tr>
			<td colspan="2">
				<div><?php if(isset($erreur)) echo $erreur; ?></div>
			<td>
		</tr>
	</table>
</form>

**Celira** · 01/10/2013, 15h34

Une page blanche, c'est une fatal error pas affichée. Active l'affichage des erreurs et regarde le message

**Bovino** · 01/10/2013, 15h39

Code :

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

if(isset($_POST[''connexion])...)

Il n'y a rien qui te choques là ?

**oOBaalberithOo** · 01/10/2013, 16h04

Envoyé par Celira

Une page blanche, c'est une fatal error pas affichée. Active l'affichage des erreurs et regarde le message

L'affichage des erreurs est déjà activé. Justement si ça avait été un problème de syntaxe ou une erreur dans le genre au moins je l'aurais vu tout de suite

Envoyé par Bovino

Code :

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

if(isset($_POST[''connexion])...)

Il n'y a rien qui te choques là ?

Oups

Mais ce n'est qu'une erreur de recopie du code (J'ai pas pu faire de copier-coller du code, j'ai dû le retaper à la main) j'ai pas fait l'erreur dans le code d'origine.

**Bovino** · 01/10/2013, 16h14

si un mauvais login ou mot de passe est entrée je tombe sur une page blanche au lieu du message d'erreur

Tu veux dire quand tu entres dans le cas

Code :

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

1
2
3
else{
	$errreur = 'Login ou mot de passe incorrect.';
}

**ABCIWEB** · 01/10/2013, 16h20

Il y a aussi :

Code :

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

if(!empty($_POST['login']) &&  !empty($_POST['login']))

**oOBaalberithOo** · 01/10/2013, 16h26

Envoyé par Bovino

Tu veux dire quand tu entres dans le cas

Code :

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

1
2
3
else{
	$errreur = 'Login ou mot de passe incorrect.';
}

Quelle truffe je fais

. J'aurais aimé que ce soit ça mais pareil que précédemment, mal recopié.

Idem pour

Code :

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

if(!empty($_POST['login']) &&  !empty($_POST['login']))

à la place de

Code :

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

if(!empty($_POST['login']) &&  !empty($_POST['password']))

J'ai re-checké minutieusement, il n'y a normalement plus d'erreur de ce type.

Désolé

**oOBaalberithOo** · 01/10/2013, 16h34

Le PC sur lequel je bosse à ses ports USB bloqués et n'est pas online, je peux donc pas directement faire un copié-collé. Et j'ai voulu poster trop vite

Je reposte le code sans les faute de recopie pour une meilleure lecture

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
<?php
mysql_connect("localhost", "mysql_user", "mysql_password") or die("Impossible de se connecter : " . mysql_error());
 
 
if(isset($_POST['connexion']) && $_POST['connexion'] == 'Connexion'){
	if(!empty($_POST['login']) &&  !empty($_POST['password'])){
		//si une entrée de la base contient login / pass
		$mysql_result = mysql_query("SELECT login, groupe, nom, prenom, password FROM users WHERE login='".mysql_real_escape_string($_POST['login'])."' AND password='".mysql_real_escape_string($_POST['password']."'");
 
		$nbUsers = mysql_num_rows($mysql_result) or die(mysql_error());
		$ligne = mysql_fetch_array($mysql_result) or die(mysql_error());
 
		mysql_close;
 
		// si on obtient une réponse
		if($nbUsers > 0){
			session_start();
			$_SESSION['login'] = $ligne[0];
			$_SESSION['groupe'] = $ligne[1];
			$_SESSION['nom'] = $ligne[2];
			$_SESSION['prenom'] = $ligne[3];
			header('Location: index.php?page=Accueil');
			exit();
		}
		else{
			$erreur = 'Login ou mot de passe incorrect.';
		}
	}
	else{
		$erreur = 'Au moins un de ces champs est vide.';
	}
}
?>
 
<form action="index.php?page=log" method="POST" name="log">
	<table>
		<tr>
			<td>Login</td>
			<td><input type="text" name="login" /></td>
		</tr>
		<tr>
			<td>Mot de passe</td>
			<td><input type="password" name="password" /></td>
		</tr>
		<tr>
			<td colspan="2">
				<div><?php if(isset($erreur)) echo $erreur; ?></div>
			<td>
		</tr>
	</table>
</form>

Encore désolé

**Bovino** · 01/10/2013, 16h39

Code :

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

if(isset($_POST['connexion'])...)

Il est où le champ "connexion" ?

**oOBaalberithOo** · 01/10/2013, 16h46

Bovino Tu auras le droit de me châtier

J'ai sauté une ligne du tableau en recopiant aussi pffffff.

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
 
<form action="index.php?page=log" method="POST" name="log">
	<table>
		<tr>
			<td>Login</td>
			<td><input type="text" name="login" /></td>
		</tr>
		<tr>
			<td>Mot de passe</td>
			<td><input type="password" name="password" /></td>
		</tr>
		<tr>
			<td colspan="2"><input type="submit" name="connexion" value="Connexion" /></td>
		</tr>
		<tr>
			<td colspan="2">
				<div><?php if(isset($erreur)) echo $erreur; ?></div>
			<td>
		</tr>
	</table>
</form>

**papajoker** · 01/10/2013, 16h59

Bonjour,

ce n'est pas la redirection ?

Code :

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

header('Location: index.php?page=accueil');

peut être que Accueil <=> accueil si il cherche ensuite Accueil.php ??

ou alors ceci

mysql_close;

ps: au passage je mettrais le mysql_connect après les if $_POST

**oOBaalberithOo** · 01/10/2013, 17h10

Bonjour Papajoker,

Non ça ne viens pas de la redirection, elle fonctionne bien lorsque que le bon login/password est renseigné. Le fait qu'il n'y ai pas l'extension .php n'est pas une erreur, c'est juste géré dans la page index.php dans laquelle je fais un include des pages du site dont celle-ci.

mysql_close(); j'y avais pensé aussi en le déplaçant à la fin du code et même en le supprimant, mais ça ne vient pas de là.

**ABCIWEB** · 01/10/2013, 18h26

Envoyé par oOBaalberithOo

mysql_close(); j'y avais pensé aussi en le déplaçant à la fin du code et même en le supprimant, mais ça ne vient pas de là.

Cela ne va pas t'aider pour le pb présent mais d'une manière générale n'utilise pas mysql_close(). PHP se débrouille bien tout seul et sauf rares exceptions cela apporte bien plus souvent des problèmes qu'une optimisation quelconque.

**oOBaalberithOo** · 02/10/2013, 09h27

Ok, merci du conseil.

Formulaire de connexion

Langage PHP

Discussions similaires

Partager

Partager