Boucler via un Record

**ratata** · 12/04/2011, 19h15

Bonjour,

en cours de mon stage je suis amené à réaliser des procédures stockées via postgreSQL 8.4, et ce pour une performance d'appel des requettes en terme de temps d’exécution, dans le script que je vous expose, j'ai du pour réduire le temps d'execution stocker le résultat d'un select dans un Record afin de boucler avec au lieu de l'utiliser directement au niveau de la boucle , chose qui me retourne l'erreur en dessous :calim2 :, quand j'appelle ma fonction au niveau de la console:

script:
------

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
drop function deleteuser(integer);
CREATE OR REPLACE FUNCTION deleteuser(userid integer) RETURNS integer AS $BODY$
DECLARE
i integer;
tab record;
BEGIN
update billing_user set deleted=true where id_user=userid;
select into tab id_user from billing_reseller where id_super_user=userid;
for i in select tab LOOP
PERFORM deleteuser(i);
END LOOP;
return 1;
END;
$BODY$
LANGUAGE plpgsql VOLATILE
COST 100;
ALTER FUNCTION "deleteuser"(integer) OWNER TO postgres;

Requêttes d'appel:
------------------
SELECT deleteuser(2);

Erruer retournée:
-----------------

______________________________________________________________
ERREUR: syntaxe en entrée invalide pour l'entier : « (4) »
CONTEXT: PL/pgSQL function "deleteuser" line 7 at FOR sur des lignes de SELECT

********** Erreur **********

ERREUR: syntaxe en entrée invalide pour l'entier : « (4) »
État SQL :22P02
Contexte : PL/pgSQL function "deleteuser" line 7 at FOR sur des lignes de SELECT

______________________________________________________________
vous trouverez ci-joint les images correspondant aux tables, je serais très reconnaissant pour tout aide, car je suis affreusement coincé.

**estofilo** · 13/04/2011, 14h15

Et pourquoi ne pas faire plus simplement:

Code :

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

1
2
3
4
5
6
 
...
FOR i IN select id_user FROM billing_reseller WHERE id_super_user=userid LOOP
PERFORM deleteuser(i);
END LOOP;
...

**ratata** · 14/04/2011, 12h52

Bonjour,

Merci pour votre réponse estofilo, je ne veux pas faire de la sorte parce que ca consomme déjà en temps d'exécution et en pile du SGBD(Recursivité), et imaginez combien ca va encore consommer l'exécution du select pour chaque itération de la boucle, parce que par la suite je serais amené à gérer des tables qui peuvent contenir plus que 1000 lignes, dont le temps d'exécution et pour moi un élément très primordiale. c'est pour ca qui j'ai pensé à enregistrer le résultat du select dans un record en premier temps et boucler avec par la suite , mais ca a donnée ce que j'avais décrit dans mon premier message.

Excellente journée.

**estofilo** · 14/04/2011, 13h14

Au lieu de faire un seul SELECT tu en fais deux et avec un passage de variable entre les deux. Je ne vois pas ce qui te fais espérer que ça va aller plus vite. Si ça fait une différence notable ce qui déjà est à voir, ça sera plutôt dans le sens de consommer plus de mémoire et d'aller moins vite.