unset sans isset ?

Bonjour,

Je suis en train de développer un site.
Je dois régulièrement, au fil du programme, supprimer des variables de sessions.

Donc je fais :

Code:

---

1 2 3 4

if (isset($_SESSION['monTrucAEffacer'])) { unset($_SESSION['monTrucAEffacer']); }

---

C'est un peu redondant.
J'ai lu le manuel, si on fait un unset sur une variable qui n'existe pas, il n'y a pas d'erreur retournée.

Je pourrai donc me contenter de :

Code:

---

1 2	unset($_SESSION['monTrucAEffacer']);

---

et puis c'est tout.

Mais ... et dans l'avenir ? Le site durera (je l'espère) quelques années, j'aimerai m'éviter de devoir remettre les mains dans le cambouis pour ça.

Vous faites comment vous ? (vous aussi vous vous faites des noeuds dans le cerveau pour des broutilles de ce genre :mouarf: ?)

D'avance merci pour vos avis :D

Bonjour,

ce que je ferais pour éviter d'avoir plus tard 50000 modifs à faire (et bien sûr d'oublier d'en faire), je créerais une fonction :

Code:

---

1 2 3

fct_unset($var){ unset($var); }

---

et partout où tu dois supprimer une variable : fct_unset($variable_a_supprimer);. Ainsi, si un jour, il faut modifier la façon de supprimer une variable, y a juste la fonction à mettre à jour.

Citation:

---

Vous faites comment vous ? (vous aussi vous vous faites des nœuds dans le cerveau pour des broutilles de ce genre ?)

---

Non.

Aussi, un site internet est quelque chose d'éphémère, sa durée de vie est courte et encore plus courte sans qu'il soit modifié voire complètement refait. Donc en partant de ce constat, inutile de trop te préoccuper de ce genre de choses, et personne n'a de boule de cristal pour savoir si telle ou telle pratique sera toujours possible dans le futur. (sauf mention contraire du manuel PHP, qui peut décourager certains usages ou fonctions voués à disparaître dans les prochaines versions). Tu peux par contre regarder sur le manuel, si ce que tu utilises avec la version de PHP que tu pratiques a toujours court dans les versions plus récentes.

Maintenant l'un comme l'autre ne me choque pas plus que ça: tester si une variable existe avant d'appliquer unset découle d'un certain bon sens, mais il se trouve que ce n'est pas nécessaire donc tu peux sucrer le test. Personnellement je pencherais plutôt pour la solution courte, mais c'est une question de goût (certains trouvent plus lisible un code très verbeux, ce n'est pas du tout mon cas, pour moi la lisibilité est dans la concision du code).

Citation:

---

Envoyé par laurentSc

Bonjour,

ce que je ferais pour éviter d'avoir plus tard 50000 modifs à faire (et bien sûr d'oublier d'en faire), je créerais une fonction :

Code:

---

1 2 3

fct_unset($var){ unset($var); }

---

et partout où tu dois supprimer une variable : fct_unset($variable_a_supprimer);. Ainsi, si un jour, il faut modifier la façon de supprimer une variable, y a juste la fonction à mettre à jour.

---

Que nenni! unset dans une fonction se contentera de détruire la copie locale de $var dans le scope de la fonction, elle laissera la variable passée en paramètre inchangée.

En effet je viens de tomber dessus, c'est le 1er avertissement dans le manuel PHP. :aie:

Bon je vais laisser mes isset finalement. C'est lourd, mais au moins j'aurai pas de mauvaises surprises (j'en aurai ailleurs à n'en pas douter :mrgreen:)

Merci pour vos avis :ccool:

Citation:

---

Envoyé par RowanMayfair

Mais ... et dans l'avenir ? Le site durera (je l'espère) quelques années, j'aimerai m'éviter de devoir remettre les mains dans le cambouis pour ça.

---

Les probabilités que le fonctionnement d'unset change sont quand même minime. Je m'embêterais pas trop avec le test précédent le unset. Et quand bien même un jour ca arrive des outils de migration permettent de faire ce genre de modif à ta place (Rector par exemple)

Citation:

---

Envoyé par CosmoKnacki

