Commentaires sur : Instancier un objet dynamiquement sans eval()

Par : Martin

Martin — Thu, 06 Mar 2008 08:24:11 +0000

En même temps, il faut bien garder à l’esprit que l’on est sur une boucle de 400 000 itérations, et donc que toutes les solutions sont envisageable dans un cas bien précis. On ne peut pas toujours utiliser la méthode la plus rapide, et même la moins rapide utilisée avec parcimonie est tout à fait correcte.

Par : Laurent Deséchalliers : “Tech”Blog » [Veille>Dev] PHP : Bench de la function eval()

Thu, 06 Mar 2008 08:19:56 +0000

[...] A voir sur glagla.org [...]

Par : Olivier Mansour

Olivier Mansour — Thu, 06 Mar 2008 07:06:23 +0000

Merci pour vos commentaires et les compléments de test.

@LaurentJ, je suis tout de même étonné du rapport entre les deux temps de mon script, je m’attendais à un rapport 2, pas 6 !

Par : Gérald

Gérald — Wed, 05 Mar 2008 13:40:52 +0000

Tient, le code est mal passé dans les commentaires….. un strip tags qui doit traîner :-)

Pour résumer, dans le premier cas j’effectue un new ReflectionClass ($class_name) à chaque passage dans le foreach, en plus de l’appel à la méthode newInstance (). Dans le deuxième cas, je ne fais qu’appeler la méthode newInstance () en ayant au préalable instancié la ReflectionClass.

Dans tous les cas donc, new $class_name est plus rapide.

Par : Antoine

Antoine — Wed, 05 Mar 2008 11:31:25 +0000

@Gerald, pris de cours, je terminais mes tests :)

Par : Antoine

Antoine — Wed, 05 Mar 2008 11:13:31 +0000

@Chris, je n’ai pas de retour chariot si je termine mes scripts. Je ne dirais pas que c’est mieux de ne pas fermer ses scripts par contre…

Par : Gérald

Gérald — Wed, 05 Mar 2008 10:52:24 +0000

Pour compléter les tests, voici ce que ça donne avec l’api Reflection

$start_time = microtime(true);
for ($i=0; $i newInstance ();
}
$end_time = microtime(true);

echo ‘time taken (new Reflection) : ‘.($end_time-$start_time).’ s’. »\n »;

$start_time = microtime(true);
$reflection = new ReflectionClass ($class_name);
for ($i=0; $i newInstance ();
}
$end_time = microtime(true);

echo ‘time taken (keeped Reflection) : ‘.($end_time-$start_time).’ s’. »\n »;

time taken (with eval) : 9.55426383018 s
time taken (no eval) : 0.662239074707 s
time taken (new Reflection) : 4.30600905418 s
time taken (keeped Reflection object) : 1.75258493423 s

Par : Chris

Chris — Wed, 05 Mar 2008 10:11:38 +0000

@Antoine : la balise de fermeture n’est pas obligatoire. C’est même mieux d’ailleurs de ne pas l’utiliser pour éviter un retour chariot intempestif en fin de fichier, qui serait envoyé directement sur la sortie standard. Symfony a adopté cette pratique par exemple.

Par : Laurentj

Laurentj — Wed, 05 Mar 2008 09:21:10 +0000

Oui c’est totalement logique, quand on sait que l’évaluation d’un script PHP se fait en deux phases : transformation du code source en bytecode, puis évaluation du bytecode.

Avec eval, on a à chaque itération les deux phases. Tandis qu’en faisant un new direct, on a que la deuxième phase, l’instruction dans la boucle ayant déjà été transformé en bytecode avant l’execution globale du bytecode du script.

Par : Antoine

Antoine — Wed, 05 Mar 2008 08:27:25 +0000

Salut,

il manque une balise de fermeture à ton script. Aussi, il est sans doute utile de préciser que faire new $class_name() ne prends pas plus de temps que de faire new toto().

La fonction call_user_func est un bon compromis pour ce genre de chose. Par exemple, sur mon serveur :
time taken (with eval) : 2.98381304741 s
time taken (no eval) : 0.30176615715 s
time taken (call_user_func) : 0.819909095764 s

Antoine