problème de header [Résolu]

yvonh · 05/06/2007, 10h08

bonjour,
voici le script

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
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<body>
<?php
@$password=$_POST['password'];
 
if (empty($password)) { ?>
 
<form method="POST" action="index.php" enctype="multipart/form-data">
<input type="password" name="password" value="">
<input type="submit" value="Login"></form>
 
<?php
} 
else if (sha1($password)=='06381a70e1eb7eb2c16662e1f60b591dbfa11c87'){
echo 'Good !';
// Change to the URL you want to redirect to
$URL="http://www.monweb.com/fichier.php";
 
header ("Location: $URL");
}
else {
echo 'Go Away !';
}
?> 
</body>
</html>

chaque fois que je rentre le bon mot de passe j'ai le message suivant :

Good !
Warning: Cannot modify header information - headers already sent by (output started at /homepages/xx/-------/htdocs/admin/index.php:5) in /homepages/xx/-------/htdocs/admin/index.php on line 21

a quel endroit ça cloche?
Merci

koopajah · 05/06/2007, 10h14

Tu fais un echo "good" avant d'utiliser les headers et c'est ce qui provoque ce message d'erreur.
Aucun caractère ne doit être envoyé au navigateur avant l'utilisation des headers (comme pour les sessions en fait) donc pas de balises html ou de echo dans le code php.

Il faut donc que tu modifies ton code en conséquence: Tu commences par du code php qui regardes la valeur de $_POST[], fais les vérifications de login/mdp etc, si c'est bon tu utilises header ("Location: $URL");
Sinon, tu affiches le formulaire.

viviboss · 05/06/2007, 10h37

...En sachant que si tu peux, (quoique pour un formulaire aussi petit, on comprend le fait e vouloir mettre le PHP dans le script de départ....), il vaut mieux pour ne pas faire de confusion et ne pas se perdre dans son code, faire appelle a un autre script auquel tu délegueras les actions de formulaire....

Crois par experience, c'est plus propre et ca permet de se retrouver beaucoup plus facilement

Mais effectivement, si tu veux mélanger PHP/HTML, autant mettre les codes de traitements en haut de page (tu y serais de toute les facons obligé si tu traitais des sessions...

)

yvonh · 05/06/2007, 15h37

ok ca marche merci!
>viviboss ok pour le tuyau je l'appliquerai la prochaine fois pour des pages plus conséquentes.

05/06/2007, 10h37	#3
viviboss Membre émérite Inscription : août 2006 Messages : 943 Détails du profil Informations personnelles : Âge : 30 Localisation : France, Gard (Languedoc Roussillon) Informations forums : Inscription : août 2006 Messages : 943 Points : 941 Points : 941	...En sachant que si tu peux, (quoique pour un formulaire aussi petit, on comprend le fait e vouloir mettre le PHP dans le script de départ....), il vaut mieux pour ne pas faire de confusion et ne pas se perdre dans son code, faire appelle a un autre script auquel tu délegueras les actions de formulaire.... Crois par experience, c'est plus propre et ca permet de se retrouver beaucoup plus facilement Mais effectivement, si tu veux mélanger PHP/HTML, autant mettre les codes de traitements en haut de page (tu y serais de toute les facons obligé si tu traitais des sessions... ) __________________ Veni Vidi Vici ------------------------- Mes articles : developpez.com ou bien vbview.net ------------------------- Et SURTOUT ne pas oublier la bible PHP : --> php_manual_fr.chm!!! Et aussi : --> pear_manual_fr.chm!!! Ou encore : --> Les tutoriaux & cours PHP de Développez.com -------------------------
	00

05/06/2007, 15h37	#4
yvonh Membre du Club Inscription : mai 2006 Messages : 137 Détails du profil Informations personnelles : Localisation : France, Paris (Île de France) Informations forums : Inscription : mai 2006 Messages : 137 Points : 52 Points : 52	ok ca marche merci! >viviboss ok pour le tuyau je l'appliquerai la prochaine fois pour des pages plus conséquentes. __________________ http://yvonh.com
	00

05/06/2007, 10h14	#2
koopajah Membre Expert Inscription : janvier 2005 Messages : 2 288 Détails du profil Informations forums : Inscription : janvier 2005 Messages : 2 288 Points : 2 287 Points : 2 287	Tu fais un echo "good" avant d'utiliser les headers et c'est ce qui provoque ce message d'erreur. Aucun caractère ne doit être envoyé au navigateur avant l'utilisation des headers (comme pour les sessions en fait) donc pas de balises html ou de echo dans le code php. Il faut donc que tu modifies ton code en conséquence: Tu commences par du code php qui regardes la valeur de $_POST[], fais les vérifications de login/mdp etc, si c'est bon tu utilises header ("Location: $URL"); Sinon, tu affiches le formulaire.
	00

Outils de la discussion
Afficher une version imprimable Envoyer un lien vers cette page par email