[MySQL] Problema stupido con trigger

toyo · **Inviato:** mer giu 15, 2011 6:28 pm

allora,

Codice:

CREATE TRIGGER tr_ins_animale BEFORE INSERT ON Animali
   FOR EACH ROW
   BEGIN 
   DECLARE foo INT;
   IF(NEW.data_arrivo > CURDATE())
      select 'la data di inserimento non può essere maggiore di quella odierna' into foo; 
   END IF;
   END;

è un trigger che fa finta di fare il raise_application_error degli altri dbms in mysql, alla fine dovrebbe buttare fuori un errore del genere "invalid integer value: "la data di inserimento non può essere maggiore di quella odierna", bloccando l'inserimento.

Ma mi da errore di sintassi, dove lo vedete voi?

Inb4 manca una semicolon

toyo · **Inviato:** mer giu 15, 2011 6:30 pm

mancava il then :asd:

toyo · **Inviato:** mer giu 15, 2011 6:45 pm

Lancia un warning e inserisce lo stesso :whistler:

toyo · **Inviato:** mer giu 15, 2011 7:00 pm

si ma ti basta un check, sei scemo?

Nex · **Inviato:** mer giu 15, 2011 7:02 pm

La DECLARE dentro al ciclo FOR è una chicca...

:pua:

LilithSChild · **Inviato:** mer giu 15, 2011 7:04 pm

Nex ? ha scritto:

La DECLARE dentro al ciclo FOR è una chicca...

:pua:

Mai usati i trigger, ma tecnicamente quello non è un vero e proprio ciclo for o sbaglio ?

Nex · **Inviato:** mer giu 15, 2011 7:05 pm

Non sbagli, ma io non avrei mai messo la DECLARE lì dentro...

toyo · **Inviato:** mer giu 15, 2011 7:18 pm

Nex ? ha scritto:

Non sbagli, ma io non avrei mai messo la DECLARE lì dentro...

guarda che non è un for, è un "fai sta roba per tutte le righe che vengono cambiate", e di solito è una sola :wat:

LilithSChild · **Inviato:** mer giu 15, 2011 7:20 pm

toyo ? ha scritto:

Nex ? ha scritto:

Non sbagli, ma io non avrei mai messo la DECLARE lì dentro...

guarda che non è un for, è un "fai sta roba per tutte le righe che vengono cambiate", e di solito è una sola :wat:

in b4 caricamento ODI/SSIS con tabella da 30 milioni di record per 150 campi.

Nex · **Inviato:** mer giu 15, 2011 7:23 pm

toyo ? ha scritto:

guarda che non è un for, è un "fai sta roba per tutte le righe che vengono cambiate", e di solito è una sola :wat:

A parte che non è esattamente così, comunque l'avevo scritto sopra.

Il punto è che a livello logico è sbagliato e se fai 'ste cappellate poi ti capita di infilarle anche nel codice normale...

toyo · **Inviato:** mer giu 15, 2011 7:26 pm

Nex ? ha scritto:

toyo ? ha scritto:

guarda che non è un for, è un "fai sta roba per tutte le righe che vengono cambiate", e di solito è una sola :wat:

A parte che non è esattamente così, comunque l'avevo scritto sopra.

Il punto è che a livello logico è sbagliato e se fai 'ste cappellate poi ti capita di infilarle anche nel codice normale...

ok, ma per una roba così

Codice:

CREATE TRIGGER tr_incasso BEFORE UPDATE ON Storico_vendite
FOR EACH ROW
   BEGIN
   DECLARE prezzo_n FLOAT;
   DECLARE prezzo_ns FLOAT;
   DECLARE prezzo_r FLOAT;
   DECLARE prezzo_rs FLOAT;
   DECLARE prezzo_c FLOAT;
   DECLARE prezzo_cs FLOAT;
   DECLARE totale FLOAT;
   SELECT prezzo FROM Biglietti WHERE tipo="normale" into prezzo_n;
   SELECT prezzo FROM Biglietti WHERE tipo="normale_s" into prezzo_ns;
   SELECT prezzo FROM Biglietti WHERE tipo="ridotto" into prezzo_r;
   SELECT prezzo FROM Biglietti WHERE tipo="ridotto_s" into prezzo_rs;
   SELECT prezzo FROM Biglietti WHERE tipo="comitiva" into prezzo_c;
   SELECT prezzo FROM Biglietti WHERE tipo="comitiva_s" into prezzo_cs;
   SET NEW.incasso=(prezzo_n*NEW.normale) + (prezzo_ns*NEW.normale_s) + (prezzo_r*NEW.ridotto) + (prezzo_rs*NEW.ridotto_s) + (prezzo_c*NEW.comitiva) + (prezzo_cs*NEW.comitiva_s);
   END;

