Discussion :

[bat11]

Bonjour à tous,

je n'ai que peu d'expérience avec Virtuemart (et plus généralement avec Mysql), mais je tombe parfois sur des portions de code qui me paraissent peu optimisées ...
A votre avis, dans le cas suivant (fichier ps_order.php > fonction order_status_update) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
// Update the Order Items' status 
$q = "SELECT order_item_id FROM #__{vm}_order_item WHERE order_id=".$d['order_id'];
$db->query($q);
$dbu = new ps_DB;
while ($db->next_record()) {
    $item_id = $db->f("order_item_id");
    $q  = "UPDATE #__{vm}_order_item SET order_status='".$d["order_status"]."'"
    . "\n, mdate='" . $timestamp . "' "
    . "\n WHERE order_item_id=".$item_id;
    $dbu->query( $q );
}

ne serait-il pas plus économique en requêtes de faire directement un UPDATE de tous les items correspondants à l'id de commande ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
// Update the Order Items' status 
$q  = "UPDATE #__{vm}_order_item SET order_status='".$d["order_status"]."'"
. "\n, mdate='" . $timestamp . "' "
. "\n WHERE order_id=".$d['order_id'];
$dbu->query( $q );

Quelle serait l'inconvénient possible (incompatilibité MySql ?) ?

Merci d'avance

[opware2000]

VM a été à l'origine développé par quelqu'un qui codait sans réelle optimisation, du coup ca peut expliquer pourquoi il en reste des traces
c'est vrai que dans ce cas, je ne vois pas pourquoi il n'a pas fait un update directement. Mais bon ce n'est que mon avis

[Cold Hand]

Salut,

Ton second code est clairement meilleur et plus clair. Je ne vois pas d'effet de bord.

La première manière de aurait pu se justifier si l'update s'effectuait sur une table différente que celle utilisée dans le select (et encore, en définissant correctement la clause WHERE, c'est même pas dit). Mais dans le cas présent, je ne comprends pas pourquoi utiliser SELECT et UPDATE.

[bat11]

Merci de votre réponse.
Pour info j'ai vu dans le code d'autres portions vraiment pas très catholiques
Par exemple dans la récupération des états par pays, le SELECT est lancé 234 fois (un select par pays ) !