Discussion :

[Shiva]

Bonjour,
J'ai un petit soucis pour accéder aux valeurs d'un tableau qui provient d'une fonction perso.
Cette fonction retourne le tableau de données suivant :

Code :

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

[0:2]={"{68,3}","{92,1}","{93,7}"}

Je cherche à lire ces infos dans une autre fonction (fonction de test) :

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
 
CREATE OR REPLACE FUNCTION test.stock(p_id_article INTEGER) RETURNS VOID AS $$
DECLARE
    v_data_stock TEXT[];
    i            SMALLINT;
    v_id_stock   INTEGER;
    v_quantite   INTEGER;
BEGIN
    SELECT test.verif_stock(p_id_article) INTO v_data_stock;
 
    i = 0;
    WHILE v_data_stock[i] IS NOT NULL LOOP
        v_id_stock = v_data_stock[i][0];
        v_quantite = v_data_stock[i][1];
        RAISE NOTICE 'id_stock=% - quantite_stock=%', v_id_stock, v_quantite;
        i = i + 1;
    END LOOP;
END;
$$ LANGUAGE plpgsql;
 
SELECT test.stock(4555846);

Pour le moment les variables v_id_stock et v_quantite restent vide, je ne vois pas trop comment accéder aux éléments {68,3}, {92,1} et {93,7}
Une idée ?

Merci

[alassanediakite]

Salut
L'indice de tableaux commence à 1 avec PostgreSQL. (voir ici paragraphe "8.15.3. Accès aux tableaux")
Donc, dès le premier pas il obtient null (ou une exception) et il n'exécutera ni le contenu du pas ni aucun pas suivant.
@+

[Shiva]

Donc ceci :

Code :

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

1
2
 
RAISE NOTICE 'v_data_stock=%', v_data_stock[1][1];

Devrait m'afficher 68 si je prend le 1er élément du 1er tableau de {"{68,3}","{92,1}","{93,7}"}
Marche pas... La manière de l'écrire n'est pas bonne peut-être ?

[alassanediakite]

Salut
Donne le code de la fonction tel que tu l'as réécris et dit nous ce qui se passe réellement. "Marche pas ..." n'est pas suffisant pour comprendre le problème!
@+

[Shiva]

Oui, avec le code c'est mieux :-)

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
CREATE OR REPLACE FUNCTION test.stock(p_id_article INTEGER) RETURNS VOID AS $$
DECLARE
    v_data_stock TEXT[];
    i            SMALLINT;
    v_id_stock   INTEGER;
    v_quantite   INTEGER;
BEGIN
    SELECT test.verif_stock(p_id_article) INTO v_data_stock; -- >> retourne ceci : [0:2]={"{68,3}","{92,1}","{93,7}"}
 
    i = 0;
    WHILE v_data_stock[i] IS NOT NULL LOOP
        v_id_stock = v_data_stock[i][1];
        v_quantite = v_data_stock[i][2];
        RAISE NOTICE 'id_stock=% - quantite_stock=%', v_id_stock, v_quantite;
        i = i + 1;
    END LOOP;
END;
$$ LANGUAGE plpgsql;

Petite explication de ce que je cherche à faire :
je veux retourner en un tableau tous les articles et leur quantité de la table "article.stock" dont l'id = celui fourni en paramètre.
avec l'exemple donné j'ai 3 lignes dans la table qui correspondent à l'article :

Code :

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

1
2
3
4
5
 id | quantite | id_article 
----+----------+------------
 68 |        3 |    4555846
 92 |        1 |    4555846
 93 |        7 |    4555846

[alassanediakite]

Salut
J'ai demandé à avoir le résultat de la fonction stock(integer)!!! Quand tu l'exécute qu'est ce qu'elle donne ou fait réellement???
Par ailleurs il faut le code de la fonction verif_stock car je ne comprend le format de sortie

Code :

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

[0:2]={"{68,3}","{92,1}","{93,7}"}

.
@+

[Shiva]

Voilà ce que retourne test.stock() pour l'article dont l'id = 4555846 :

Code :

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

1
2
3
4
5
6
7
8
he_livre=# SELECT test.stock(4555846);
NOTICE:  v_data_stock=[0:2]={"{68,3}","{92,1}","{93,7}"} - id_stock=<NULL> - quantite_stock=<NULL>
NOTICE:  v_data_stock=[0:2]={"{68,3}","{92,1}","{93,7}"} - id_stock=<NULL> - quantite_stock=<NULL>
NOTICE:  v_data_stock=[0:2]={"{68,3}","{92,1}","{93,7}"} - id_stock=<NULL> - quantite_stock=<NULL>
 stock 
-------
 
(1 ligne)

Et voici le code de la fonction en question :

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
CREATE OR REPLACE FUNCTION test.verif_stock(p_id_article INTEGER) RETURNS TEXT[] AS $$
DECLARE
    cursor_STOCK                CURSOR FOR SELECT id, quantite FROM article.stock WHERE id_article = p_id_article AND quantite > 0;
    v_id_stock                  INTEGER;
    v_quantite_stock            SMALLINT;
    p_data_stock                TEXT[];
    i                           SMALLINT = 0;
BEGIN
    OPEN cursor_STOCK;
    LOOP
        FETCH cursor_STOCK INTO v_id_stock, v_quantite_stock;
        EXIT WHEN NOT FOUND;
        p_data_stock[i] = ARRAY[v_id_stock,v_quantite_stock];
        i = i + 1;
    END LOOP;
    CLOSE cursor_STOCK;
    RETURN p_data_stock;
END;
$$ LANGUAGE plpgsql;

Possible que je me sois planté ici ?

[alassanediakite]

