Commentaires sur : Manipulation de tableaux associatifs

Par : Calak

Calak — Fri, 28 Jan 2005 22:20:43 +0000

awi!
Et du coup, on gagne encore près de 10% de temps d’exécution ^_^;
C’est vrai, je n’y aurais pas pensé à mettre des quote pour les int aussi…
Ou bien c’était trop simple pour y penser, ou bien le c/c++ et autres languages « évolués » font que perd certaines habitudes ^^

Par : solo

solo — Fri, 28 Jan 2005 15:45:43 +0000

Que penserais-tu de toujours mettre des « ‘ » pour englober tes valeurs. C’est tout à fait jouable si on prend en compte que les champs int de MySQL sont alimentables même si les valeurs insérées sont entourées de simples quotes.

Je ne pense pas que ça soit aussi bourrin que ça en a l’air étant donné que MySQL et PHP(4) ne sont pas pas si sévères quand au typage des données traitées.

Par : Calak

Calak — Fri, 28 Jan 2005 15:40:41 +0000

J’ai essayé mais n’y suis pas arrivé. En fait, array_map attend une fonction.
La fonction en elle même j’ai essayé de l’otimiser.

Sinon, si je ne voulais pas passer par array_map, je devais faire une boucle, et ça aurait bouffé du temps aussi.

Quel serait ta proposition?
( à noter que mon code est approximativement 2fois plus rapide que le sien :p )

Par : solo

solo — Fri, 28 Jan 2005 12:47:11 +0000

Bien vu pour le implode.

Par contre tu pourrais remplacer ta fonction par une écriture conditionnelle de la forme:

« (expression) ? val1 : val2; » ^^

Par : Calak

Calak — Fri, 28 Jan 2005 12:40:19 +0000

désolé, 3 commentaires à la suite

concernant ton premier post, il pourrait être par mal optimisé aussi. Je ne fais pas de tests de bench car là, ça sert même à rien:

// On mémorise la configuration concerant les magic_quotes_gpc
define(« __CFG__MQGPC__ », get_magic_quotes_gpc());
// On crée un tableau avec les différents noms de champs de formulaires postés
$tableau2cles = array_keys($_POST);
// Si les magic_quotes sont activées
if ((__CFG__MQGPC__))
{
// On parcourt le tableau
foreach($tableau2cles as $cle) {
$_POST[$cle] = addslashes($_POST[$cle]);
}
}

j’ai corrigé ça en live, donc j’ai p-e loupé un point-virgule ou une bétise comme ça :p

Par : Calak

Calak — Fri, 28 Jan 2005 12:31:31 +0000

oops j’oubliais une fonction, celle qui rajoute les ‘ ‘ dans le cas où la valeur est une string.
C’est la partie qui bouffe le plus de temps dans tout l’algo.
Au début j’étais passé par « create_function » mais ca prend bcp trop de ressources. Je suis donc passé par une fonction extérieure.

function protect_quote(&$value)
{
if (is_string($value)) $value = « ‘ » . $value . « ‘ »;
}

Par : Calak

Calak — Fri, 28 Jan 2005 12:29:21 +0000

Pourquoi utiliser de grosses boucles bien gourmandes?

Voici le code que j’utilise.
J’ai fais des tests par rapport à ton code, en retirant la partie « mysql » ( afin de mettre en avant la vitesse de l’algo de parsing )
Voici deux version de mon code, une version lisible et une autre bcp moins. Mais j’ai remarqué lors de mes tests que la version « all in-line » est sensiblement plus rapide

function insert_array($table, $array)
{
$sql = ‘INSERT INTO `’ . $this->add_prefix($table) . ‘` (`’ . implode(‘`, `’, array_keys($array)) . ‘`) VALUES (‘ . implode(‘, ‘,array_map($this->protect_quote($value), $array)) . ‘)’;
return $this->query($sql);
}
function insert_array2($table, $array)
{
$sql = ‘INSERT INTO `’ . $this->add_prefix($table) . ‘` ‘;
$sql .= ‘(`’ . implode(‘`, `’, array_keys($array)) . ‘`) ‘;
$sql .= ‘VALUES (‘ . implode(‘, ‘,array_map($this->protect_quote($value), $array)) . ‘)’;
return $this->query($sql);
}