Discussion :

[jlb59]

Bonjour à tous,

En relisant tous mes fichiers de l'administration d'un moteur de recherche, j'ai vu ça

Code :

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

if ((isset($_POST['base'])) && ($_POST['base'] != ""))

Code :

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

if ((isset($_POST['base']) && !empty($_POST['base']))

Ce sont des codes récupérés ou qu'on m'a donnés, je sais plus !

Ces 2 formes de codes testant la valeur d'un champ, quelle serait la meilleure des deux ?

Merci de votre réponse.

[SphynXz]

cela va dépendre de ce que peut contenir ta variable, mais dans les cas les plus général, je conseille la deuxième méthodes.

empty ne renverra pas false si et seulement si la variable est vide !

""
"0"
0
false
null

toute ces cas feront qu'empty renverra false!

empty n'étant pas une fonction mais une structure du langage, ton code sera plus performant avec empty, mais c'est négligeable!

Pourquoi cela va dépendre du contenu? c'est justement parce que pour certains cas, "0", 0, ou false sont des valeurs que tu as peut être besoin de savoir

[Jumano]

Bonjour,
Tu peux toujours faire un petit test avec le code suivant :

Code :

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

1
2
3
4
5
6
<?php if($_POST['test'] != "") {echo "test ok" ;} ?>
<?php if(!empty($_POST['test'])) {echo "test_empty ok" ;} ?>
<form name="form1" method="post" action="test.php">
  <input type="text" name="test" id="test">
  <input type="submit" name="button" id="button" value="Envoyer">
</form>

Essaye de mettre un "0" (zéro) dans le champs et regarde le résultat ...
Personnellement j'utilise la première solution !="" ...
Maintenant comme le dit SphynXz si tu es sûr que ta valeur ne soit jamais égale à zéro ... tu peux utiliser la deuxième solution.

[jlb59]

Merci de tes explications SphynXz,

Comme j'ai pas mal de fichiers qui testent la valeur de certains champs, et qui font par exemple

Code :

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

1
2
3
if ((isset($_POST['url']) && ($_POST['url'] != "")) && 
   ((isset($_POST['titre']) && ($_POST['titre'] != "")) && 
   ((isset($_POST['motscles']) && ($_POST['motscles'] != ""))

il faudrait que je change tout ça en

Code :

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

1
2
3
if ((isset($_POST['url']) && !empty($_POST['url'])) && 
    (isset($_POST['titre']) && !empty($_POST['titre'])) && 
    (isset($_POST['motscles']) && !empty($_POST['motscles'])))

En principe, il ne devrait pas y avoir de problèmes !

La syntaxe ne change pas pour ce qui suit ?

[jlb59]

Bonjour,
Tu peux toujours faire un petit test avec le code suivant :

Bonjour Jumano,

Quand j'ai voulu faire ton test, qui est très bien expliqué d'ailleurs, j'ai eu un Parse Error

Code :

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

Notice: Undefined index: test in G:\SiteQI98\exemples_forum\moteur_recherche_admin\ajout\test.php on line 7

Là, je ne percute pas ! J'ai du mal à comprendre cette histoire d'index.

Essaye de mettre un "0" (zéro) dans le champs et regarde le résultat ...
Personnellement j'utilise la première solution !="" ...
Maintenant comme le dit SphynXz si tu es sûr que ta valeur ne soit jamais égale à zéro ... tu peux utiliser la deuxième solution.

Il est vrai que ce que vous m'avez dit et très explicatif, mais un petit renseignement si tu veux bien :
Si le champ est alphanumérique ou numérique, la syntaxe est la même n'est-ce pas ?

Je croyais avoir compris les explications de SphynXz, mais je me suis posé la question après !

[Jumano]

Pour le message d'erreur essaye de rajouter un isset() :

Code :

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

1
2
3
4
5
<?php if (isset($_POST['test'])) { 
      if($_POST['test'] != "") {echo "test ok" ;} 
      if(!empty($_POST['test'])) {echo "test_empty ok" ;}
	  }
?>

Si le champ est alphanumérique ou numérique, la syntaxe est la même n'est-ce pas ?

La syntaxe sera la même, maintenant si ta variable peut être égale à "0"(zéro) avec §empty çà ne fonctionnera pas, puisqu'il la considèrera comme vide.
Sinon regarde la doc ...

[SphynXz]

Pour ce qui est de l'erreur "Undefined Index", c'est que tu as très simplement copier/coller le bout de code que t'a gracieusement donné jumano. Il esperait que tu l'adapterait à ton code à toi

Si une variable est vide : empty renvoit false
Si une variable contient des données numérique :
-Si elle contient 0 : empty renvoit false
-Sinon empty renvoit true
Si une variable contient "0" : empty renvoit false
Si une variable contient NULL : empty renvoit false
Si ta variable est un tableau dans lequel aucune données n'est présente : empty renvoit false aussi
Si une variable n'est que déclaré : empty renvoit false

Pour empty : le 0 pour le numérique c'est l'équivalent du vide pour du texte

[Jumano]

Il esperait que tu l'adapterait à ton code à toi

Non je penssais juste qu'on pouvait le mettre dans une page vierge pour faire le test ... et ainsi voir le résultat ...

[SphynXz]

je comprend, mais dans ce cas vaux mieux éviter d'utiliser des superglobals pour faire des test, autant tester de simple variables

[jlb59]

Je sais pas si c'est bien ça parce dans les tests de la base de données je n'ai pas de "isset" et ça marche !

Ouh ! Là je m'embrouille...

[SphynXz]

sans "isset" ton contrôle fonctionnera tant que les variables que tu teste existent réellement!

mais une fois que tu testerais une variables qui n'existe pas Aie : Warning!

[jlb59]

Je crois comprendre SphynXz, j'ai bien dis, je crois hein !

Dans un formulaire, si je saisis les champs et que je les teste pour la requête, ça va marcher.
Par contre, si les champs du formulaire restent vides et que j'envoie, c'est là qu'il risque d'y avoir des problèmes.

Je vois la chose comme ça d'après tes explications.

[SphynXz]

C'est exactement çà !

[jlb59]

D'accord, je vérifie tout ça.
Merci SphynXz.

[jlb59]

sans "isset" ton contrôle fonctionnera tant que les variables que tu teste existent réellement!

mais une fois que tu testerais une variables qui n'existe pas Aie : Warning!

Il y a quelque chose qui m'échappe totalement !

Tu m'as dis que si des champs d'un formulaire restaient vide, j'allais avoir des problèmes dans le test avant la requête.

Explications :
Dans un formulaire, j'ai 3 champs

Si j'insère sans qu'aucun champ ne soit rempli, voilà ce que j'ai fait


C'est la même chose s'il manque 1 ou 2 saisie.

Et mon code se résume à ça :

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
<?php
require_once('../config/config_inc.php');
echo '<fieldset>';
echo '<legend>Ajout de mots-clés danse la base de données [<strong>',$nombase,'</strong>]</legend>';
// Vérification de la saisie des champs
if (($_POST['url'] !="") && 
    ($_POST['titre'] !="") && 
    ($_POST['motscles'] !=""))
{
$url     = $_POST["url"] ;
$titre   = $_POST["titre"] ;
$motscles = $_POST["motscles"] ;
 
$sql = "INSERT INTO $nomtable (url,titre,motscles) VALUES ( '$url','$titre','$motscles')";
$requete = mysql_query($sql, $lien) or die('Erreur dans la requête : ' . mysql_error() . ""); 
...

Je ne comprends plus rien, parce que ça marche très bien sans les "ISSET".

[darkstar123456]

Tu ne peux pas tester un champ de formulaire avec isset() car il le sera forcément (théoriquement... parce que ce n'est pas le cas avec IE qd on valide en faisant "enter").

isset() ==> $_GET et variables que tu définis toi-même

empty() ==> $_POST

[jlb59]

Alors comme je fais mes tests avec FF, j'aurais plus de problèmes avec IE ?

[darkstar123456]

Oui et non... vu que si le champ n'est pas rempli, IE (7 ?) te dira que on isset() est "false"

Vu que tu test isset() & empty() et que pour que la condition soit vraie il faut que les 2 retourne true(), la condition ne passera pas et renverra vers l'erreur

[jlb59]

D'accord, d'accord !

En définitive, il faut mettre ISSET et EMPTY dans tous les tests des variables saisies dans un formulaire ?

[SphynXz]

personnellement c'est ce que je fais oui