Toujours dans le but d'améliorer le code de Phyxo, après avoir supprimé les fonctions globales (au moins en majeure partie), j'ai voulu m'attaquer à la suppression de l'utilisation de global. Cette tâche étant finalement trop complexe à ne niveau, j'ai plutôt décidé de regrouper toutes les requêtes SQL au sein de classes dédiées.
Cela a plusieurs avantages :
- cela permet d'identifier les requêtes similaires et du coup de simplifier le code global en ne répétant pas le code d'une requête,
- cela permet de s'assurer plus facilement que tous les paramètres des requêtes sont protégés contre les injections SQL,
- et cela permet d'optimiser certaines requêtes
Cette tâche est longue et répétitive. Je n'ai pas fini et je n'ai pas encore refactorisé certaines requêtes similaires. Mais j'ai déjà pu éliminé des requêtes identiques, notamment des requêtes complexes faites dans la partie administration et d'autres faites depuis l'API.