Discussion :

[in]

Bonjour,

il s'agit sûrement d'une question assez bête mais je n'arriva pas trop à) m'en sortir.

C'est au sujet d'un script de création de tables. Au début de ce script j'ai mis des instructions DROP TABLE ... mais pour éviter d'avoir les messages d'erreur "TABLE OR VIEW DOES NOT EXIST ..." on ma donné ce script à mettre au début :

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
40
41
42
43
44
45
46
47
WHENEVER SQLERROR CONTINUE;
WHENEVER OSERROR CONTINUE;
 
 
declare
 
c number;
n number;
texte_sql varchar2(2000);
 
exp_existe_deja EXCEPTION;
PRAGMA EXCEPTION_INIT(exp_existe_deja, -00955);
 
begin
 
    begin
 
        c:= dbms_sql.open_cursor;
        texte_sql := 'DROP TABLE T1';
 
        dbms_sql.parse(c,texte_sql, dbms_sql.native);
        n := dbms_sql.execute(c);
        dbms_sql.close_cursor(c);
 
    exception
        when others then
            null;
    end;
 
    begin
 
        c:= dbms_sql.open_cursor;
        texte_sql := 'DROP TABLE t2';
 
        dbms_sql.parse(c,texte_sql, dbms_sql.native);
        n := dbms_sql.execute(c);
        dbms_sql.close_cursor(c);
 
        /**** BREF autant de fois qu'il y a de tables .... ****/
 
 
exception
    when Others then
        dbms_output.put_line(SQLERRM);
        null;
end;
/

Le truc c'est que je ne le trouve pas très joli. Je voudrais si possible le simplifier en créant une liste des tables et itérer dessus pour exécuter le code .. mais je ne sais pas comment m'y prendre. De plus je ne sais pas si c'est la meilleure manière de procéder ...

J'en appelle donc à vos connaissances ... comment m'y prendre pour avoir un truc propre et facilement maintenable et transposable à d'autres scripts de ce genre ?

merci d'avance

[Oraman]

pourquoi tu ne teste pas sur l'existance de la table que tu veux créer dans user_tables ou all_tables

Code :

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

select table_name from user_tables where table_name = upper('nomdetable')

[in]

pourquoi tu ne teste pas sur l'existance de la table que tu veux créer dans user_tables ou all_tables

Code :

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

select table_name from user_tables where table_name = upper('nomdetable')

Ben oui mais je ne vois pas trop comment appliquer ceci à un traitement "générique"

Tes tables s'appellent t1, t2, tn ? ou elles ont des noms différents (ex: aaa, toto, titi, matable) ?

non, des nom différents. enfin elles ont toutes le meme prefixe quand meme mais derrière ça varie beaucoup.

En fait, vos idées me font penser à qqchose. Si je remplis un curseur avec mes tables (d'apres le prefixe) je peux itérer dessus comme expliqué précedemment ?

Par contre seule hic, si jamais une des autres tables de la base commence pareil ...

-> y a t'il moyen de remplir un curseur (ou autre objet itérable) à la main. Genre remplir manuellement une liste des nom de tables à dropper et la parcourir ??

[Oraman]

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
create or replace procedure chercher_table
is
declare 
v varchar2(50);
begin
v:='error';
select table_name into v from user_tables where table_name = upper('TEST');
EXCEPTION WHEN NO_DATA_FOUND THEN
   v:='error';
 
   if v='error' then
   	  dbms_output.put_line('table introuvale ');
   else
   	   dbms_output.put_line('table exite déjà ');
   end if ;
end;

tu n'as qu'as passer le nom de ta table en paramètre et remplacer les dbms_output par tes DDL

[plaineR]

Voici une solution que j'avais adopté :

Code :

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

1
2
3
4
5
6
7
8
9
create or replace procedure drop_table (p_table in varchar2) is
  not_dropable exception;
  pragma exception_init (not_dropable, -942);
begin
execute immediate 'drop table ' || p_table;
exception
   when not_dropable then null;
end;
/

et au lieu de faire "drop table toto;" je faisais :

Code :

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

exec drop_table ('toto')

Après généraliser le principe avec un curseur cela me semble très risqué et peu recommandé.

[in]

c'est vrai que la solution du curseur est peut être (surement) plus dangereuse ...

du coup, désolé mais je te pique ta solution PlaineR.

Merci à vous deux en tous cas.

A charge de revanche j'espère !

[plaineR]

Tes tables s'appellent t1, t2, tn ? ou elles ont des noms différents (ex: aaa, toto, titi, matable) ?