Boucle WHILE qui ne boucle pas

Version imprimable

Bonjour à tous.
La boucle while en ligne 14 fonctionne pour var_i = 1 : La LOOP de la ligne 16 insère 60 enregistrements avec var_i = 1, puis la procédure s’arrête au lieu d'incrémenter var_i de 1 en ligne 25.

Code:

1
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
DELIMITER $$
drop procedure if exists sujets_importants $$
CREATE PROCEDURE sujets_importants()
BEGIN
	declare var_i INT default 1;
	DECLARE done INT DEFAULT FALSE;
	DEclare v_id INT;
	DECLARE v_sujet CHAR(250); 
		declare cur1 CURSOR FOR
		SELECT id , Avenir__Cinq_sujets_les_plus_prioritaires FROM `20231009_sainte_gemmes_en_2035`
		WHERE Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',(SELECT sp.intitule  from sujets_prioritaires sp WHERE sp.id=var_i),'%');
		DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = TRUE;
 
	read_while: WHILE (var_i < 11) DO
		OPEN cur1;
		read_loop: LOOP
			FETCH cur1 INTO v_id,v_sujet;
			IF done THEN
				LEAVE read_loop;
			ELSE
				INSERT INTO lien__personne__sujet_prioritaire VALUES(v_id,var_i);
			END IF;
		END LOOP read_loop;
		close cur1;
		SET var_i := var_i+1;
	END WHILE read_while;
END$$
DELIMITER ;
call sujets_importants();

Je dois louper un truc :?
Merci pour vos lumières

06/11/2023, 15h44
JeitEmgie

Ne serait-ce tout simplement pas que le "var_i" dans le CURSOR n'est évalué qu'une seule fois à la déclaration du CURSOR ?
Ce dont vous avez besoin est d'un CURSOR paramétré.
06/11/2023, 16h46
ypcman

"CURSOR paramétré" ?
J'ai trouvé un lien en PL/SQL mais pas avec MySQL ... Je me trompe ? => oui finalement ça existe depuis MariaDB 10 !

Du coup, j'ai ajouté un paramêtre au curseur mais ça ne change rien ... Il n'y a toujours que les 60 premiers INSERT avec l'indice 1 qui sont enregistrés

Code:

1
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
DELIMITER $$
drop procedure if exists sujets_importants $$
CREATE PROCEDURE sujets_importants()
BEGIN
	declare var_sujet_id INT default 1;
	DECLARE done INT DEFAULT FALSE;
	DEclare v_id INT;
	DECLARE v_sujet CHAR(250); 
		declare cur1  CURSOR  (var_i INT) FOR
		SELECT id , Avenir__Cinq_sujets_les_plus_prioritaires FROM `20231009_sainte_gemmes_en_2035`
		WHERE Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',(SELECT sp.intitule  from sujets_prioritaires sp WHERE sp.id=var_i),'%');
		DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = TRUE;
 
	read_while: WHILE (var_sujet_id < 11) DO
		OPEN cur1(var_sujet_id);
		read_loop: LOOP
			FETCH cur1 INTO v_id,v_sujet;
			IF done THEN
				LEAVE read_loop;
			ELSE
				INSERT INTO lien__personne__sujet_prioritaire VALUES(v_id,var_sujet_id);
			END IF;
		END LOOP read_loop;
		close cur1;
		SET var_sujet_id := var_sujet_id+1;
	END WHILE read_while;
END$$
DELIMITER ;
call sujets_importants();

J'ai simplifié le code avec une boucle for en ligne 12:

Code:

1
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
DELIMITER $$
drop procedure if exists sujets_importants $$
CREATE PROCEDURE sujets_importants()
BEGIN
	DECLARE done INT DEFAULT FALSE;
	DEclare v_id INT;
	declare cur1  CURSOR  (var_i INT) FOR
		SELECT id  FROM `20231009_sainte_gemmes_en_2035`
		WHERE Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',(SELECT sp.intitule  from sujets_prioritaires sp WHERE sp.id=var_i),'%');
	DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = TRUE;
 
	FOR v_sujet_id IN 1..10 DO
		OPEN cur1(v_sujet_id);
		read_loop: LOOP
			FETCH cur1 INTO v_id;
			IF done THEN
				LEAVE read_loop;
			END IF;
			INSERT INTO lien__personne__sujet_prioritaire VALUES(v_id,v_sujet_id);
		END LOOP read_loop;
		close cur1;
	END FOR;
 
END $$
DELIMITER ;
call sujets_importants();

Mais, manifestement, cette boucle n'est effectuée qu'une fois, avec 60 insertions pour la valeur 1 de v_sujet_id. Pourtant, la requête du curseur, exécutée seule, retourne bien des valeurs pour v_sujet_id variant de 1 à 10,

Si ce SELECT, à adapter si besoin, renvoie bien les données souhaitées à insérer :

Code:

1
2
3
4
5
6
7
SELECT t1.id as v_id, t2.id as v_sujet_id  
  FROM 20231009_sainte_gemmes_en_2035 t1
  join (SELECT sp.id, sp.intitule 
          from sujets_prioritaires sp 
		 WHERE sp.id between 1 and 10
		) t2
    on t1.Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',t2.intitule,'%');

Alors il suffit de faire un INSERT INTO SELECT :

Code:

1
2
3
4
5
6
7
8
INSERT INTO lien__personne__sujet_prioritaire (la_col_v_id, la_col_v_sujet_id)
SELECT t1.id as v_id, t2.id as v_sujet_id  
  FROM 20231009_sainte_gemmes_en_2035 t1
  join (SELECT sp.id, sp.intitule 
          from sujets_prioritaires sp 
		 WHERE sp.id between 1 and 10
		) t2
    on t1.Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',t2.intitule,'%');

Vous avez tout à fait raison skuatamad, ça fonctionne parfaitement avec votre élégante solution

Pour ceux que ça intéresse, j'ai résolu ma question employant un curseur en sortant la boucle for de la 1° procédure :

Code:

1
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
DELIMITER $$
drop procedure if exists sujets_importants $$
CREATE PROCEDURE sujets_importants(v_sp_id INT)
BEGIN
	DECLARE done INT DEFAULT FALSE;
	DEclare v_id INT;
	declare cur1  CURSOR  FOR
		SELECT id  FROM `20231009_sainte_gemmes_en_2035`
		WHERE Avenir__Cinq_sujets_les_plus_prioritaires LIKE CONCAT('%',(SELECT sp.intitule  from sujets_prioritaires sp WHERE sp.id=v_sp_id),'%');
	DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = TRUE;
 
		OPEN cur1;
		read_loop: LOOP
			FETCH cur1 INTO v_id;
			IF done THEN
				LEAVE read_loop;
 
END IF;
			INSERT INTO lien__personne__sujet_prioritaire VALUES(v_id,v_sp_id);
		END LOOP read_loop;
		close cur1;
 
END $$
 
drop procedure if exists insert_sujets_importants $$
CREATE PROCEDURE insert_sujets_importants()
BEGIN
	FOR v_sujet_id IN 1..10 DO
		call sujets_importants(v_sujet_id);
	END FOR;
END $$
DELIMITER ;
 
call insert_sujets_importants;

13/11/2023, 02h40
Artemus24

Salut à tous.

Je ne l'ai pas vérifié mais il me semble que l'erreur se situe à cette ligne :

Code:

SET var_i := var_i+1;

Il ne faut pas mettre de : devant le =

@+