Petit soucis de validation

**philippef** · 15/02/2011, 12h25

Bonjour,

C'est surement simple !! mais je vois pas

Quand je valide le formulaire et que je n'ai pas de message $id_proprio ne s'affiche pas

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
if (isset($_SESSION['id_proprio'])){
	$id_proprio = $_SESSION['id_proprio'];
}
$message = array();
 
if (isset($_POST['valider'])){
	// Affichage des messages/infos
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}
	//si pas de message, enregistrement de l'annonce
	if (empty($message)) {
		echo "$id_proprio";
	}
}

Je fais un echo de $id_proprio juste pour voir si mes valeurs sont correctes

**sabotage** · 15/02/2011, 13h55

Assure toi d'avoir bien démarré la session et controle son contenu.

Code :

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

var_dump($_SESSION);

Au passage simplement

Code :

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

echo $id_proprio;

**philippef** · 15/02/2011, 19h39

Var_dump($_SESSSION) me renvoie bien tous les paramètres passés en session dont id_proprio.

Si je fais un echo $id_proprio juste après if (isset($_POST['valider'])) l'id_proprio apparait bien.

Si je provoque une erreur j'entre bien dans la boucle foreach

Est ce que ce n'est pas dû à $message=array(); ?

**ABCIWEB** · 15/02/2011, 19h55

Envoyé par philippef

Est ce que ce n'est pas dû à $message=array(); ?

Logiquement il faudrait définir le contenu de $message à l'intérieur de la condition if (isset($_POST['valider']))

**sabotage** · 15/02/2011, 20h34

Est ce que ce n'est pas dû à $message=array(); ?

Je pensais que cette ligne n'était pas que pour le test.
Si tu définis manuellement $message, a quoi cela sert-il de tester sa valeur juste en dessous ?

**philippef** · 15/02/2011, 20h51

Envoyé par sabotage

Je pensais que cette ligne n'était pas que pour le test.
Si tu définis manuellement $message, a quoi cela sert-il de tester sa valeur juste en dessous ?

Dans une autre page je controle que le formulaire est correctement rempli et j'utilise $message[] = 'mon texte'; J'ai environ 15 controles

Avant d'enregistrer le formulaire je controle $message . Si il y a des messages je les affiche, s'il n'y a pas de message alors j'enregistre.

**sabotage** · 15/02/2011, 20h53

Pourquoi dans le code que tu nous montres définis tu $message comme un tableau vide et juste en dessous controles-tu s'il est vide ou non ?

**philippef** · 15/02/2011, 21h08

Pardon si je ne suis pas clair dans mon explication.
Je redonnes le code complet de ma page

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
52
53
54
55
56
57
58
59
60
<?php
require('proprietaires/annonces/fonctionsphp.php');
require ('connect.php');
if (isset($_SESSION['id_proprio'])){
	$id_proprio = $_SESSION['id_proprio'];
}
$message = array();
 
$departement = "";
//
$displayDiv0 = 'none';
$displayDiv1 = 'block';
$displayDiv2 = 'none';
$displayDiv3 = 'none';
$displayDivSaison = 'none';
$semaine = array();
$Disponibilite = array();
 
$year = date('Y');
$date = date("Y-m-d");
//récupération du n° de la dernière annonce
$sql = mysql_query("SELECT ref_annonce FROM logements ORDER BY ref_annonce DESC Limit 1");
$nombreligne = mysql_num_rows($sql);
if ($nombreligne > 0){
	$res = mysql_fetch_array($sql);
	$ref_annonce = $res['ref_annonce'];
	$ref_annonce = $ref_annonce + 1;
}
else {
	$ref_annonce = "000001";
}
include ('b_control_formulare.php');
 
//*************************************************************
//   Enregistrements dans la table logements
//*************************************************************
if (isset($_POST['valider'])){
	// Affichage des messages/infos
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}
	//si pas de message, enregistrement de l'annonce
	if (empty($message)) {
		echo $id_proprio $type_logement $codep $ville;
	}
}
//******** FIN *************
if (empty($_SESSION['id_proprio'])){
	$displayDiv1='none'; 
	echo "<div style='position:relative;width:470px;margin-left:10px;background-color:#ffffff;'>";
	echo "<p align='center'><span style='color:red;'><br /><br />Merci de vous enregistrer ou de vous identifier pour déposer une annonce</span></p></div>";
}
else {
	include ('c_insert_update.php');
	include ('d_property_formulare.php');
	include ('e_calendar_formulare.php');
}
?>