Salut
Dans le code...

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
CREATE OR REPLACE FUNCTION test.verif_stock(p_id_article INTEGER) RETURNS TEXT[] AS $$
DECLARE
    cursor_STOCK                CURSOR FOR SELECT id, quantite FROM article.stock WHERE id_article = p_id_article AND quantite > 0;
    v_id_stock                  INTEGER;
    v_quantite_stock            SMALLINT;
    p_data_stock                TEXT[];
    i                           SMALLINT = 0;
BEGIN
    OPEN cursor_STOCK;
    LOOP
        FETCH cursor_STOCK INTO v_id_stock, v_quantite_stock;
        EXIT WHEN NOT FOUND;
        p_data_stock[i] = ARRAY[v_id_stock,v_quantite_stock];
        i = i + 1;
    END LOOP;
    CLOSE cursor_STOCK;
    RETURN p_data_stock;
END;
$$ LANGUAGE plpgsql;

• TEXT[] désigne un tableau de TEXT à UNE DIMENSION
• la ligne [I]p_data_stock = ARRAY[v_id_stock,v_quantite_stock]; place un tableau dans une case , le tableau sera donc converti en text.

Faisons table rase de tous ces problèmes.
Peux-tu donner

• le code des deux fonctions
• le DDL des tables
• l'objectif cherché

Par ailleurs, je soupçonne que vous utilisez le type tableau pour modéliser des lignes de table sql ce qui est TRÈS MAUVAIS!!!
@+

[Shiva]

La table est d'une grande simplicité :

Code :

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

1
2
3
4
5
6
7
8
CREATE TABLE article.stock
(
    id         SERIAL,
    id_article INTEGER NOT NULL DEFAULT 1,
    quantite   SMALLINT NOT NULL DEFAULT 1,
    adresse    SMALLINT NOT NULL DEFAULT 1,
    PRIMARY KEY(id)
);

Dans cette table hors de question d'additionner les quantités de plusieurs articles identique car l'adresse en stock est différente pour chaque lignes d'article.

La première fonction n'a pour but que de me retourner toutes les quantités d'un article recherché ainsi que l'identifiant de ces lignes dans la table "article.stock":

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
CREATE OR REPLACE FUNCTION test.verif_stock(p_id_article INTEGER) RETURNS TEXT[] AS $$
DECLARE
    cursor_STOCK                CURSOR FOR SELECT id, quantite FROM article.stock WHERE id_article = p_id_article AND quantite > 0;
    v_id_stock                  INTEGER;
    v_quantite_stock            SMALLINT;
    p_data_stock                TEXT[];
    i                           SMALLINT = 0;
BEGIN
    OPEN cursor_STOCK;
    LOOP
        FETCH cursor_STOCK INTO v_id_stock, v_quantite_stock;
        EXIT WHEN NOT FOUND;
        p_data_stock[i] = ARRAY[v_id_stock,v_quantite_stock];
        i = i + 1;
    END LOOP;
    CLOSE cursor_STOCK;
    RETURN p_data_stock;
END;
$$ LANGUAGE plpgsql;

La seconde fonction appel test.verif_stock() qui sera au final utilisée dans plusieurs autres fonctions d'ou la création de cette première fonction.

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
CREATE OR REPLACE FUNCTION test.stock(p_id_article INTEGER) RETURNS VOID AS $$
DECLARE
    v_data_stock TEXT[];
    i            SMALLINT;
    v_id_stock   INTEGER;
    v_quantite   INTEGER;
BEGIN
    SELECT test.verif_stock(p_id_article) INTO v_data_stock;
 
    i = 0;
    WHILE v_data_stock[i] IS NOT NULL LOOP
        -- C'est ici que ça ne va pas...
        v_id_stock = v_data_stock[i][1];
        v_quantite = v_data_stock[i][2];
        RAISE NOTICE 'id_stock=% - quantite_stock=%', v_id_stock, v_quantite;
        i = i + 1;
    END LOOP;
END;
$$ LANGUAGE plpgsql;

Ce que je cherche à faire : récupérer plusieurs lignes avec la 1ère fonction "test.verif_stock()" et retourner le résultat obtenu à la 2ème fonction "test.stock()" qui effectuera des traitements avec les données ainsi reçue, mais là n'est pas le problème. Je ne vois pas d'autres solutions que de retourner un tableau.

[alassanediakite]

Salut

La première fonction n'a pour but que de me retourner toutes les quantités d'un article recherché ainsi que l'identifiant de ces lignes dans la table "article.stock":

Éviter les approches des langages procédurales en voulant placer les données SQL dans un tableau.
Avec les SGBDR ces genres de choses se font UNIQUEMENT avec un SELECT pas une fonction qui retourne un tableau!!!
A tout instant, lancer

Code :

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

SELECT id, quantite FROM article.stock WHERE id_article = p_id_article AND quantite > 0;

suffira
A la limite, utilisez une fonction qui renvoie UNE TABLE SQL et non TABLEAU!...

Code :

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

1
2
3
4
5
 
CREATE OR REPLACE FUNCTION test.verif_stock(p_id_article INTEGER) RETURNS table(id integer, quantite SMALLINT)
AS $$
SELECT id, quantite FROM article.stock WHERE id_article = p_id_article AND quantite > 0;
$$ LANGUAGE sql;

@+

[Waldar]

Même une vue suffirait.

[Shiva]

J'ai opté pour la première solution qui est plus simple et j'ai virer la fonction verif_stock() qui ne sert plus à rien.
Si je dois rajouter des traitements lors de la vérification de la présence de produit en stock ils seront multipliés autant de fois que ce code est utilisé, mais bon tant pis. En tout cas mon petit soucis est résolu.
Merci à toi :-)