SÉLECTIONNEZ ou d'EFFECTUER, dans un fonctions PL/pgSQL

J'ai cette fonction dans ma base de données:

CREATE OR REPLACE FUNCTION "insertarNuevoArticulo"(nombrearticulo character varying, descripcion text, idtipo integer, idfamilia bigint, artstock integer, minstock integer, maxstock integer, idmarca bigint, precio real, marcastock integer)
RETURNS boolean AS
$BODY$
DECLARE
articulo "Articulo"%ROWTYPE;
BEGIN
SELECT * INTO articulo FROM "Articulo" WHERE "Nombre" = $1 AND "idTipo"=$3 AND "idFamilia"=$4;
IF NOT FOUND THEN
INSERT INTO "Articulo" ("Nombre", "Descripcion", "idTipo", "idFamilia", "Stock", "MinStock", "MaxStock") Values ($1, $2, $3, $4, $5, $6, $7);
SELECT last_value
INTO articulo."idArticulo"
FROM "public"."Articulo_idArticulo_seq";
END IF;
SELECT * FROM "ArticuloMarca" AS am WHERE am."idArticulo" = articulo."idArticulo" and am."idMarca" = $8;
IF NOT FOUND THEN
Insert into "ArticuloMarca"("idArticulo", "idMarca", "PrecioReferencial", "Stock") Values (articulo."idArticulo", $8, $9, $10);
RETURN TRUE;
END IF;
RETURN FALSE;
END;
$BODY$
LANGUAGE plpgsql VOLATILE
COST 100;
ALTER FUNCTION "insertarNuevoArticulo"(character varying, text, integer, bigint, integer, integer, integer, bigint, real, integer)
OWNER TO postgres;

Mais dès que j'essaie de l'utiliser, il dit que j'ai besoin d'utiliser PERFORM si je veux ignorer les résultats! Le problème ici est que je ne veux pas! Je les veux dans le articulo ligne, j'ai déclaré!

Je suis l'aide de cette instruction: 

SELECT "insertarNuevoArticulo"('Acetaminofen', 'caro', '1' , '1', '8', '1', '10', '1', '150.7', '10');

Et l'erreur que j'obtiens est 42601, une erreur de syntaxe! Comment pourrait-il en être si je suis à l'aide de l'IDE pour le créer? Une idée sur le problème?

OriginalL'auteur dbncourt | 2012-04-12

  1. 13

    En plpgsql code, SELECT sans cible déclenche une erreur. Mais de toute évidence vous ne pas voulez SELECT INTO, vous voulez juste de définir le statut de FOUND. Vous utilisez EXÉCUTER.

    Mieux, mais, utilisez IF EXISTS .... Considérer cette réécriture de votre fonction:

    CREATE OR REPLACE FUNCTION "insertarNuevoArticulo"
(nombrearticulo text, descripcion text, idtipo int, idfamilia bigint, artstock int
, minstock int, maxstock int, idmarca bigint, precio real, marcastock int)
RETURNS boolean AS
$func$
DECLARE
_id_articulo "Articulo"."idArticulo"%TYPE;
BEGIN
SELECT a."idArticulo" INTO _id_articulo
FROM   "Articulo" a
WHERE  a."Nombre" = $1 AND a."idTipo" = $3 AND a."idFamilia" = $4;
IF NOT FOUND THEN
INSERT INTO "Articulo"("Nombre", "Descripcion", "idTipo"
, "idFamilia", "Stock", "MinStock", "MaxStock")
VALUES ($1, $2, $3, $4, $5, $6, $7)
RETURNING "Articulo"."idArticulo" INTO _id_articulo;
END IF;
IF EXISTS (SELECT FROM "ArticuloMarca" a
WHERE a."idArticulo" = _id_articulo AND a."idMarca" = $8) THEN
RETURN FALSE;
ELSE
INSERT INTO "ArticuloMarca"("idArticulo", "idMarca", "PrecioReferencial", "Stock")
VALUES (_id_articulo, $8, $9, $10);
RETURN TRUE;
END IF;
END
$func$  LANGUAGE plpgsql;

    Sur EXISTS:

    La autre point essentiel:

    • Utiliser le RETOUR l'article de la INSERT instruction au lieu d'un autre SELECT.

    Postgres 9.5+

    Dans Postgres 9.5 ou plus tard plutôt l'utilisation d'un UPSERT: (INSERT ... ON CONFLICT DO NOTHING).

    Vous auriez UNIQUE contraintes sur "Articulo"("Nombre", "idTipo", "idFamilia") et "ArticuloMarca"("idArticulo", "idMarca") et puis:

    CREATE OR REPLACE FUNCTION insert_new_articulo