j'espère que c'est plus clair comme ça

**sabotage** · 15/02/2011, 21h12

tu as un problème de syntaxe :

Code :

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

1
2
3
if (empty($message)) {
	echo $id_proprio . ' ' . $type_logement . ' ' . $codep $ville;
}

**philippef** · 15/02/2011, 21h27

Même si je ne mets que echo $id_proprio; rien ne s'affiche.

C'est bien ce code qui pose problème

Code :

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

1
2
3
if (empty($message)) {
           echo $id_propio;
}

Puisque ce code fonctionne et il affiche les messages

Code :

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

1
2
3
4
5
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}

Est ce qu'il y a une autre solution ?

**micetf** · 15/02/2011, 21h28

Je ne serais pas étonné si $message n'était pas vide.
Deux manières de le savoir :
- afficher le code source généré et voir si cette ligne html apparait (le contenu du tableau $message a été parcouru, mais il contient un espace) :

Code :

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

<p align="center"><span style="color:red"> </span></p>

- ajouter cette ligne avant le test sur le $_POST['valider'] (test explicite) :

Code :

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

echo (empty($message)) ? '<p>Message vide</p>' : '<p>Message non vide</p>';

Si mon hypothèse se confirme, il faudra aller faire un tour dans b_control_formulare.php et voir comment tu gères $message dans ce script.
Fred

**philippef** · 15/02/2011, 21h45

Je viens de regarder le code source et je n'ai rien vu d'anormal (pas d'espace)

par contre le 2è test affiche 'Message non vide'.

Pour les zones obligatoires mes messages sont construits de la même façon

Code :

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

$message[] = "La zone XX n'est pas renseignée.";

Est ce que ' peut créer un problème?

**micetf** · 15/02/2011, 21h53

Je persiste.
Quelque part dans b_control_formulare.php,
tu dois faire ceci :

Code :

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

$message[]='';

et, dans ce cas, $message n'est pas vide :
il contient un élément vide (ce qui n'est pas pareil).

D'où la ligne html dans le code source :

Code :

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

<p align="center"><span style="color:red"></span></p>

sans espace (je fais une concession) entre les balises <span>.
Si $message était réellement vide, cette ligne n'existerait pas.

Me trompe-je ?

**philippef** · 15/02/2011, 22h01

Tu veux dire que je dois traiter ce code autrement ?

Code :

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

1
2
3
4
5
6
7
if (!empty($_POST['nbre_pieces'])) {
	$chaine = $_POST['nbre_pieces'];
	if (tout_numerique($chaine))
		$message[] = "";
	else
		$message[] = "Le nombre de pièces n'est pas valide.";
}

Je n'ai pas d'espace entre <span> et </span>

**sabotage** · 15/02/2011, 22h07

Tout simplement :

Code :

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

1
2
3
if (isset($_POST['nbre_pieces']) && !tout_numerique($_POST['nbre_pieces'])) {
	$message[] = "Le nombre de pièces n'est pas valide.";
}

**micetf** · 15/02/2011, 22h08

Effectivement,
en remplaçant :

Code :

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

1
2
3
4
	if (tout_numerique($chaine))
		$message[] = "";
	else
		$message[] = "Le nombre de pièces n'est pas valide.";

par

Code :

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

1
2
	if (!tout_numerique($chaine))
		$message[] = "Le nombre de pièces n'est pas valide.";

Lorsqu'il n'y a pas de message, tu devrais voir ton id_proprio s'afficher.
Fred

**philippef** · 15/02/2011, 22h10

Merci à vous Sabotage et Micetf, c'est bien $message= "" qui m'ennuie.

Je viens de corriger et ça marche

Petit soucis de validation

Langage PHP

Discussions similaires

Partager

Partager