CactusCrew

Voici une proposition de code afin de remplacer l'appel à eval dans HTML_QuickForm_element::_findValue()

Suite à mon précédent billet.

Code de la méthode

<?php
function _findValue(&$values)
    {
        if (empty($values)) {
            return null;
        }
        // extrait les sous elements de l'element : element[foo][bar]
	preg_match_all('/([^\[|\]]+)/', $this->getName(), $A_element);
        $return	= $values;
        // boucle sur la liste  des "elements"
        foreach ($A_element[0] as $S_element) {
        	// si l'element existe dans le tableau de retour
		if (isset($return[$S_element])) {
			$return	= $return[$S_element];
		} else {
			$return	= null;
			break;
		}
        }
        return $return;
    }
?>

Performances

Au final, cette méthode s'avère moins gourmande en ressource qu'avec eval.
J'ai effectué des benchs(source) des 2 méthodes, avec PEAR Benchmark, sur une centaine d'itérations :

J'obtiens une meilleure moyenne d'exécution sans eval.

Peut cela peut-il convaincre un éventuel développeur chez PEAR ;)...