Discussion :

[F.Salenko]

Bonjour a tous,

Je voudrais checker une variable du style :

Code :

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

1
2
3
4
5
 
If ($a == 1)
{
...
}

Le truc cest quil est possible que $a ne soit pas sette et cela me gave de faire a chaque fois :

Code :

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

1
2
3
4
5
6
7
8
 
if (isset($s))
{
   If ($a == 1)
   {
   ...
   }
}

Pas reussi a trouver une maniere de detourner le probleme. Auirez vous une idee ?

[rawsrc]

Salut,

Pour garder du code propre, t'as pas trop le choix à vrai dire.
Par contre je me pose la question suivante :
comment tu te débrouilles pour avoir à vérifier l'existence d'une variable ?

Généralement, le seuls isset() se rapportent aux tableaux et tableaux super-globaux genre $_POST, mais une fois dans le code pur et dur, tu ne devrais pas avoir à vérifier ce genre de chose sur tes variables.

Si tu veux éviter ces tests, prends l'habitude d'initialiser tes variables en début de traitement et comme ça tu auras la certitude que PHP ne couinera plus.
En plus c'est une très bonne pratique, je te la conseille très fortement.

[F.Salenko]

En gros, c'est de la requete SQL.

Sous certaines conditions, des queries se declenchent.
Mais ils arrivent que celles ci ne soient pas declenchees et du coup je me retrouve a faire des isset un peu partout.

Je ne peux pas definir mes variables aux debuts car la variable que je mentionne est en realite le resultat d'une query

[Invité]

Si tous tes résultats sont connus, tu peux les stocker dans un tableaux et y faire appel à chaque fois que tu as besoin. Sous forme de fonction peut-être ?

[F.Salenko]

Hmm, ce serait un peu le bazar mais ca pourrait etre une piste...

Je vous tiens informé.

Merci pour vos conseils.

[imikado]

Une solution peut être au départ du script de créer l'ensemble de ces variables à null

Dans certains langages, il est nécessaire d'initialiser toutes les variables en début de script.

[gentil2005]

Dans certains langages, il est nécessaire d'initialiser toutes les variables en début de script.

En php aussi, même si le langage ne met pas une contrainte là dessus! dans le cas présent, si la variable était initialisé, il n'aurait plus à tester si elle existe parce qu'elle le serait forcement, il aurait plutôt à tester si elle contient quelque chose quand il pense y avoir affecté quelque chose...

[imikado]

En php aussi, même si le langage ne met pas une contrainte là dessus!

Php demande à minima que la variale soit initialisé avant de la concaténer, l'utiliser, mais il n'exige pas une déclaration en début de script contrairement à COBOL

dans le cas présent, si la variable était initialisé, il n'aurait plus à tester si elle existe parce qu'elle le serait forcement, il aurait plutôt à tester si elle contient quelque chose quand il pense y avoir affecté quelque chose...

C'est ce qu'il demande: de na pas avoir à faire un isset() sur chacune de ses variables

[Zac699]

Une méthode qui ne satisfera pas les puristes pour la question initiale :

Code :

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

1
2
3
4
5
 
If (@$a == 1)
{
...
}

[jb_gfx]

Si ton but est de tuer les performances de ton script, c'est une bonne méthode.

[Zac699]

Tuer, vraiment ?
J'ai essayé sur une boucle de 100 000 tests, c'est vrai que le @ amène une durée d'exécution de 0.18 s contre 0.06 avec un isset.
Mais sur trois quatre tests la différence est imperceptible.

[grunk]

Y'a pas de miracle.

Soit tu initialise correctement tes variables en début de script :

Code :

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

1
2
3
4
5
$a = 0;
[...]
$a = mafonction();
 
if($a == 1)

Soit tu vérifies la validité de $a

Code :

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

if(isset($a) && $a == 1)

Dans tous les cas faut un minimum de rigueur dans le code écrit