Discussion :

[popy67]

Bonjour,

Je viens de créer un formulaire et je souhaite que les champs soient autocomplétés en cas d'erreur ainsi j'ai créer le script suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
...
else
{
echo "
<fieldset><legend>Formulaire de contact</legend>
<br />
<form name=\"".$_SERVER["PHP_SELF"]."\" method=\"post\">
<table>
<tr>
<td><label>Nom ou Raison Sociale</label></td>
<td> : <input type=\"text\" name=\"nom\" style=\"border:1px #C9C9C9 solid; width:150px;\" value=\"".if($_POST["nom"] != \"\"){ print($_POST["nom"]);}."\" /> *</td>						 </tr>
...

Le if figurant dans le champ "nom" créer une erreur, comment faire ?

Merci d'avance pour votre aide :-)

[sebhm]

Bonjour,

il faut découper ton instruction en plusieurs

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
 
...
else
{
echo "
<fieldset><legend>Formulaire de contact</legend>
<br />
<form name=\"".$_SERVER["PHP_SELF"]."\" method=\"post\">
<table>
<tr>
<td><label>Nom ou Raison Sociale</label></td>
<td> : ";
echo "<input type=\"text\" name=\"nom\" style=\"border:1px #C9C9C9 solid; width:150px;\" value=\"";
if($_POST["nom"] != "") {
  print($_POST["nom"]);
}
echo "\" /> *</td>
</tr>
...

je te conseillerais ensuite de découper mieux que ca tes instructions (question de lisibilité).
Et d'utiliser des fonctions comme 'empty()' pour tes tests

[popy67]

Bonjour,
je te conseillerais ensuite de découper mieux que ca tes instructions (question de lisibilité).

Et d'utiliser des fonctions comme 'empty()' pour tes tests

Bonjour,

Ok, je pensais qu'il était possible de faire des concatenations pour les tests.

En ce qui concerne empty() , fut il une époque ou j'utilisais isset() pour les tests mais suite à plusieurs renvois incorrect (la méthode m'indiquait un champ complété alors qu'il ne l'était pas) j'ai trouvé la solution non fiable et ai preferé utiliser ma méthode. Cela post t-il un problème ?

Cordialement,

[Petibidon]

Sinon tu peux t'arranger avec l'opérateur ternaire

Code :

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

condition ? true : false

http://fr.php.net/manual/fr/language...comparison.php

C'est assez pratique mais a tendance à rendre le code moins lisible. A utiliser avec parcimonie.

[sabotage]

(la méthode m'indiquait un champ complété alors qu'il ne l'était pas) j'ai trouvé la solution non fiable

empty() et isset() sont fiables dans le sens ou ils font exactement ce qu'ils sont censé faire.
Apres c'est plutot le comportement du HTML qu'il faut bien comprendre : par exemple un champ texte non rempli est TRUE sur isset() et TRUE sur empty().

Après y a t'il dans ton cas un interet a utiliser empty ...