WÄHLEN Sie oder FÜHREN Sie in eine PL/pgSQL-Funktion

Habe ich diese Funktion in meine Datenbank:

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;

Aber sobald ich versuche, es zu benutzen, es sagt ich PERFORM wenn ich will, Sie zu verwerfen, die Ergebnisse! Das problem hier ist, dass ich nicht wollen! Ich möchte, dass Sie in der articulo Zeile habe ich erklärt!

Ich bin mit dieser Aussage: 

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

Und der Fehler den ich bekomme, ist 42601 einen syntax-Fehler! Wie kann es sein, wenn ich mit der IDE zu erstellen? Irgendeine Idee zum problem?

InformationsquelleAutor dbncourt | 2012-04-12
  1. 14

    In plpgsql-code SELECT ohne ein Ziel löst einen Fehler aus. Aber Sie haben natürlich nicht wollen SELECT INTO Sie wollen einfach nur, um den status der FOUND. Verwenden Sie FÜHREN für, die.

    Besser, aber, verwenden Sie IF EXISTS .... Betrachten Sie dieses umschreiben der Funktion:

    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;

    Über EXISTS:

    Den anderen wichtigen Punkt:

    • Verwenden Sie die GIBT - Klausel der INSERT - Anweisung anstelle von einer zusätzlichen SELECT.

    Postgres 9.5+

    In PostgreSQL 9.5 oder höher verwenden Sie lieber ein einem einfachen Beispiel: statt: (INSERT ... ON CONFLICT DO NOTHING).

    Sie hätte UNIQUE Einschränkungen auf "Articulo"("Nombre", "idTipo", "idFamilia") und "ArticuloMarca"("idArticulo", "idMarca") und dann:

    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;

    Dies ist schneller, einfacher und zuverlässiger. Die zusätzlichen loops Regel verbliebene race-conditions bei gleichzeitigem schreibt (beim hinzufügen der kaum Kosten). Ohne gleichzeitige schreibt, Sie vereinfachen können. Ausführliche Erklärung:

    Beiseite: rechtliche, lower-case-IDS zu vermeiden alle hässlich Doppel-Zitat.

    • yeah! thx dude! ill give it a try! dies ist meine erste Funktion und es gibt tatsächlich zu wenige Beispiele im internet! und die Dokumentation ist irgendwie seltsam, zumindest für mich, im neuen mit dieser ganzen Programmierung/Datenbank-Zeug! ich habe es bereits gelöst, aber die Lösung ist nicht gut, es tut was ich will, aber im hinzufügen-Zeug bin ich nicht wirklich mit sondern beheben Sie die Fehler. Ill versuchen Sie dieses! und thx nochmal! 😀
    InformationsquelleAutor Erwin Brandstetter
  2. 4

    Diese Zeile sieht mir suspekt und ist es wahrscheinlich, was die Ursache Ihrer Trauer:

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

    Sind Sie die Ausführung einer SELECT-in Ihrer Funktion, aber nicht etwas zu tun mit den Ergebnissen. Sie müssen zum ausführen einer SELECT INTO-wie Sie früher in Ihrer Funktion.

    • yeah! ich fand ich die Möglichkeit zum Debuggen der Funktion! und ich bekam eine Fehlermeldung gibt! ich habe es bereits gelöst! war das 2. gefunden hatte, die wie keine Referenz oder sowas, auch das ist, was ich aus der Fehlermeldung, und löste es, indem eine weitere variable "Articulo"%rowtype-und putting-Select * Into thatvariable. Ill versuchen Sie es auf die erste Antwort! Thx u Jungs! 😀
    InformationsquelleAutor Glenn
Schreibe einen Kommentar