come fai senza i declare?

Nex · **Inviato:** mer giu 15, 2011 8:29 pm

Li metti subito dopo la CREATE TRIGGER e fuori dal ciclo.

toyo · **Inviato:** mer giu 15, 2011 10:33 pm

Nex ? ha scritto:

Li metti subito dopo la CREATE TRIGGER e fuori dal ciclo.

ah si può? in tutti gli esempi che ho visto il for each era sempre sotto il create trigger.

provo subito.

toyo · **Inviato:** mer giu 15, 2011 10:38 pm

intendi così giusto?

Codice:

CREATE TRIGGER tr_incasso BEFORE UPDATE ON Storico_vendite
  DECLARE prezzo_n FLOAT;
  DECLARE prezzo_ns FLOAT;
  DECLARE prezzo_r FLOAT;
  DECLARE prezzo_rs FLOAT;
  DECLARE prezzo_c FLOAT;
  DECLARE prezzo_cs FLOAT;
  DECLARE totale FLOAT;
  SELECT prezzo FROM Biglietti WHERE tipo="normale" into prezzo_n;
  SELECT prezzo FROM Biglietti WHERE tipo="normale_s" into prezzo_ns;
  SELECT prezzo FROM Biglietti WHERE tipo="ridotto" into prezzo_r;
  SELECT prezzo FROM Biglietti WHERE tipo="ridotto_s" into prezzo_rs;
  SELECT prezzo FROM Biglietti WHERE tipo="comitiva" into prezzo_c;
  SELECT prezzo FROM Biglietti WHERE tipo="comitiva_s" into prezzo_cs;
   FOR EACH ROW
   BEGIN
   SET NEW.incasso=(prezzo_n*NEW.normale) + (prezzo_ns*NEW.normale_s) + (prezzo_r*NEW.ridotto) + (prezzo_rs*NEW.ridotto_s) + (prezzo_c*NEW.comitiva) + (prezzo_cs*NEW.comitiva_s);
   END;

o serve un altro begin/end?

magari è una domanda straniubba ma iniziato stamattina con pl/sql

Nex · **Inviato:** mer giu 15, 2011 10:43 pm

Le DECLARE stanno fuori dal ciclo, ma le SELECT devono stare dentro, altrimenti non funziona niente.

I BEGIN/END li usi solo per i cicli, gli IF/CASE o quando inizi una transazione, ecc...

E solo se ci sono più righe di istruzione, altrimenti puoi fare senza, es:

IF @MY_VAR = 'Ciao' THEN SET @YOUR_VAR = 'Ok'

toyo · **Inviato:** mer giu 15, 2011 10:53 pm

Ok grazie.
Ma se posso dichiarare una variabile fuori dal ciclo, perché non posso inizializzarla?

Nex · **Inviato:** mer giu 15, 2011 10:57 pm

Puoi inizializzarle dove vuoi, ma in questo caso specifico non sapendo se la tabella "BIGLIETTI" ha dei dati che non sono influenzati dal trigger che stai lanciando o comunque che non mutano mentre il trigger viene innescato, conviene mettere le SELECT dentro.

Mi spiego: mettiamo che il TRIGGER parta per ogni UPDATE che fai. Se le tue SELECT vanno a testare le stesse UPDATE e tu le mettessi fuori dal ciclo, nel caso in cui il ciclo agisse su più ROW modificate, avresti una situazione incoerente...

Dipende quindi dai casi.

toyo · **Inviato:** mer giu 15, 2011 11:03 pm

Nex ? ha scritto:

Puoi inizializzarle dove vuoi, ma in questo caso specifico non sapendo se la tabella "BIGLIETTI" ha dei dati che non sono influenzati dal trigger che stai lanciando o comunque che non mutano mentre il trigger viene innescato, conviene mettere le SELECT dentro.

Mi spiego: mettiamo che il TRIGGER parta per ogni UPDATE che fai. Se le tue SELECT vanno a testare le stesse UPDATE e tu le mettessi fuori dal ciclo, nel caso in cui il ciclo agisse su più ROW modificate, avresti una situazione incoerente...

