[PHP-JS] Pb d'affichage de message d'erreur

**yakup.67** · 08/06/2007, 10h13

Bonjour à tous,
Voilà j'ai un petit problème, je suis en train de créer un site et j'ai une page intitulée cnx.php pour que les utilisateurs puissent se connecter.
Mais quand il y a une erreur lors de la saisie du login ou de mot de passe, le message d'erreur ne s'affiche.
Je pense que c'est à cause du header qui est trop rapide et ne peut donc pas afficher le message d'erreur.

Voici donc le 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
<html>
<head></head>
<body>

<?php

// Récupération des données du formulaire
$login=$_POST['GINLOGIN'];
$mdp=$_POST['GINPASS'];
echo $login, $mdp;
//Appel de la classe Oracle et du fichier de connexion
require_once("../PAGES/include/oracle.class.php");
require_once("../PAGES/include/connect.php");
//Connexion à la BDD Oracle
$bdd = new oracle (NOMUBD,PASSEUBD,SERVEURBD);
$bdd->connecteBDD();


$req="Select * from grcintervenant where GINLOGIN='".$login."'";
$bdd->requeteLMD($req);
$nblignes=$bdd->retourneNombreLigne();
$resultat= $bdd->resultat;
echo $nblignes;
for ($x=0;$x < $nblignes; $x++) {
	$nomint=$resultat[$x]["GINNON"];
	$pass=$resultat[$x]["GINPASS"];
	$nomLogin=$resultat[$x]["GINLOGIN"];
	}
	
		
// Vérification des données dans la base
if (isset($login)) {
	if (isset($pass)) {
		if ($mdp==$pass) {
			$chemin="../PAGES/index.php";
			header("Location:".$chemin);
			exit();
		}
	}
}
	echo 'alert("Erreur! Login ou mot de passe incorrect!")';
	header("location:cnx.php");
	exit();

?>

</body>
</html>

**koopajah** · 08/06/2007, 10h17

Je ne suis pas sur mais je pense que le problème vient de la manière dont tu gère l'affichage du message d'erreur. alert() est une fonction javascript qui va t'afficher un boite de message dans le navigateur (et non pas un texte dans la page html).

Peut etre si tu ajoutes ca :

Code :

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

<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!");</script>

**Celira** · 08/06/2007, 10h44

J'aurais suggéré la même chose que koopajah, en n'oubliant pas les echos autour

Code php :

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

echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!");</script>';

**yakup.67** · 08/06/2007, 11h00

Désolé mais c'est pareil, y a rien qui s'affiche!!
Donc si vous avez une autre solution, je suis preneur.
Merci!!

**mathieugamin** · 12/06/2007, 17h01

Salut,

Ca corrigera pas ton problème, mais tu as une autre erreur importante !

Il ne faut aucun code html (ni même aucun echo) avant header(). Cela génère une erreur et tu ne bougeras pas de la page où tu as lancé le script...

Finalement, je crois que les 2 exit() ne sont pas utiles puisque tu changes de page et que tu arrives à la fin de ton script.

**yakup.67** · 13/06/2007, 15h35

Exact, je l'ai tout de suite enlevé!!

**koopajah** · 13/06/2007, 15h55

Tu as mis echo "alert..." mais tu n'as pas mis les balises <script> avant et apres comme on te l'a indiqué.

Par ailleurs tu n'as pas modifié le probleme pour les headers, tu as tjs des balises html au tout début de ton fichier donc ca ne fonctionnera pas

**mathieugamin** · 13/06/2007, 16h17

Et maintenant, ça fonctionne ?

**yakup.67** · 14/06/2007, 11h33

Non ça ne fonctionne toujours pas .

voici le code après modification :

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
<?php
 
// Récupération des données du formulaire
$login=$_POST['GINLOGIN'];
$mdp=$_POST['GINPASS'];
echo $login, $mdp;
//Appel de la classe Oracle et du fichier de connexion
require_once("../PAGES/include/oracle.class.php");
require_once("../PAGES/include/connect.php");
//Connexion à la BDD Oracle
$bdd = new oracle (NOMUBD,PASSEUBD,SERVEURBD);
$bdd->connecteBDD();
 
 
$req="Select * from grcintervenant where GINLOGIN='".$login."'";
$bdd->requeteLMD($req);
$nblignes=$bdd->retourneNombreLigne();
$resultat= $bdd->resultat;
echo $nblignes;
for ($x=0;$x < $nblignes; $x++) {
	$nomint=$resultat[$x]["GINNON"];
	$pass=$resultat[$x]["GINPASS"];
	$nomLogin=$resultat[$x]["GINLOGIN"];
	}
 
 
// Vérification des données dans la base																					
if (isset($login)) {
	if (isset($pass)) {
		if ($mdp==$pass) {
			$chemin="../PAGES/index.php";
			header("Location:".$chemin);
		}
	}
}
	header("location:cnx.php");
	echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!")</script>';
 
 
?>

**yakup.67** · 14/06/2007, 11h35

Comme vous le voyez, j'ai enlevé les balises html .
Mais ce que je ne comprend pas c'est que les header marchent.
Dès que je met un login ou mdp faux il me rafraichi la page cnx.php
et si le login est juste ça me renvoie à l'index.
Mais le message d'erreur ne s'affiche pas.

**koopajah** · 14/06/2007, 11h42

C'est normal qu'il ne s'affiche pas, rien de ce qui est après un header n'est "exécuté", un header doit être vu comme un die() (hormis que tu lances un autre script a la place).
Tu peux faire qqch comme ca par contre:

Code :

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

header("location:cnx.php?valid=0");

et dans ta page cnx.php, au début, tu fais ca:

Code :

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