(nombrearticulo text, descripcion text, idtipo int, idfamilia bigint, artstock int
, minstock int, maxstock int, idmarca bigint, precio real, marcastock int)
RETURNS boolean AS
$func$
DECLARE
_id_articulo "Articulo"."idArticulo"%TYPE;
BEGIN
LOOP
SELECT "idArticulo" INTO _id_articulo
FROM   "Articulo"
WHERE  "Nombre" = $1 AND "idTipo" = $3 AND "idFamilia" = $4;
EXIT WHEN FOUND;
INSERT INTO "Articulo"("Nombre", "Descripcion", "idTipo"
, "idFamilia", "Stock", "MinStock", "MaxStock")
VALUES ($1, $2, $3, $4, $5, $6, $7)
ON     CONFLICT (tag) DO NOTHING
RETURNING "idArticulo" INTO _id_articulo;
EXIT WHEN FOUND;
END LOOP;
LOOP
INSERT INTO "ArticuloMarca"("idArticulo", "idMarca", "PrecioReferencial", "Stock")
VALUES (_id_articulo, $8, $9, $10)
ON     CONFLICT ("idArticulo", "idMarca") DO NOTHING;
IF FOUND THEN
RETURN TRUE;
END IF;
IF EXISTS (SELECT FROM "ArticuloMarca"
WHERE "idArticulo" = _id_articulo AND "idMarca" = $8) THEN
RETURN FALSE;
END IF;
END LOOP;
END
$func$ LANGUAGE plpgsql;

    C'est plus rapide, plus simple et plus fiable. L'ajout de boucles règle les conditions de course avec des écritures simultanées (tout en ajoutant de la peine à tout prix). Sans écritures simultanées, vous pouvez simplifier. Explication détaillée:

    De côté: recours à la justice, bas-de-casse identifiants pour éviter tout moche double-échappement.

    ouais! thx dude! mal lui donner un essai! c'est ma première fonction et il y a effectivement trop peu d'exemples sur internet! et de la documentation est une sorte de bizarre, au moins pour moi, im nouveau avec toute cette programmation/base de données des trucs! j'ai déjà résolu, mais la solution n'est pas bonne, c'est ce que je veux, mais im ajouter des trucs im pas réellement de l'aide, mais pour effacer les erreurs. Vais essayer ça! et merci encore une fois! 😀

    OriginalL'auteur Erwin Brandstetter

  2. 4

    Cette ligne semble suspect à moi et c'est probablement ce qui est la cause de votre chagrin:

    SELECT * FROM "ArticuloMarca" AS am WHERE am."idArticulo" = articulo."idArticulo" and am."idMarca" = $8;

    L'exécution d'une sélection au sein de votre fonction, mais ne pas faire n'importe quoi avec les résultats. Vous avez besoin de SÉLECTIONNER un DANS comme vous l'avez fait plus tôt dans votre fonction.

    ouais! j'ai trouvé j'ai pour déboguer la fonction! et j'ai eu une erreur là! j'ai déjà résolu! était que la 2ème trouvé en avait presque pas de référence ou quelque chose comme ça, bien c'est ce que je reçois le message d'erreur, et résolu à l'ajout d'une autre variable "Articulo"%rowtype et la mise Select * Into thatvariable. Ill essayer, pour la première réponse! Thx u les gars! 😀

    OriginalL'auteur Glenn