Echadme una mano con triggers, estoy aprendiendo

[manurodri189]

Pues a ver si alguno de vosotros me echa una mano con esto.

Tengo 2 tablas, y se trata de hacer un trigger que si se actualiza una, tenga que hacer un registro en otra, con la hora de modificación, el valor anterior y el nuevo, y poco más.

Pero he hecho el trigger, y me dice errores de compilación, osea que está mal. Me vuelvo loco con esto, y tiene pinta de ser chorrada, pero no doy con la tecla.

Hasta ahora he hecho esto, pero vamos que parece que no funciona el trigger.

Código:

CREATE TABLE Cuenta (nro_cuenta varchar(10),
balance integer,
PRIMARY KEY(nro_cuenta));
 
CREATE TABLE Transaccion(nro_cuenta varchar(10),
hora_mod date,
id_cliente varchar(10),
ant_balance integer,
act_balance integer,
PRIMARY KEY(nro_cuenta,hora_mod));
 
CREATE OR REPLACE TRIGGER actualizar
AFTER UPDATE ON Cuenta
BEGIN
    INSERT INTO Transaccion
    (hora_mod, ant_balance, act_balance)
    VALUES(SYSDATE, OLD.balance, NEW.balance)
END actualizar;

Salu2

[huesos52]

Si en la tabla transaccion lleva un primary key, por defecto es not null y es necesario incluirlo en cualquier inserciòn.

Con respecto al trigger faltan algunos detalles... como el for each row

Código SQL:

SQL> CREATE OR REPLACE TRIGGER actualizar 
AFTER UPDATE ON Cuenta
FOR each ROW WHEN (NEW.balance <> 999)
BEGIN
INSERT INTO Transaccion (nro_cuenta,hora_mod, ant_balance, act_balance) VALUES(:OLD.nro_cuenta,SYSDATE, :OLD.balance, :NEW.balance);
END actualizar;
/  2    3    4    5    6    7  
 
Disparador creado.
 
SQL> SELECT *FROM cuenta;
 
NRO_CUENTA    BALANCE
---------- ----------
1254120            20
 
SQL> SELECT *FROM transaccion;
 
ninguna fila seleccionada
 
SQL> UPDATE cuenta SET balance=25;
 
1 fila actualizada.
 
SQL> SELECT *FROM transaccion;
 
NRO_CUENTA HORA_MOD ID_CLIENTE ANT_BALANCE ACT_BALANCE
---------- -------- ---------- ----------- -----------
1254120    28/07/09                     20          25
 

saludos

[manurodri189]

Tengo unas dudillas con respecto al codigo que me pasaste, en primer lugar esta sentencia:

(new.balance <> 999) que significa exactamente? que el nuevo balance sea diferente de 999? porque? y otra cosa el when es obligatorio?

y otra cosa, porque el : delante de NEW y OLD, y en esa sentencia no? muchas gracias tío.

Si me surgen más dudillas te puedo preguntar?

Salu2

[huesos52]

Tengo unas dudillas con respecto al codigo que me pasaste, en primer lugar esta sentencia:

(new.balance <> 999) que significa exactamente? que el nuevo balance sea diferente de 999? porque? y otra cosa el when es obligatorio?

creo que esto puedes probarlo tu mismo.
Si el trigger se crea con errores puedes digitar en consola SHOW ERRORS; para ver en que parte se generan.

y otra cosa, porque el : delante de NEW y OLD, y en esa sentencia no? muchas gracias tío.

Sino estoy mal, el : en una variable indica asignación, pero para efectos de comparación no es necesaria.


saludos

[jc3000]

Iniciado por huesos52

Sino estoy mal, el : en una variable indica asignación, pero para efectos de comparación no es necesaria.


saludos

Compi huesos, te confundes. La asignacion es esta :=

VAR_CODIGO := 98;

y los dos puntos tienen que ir tanto en el old como en el new

:NEW.SALARIO := :OLD.SALARIO * 10;

[huesos52]

manurodri189 ahí tienes tu respuesta.


Saludos

[manurodri189]

Muchas gracias, ya entendí lo del when y lo de los :.

Ahora intento lo siguiente:

Código SQL:

CREATE TABLE distancias(ruta VARCHAR(10),
distancia_k INTEGER,
distancia_m INTEGER,
PRIMARY KEY(ruta));
 
Tabla creada.
 
 
CREATE OR REPLACE TRIGGER up_km
AFTER UPDATE ON distancias ON distancia_k
FOR each ROW
BEGIN
UPDATE transaccion SET distancia_m=:NEW.distancia_k*0.621371;
END up_km;
/
 

Este trigger lo que hace es si se actualiza la distancia en km automaticamente lo pone en millas. Pero no se como acceder en esta linea al altributo de la tabla:

Código SQL:

AFTER UPDATE ON distancias ON distancia_k
 

Se supone que si inserto una nueva fila con km o millas, tiene que actualizar el otro atributo, eso si que no se por donde cogerlo. Muchas gracias.

Salu2

[manurodri189]

