Manipuler les enregistrements dans un curseur

Version imprimable

Bonjour à tous,
j' aimerai obtenir un p' tit coup de pouce sur un pb qui me bloque depuis plusieurs jours. J' ai dans une procédure le curseur suivant:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 create or replace maproc(nr_courant number, nr_ressource) DECLARE CURSOR cur_ress IS SELECT id FROM table a, table b WHERE a.nr_courant = nr_courant and a.nr_ressource = nr_ressource and a.id = b.id ; v_ress cur_ress %ROWTYPE; v_rem varchar2(255); v_test boolean;
Ce curseur me renvoie deux lignes. Pour appliquer un traitement à une ligne précise mais seulement à cette ligne là j' ai écrit le code suivant:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 BEGIN v_test:= false; v_rem:= null; open cur_ress; loop fetch cur_ress into v_ress; exit when cur_ress%notfound; end loop; close cur_ress; if v_ress = 44 then v_test := true; else v_test := false; end if; if v_test = true then v_rem := 'mon texte'; dbms_output.put_line(v_rem); end if; END;
Seulement le texte est imprimé pour chacune des lignes contenues dans le curseur. Alors que je pensais avoir restraint l' impression du texte pour la ligne du curseur ayant la valeur 44.
Merci de votre aide

01/08/2013, 13h38
orafrance
Bonjour,

il vous manque un passage à false de la variable pour éviter qu'une fois que ça passe à true, la valeur ne reste true.
Code:

1 2 3 4 5 IF v_test = true then v_rem := 'mon texte'; dbms_output.put_line(v_rem); v_test := false; end IF;
01/08/2013, 13h41
teddi

merci pour la réponse Orafrance,

c´était une erreur de frappe de ma part. le passage à false je l' ai effectivement et c' est tout cela qui m' embête.

Je réédite mon message de suite merci.

Désolée orafrance je viens de voir la nuance avec ton code. j' essaie ca de suite et je te fais signe

merci pour tout
01/08/2013, 13h52
teddi

non malheureusement rien n' y fait :oops:.
Le texte est malgré tout imprimé deux fois. :arf:

Please Help

C'est pas clair du tout là... ton code ne devrait même être exécuté qu'une fois vu qu'il est après le loop... c'est pas du tout sensé marché en fait :aie:

Essaye ceci :

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
CREATE OR REPLACE maproc(nr_courant number, nr_ressource)
DECLARE  
 
v_rem varchar2(255);
v_test BOOLEAN;
 
BEGIN
v_test:= false;
v_rem:= NULL;
FOR cur_ress in (SELECT id 
FROM TABLE a,
        TABLE b
WHERE a.nr_courant = nr_courant
AND a.nr_ressource = nr_ressource
AND a.id = b.id) loop
 
	IF   cur_ress.id = 44 then
		v_test := true;
	ELSE
		v_test := false;
	END IF;
 
	IF v_test = true  then 
		v_rem := 'mon texte';
		dbms_output.put_line(v_rem);
	END IF;
 
END LOOP;
 
END;

Ou nettement plus simple :

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
CREATE OR REPLACE maproc(nr_courant number, nr_ressource)
DECLARE  
 
v_rem varchar2(255);
 
BEGIN
v_rem:= 'mon texte';
FOR cur_ress in (
SELECT DECODE(id , 44, v_rem, NULL) text2display
FROM TABLE a,
        TABLE b
WHERE a.nr_courant = nr_courant
AND a.nr_ressource = nr_ressource
AND a.id = b.id
) 
LOOP
 
	IF cur_ress.text2display IS NOT NULL  then 
		dbms_output.put_line(text2display);
	END IF;
 
END LOOP;
 
END;

01/08/2013, 14h17
teddi

désolée orafrance mais toujours rien. Je ne comprends pas pourquoi le code est exécuté 2 fois.

Et pour être sincère j' avais déjà fais le test avec une boucle for (donc pratiquement ce test). Est ce que je bloque quelque part dans ma logique?

merci
01/08/2013, 14h22
orafrance
Comment la procédure est-elle exécutée ?

que donne :
Code:

1 2 3 4 5 6 SELECT DECODE(id , 44, v_rem, NULL) text2display FROM TABLE a, TABLE b WHERE a.nr_courant = nr_courant AND a.nr_ressource = nr_ressource AND a.id = b.id
avec nr_courant et nr_ressource qui sont transmis à la procédure ?

Es tu bien certain de n'avoir qu'un seul dbms_output ?

Citation:
Envoyé par orafrance
Ou nettement plus simple :
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 CREATE OR REPLACE maproc(nr_courant number, nr_ressource) DECLARE v_rem varchar2(255); BEGIN v_rem:= 'mon texte'; FOR cur_ress in ( SELECT DECODE(id , 44, v_rem, NULL) text2display FROM TABLE a, TABLE b WHERE a.nr_courant = nr_courant AND a.nr_ressource = nr_ressource AND a.id = b.id ) LOOP IF cur_ress.text2display IS NOT NULL then dbms_output.put_line(text2display); END IF; END LOOP; END;
Cette variante non plus ne me donne pas le résultat escompté. Et pourtant elle est effectivement nettement plus simple en plus je ne l' avais pas encore testé.
Dommage:cry:

01/08/2013, 14h35
orafrance

Alors il y a un autre problème... probablement dans l'appel de la procédure.
01/08/2013, 14h50
teddi
Citation:
Envoyé par orafrance

Comment la procédure est-elle exécutée ?

que donne :

Code:

1 2 3 4 5 6 SELECT DECODE(id , 44, v_rem, NULL) text2display FROM TABLE a, TABLE b WHERE a.nr_courant = nr_courant AND a.nr_ressource = nr_ressource AND a.id = b.id

avec nr_courant et nr_ressource qui sont transmis à la procédure ?
tu as raison le select decode(lui seul lorsque je l' isole) m' imprime une seule fois le texte.

Citation:

Es tu bien certain de n'avoir qu'un seul dbms_output ?

oui je n' en ai qu' un.

Citation:

Comment la procédure est-elle exécutée ?

elle est appelé dans un état. Là aussi j' ai vérifié l' appel est correct. Surtout je peux exclure un quelconque problème au niveau de l' appel car lorsque je commente mon bout de code(celui qui pose probème) elle fonctionne parfaitement.
01/08/2013, 15h15
orafrance

L'état doit exécuter 2 fois la procédure, il n'y a pas d'autre explication ;)

Genre une repeting frame pour avoir une ligne par ligne dans le select alors que tu veux afficher le texte si pour une ligne.
01/08/2013, 15h25
teddi

Merci pour la piste orafrance j'y avais pas du tout pensé!!! oui ca doit en effet être le cas.
Dans ce cas comment pourrais-je éviter cela car le repeating frame est voulu.
Dans la mesure où pour un id!=44 il y aurait autre chose à imprimer.
Que puis-je faire dans ce cas là (au niveau de l'état) pour que mon texte ne soit imprimé qu' une fois?
merci
01/08/2013, 16h01
orafrance

Utiliser une procédure pour afficher un truc dans l'état, c'est une erreur.

Dans l'état, il y a moyen en principe d'ajouter un champ que tu affiches en fonction d'une valeur dans la ligne courante sans avoir à passer par une procédure externe.

Rien qu'en faisant le DECODE dans la requête SQL de l'état, tu dois avoir moyen de t'en sortir ;)
01/08/2013, 16h21
teddi

merci beaucoup