Discussion :

[mcorgnet]

Bonjour,

J'ai une procédure dans laquelle une requête de sélection parse une table, réalise une extraction, puis écrit les résultats dans une autre table.
Actuellement, pour chaque résultat, je génère une requête "insert into". Je trouvais pas ça très optimisé, dans le cas ou je dépassais un certain nombre de résultats, autant réaliser un insert multiple.

En l'occurence, je ne vois pas trop comment le réaliser vu que je génère l'intégralité de la requête à la volée.

Voici le code de la procédure :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
35
36
37
38
39
CREATE PROCEDURE CREATE_BRANDS()
BEGIN 
  /**
   * Flag : définit la fin de la lecture de la table CONTENT.
   */
  DECLARE done INT DEFAULT 0;
  /**
   * Variable MYBRANDS
   * Liste les marques.
   */
  DECLARE myBrand VARCHAR(35);
  /**
   * Création d'un curseur pour lecture de la table CONTENT.
   * Ce curseur stocke l'ensemble des marques.
   */
  DECLARE BRANDCURSOR CURSOR FOR SELECT BRAND FROM CONTENT GROUP BY BRAND; 
  /**
   * Gestionnaire de fin d'exécution du traitement de la requête.
   */
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
  /**
   * Ouverture du curseur.
   */
  OPEN BRANDCURSOR; 
  /**
   * Lit les résultats tant que done = 1, les écrits dans la marque.
   */
  REPEAT
    FETCH BRANDCURSOR INTO myBrand;
	IF done = 0 THEN 
		INSERT INTO BRAND (LIBELLE) VALUES (myBrand) ON DUPLICATE KEY UPDATE LIBELLE = myBrand;
	END IF;
  UNTIL done
  END REPEAT;
  /**
  * Fermeture du curseur.
  */
  CLOSE BRANDCURSOR;
END

On constate que dans mon FETCH, pour chaque résultat je génère une requête. Comment faire pour générer une seule requête d'insertion (multi insert) dans ce cas précis ?

Merci beaucoup de votre aide !

M.

[tse_jc]

Bonjour,

Il te suffit de faire un CONCAT dans la boucle avec ton curseur sur le/les champ(s) a extraire, et de construire ta requête statique générée à la volée, puis l'executer via une requête préparée (tu n'as pas le choix en MySQL).

[ericd69]

salut,

en faisant attention à la taille que ça représente en fonction du type de variable que tu vas utiliser

text te permet 65k par exemple... donc il faut faire gaffe à ça en choisissant le type si tu utilises une variable locale (le moins gourmand)...sinon si tu utilises une variable globale (genre @t) tu ne te poseras pas le soucis de la contenance et du type...

[CinePhil]

J'ai l'impression qu'une simple requête INSERT SELECT ferait l'affaire !

D'autre part...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

DECLARE BRANDCURSOR CURSOR FOR SELECT BRAND FROM CONTENT GROUP BY BRAND

GROUP BY s'utilise avec des fonctions de groupage dans le SELECT. Ici, vous n'en avez pas. Utilisez plutôt DISTINCT.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

INSERT INTO BRAND (LIBELLE) VALUES (myBrand) ON DUPLICATE KEY UPDATE LIBELLE = myBrand;

Quelle est la clé primaire de la table BRAND ?
Si c'est une clé entière auto-incrémentée, votre INSERT ne fera jamais de DUPLICATE KEY.
Si c'est LIBELLE, c'est une mauvaise clé !

Cette requête ne répondrait-elle pas à votre besoin ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
INSERT INTO BRAND (LIBELLE)
SELECT DISTINCT BRAND
FROM CONTENT

[ericd69]

oui cinéphil à raison, vu qu'il n'y a pas plusieurs insert ou update dans la boucle de ton curseur ou de récursion, ton code revient à un simple insert...select

[Fred_34]

Si c'est LIBELLE, c'est une mauvaise clé !

J'ai l'impression que c'est le cas...
Il faut juste qu'il ajoute IGNORE :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
INSERT IGNORE INTO BRAND (LIBELLE)
SELECT DISTINCT BRAND
FROM CONTENT