Aussi, un site internet est quelque chose d'éphémère, sa durée de vie est courte et encore plus courte sans qu'il soit modifié voire complètement refait.

---

Tu as une vision surement plus consumer/marketing du web , mais j'ai des applis web qui fonctionnent depuis des années dans l'industrie et qu'il n'est absolument pas question de modifier tant que les choses auxquels elles sont connectés n'évoluent pas (coût du dév, coût de la formation utilisateur, etc ...). Donc il faut maintenir à jour avec les technos serveurs récentes mais certainement pas tout réécrire et ce pendant de (trop) nombreuses années.

Citation:

---

Envoyé par CosmoKnacki

Que nenni! unset dans une fonction se contentera de détruire la copie locale de $var dans le scope de la fonction, elle laissera la variable passée en paramètre inchangée.

---

OK, je savais pas ; alors autre possibilité (pour mettre le unset directement dans la page où a été créée la variable, sans pour autant avoir des milliers de modifs à faire) : on écrit ce fichier :
fct_unset.php

Code:

---

unset($var);

---

et partout où on a une variable à effacer :

Code:

---

1 2	&$var=&$variable_a_supprimer; include (path)/fct_unset.php

---

Est-ce que ça irait ?

Le but d'unset est que la variable n'existe plus dans la suite du code. Si elle n'existait pas déjà, on s'en fout. Donc je doute aussi que le changement d'unset change et que les dev de php obligent à vérifier l'existence de la variable, et j'enlèverais les tests.

Citation:

---

Envoyé par laurentSc

Est-ce que ça irait ?

---

En théorie, c'est bien tenté ! En pratique, cette utilisation me semble horrible et à priori ça ne va détruire que la variable $var dans ton exemple : https://www.php.net/manual/fr/langua...nces.unset.php

Dans les commentaires utilisateur sur la documentation php, il est également recommandé de ne pas faire un test avec isset() avant de détruire une variable avec unset() : https://www.php.net/manual/fr/functi....php#usernotes

PHP pourra déclencher une notice/un warning si on essaye de supprimer un élément d'un tableau qui n'existe pas (ou la propriété d'un objet qui n'existe pas) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

unset($unexist); // pas d'erreur   $exist = []; unset($exist['nokey']); // pas d'erreur   unset($unexist['nokey']); // Undefined variable unexist   $exist = new StdClass(); unset($exist->noprop); // pas d'erreur   unset($unexist->noprop); // Undefined variable unexist si php <= 7.3, pas d'erreur si php >= 7.4

---

Bonsoir,

pourquoi mon code te semble horrible ?

Cela dit, ton lien semble effectivement dire que le but ne serait pas atteint :aie:

Enfin, selon ce que je comprends dans l'autre lien, le test isset n'est pas déconseillé ; ils disent seulement qu'il est inutile.

Ce qui est inutile est déconseillé puisque c'est plus de code à lire et à maintenir et c'est un test de plus donc c'est mauvais pour les perfs (même si ça ne va pas se connaître de façon visible). Le plus simple est le mieux.

En toute honnêteté unset des variables reste un cas très particulier. En général on préfère mettre la variable à null (par exemple). Plutôt que de s'embêter à la supprimer et donc induire des tests d'existence.

Citation:

---

Envoyé par laurentSc

pourquoi mon code te semble horrible ?

---

Je n'ai pas dis que ton code est horrible, le fait de passer la variable par référence était une bonne idée sur le principe (en tout cas c'est original comme solution, même si je préfère ta première proposition concernant l'utilisation d'une fonction). Mais son utilisation serait un vrai casse tête, pour t'en convaincre essaye de reprendre un projet PHP et de remplacer toutes les fonctions par des scripts avec un include (et attention au nommage des variables dans le contexte global !).

De plus, si l'objectif de cette solution est d'éviter de modifier le code si jamais la fonction unset() venait à changer, ceci ne fonctionnera pas dans tous les cas. Par exemple, imaginons que demain la fonction unset() requiert un 2e paramètre, il faudra alors modifier l'utilisation de cette fonction dans le script à inclure mais il faudra également modifier tous les endroits où ce script est inclus pour ajouter le deuxième paramètre lors de l'appel.