Bueno, he leido un tutorial que me he bajado muy chulo, y he hecho lo siguiente:

Código SQL:

CREATE OR REPLACE TRIGGER up_km
AFTER UPDATE OF distancia_k, distancia_m ON distancias 
FOR each ROW
BEGIN
IF UPDATING ('distancia_k') THEN UPDATE distancias SET distancia_m=:NEW.distancia_k*0.621371;
ELSIF UPDATING ('distancia_m') THEN UPDATE distancias SET distancia_k=:NEW.distancia_m*1.609344;
END IF;
END up_km;
 

Pero al hacer un set, da problemas de tablas mutantes, ahi si que no tengo ni idea de como solucionarlo.

Salu2

[jc3000]

Las tablas mutantes en algunos casos tienen solucion, pero antes hay algo que no veo.

¿ haces un update sin where ?

THEN UPDATE distancias SET distancia_m=:NEW.distancia_k*0.621371;
THEN UPDATE distancias SET distancia_k=:NEW.distancia_m*1.609344;

[manurodri189]

Pues si, no? se supone que lo que quiero hacer es modificar la misma fila que me llegó, y solo tengo una, porque si pusiera el where como distingo la fila?

Salu2

[huesos52]

manurodri189

Con ese update vas a modificar todas las filas de tu tabla.

debes poner el where para distinguir la ruta.
Ademas, puedes hacerlo en una misma sentencia update.

Código SQL:

UPDATE distancias 
SET 
distancia_m=:NEW.distancia_k*0.621371,
distancia_k=:NEW.distancia_m*1.609344 
WHERE ruta=:OLD.ruta;
 

[manurodri189]

No podría ponerlo en la misma sentencia no? osea, si quiero actualizar los km, entonces se debe cambiar solo las millas, y viceversa, si me viene un parametro null en las millas o km eso no iria, que no?

Salu2

[jc3000]

Iniciado por huesos52

manurodri189

Con ese update vas a modificar todas las filas de tu tabla.

debes poner el where para distinguir la ruta.
Ademas, puedes hacerlo en una misma sentencia update.

Código SQL:

UPDATE distancias 
SET 
distancia_m=:NEW.distancia_k*0.621371,
distancia_k=:NEW.distancia_m*1.609344 
WHERE ruta=:OLD.ruta;
 

Compi, ese update me dá que no funcionaria. Si no estoy equivocado ese trigger estaría saltando de manera infinita o daría error por tabla mutante.

Creo que lo correcto sería algo parecido a esto:

if UPDATING distancia_k THEN
:new.distancia_m=:NEW.distancia_k*0.621371;
ELSIF UPDATING distancia_m THEN
:new.distancia_k=:NEW.distancia_m*1.609344;
end if;

Todo es probar.

Ya nos dirás.

[manurodri189]

Pero así es como yo lo tenia, y da problemas de tablas mutantes.

Salu2

[jc3000]

Iniciado por manurodri189

Pero así es como yo lo tenia, y da problemas de tablas mutantes.

Salu2

Ahora me doy cuenta de que es posible ya que o bien actualizas un campo o bien el otro.

Mmmmmmmmmmm, mala solucion...........

Tengo una idea ¿ como funcionaria un AFTER UPDATE para un campo y un BEFORE UPDATE para el otro con un AUTONOMOUS TRANSACTION en cada uno ?, pero le doy una pensada a ver si se me ocurre algo mas decente y lo comentamos, ahora me voy a cenar que la parienta me va a poner la almohada en la bañera.........

Mañana seguimos.

[manurodri189]

Eso de AUTONOMOUS TRANSACTION no se ni lo que es xDDD, voy a investigar a ver que sale.

Salu2

[jc3000]

Hoy tengo un dia muy feo en el curro, a ver si a ultima hora le puedo dar una vuelta a este tema.

[manurodri189]

Iniciado por jc3000

Hoy tengo un dia muy feo en el curro, a ver si a ultima hora le puedo dar una vuelta a este tema.

No te preocupes tio, y gracias por todo.

Salu2

[jc3000]

Iniciado por manurodri189

No te preocupes tio, y gracias por todo.

Salu2

Pues no, no he podido, a ver si mañana estoy mas tranquilo.

Te informo pa que sepas que sigo este tema, me parece un reto.......

[manurodri189]

De verdad que no te tienes por que preocupar eh. Pero vamos si te parece un reto entonces me callo, pero si no, puedes dejarlo tranquilamente.

Salu2

[jc3000]

Si los dos campos no se actualizen a la vez ( no he probado esa casuistica ) a lo mejor te valdría esto:

SQL*Plus: Release 8.0.5.0.0 - Production on Mié Mar 17 11:23:32 2010

(c) Copyright 1999 Oracle Corporation. All rights reserved.


Conectado a:
Oracle Database 10g Enterprise Edition Release 10.2.0.4.0 - 64bit Production
With the Partitioning, OLAP, Data Mining and Real Application Testing options

SQL> CREATE TABLE distancias (ruta varchar(10),
2 distancia_k integer,
3 distancia_m integer )
4 /

