Version imprimable
Bonjour à tous,
Voilà je suis en train de modifier un script
enCode:($id_category ? ' AND c.`id_category` = '.(int)$id_category : '').
Afin que faire un requete à partir d'un array(15,20,25)Code:($id_category ? (is_array($id_category) ? ' AND c.`id_category` IN ('.join(',', $id_category).')':' AND c.`id_category` = '.(int)$id_category) : '').
Mais je souhaiterais de prémunir contre tout attaque sql, mais je me demande quel test pourais je faire?
J'ai pensé à celà.
Y a t il mieux?Code:
2
3
4
5
6
7
Salut,
comme ça :
Code:
2
3
4
5
6
7
8
9
10
Merci beaucop pour ta réponse
Cela m'as permis de découvrirEn revanche je crois qu'il y avais une erreur:Code:array_filter(array_map('intval', (array)
Code:
2
3
4
Du coup je l'ai modifié pour l'addapté à mon code:
Code:
2
3
4
5
6
7
8
9
10
11
12
13
14
15if ( ! empty($id_category)) $ids = array_filter(array_map('intval', (array)$id_category)); $sql = 'SELECT p.*, product_shop.*, pl.* , m.`name` AS manufacturer_name, s.`name` AS supplier_name FROM `'._DB_PREFIX_.'product` p '.Shop::addSqlAssociation('product', 'p').' LEFT JOIN `'._DB_PREFIX_.'product_lang` pl ON (p.`id_product` = pl.`id_product` '.Shop::addSqlRestrictionOnLang('pl').') LEFT JOIN `'._DB_PREFIX_.'manufacturer` m ON (m.`id_manufacturer` = p.`id_manufacturer`) LEFT JOIN `'._DB_PREFIX_.'supplier` s ON (s.`id_supplier` = p.`id_supplier`) LEFT JOIN `'._DB_PREFIX_.'category_product` c ON (c.`id_product` = p.`id_product`) WHERE pl.`id_lang` = '.(int)$id_lang. (( ! empty($ids))?((count($ids) === 1)? ' AND c.`id_category` = '.current($ids): ' AND c.`id_category` IN ('.implode(', ', $ids).')'):''). ($front ? ' AND product_shop.`visibility` IN ("both", "catalog")' : ''). ($only_active ? ' AND product_shop.`active` = 1' : '').' ORDER BY '.(isset($order_by_prefix) ? pSQL($order_by_prefix).'.' : '').($order_by!="rand"?'`'.pSQL($order_by).'` '.pSQL($order_way):"rand()"). ($limit > 0 ? ' LIMIT '.(int)$start.','.(int)$limit : '');
Dernière petit question sachant qu 99% du temps $id_category est (int) et non un array, penser vous que cela risque de ralentir les 99% des requetes?
Le ralentissement c'est de l'ordre de quelques microsecondes, donc à moins d'avoir des dizaines de milliers de requêtes simultanées, tu ne devrais pas voir la différence.
Un conseil, tu dois toujours privilégier la lisibilité du code, toujours ! Pour y parvenir il faut absolument éviter l'imbrication de plusieurs opérateurs ternaire.
Pareil avec les longues chaînes : il est possible d'éviter les concaténations en pagailles, le bordel des guillemets : regarde du côté des notations heredoc et nowdoc.
Un code bien écrit te facilite tellement la vie par la suite..., fais l'effort maintenant et après tu n'y penseras plus.
Dans 100% des cas, les problèmes de performances ne sont pas liés à la mise en page du code...Code:
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
Et puis depuis quelques temps maintenant, avec les système de cache, le code PHP n'est parsé qu'une seule et unique fois donc une super mise en page n'est pas du tout pénalisante bien au contraire.
le delais va être au niveau du select et nul part ailleurs :p
Citation:
Envoyé par rawsrc
Un grand merci.
Cependant c'est le code presta de base. J'apporte juste un amélioration personnelle, mais je récris pas tout.
Bien que je sois entièrement d'accord.
Un grand merci pour la démonstration!