Discussion :

[bypbop]

Bonjour à tous,
Je viens de passer à php5 depuis peu et je modifie peu à peu mes pages PHP.

Voila j'ai bien compris qu'il fallait declarer toutes les variables et je m'efforce peu à peu de le faire.

Mais il y a une chose que je comprends pas comme par exemple lors d'un passe de variable via url rewritting.

Je passe ma variable et en debut de script je test si elle existe en faisant :

Code :

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

1
2
3
4
 
if (isset($_GET['nom_categorieurl'])){$nom_categorieurl = $_GET['nom_categorieurl'];}
 
if (isset($_GET['recherche'])){$recherche = $_GET['recherche'];}

ce qui supprime mon erreur Notice. Mais si aprés je fais :

Code :

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

1
2
 
echo "<title>titre de ma page $nom_categorieurl $recherche</title>";

Faut il tester à chaque fois si la variable existe ou peut on le faire une fois en debut de code et desactiver cette erreur de variable non declarée pour tt le reste du code ?

Cordialement,
bypbop

[sabotage]

Sois logique, la réponse depend donc de ton cas.
Par exemple, on peut imaginer que
- si "titre" n'est pas passé dans l'url, tu mettras un titre par défaut à la page
- si "recherche" n'est pas passé dans l'url, tu n'executeras pas la requête SQL de recherche.

[Climoo]

D'accord avec sabotage.
Il suffit que tu initialises tes valeurs dans tous les cas, avec des valeurs par défaut (chaine vide ?), et pas uniquement dans ton if.

Code :

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

1
2
3
4
$nom_categorieurl = "";
if (isset($_GET['nom_categorieurl'])) {
 $nom_categorieurl = $_GET['nom_categorieurl'];
}

Après ton isset, du coup, tu es sûr que quoi qu'il arrive, ta variable est initialisée. Et donc, jamais de warning. Moralité : ne jamais utiliser des variables tant que l'on est pas sur qu'elles soient bien initialisées.

[sabotage]

Plus court en version ternaire :

Code :

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

$nom_categorieurl = isset($_GET['nom_categorieurl']) ? $_GET['nom_categorieurl'] : 'valeur par défaut';{

Mais il ne faut pas mettre une valeur par défaut dans tous les cas comme expliqué au dessus.

[bypbop]

ok merci à tous je vais essayer tt ca ;-)
faut juste s'habituer à coder en PHP 5

Cordialement,
bypbop

[ABCIWEB]

faut juste s'habituer à coder en PHP 5

Oui enfin le principe de préciser l'origine d'une variable et de lui affecter (la plupart du temps) une valeur par défaut n'est pas spécifique à php5. On pouvait déjà faire ainsi en php4 et c'était même très recommandé. Mais php4 autorisait par facilité de ne pas le faire ce qui pouvait engendrer assez facilement des problèmes de sécurité. C'est pour avoir un code plus sûr que php5 renvoie maintenant des notices.
En d'autres termes, on dit que l'on code en php5 quand on emploie des fonctions ou classes qui sont apparues avec php5 mais sur le principe du contrôle des variables rien n'a changé : moins tu contrôle tes variables et plus tu risque d'avoir de mauvaises surprises, quelque soit les langages et leur version. Php5 incite simplement à plus de rigueur à ce niveau, mais c'est loin d'être la principale évolution par rapport à php4.