Discussion :

[ashtur]

Bonjour
Dans ma base Oracle j'ai une table contenant une liste de table
Je souhaiterai pouvoir connaitre le nombre de lignes que contiennent chacune de ces tables.
Je pensais notamment créer un curseur dans lequel je recupere le nom des tables puis ensuite faire un EXECUTE pour connaitre le nombre de ligne de la table contenu dans le curseur, mais je sais pas du tout comment on fait.
la table contenant la liste des table s'appelle REF_TABLE et contient la colonne TABLE_NAME qui contient le nom des tables.
Je veux mettre le resultat dans une autre table nommé GRS_TABLE contient la colonne TABLE_NAME pour le nom de la table et NB_LIGNE pour le nombre de ligne.
je pensais faire un truc du style(je sais que c'est completement faux mais c'est pour donner une idée) :

DECLARE
NB_LIGNE INTEGER;

BEGIN
for MTABLE in
(select NOM_TABLE
From REF_TABLE)
LOOP
NB_LIGNE = EXECUTE ("select count(*) from " || MTABLE.NOM_TABLE );
EXECUTE("INSERT INTO GRS_TABLE VALUES(" || MTABLE.NOM_TABLE || "," || NB_LIGNE ||")") ;
END LOOP;
END;

Merci de votre aide

[McM]

avec EXECUTE IMMEDIATE

http://sheikyerbouti.developpez.com/execute_immediate/

[ashtur]

Merci mais y aurait-il moyen d'avoir le code car je debute en SQL et la je vois toujours pas comment faire.

[plaineR]

Je pense qu'en faisant un petit effort de lecture et de compréhension du tutoriel tu devrais t'en sortir. Si ce n'est pas le cas, postes-nous ce que tu as fait et ce qui te pose problème.

Cela te sera beaucoup plus profitable et tu retiendras beaucoup mieux que si l'on te donne une solution toute faite.

[McM]

Cela te sera beaucoup plus profitable et tu retiendras beaucoup mieux que si l'on te donne une solution toute faite.

Exact

[ashtur]

alors voila ou j'en suis :

pour la creation de la fonction :

CREATE or REPLACE FUNCTION Compte_rec (PC$Table IN VARCHAR2)
RETURN PLS_INTEGER
IS
LC$Requete VARCHAR2(512) ;
LN$Total PLS_INTEGER ;
BEGIN
LC$Requete := ‘Select count(*) From ‘ || PC$Table ;
EXECUTE IMMEDIATE LC$Requete INTO LN$Total ;
return LN$Total ;
END ;


Pour l'insertion proprement dite

DECLARE
LN$Nombre INTEGER;
LC$Requete2 VARCHAR2(512) ;
BEGIN
for MTABLE in
(select DESTINATION
From GRS_AUTOSYS_REF_TABLE)
LOOP
LN$Nombre := Compte_rec(MTABLE.DESTINATION) ;
LC$Requete2 := ‘INSERT INTO GRS_AUTOSYS_TABLE VALUES(‘ || MTABLE.DESTINATION || ‘,‘ || LN$Nombre ‘)‘ ;
EXECUTE IMMEDIATE LC$Requete2
END LOOP;
END;

moi ca ne marche toujours pas. Apparement il y a une erreur deja sur la compilation de la fonction. J'utilise TOAD comme interface pour oracle 9.2

[McM]