Dipende quindi dai casi.

ah ok, ma nel caso in cui viene richiamato questo trigger(che è sull'update di Storico_vendite, non Biglietti), la tabella biglietti ha sempre gli stessi valori (per motivi intrinsechi a come è progettato il mio zoo :asd:

)

quindi lo lascio fuori, grazie ancora

toyo · **Inviato:** mer giu 15, 2011 11:07 pm

Codice:

CREATE TRIGGER tr_incasso BEFORE UPDATE ON Storico_vendite
DECLARE prezzo_n FLOAT;
DECLARE prezzo_ns FLOAT;
DECLARE prezzo_r FLOAT;
DECLARE prezzo_rs FLOAT;
DECLARE prezzo_c FLOAT;
DECLARE prezzo_cs FLOAT;
SELECT prezzo FROM Biglietti WHERE tipo="normale" into prezzo_n;
SELECT prezzo FROM Biglietti WHERE tipo="normale_s" into prezzo_ns;
SELECT prezzo FROM Biglietti WHERE tipo="ridotto" into prezzo_r;
SELECT prezzo FROM Biglietti WHERE tipo="ridotto_s" into prezzo_rs;
SELECT prezzo FROM Biglietti WHERE tipo="comitiva" into prezzo_c;
SELECT prezzo FROM Biglietti WHERE tipo="comitiva_s" into prezzo_cs;
FOR EACH ROW
   BEGIN
   SET NEW.incasso=(prezzo_n*NEW.normale) + (prezzo_ns*NEW.normale_s) + (prezzo_r*NEW.ridotto) + (prezzo_rs*NEW.ridotto_s) + (prezzo_c*NEW.comitiva) + (prezzo_cs*NEW.comitiva_s);
   END;

errore di sintassi alla riga 2 :wat:

Nex · **Inviato:** gio giu 16, 2011 8:33 am

CREATE TRIGGER tr_incasso BEFORE UPDATE ON Storico_vendite

AS

DECLARE ...
...

MadJackal · **Inviato:** sab giu 18, 2011 6:27 pm

Il problema stupido è usare MySQL :whistler:

LilithSChild · **Inviato:** sab giu 18, 2011 6:28 pm

MadJackal ? ha scritto:

Il problema stupido è usare MySQL :whistler:

Sto cazzo.

MadJackal · **Inviato:** sab giu 18, 2011 6:50 pm

LilithSChild ? ha scritto:

Sto cazzo.

Sto cazzo un cazzo. :asd:

LilithSChild · **Inviato:** sab giu 18, 2011 6:52 pm

MadJackal ? ha scritto:

LilithSChild ? ha scritto:

Sto cazzo.

Sto cazzo un cazzo. :asd:

torna a giocare con gli archivi. :wat:

trallallero · **Inviato:** sab giu 18, 2011 7:00 pm

toyo ha scritto:

Codice:

CREATE TRIGGER tr_incasso BEFORE UPDATE ON Storico_vendite
DECLARE prezzo_n FLOAT;
DECLARE prezzo_ns FLOAT;
DECLARE prezzo_r FLOAT;
DECLARE prezzo_rs FLOAT;
DECLARE prezzo_c FLOAT;
DECLARE prezzo_cs FLOAT;
SELECT prezzo FROM Biglietti WHERE tipo="normale" into prezzo_n;
SELECT prezzo FROM Biglietti WHERE tipo="normale_s" into prezzo_ns;
SELECT prezzo FROM Biglietti WHERE tipo="ridotto" into prezzo_r;
SELECT prezzo FROM Biglietti WHERE tipo="ridotto_s" into prezzo_rs;
SELECT prezzo FROM Biglietti WHERE tipo="comitiva" into prezzo_c;
SELECT prezzo FROM Biglietti WHERE tipo="comitiva_s" into prezzo_cs;
FOR EACH ROW
   BEGIN
   SET NEW.incasso=(prezzo_n*NEW.normale) + (prezzo_ns*NEW.normale_s) + (prezzo_r*NEW.ridotto) + (prezzo_rs*NEW.ridotto_s) + (prezzo_c*NEW.comitiva) + (prezzo_cs*NEW.comitiva_s);
   END;

errore di sintassi alla riga 2 :wat:

Si ma toyo, googlare un esempio di trigger sql prima di fare domande qui, no ? :lnrg:

The Dark Side of Hardware Upgrade Forum

[MySQL] Problema stupido con trigger

Chi c’è in linea