1
2
3
4
5
 
if(isset($_GET['valid']) && $_GET['valid'] == 0)
{
echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!")</script>';
}

**yakup.67** · 14/06/2007, 13h23

J'ai essayé mais ça ne donne rien, merci tout de même.
Sinon je vous donne également le code de la page cnx.php.
voici le 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
<html>
<head></head>
<body>
		<?php
 
		//Appel de la classe Oracle et du fichier de connexion
					require_once("../pages/include/oracle.class.php");
		require_once("../pages/include/connect.php");								
 
		//Connexion à la BDD Oracle
		$bdd = new oracle (NOMUBD,PASSEUBD,SERVEURBD);
		$bdd->connecteBDD();
		?>	
 
	        <center><b><font size="6" color="blue">Suivi de maintenance</font></center></b>
 
		<?
		if(isset($_GET['valid']) && $_GET['valid'] == 0)
		{
		echo '<script language="JavaScript"    type="text/javascript">alert("Erreur! Login ou mot de passe  
                incorrect!")</script>'; 
 
		}
 
 
		// on affiche les 2 champs nécessaires pour se connecter
		$connex='<tr><td>Login : </td><td><Input type="text" size=10 name="GINLOGIN"></td></tr>
               <tr><td>Pass : </td><td><Input type="password" size=10 name="GINPASS"></td></tr>';
 
	       echo '<form name="connex" method="POST" action="verif.php"><center><table>';
					print("<center><b>$connex</center></b></br>");
		echo '<tr><td></td><td><Input type="submit" name="valider" value="Connexion"></td></tr></table></center></form>'; //on affiche le bouton Connexion
 
				?>
 
 
</body>
</html>

**yakup.67** · 14/06/2007, 13h26

Ah! Je crois que je me suis trompé, le premier code que j'ai mis est celle de la page verif.php et non cnx.php.
Désolé pour cette erreur.

**mathieugamin** · 14/06/2007, 13h41

Change ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
// Vérification des données dans la base																					
if (isset($login)) {
	if (isset($pass)) {
		if ($mdp==$pass) {
			$chemin="../PAGES/index.php";
			header("Location:".$chemin);
		}
	}
}
	header("location:cnx.php");
	echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!")</script>';
?>

en ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
// Vérification des données dans la base																					
if (isset($login)) {
	if (isset($pass)) {
		if ($mdp==$pass) {
			$chemin="../PAGES/index.php";
			header("Location:".$chemin);
		}
	}
}
	echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!")</script>';
header("location:cnx.php");
?>

Ca devrait quand même fonctionner un peu mieux...

**koopajah** · 14/06/2007, 13h49

je sais pas si c'est lié, mais tu ne peux pas aller a la ligne dans une instruction javascript il me semble, laisse le alert('....'); sur la meme ligne si possible.

**yakup.67** · 14/06/2007, 14h02

Désolé mais je suis obligé de mettre l'exit().
Sinon il ne se connecte pas.
Et avec l'exit, il se connecte mais il n'y a toujours pas de message d'erreur quand le login et mdp est faux!!

Code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
// Vérification des données dans la base																					
if (isset($login)) {
	if (isset($pass)) {
		if ($mdp==$pass) {
			$chemin="../PAGES/index.php";
			header("Location:".$chemin);
			exit();
		}
	}
}
	echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!")</script>';
header("location:cnx.php");

**mathieugamin** · 14/06/2007, 15h27

Bon pour la forme, voilà ton code corrigé:

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
<?php
//Appel de la classe Oracle et du fichier de connexion
	require_once("../pages/include/oracle.class.php");
	require_once("../pages/include/connect.php");								
 
//Connexion à la BDD Oracle
	$bdd = new oracle (NOMUBD,PASSEUBD,SERVEURBD);
	$bdd->connecteBDD();
?>	
 
<center>
	<h2 style="color:blue; font-weight:bold;">Suivi de maintenance</h2>
</center>
 
<?php
if(isset($_GET['valid']) && $_GET['valid'] == 0){
  echo '<script language="JavaScript" type="text/javascript">alert("Erreur! Login ou mot de passe incorrect!");</script>'; 
}
?>
<form name="connex" method="post" action="verif.php">
  <label for="login">Login : </label>
  <input type="text" size="10" name="GINLOGIN" id="login" />
  <label for="pass">Pass : </label>
  <input type="password" size="10" name="GINPASS" id="pass" />
  <input type="submit" name="valider" value="Connexion" />
</form>

Ensuite, tu fais appel à des constantes dans ta classe de connexion, mais où sont-elles définies (NOMUBD,PASSEUBD,SERVEURBD)?

**yakup.67** · 14/06/2007, 15h44

les constantes sont définies dans une page à part appelé connect.php .
Et comment on fait appel à des constantes dans la classe de connexion?

Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?php
/****************************************************************

Page de connexion à la BDD

*****************************************************************/
 
	define ('NOMUBD',"ATALGRC");
	define ('PASSEUBD',"ATALGRC");
	define ('SERVEURBD', "servweb");
?>

**mathieugamin** · 14/06/2007, 15h48

Est-ce que tu peux mettre un

Code :

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

<?php error_reporting(2047); ?>

au début de ton script de vérification ?

S'il y a une erreur, on la verra à coup sûr...

**yakup.67** · 14/06/2007, 15h50

En fait c'est bon!!
Maintenant ça marche parfaitement grâce au code de mathieugamin.
et j'ai rajouté le code qu'avait proposé koopajah à propos du header :

Code :

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

header("location:cnx.php?valid=0");

Merci à tous !!

[PHP-JS] Pb d'affichage de message d'erreur

Langage PHP

Discussions similaires

Partager

Partager