Tu peux utiliser la balise CODE pour ton code stp ? (c'est le #)

C'est normal que tu mettes des ` et pas des ' pour tes chaines de caractère ?

[ashtur]

c'est bon pour la fonction c'etait effectivement les ' qui marchaient pas.
Mais le reste ne marche toujours pas

[McM]

Ensuite pour débugguer le EXECUTE IMMEDIATE (erreur très très fréquente) : Attention aux chaines de caractère.
Fais un DBMS_OUTPUT pour voir le sql que le execute va faire (sous toad penser à activer le output on) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
DECLARE
	LN$Nombre INTEGER;
	LC$Requete2 VARCHAR2(512) ;
BEGIN
FOR MTABLE IN
(SELECT DESTINATION
FROM GRS_AUTOSYS_REF_TABLE)
LOOP
	LN$Nombre := Compte_rec(MTABLE.DESTINATION) ;
	LC$Requete2 := 'INSERT INTO GRS_AUTOSYS_TABLE VALUES(' || MTABLE.DESTINATION || ',' || LN$Nombre ')' ;
	DBMS_OUTPUT.PUT_LINE(LC$Requete2);
	EXECUTE IMMEDIATE LC$Requete2
END LOOP;
END;

T'y es presque

[plaineR]

Tu peux remplacer le bloc PL par :

Code :

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

1
2
3
insert  INTO GRS_AUTOSYS_TABLE 
select DESTINATION, Compte_rec(DESTINATION) 
from GRS_AUTOSYS_REF_TABLE;

[McM]

Dans ton code, l'ordre sql sera

Code :

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

INSERT INTO GRS_AUTOSYS_TABLE VALUES(NOM_TABLE,15)

faut juste rajouter des ' devant ert derrière le nom de la table

et conseil : toujours nommer les colonnes dans l'INSERT

Code :

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

INSERT INTO MATABLE(col1, col2) values (val1, val2)

[ashtur]

ca avance , ca avance :-)
apres quelques corrections:

DECLARE
LN$Nombre INTEGER;
LC$Requete2 VARCHAR2(512) ;
BEGIN
for MTABLE in
(select DESTINATION
From GRS_AUTOSYS_REF_TABLE)
LOOP
LN$Nombre := Compte_rec(MTABLE.DESTINATION) ;
DBMS_OUTPUT.PUT_LINE(LN$Nombre);
LC$Requete2 := 'INSERT INTO GRS_AUTOSYS_TABLE VALUES(' || MTABLE.DESTINATION || ',' || LN$Nombre || ')' ;
DBMS_OUTPUT.PUT_LINE(LC$Requete2);
EXECUTE IMMEDIATE LC$Requete2;
END LOOP;
END;

en DBMS_OUTPUT j'ai bien :
504935
INSERT INTO GRS_AUTOSYS_TABLE VALUES(PS_JOB,504935)

ce qui est bon car pour le moment j'ai qu'une seule table dans ma liste

mais j'ai l'erreur:
ORA-00947: not enough values
ORA-06512: at line 13

faut que je verifie la syntaxe du INSERT ?

[plaineR]

Ta table GRS_AUTOSYS_TABLE doit comporter plus de 2 colonnes. Mets dans le insert celles que tu veux renseigner.

[ashtur]

c'est fait :
LC$Requete2 := 'INSERT INTO GRS_AUTOSYS_TABLE (NAME_TABLE,LINE_INSERTED) VALUES(' || MTABLE.DESTINATION || ',' || LN$Nombre || ')' ;

en output
INSERT INTO GRS_AUTOSYS_TABLE (NAME_TABLE,LINE_INSERTED) VALUES(PS_JOB,504935)

Mais le soucis c'est que ce qui est contenu dans MTABLE.DESTINATION (PS_JOB) devrait etre entre quote mais il est contenu dans une variable donc la je sais pas trop comment faire...

[plaineR]

Il faut doubler les côtes (comme te l'a dit McM) :

Code :

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

LC$Requete2 := 'INSERT INTO GRS_AUTOSYS_TABLE (NAME_TABLE,LINE_INSERTED) VALUES(''' || MTABLE.DESTINATION || ''',' || LN$Nombre || ')' ;

Mais encore une fois je ne comprends pas pourquoi tu fais cette partie en SQL dynamique alors que la solution que je t'ai proposée plus haut est plus simple et plus performante.

[ashtur]

oups j'avais pas vu l'autre solution.
voila ce que ca donne :

INSERT INTO GRS_AUTOSYS_TABLE (NAME_TABLE,LINE_INSERTED)
SELECT DESTINATION, Compte_rec(DESTINATION)
FROM GRS_AUTOSYS_REF_TABLE

et j'ai l'erreur
ORA-00903: invalid table name
ORA-06512: at "GRSMANAG.COMPTE_REC", line 8

EDIT: j'ai la meme erreur en utilisant le code PL

EDIT2 : c'est bon ca marche en fait y'avait des valeurs à NULL dans DESTINATION ce qui devait faire un peu planter la fonction Compte_rec donc avec l'ajout d'une petite clause Where tout est rentré dans l'ordre :-)

Merci beaucoup pour votre aide