Discussion :

[Zalawy]

Bonjour à tous,

Je voudrai faire une requête qui ajoute un nouvelle colonne si cette dernière n'existe pas dans la table si elle existe et pas ajouter.
Voici le code que j'ai fait :

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
 
DECLARE
    p_count  NUMBER;
BEGIN 
    SELECT count(*) INTO p_count
    FROM user_tab_columns
    AND table_name = 'Table1'
    AND column_name = 'col1';
 
    IF IS_OBJECT = 0
    THEN
        BEGIN
            DBMS_OUTPUT.PUT_LINE('Ajouter la colonne col1');
            EXECUTE IMMEDIATE 'ALTER TABLE Table1 ADD (col1)';
 
		ELSE 
		DBMS_OUTPUT.PUT_LINE('La colonne est déjà existée');
        END;
END IF;
END;

Le code de la condition d'ajouter si la colonne n'existe pas est-il bon?
Pour plus de précision, la langue de sql est pl/sql.

[Waldar]

Non il faut typer la colonne, c'est obligatoire.

[vanagreg]

Bonjour,

Plusieurs soucis dans ton code.

- Tu spécifies le nom de la table et de la colonne en mélange majuscules/minuscules, mais il faut le faire en majuscule (sauf si elles ont été créées avec des guillemets, ce qui n'est pas souhaitable)
- Tu utilises la variable p_count pour vérifier si la colonne est présente mais ensuite tu utilises IS_OBJECT?
- Le bloc BEGIN END interne est mal placé, il est d'ailleurs inutile ici
- Il faut préciser un type de données pour la colonne
- "La colonne est déjà existée" n'est pas correct, ce serait plutôt "La colonne existe déjà"

[Ikebukuro]

Il manque la clause WHERE dans

Code :

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

1
2
3
4
    SELECT count(*) INTO p_count
    FROM user_tab_columns
    AND table_name = 'Table1'
    AND column_name = 'col1';

C'est quoi IS_OBJECT? Il est renseigné où?

Je ne sais pas quel est ton métier mais quand je lis "la langue de sql est pl/sql" désolé mais ça ne veut rien dire; ton code est écrit en SQL, en PL/SQL, en Java, en C mais "la langue de sql" ne veut rie dire.

[Zalawy]

Bonjour,
Je vous remercie de vos réponses.

@Waldar : typer la colonne, cela signifie la propriété de la colonne?

@Ikebukuro : effectivement, j'ai mal placé pour écrire "la langue de sql", c'est plutôt la langue de programmation. ^^

@vanagreg : Pour le nom de table ne majuscule/minuscule, j'ai vu sur des sites qui indiquent un nom de table en majuscule et minuscule.
IS_OBJECT est un variable que je me suis trompée de nommer comme j'ai déjà nommer p_count.
J'ai enlevé BEGIN et END inutiles.
J'ai défini le type de colonne.

Voici le code corrigée :

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
 
DECLARE
    p_count  NUMBER;
 
    SELECT count(*) INTO p_count
    FROM user_tab_columns
    AND TABLE_NAME = 'TABLE1'
    AND COLUMN_NAME= 'col1';
 
    IF p_count  = 0
    THEN
        BEGIN
            DBMS_OUTPUT.PUT_LINE('Ajouter la colonne col1');
            EXECUTE IMMEDIATE 'ALTER TABLE TABLE1 ADD (col1 NUMBER(12,2))';
 
		ELSE 
		DBMS_OUTPUT.PUT_LINE('La colonne existe déjà');
        END;
END IF;

[Ikebukuro]

Il manque encore la clause WHERE.

[McM]

Attention avec tes DECLARE, BEGIN, END

Code :

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

1
2
3
4
5
DECLARE 
 définitions de variables
BEGIN
code 
END;

Ne pas confondre avec les tests de conditions

Code :

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

1
2
3
4
IF .. THEN ..
ELSIF .. THEN  ..
ELSE ..
END IF;

[vanagreg]

Oui c'était du bloc BEGIN..END interne que je parlais...

[Zalawy]

Merci à tous pour vos réponses,
J'ai corrigé le code mais lors de test, il n'y a pas la fin de processus dès que le traitement termine, c'est à dire que j'ai lancé le test et ce dernier qui est toujours en cours même s'il n'y a pas d'erreur à afficher...

Voici le code modifié :

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
DECLARE
    p_count  NUMBER;
 
    SELECT count(*) INTO p_count
    FROM user_tab_columns
    WHERE TABLE_NAME = 'TABLE1'
    AND COLUMN_NAME= 'col1';
 
    IF p_count  = 0
    THEN
        BEGIN
            DBMS_OUTPUT.PUT_LINE('Ajouter la colonne col1');
            EXECUTE IMMEDIATE 'ALTER TABLE TABLE1 ADD (col1 NUMBER(12,2))';
 
		ELSE 
		DBMS_OUTPUT.PUT_LINE('La colonne existe déjà');
        END;
   END IF;

[Waldar]

Presque, mais vos BEGIN END au milieu n'ont toujours aucun sens ; de surcroît vous ne pouvez pas l'imbriquer en plein milieu d'un IF THEN ELSE.

[McM]

2 choses :
1/ il y a incohérence entre le test de colonne et l'ordre de création de la colonne
AND COLUMN_NAME= 'col1'; <= Colonne créée en forçant la minuscule
ALTER TABLE TABLE1 ADD col1 NUMBER(12,2) <= Création de colonne en casse par défaut => en majuscule

Si la colonne existante est en majuscule, tester column_name = 'COL1'
Si le colonne existante est en minuscule, créer en forçant la minuscule : ADD "col1" NUMBER(12,2)

2/ On peut aussi le faire sans les tests en gérant l'exception

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
DECLARE
    v  NUMBER;
 BEGIN
    SELECT 1 INTO v
    FROM user_tab_columns
    WHERE TABLE_NAME = 'TABLE1'
    AND COLUMN_NAME= 'col1';
 
	DBMS_OUTPUT.PUT_LINE('Ajouter la colonne col1');
	EXECUTE IMMEDIATE 'ALTER TABLE TABLE1 ADD col1 NUMBER(12,2)';
 
EXCEPTION WHEN NO_DATA_FOUND
THEN
	DBMS_OUTPUT.PUT_LINE('La colonne existe déjà');
END;

[Zalawy]

@McM et @Waldar : merci de vos réponses, c'est noté. J'ai modifié et testé, c'est impeccable. Je ferme ce sujet et merci encore d'avoir pris le temps de répondre à ce sujet.

Voici mon code corrigé qui pourrait aider des autres :

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
 
DECLARE
    p_count  NUMBER;
 
BEGIN 
    SELECT count(*) INTO p_count
    FROM ALL_TAB_COLUMNS
    WHERE TABLE_NAME = 'TABLE1'
    AND COLUMN_NAME= 'COL1';
 
    IF (p_count  = 0) THEN
            DBMS_OUTPUT.PUT_LINE('Ajouter la colonne col1');
            EXECUTE IMMEDIATE 'ALTER TABLE TABLE1 ADD (COL1 NUMBER(12,2))';
 
	ELSE 
		DBMS_OUTPUT.PUT_LINE('La colonne existe déjà');
     END IF;
END;