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.