Discussion :

[jrm83]

Bonjour,

Je viens de créer mon premier script dynamique (Faut bien débuter un jour !), mais il ne me renvoie rien. Si j'enlève la boucle while, j'obtiens la première ligne de ma requête.

Mon objectif est d'obtenir la liste des personnes présentes par 1/4h. A terme la requête sera exploi

Merci par avance pour votre retour.

[iberserk]

Oulà vous partez dans du code extrémement complexe et peu performant ce genre de requète peut se faire beaucoups plus simplement...

Mais pour répondre à votre question a chaque boucle vous écrasez votre variable @SQL en faisant @SQL=.... il faut la concaténer ainsi SET @SQL=@SQL +...

[jrm83]

Merci pour votre retour.

J'ai rajouté "set @SQL = @SQL + ' union ..." dans le "else".

Mais cela ne change rien au résultat :
- La requête SQL ne s'affiche pas (Print)
- Le résultat est vide.

Avec mon script de départ même si j'écrase la requête à chaque boucle, je devrais avoir au final la dernière partie de la requête qui comme par un "union" et avoir un message d'erreur. Non ? Ce n'est pas le cas

Je suis également preneur pour simplifier ma requête.

[escartefigue]

Bonjour,

Vérifiez que l'une des composantes de la concaténation n'est pas vide, c'est le cas classique de chaine finale vide

Par ailleurs, il est préférable de construire une solution ensembliste plutôt que de procéder par boucle, votre traitement sera beaucoup plus performant

[iberserk]

vous faites des comparaison sur des chaines ca ne peut pas marcher comme vous le souhaitez! : (boucle while)
vous etes vous assuré que vous rentriez bien dans votre boucle?

[jrm83]

Bonjour escartefigue,

Etant novice, je ne sais pas ce que veut dire "une solution ensembliste". Pouvez-vous me préciser ? Merci par avance.



iberserk,

Mes deux données sont bien des Varchar(). Les heures dans les tables de l'application sur laquelle je dois faire mon édition sont sous ce format.

[escartefigue]

C'est à dire que plutôt que de construire une boucle, et d'exécuter une requête à chaque itération, il est préférable de construire une requête qui sélectionne toutes les lignes dont vous avez besoin en une seule fois.
Vu que dans votre cas vous voulez une ligne pour chaque plage de 15 minutes, le plus simple est de construire une table calendrier qui contient les plages dont vous avez besoin, et de faire une jointure avec les tables de votre requête.

EDIT : je viens de voir votre réponse

iberserk,

Mes deux données sont bien des Varchar(). Les heures dans les tables de l'application sur laquelle je dois faire mon édition sont sous ce format.

Si vos dates et/ou heures sont au format Varchar dans votre base de données, alors allez tirer les oreilles du coupable et demandez lui de corriger ca d'urgence, car c'est la source de très nombreux problèmes quasi insurmontables : résultats faux, performances désastreuses, requêtes alambiquées, portabilité sur d'autres SGBD compliquée etc...
Une date c'est un type date (ou date time ou timestamp), une heure c'est un type time, point barre !

[jrm83]

Merci à vous deux pour votre aide.

Mon script fonctionne.

Escartefigue,

Je vais étudier votre solution avec une table calendrier. En revanche, je ne dirais rien à l'éditeur du soft .

Bonne journée à vous deux et encore merci.

[escartefigue]

En revanche, je ne dirais rien à l'éditeur du soft .

Et pourquoi donc ? c'est une très grave erreur de modélisation, un logiciel modélisé ainsi ne devrait même pas être sur le marché.
Si votre fournisseur est intelligent, il comprendra les remarques constructives et argumentées, et prendra peut être en compte (là c'est beaucoup moins sur) ces pistes d'amélioration en compte, pour une version future.
Dans le cas contraire, changez de fournisseur