Pourquoi avoir mis lire_meta() en vieille def, quand on voit la simplification de code que son usage donne ?
Je ne sais pas ce qui a motivé cette mise à l'écart.
Je pense aussi que c'était bien plus pratique.
Et même, je voudrais bien, à la manière de lire_config() de cfg (qui n'est finalement qu'un extension de lire_meta()) une interface du genre:
function lire_meta($nom, $defaut)
Qui éviterait de nombreux tests d'existence.
--
toggg
Pourquoi avoir mis lire_meta() en vieille def, quand on voit la
simplification de code que son usage donne ?
Normalement on se contente de
return $GLOBALS['meta']['kitspip_serveur'];
C'est vrai que lire_meta() était bien jolie (l'argument d'Emmanuel
était que ça faisait un appel de fonction, a priori plus coûteux
qu'une lecture directe dans le tableau).
Pourquoi avoir mis lire_meta() en vieille def, quand on voit la
simplification de code que son usage donne ?
Normalement on se contente de
return $GLOBALS['meta']['kitspip_serveur'];
Avec des warnings à la clef, pas très propre...
C'est vrai que lire_meta() était bien jolie (l'argument d'Emmanuel
était que ça faisait un appel de fonction, a priori plus coûteux
qu'une lecture directe dans le tableau).
Je viens de regarder le code dans vieilles_defs.php
Il me semblait que lire_meta() allait chercher les données en base si elles n'étaient pas encore chargées, mais ce n'est pas (plus ?) le cas.
Si on utilisait lire_meta() partout, on pourrait y avoir un tableau en variable statique avec les données de meta, ça viderait un peu $GLOBALS et ça permettrait le chargement dynamique, à la première demande, et non le chargement systématique de lire_metas() quand on n'en aura peut-être jamais besoin...
Il me semblait que lire_meta() allait chercher les données en base si
elles n'étaient pas encore chargées, mais ce n'est pas (plus ?) le cas.
Sauf erreur, ça n'a jamais été le cas.
Si on utilisait lire_meta() partout, on pourrait y avoir un tableau
en variable statique avec les données de meta, ça viderait un peu
$GLOBALS et ça permettrait le chargement dynamique, à la première
demande, et non le chargement systématique de lire_metas() quand on
n'en aura peut-être jamais besoin...
Il me semblait que lire_meta() allait chercher les données en base si
elles n'étaient pas encore chargées, mais ce n'est pas (plus ?) le cas.
Sauf erreur, ça n'a jamais été le cas.
OK, je dois confondre avec autre chose.
Si on utilisait lire_meta() partout, on pourrait y avoir un tableau
en variable statique avec les données de meta, ça viderait un peu
$GLOBALS et ça permettrait le chargement dynamique, à la première
demande, et non le chargement systématique de lire_metas() quand on
n'en aura peut-être jamais besoin...
On en a *toujours* besoin (vraiment).
OK. Reste à savoir si on a toujours besoin de deux ou trois metas, ou vraiment de toutes, parce que ça commence à bien se remplir.
Mais bon, charger les metas unes par unes ne serait sans doute pas performant de toute façon...
Emmanuel, c'est vraiment si pénalisant que ça, en termes de perf, l'appel de lire_meta() ?
