EXECUTER UNE REQUÊTE CONTENANT PLUSIEURS SOUS-REQUÊTES
coucou747
Messages postés
12303
Date d'inscription
mardi 10 février 2004
Statut
Membre
Dernière intervention
30 juillet 2012
-
5 févr. 2008 à 08:09
cs_morpheus57 Messages postés 121 Date d'inscription vendredi 31 mars 2006 Statut Membre Dernière intervention 30 décembre 2010 - 6 févr. 2008 à 10:24
cs_morpheus57 Messages postés 121 Date d'inscription vendredi 31 mars 2006 Statut Membre Dernière intervention 30 décembre 2010 - 6 févr. 2008 à 10:24
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/45628-executer-une-requete-contenant-plusieurs-sous-requetes
https://codes-sources.commentcamarche.net/source/45628-executer-une-requete-contenant-plusieurs-sous-requetes
6 févr. 2008 à 10:24
Je ne trouve pas que les notes servent à rien, je m'explique : Parfois je suis à la recherche d'un code permettant une fonctionnalité précise. Si je vois que ce code a une note de 1, je sais que je ne pourrais pas l'utiliser sans apporter des changements majeurs... donc je préfère ne pas l'utiliser.
MadM@tt => Je suis désolé, je ne me permet pas de juger tes capacités de programmeur, j'ai simplement donné mon avis sur ce code
6 févr. 2008 à 10:05
une liste de bugs
une liste d'ameliorations
une liste de defauts dans le code
des felicitations
les fond d'ecrans des commentaires seraient classes selon leur utilite
et les notes ne servent tout simplement a rien...
la, si j'ai mis 1 c'est parce-que je trouve ca pretentieux de parler de partage pour 9 lignes de code... Le code ne vaut pas plus et l'auteur n'en a pas conscience, alors...
6 févr. 2008 à 09:45
Note: 1/10
Je plussoie avec vous cher MadM@tt, c'est infame comme note!! beaucoup d'égo de la part des votants... L'administrateur devrait laisser la liberté aux débutants d'accepter d'être noté ou pas par les membres.
6 févr. 2008 à 09:31
5 févr. 2008 à 21:15
Et je n'ai pas de shell mysql sur mon hébergeur.
Je ne peux pas "fuire" le problème, le fait de parser la requête serait une solution.
5 févr. 2008 à 21:01
INSERT INTO table (...) VALUES (...), (...) ;
t'as un seul insert
ensuite, si t'as un ; dans une data ca merde donc bon...
ce que je ferais pour ce cas : je decouperais selon les ; qui ne sont pas dans des chaines, ou mieux, j'ouvrirais une pipe vers le programme shell mysql... et il ferait tout le travail a ma place
5 févr. 2008 à 20:53
Si je veux faire un script qui va insérer des lignes dans un table exportés préalablement d'une autre table (donc sous forme de requetes INSERT), et que j'ai 50 lignes à insérer, je vais pas me les décomposer à la main une à une. (sachant que je ne peux pas faire la copie des lignes "à la main")
5 févr. 2008 à 20:49
si t'as plusieurs requetes a effectuer, tu fais plusieurs query, c'est tout...
5 févr. 2008 à 20:43
5 févr. 2008 à 20:41
/**
* @bug si les ; servent a autre chose que separer les requettes, ca fout la merde et renvoie false en executant la moitie des requetes
*/
pour contourner ca, t'aurais du parser les requetes, je te laisse imaginer la fiabilite du systeme et la consomation de ressources...
bref, non ca n'aurait pas servi...
5 févr. 2008 à 20:37
Et je sais d'expérience qu'il n'est pas évident de savoir qu'on ne peut pas exécuter plusieurs requête dans une seule en php, et comment faire pour contourner ça, donc je pense que ça aurait pu servir à quelques uns.
5 févr. 2008 à 20:14
l'amelioration ici se resumerait a :
- faire un vrai parsing de la requete sql pour savoir ou couper
- faire une gestion des requetes avec echappement anti XSS
- faire une gestion des erreurs avec des Exceptions
- faire une gestion des retours avec des ArrayItterators (un pour les requetes, un pour les results)
si je conclue : t'en as pour au moins 100 fois plus de lignes pour faire un truc correct... t'aurais poste 100 lignes, j'aurais pu penser qu'une amelioration fut possible, mais la : t'as poste 9 lignes, c'est pas une amelioration qu'il faudrait... c'est une refonte complete du code...
tu postes 9 lignes mal documentes, bugges et qui ne servent a rien... c'est gentil, mais tu t'attendais vraiment a recevoir plus de 3/10 ?
5 févr. 2008 à 19:59
Je sais pas trop comment ça se passe sur PHPCS, mais je vais vous expliquer comment ça se passe sur d'autres site de Codes sources (vbfrance par exemple).
Si un code est mauvais, on explique pourquoi et on laisse le temps à l'auteur d'améliorer son code. Je ne cours pas après les bonnes notes, au contraire je part du principe que ce genre de site est la pour aider à progresser et partager des éléments intéressants plutôt que de juger en soi les codes sources de chacun. Mais recevoir 3 notes comme ça ça motive vraiment PAS DU TOUT pour modifier ce code source.
Donc plomber quelqu'un en lui balançant des notes de merde le jour même il poste ce code (et je rappelle que ce code je le met car j'ai passé assez longtemps sur google à chercher pourquoi on pouvait pas exécuter plusieurs requête en un seul mysql_query() et comment faire pour contourner ça facilement, donc j'ai posté ça uniquement dans le but d'aider), ben c'est pas constructif du tout (même si vos commentaires contiennent bien des idées d'améliorations).
Alors ce codes sources j'en ai plus rien à faire, je vais faire l'égoïste de base : pour moi il me convient bien. J'ai plus la motiv du tout
Bonne soirée
5 févr. 2008 à 15:38
5 févr. 2008 à 14:01
5 févr. 2008 à 13:53
Je vais peut être dire une connerie, mais en SQL, IN() permet d'effectuer une sous requête non?
C'est déja ça...
5 févr. 2008 à 13:22
ta fonction n'est meme pas documente aux normes doxygen ou autreelle est mal codee (utilisation des et pas)
elle n'utilise pas les exceptions alors que les requetes sql sont des endroits ou les exceptions sont tres tres pratiques...
tu ne verifies rien de special (connexion a la db, XSS, ...), et ta fonction est buggee
bref, je mets moi aussi 1/10
5 févr. 2008 à 13:10
morpheus57 > Erreur de frappe de ma part. Effectivement je peux ajouter des amélioration, mais bon merci pour le 1, ça fait toujours plaisir de vouloir partager des codes sources
yoman64 > Tu as raison, si un admin passe par la merci de supprimer la source.
5 févr. 2008 à 11:55
Savoir à quel requête ça a planté serait interessant aussi...
5 févr. 2008 à 09:13
Dans la description, tu mets "La fonction renvoie false si un erreur s'est produite." et dans les commentaires tu mets "renvoie l'erreur SQL dans le cas contraire ".
En fait cela renvoi bien false -> donc impossible de savoir quelle est la requête qui a planté.
Je prends un exemple : tu as 25 requêtes et une erreur à la 12ème... il y aura donc 13 requêtes non exécutés et aucun moyen de le savoir ! ! !
Je pense que c'est clairement le genre de script qui doit utiliser les transactions...
De plus, il faut corriger le bug relevé par coucou747.
Bonne chance ! ! !
5 févr. 2008 à 08:09
ca fait planter ton code