Tabla creada.

SQL> insert into distancias
2 values
3 ('1',2,3)
4 /

1 fila creada.

SQL>
SQL>
SQL> commit
2 /

Validación terminada.

SQL>
SQL> insert into distancias
2 values
3 ('2',4,5)
4 /

1 fila creada.

SQL>
SQL>
SQL> commit
2 /

Validación terminada.

SQL> CREATE OR REPLACE TRIGGER up_km
2 before UPDATE Of distancia_k,distancia_m on distancias
3 FOR each row
4 begin
5 if :new.distancia_k <> :old.distancia_k THEN
6 :new.distancia_m := :NEW.distancia_k* 2;
7 elsif :new.distancia_m <> :old.distancia_m THEN
8 :new.distancia_k := :NEW.distancia_m* 4;
9 end if;
10 end;
11 /

Disparador creado.

SQL> select * from distancias
2 ;

RUTA DISTANCIA_K DISTANCIA_M
---------- ----------- -----------
1 2 3
2 4 5

SQL> update distancias
2 set distancia_k = 6
3 where ruta = '1';

1 fila actualizada.

SQL> commit;

Validación terminada.

SQL> select * from distancias;

RUTA DISTANCIA_K DISTANCIA_M
---------- ----------- -----------
1 6 12
2 4 5

SQL> drop table distancias purge;

Tabla borrada.

SQL>

[manurodri189]

Muchas gracias tio, pues seguramente si me valga, gracias por todo.

Salu2

[huesos52]

Acabo de tener un problema con tablas mutantes en el uso de unos triggers y recordé este tema. Me ayudó bastante.

Considerando que he ganado un poco de habilidad en el manejo de PL (hace unos cuantos meses no tenía idea de que estaban hablando ) empecé a mirar detalladamente el problema y noté que la primera solución que dió jc3000 era la correcta y no era necesario dar tantas vueltas (Por cierto, muy ingeniosas)

Retomando el ejemplo de jc3000

Código SQL:

SQL> CREATE TABLE distancias (ruta VARCHAR(10),
distancia_k INTEGER,
distancia_m INTEGER )
/
 
INSERT INTO distancias
VALUES
('1',2,3)
/
 
INSERT INTO distancias
VALUES
('2',4,5)
/
commit;  2    3    4
Tabla creada.
 
SQL> SQL>   2    3    4
1 fila creada.
 
SQL> SQL>   2    3    4
1 fila creada.
 
SQL> CREATE OR REPLACE TRIGGER up_km
  2  BEFORE UPDATE OF distancia_k,distancia_m ON distancias
  3  FOR each ROW
  4  BEGIN
  5  IF UPDATING ('distancia_k') THEN
  6  :NEW.distancia_m := :NEW.distancia_k*2;
  7  ELSIF UPDATING ('distancia_m') THEN
  8  :NEW.distancia_k := :NEW.distancia_m*4;
  9  END IF;
 10  END;
 11  /
 
Disparador Creado.
 
SQL> SELECT * FROM distancias;
 
RUTA       DISTANCIA_K DISTANCIA_M
---------- ----------- -----------
1                    2           3
2                    4           5
 
SQL> UPDATE distancias
SET distancia_k = 6
WHERE ruta = '1';  2    3
 
1 fila actualizada.
 
SQL> SELECT * FROM distancias;
 
RUTA       DISTANCIA_K DISTANCIA_M
---------- ----------- -----------
1                    6          12
2                    4           5
 

Creo que el problema de manurodri189 era que en vez de asignar los valores a las variables :NEW lo hacía pero en la ejecución de una sentencia update de la misma tabla que ejecuta el trigger (Por ello el error de tablas mutantes) y no como inicialmente se lo recomendaba jc3000.

Por cierto... me ha gustado mucho el tema de AUTONOMOUS TRANSACTION que de seguro me va a ser muy util para un proyecto que tengo.

Gracias por información tan valiosa jc3000

[jc3000]

Iniciado por huesos52

Por cierto... me ha gustado mucho el tema de AUTONOMOUS TRANSACTION que de seguro me va a ser muy util para un proyecto que tengo.

Que tal Huesos ?

Me alegro de haber aportado algo.

Respecto a lo de " AUTONOMOUS TRANSACTION ", lee esto : MUCHO OJO CON ELLO, mal manejado es peligroso.

Como no lo tengas todo, muy, muy controlado puedes dejar en la BBDD inconsistencia de datos.

Iniciado por huesos52

Gracias por información tan valiosa jc3000

De nada tio, con peña como tu, da gusto venir por aquí.

[huesos52]

Respecto a lo de " AUTONOMOUS TRANSACTION ", lee esto : MUCHO OJO CON ELLO, mal manejado es peligroso.

Como no lo tengas todo, muy, muy controlado puedes dejar en la BBDD inconsistencia de datos.

Lo tendré muy en cuenta.

[anahim]

Gracias a todos los que participaron de este post me sirvio mucho en lo actualmente estoy desarrollando.

Espero pronto también colaborar en algo.

Saludos a todos.