ALTER TABLE ajouter une colonne s'elle n'existe pas

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:

---

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.

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

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à"

Il manque la clause WHERE dans

Code:

---

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" :aie::aie: 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.

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:

---

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;

---

Il manque encore la clause WHERE.

Attention avec tes DECLARE, BEGIN, END

Code:

---

1 2 3 4 5

DECLARE définitions de variables BEGIN code END;

---

Ne pas confondre avec les tests de conditions

Code:

---

1 2 3 4

IF .. THEN .. ELSIF .. THEN .. ELSE .. END IF;

---

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

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:

---

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;

---

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.

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:

---

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;

---

@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:

---

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;

---