[SOLUCIONADO] Problema con triggers

[Begotten]

Buenas, necesito desarrollar un trigger para una bd en la cual tengo una tabla llamada detalle_Obra en la cual al momento de agregar un valor a las variables egresos, ingresos y recepcion, la respectiva obra actualice sus datos también (egresos_t, ingresos_t, recepcion_t). creo que debería ser algo así:

Código MySQL:

UPDATE obra SET(`egresos_t`, `ingresos_t`, `recepcion_t`, `combustible_recepcion_t`, 
`combustible_emision_t`) VALUES (obra.egresos_t +
NEW.egresos, obra.ingresos_t + NEW.ingresos, obra.recepcion_t +  NEW.recepcion, 
obra.combustible_recepcion_t + NEW.combustible_recepcion, obra.combustible_emision_t + NEW.combustible_emision) 
WHERE obra.codigo = NEW.cod_obra;
 

No puse delimiters y todo el protocolo porque estoy trabajando con php myadmin
BD:

[gnzsoloyo]

Lo que tienes es errores en la sentencia UPDATE. Está simplemente mal escrita.
Vamos al manual de referencia:

13.2.11 UPDATE Syntax

Single-table syntax:

Código MySQL:

UPDATE [LOW_PRIORITY] [IGNORE] table_reference
    SET col_name1={expr1|DEFAULT} [, col_name2={expr2|DEFAULT}] ...
    [WHERE where_condition]
    [ORDER BY ...]
    [LIMIT row_count]
 

Multiple-table syntax:

Código MySQL:

UPDATE [LOW_PRIORITY] [IGNORE] table_references
    SET col_name1={expr1|DEFAULT} [, col_name2={expr2|DEFAULT}] ...
    [WHERE where_condition]
 

¿Dónde ves que esté la cláusula VALUES?

...



Exacto: En ninguna parte.

Corrige la sintaxis, prueba de nuevo y si aparecen problemas, dinos qué emnsajes se visualizan.

[Begotten]

Código MySQL:

UPDATE obra SET(`egresos_t`= obra.egresos_t + NEW.egresos, `ingresos_t` = obra.ingresos_t + 
NEW.ingresos, 
`recepcion_t`= obra.recepcion_t +  NEW.recepcion, `combustible_recepcion_t`= obra.combustible_recepcion_t + 
NEW.combustible_recepcion, `combustible_emision_t` = obra.combustible_emision_t + 
NEW.combustible_emision)
 WHERE obra.codigo = NEW.cod_obra;
 

me da error de sintaxis (error 1064), al parecer es dentro del "SET"

[gnzsoloyo]

Vuelvo a lo que te dije: Manual de referencia...

¿Donde ves que se usen paréntesis para agrupar los campos afectadospor el SET?

En ninguna parte.

La sintaxis de UPDATE en MySQL es del ANSI-SQL, es decir, es estandar. Se usa igual en todos los DBMS...

Por favor, lee con cuidado los ejemplos del manual, y algun que otro tutorial.

[Begotten]

Iniciado por gnzsoloyo

Vuelvo a lo que te dije: Manual de referencia...

¿Donde ves que se usen paréntesis para agrupar los campos afectadospor el SET?

En ninguna parte.

La sintaxis de UPDATE en MySQL es del ANSI-SQL, es decir, es estandar. Se usa igual en todos los DBMS...

Por favor, lee con cuidado los ejemplos del manual, y algun que otro tutorial.

ya lo solucione, me estaba confundiendo ya que lo estaba haciendo de otra forma anteriormente, ademas los manuales son bastante complicados de entender, es mejor un ejemplo y se entiende mucho mejor. Gracias por la ayuda, dejo la solucion:

Código MySQL:

UPDATE obra SET `egresos_t`= obra.egresos_t + NEW.egresos, `ingresos_t` = obra.ingresos_t + NEW.ingresos, 
`recepcion_t`= obra.recepcion_t +  NEW.recepcion, `combustible_recepcion_t`= obra.combustible_recepcion_t + 
NEW.combustible_recepcion, `combustible_emision_t` = obra.combustible_emision_t + NEW.combustible_emision 
WHERE obra.codigo = NEW.cod_obra
 

[Begotten]

Vuelvo a consultar acá ya que me surgió un inconveniente, cree otro trigger para cuando la tabla detalle_Obra fuera actualizada, los cambios se vieran en obra hasta ahí todo bien pero el problema surgió al crear el trigger para cuando las tablas detalle_Obra fueran eliminadas ya que en ningún momento elimino realmente los valores si no que cambio el atributo "estado" a cero y no se vuelven a visualizar. Esto me genera problemas ya que existen 2 tipos de actualización (la real y la supuesta eliminación). ¿Como puedo diferenciar cada UPDATE?

[gnzsoloyo]

Tendrás que programar todo dentro del mismo trigger, lo que significa que deberás analizar los datos entrantes y determinar a partir de eso qué UPDATE debe ejecutarse.
No se admiten triggers sobre el mismo evento, por lo que todo pasará por lo que recibe la tabla.

[Begotten]

Iniciado por gnzsoloyo

Tendrás que programar todo dentro del mismo trigger, lo que significa que deberás analizar los datos entrantes y determinar a partir de eso qué UPDATE debe ejecutarse.
No se admiten triggers sobre el mismo evento, por lo que todo pasará por lo que recibe la tabla.

entendido, ¿como se puede saber si un atributo fue editado o no? para hacer una condicion

[gnzsoloyo]

¿como se puede saber si un atributo fue editado o no?

No puedes. Sólo podrás ver si hay un datos entrantes válidos en el UPDATE que haces...
Pero creo que si piensas un momento lo que tu mismo dices, verás la respuesta: Si tienes un campo que sólo cargas cuando haces el borrado lógico (así se denomina), entonces ese campo será NULL en todos los UPDATE excepto en ese caso...
Me parece que con sólo razonarlo ya puedes ver cuál es la condición a cumplir. ¿no?

[Begotten]

Iniciado por gnzsoloyo

No puedes. Sólo podrás ver si hay un datos entrantes válidos en el UPDATE que haces...
Pero creo que si piensas un momento lo que tu mismo dices, verás la respuesta: Si tienes un campo que sólo cargas cuando haces el borrado lógico (así se denomina), entonces ese campo será NULL en todos los UPDATE excepto en ese caso...
Me parece que con sólo razonarlo ya puedes ver cuál es la condición a cumplir. ¿no?

No se si en este caso el valor cero se considera NULL pero se podría poner una condición para el nuevo valor de "estado" (si es uno hace las operaciones de actualización y si es cero hace las operaciones del borrado lógico) pero de nuevo tengo problemas con la sintaxis, ¿Podrías ayudarme?

Código MySQL:

if NEW.estado == 1 THEN BEGIN //actualizacion
UPDATE obra SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos), `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos), `recepcion_t`= obra.recepcion_t +  (NEW.recepcion - OLD.recepcion) , `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion), `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision) WHERE obra.codigo = OLD.cod_obra;
END; END IF;
if NEW.estado == 0 THEN BEGIN //borrado logico
UPDATE obra SET `egresos_t`= obra.egresos_t - OLD.egresos, `ingresos_t` = obra.ingresos_t - OLD.ingresos, `recepcion_t`= obra.recepcion_t - OLD.recepcion, `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion, `combustible_emision_t` = obra.combustible_emision_t - OLD.combustible_emision WHERE obra.codigo = OLD.cod_obra;
WHERE obra.codigo = NEW.cod_obra
END; END IF;
 

[gnzsoloyo]

Por favor, usa los Highligt. Hay uno específico para MySQL.

Por otro lado, acuerdate que no estás esribiendo PHP, ni C#. Las sintaxis ecada DBMS es propia y el"==", no existe en el procedural de MySQL.
Además de eso, NULL no es cero. No es siquiera un dato. es un estado que se evalua con IS NULL o IS NOT NULL.

[Begotten]

Iniciado por gnzsoloyo

Por favor, usa los Highligt. Hay uno específico para MySQL.

Por otro lado, acuerdate que no estás esribiendo PHP, ni C#. Las sintaxis ecada DBMS es propia y el"==", no existe en el procedural de MySQL.
Además de eso, NULL no es cero. No es siquiera un dato. es un estado que se evalua con IS NULL o IS NOT NULL.

el error no es ese, tampoco puedo ver bien el error ya que el popup se sale de la pantalla y no puedo bajar.

[gnzsoloyo]

¿Qué herramienta estás usando para crear ese trigger?

No la reconozco...

En cuanto al popup, debería ser perfectamente visible, incluso trabajando desde un telefono (cosa quye yo a veces hago). Tienes algún problema de navegador u otro componente...

[Begotten]

es phpmyadmin la version que se instala junto a wamp y no creo que sea problema de navegador ya que tanto en chrome como mozilla ocurre lo mismo pero revisando el codigo con firebug pude visualizar el error:

Falló la siguiente consulta: "CREATE TRIGGER `actualizarObra2` AFTER UPDATE ON `detalle_obra` FOR EACH ROW if NEW.estado = 1 THEN UPDATE obra SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos), `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos), `recepcion_t`= obra.recepcion_t + (NEW.recepcion - OLD.recepcion) , `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion), `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision) WHERE obra.codigo = OLD.cod_obra END; END IF; if NEW.estado = 0 THEN UPDATE obra SET `egresos_t`= obra.egresos_t - OLD.egresos, `ingresos_t` = obra.ingresos_t - OLD.ingresos, `recepcion_t`= obra.recepcion_t - OLD.recepcion, `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion, `combustible_emision_t` = obra.combustible_emision_t - OLD.combustible_emision WHERE obra.codigo = OLD.cod_obra WHERE obra.codigo = NEW.cod_obra END; END IF;"
<br/>
MySQL ha dicho: #1064 - Erreur de syntaxe près de 'END; END IF; if NEW.estado = 0 THEN UPDATE obra SET `egresos_t`= obra.egresos_' à la ligne 3

[gnzsoloyo]

Disculpa,pero el problema es que no estás estudiando el manual de referencia para escribir rutinas almacenadas... En serio. Son errores de manual.

Código MySQL:

CREATE TRIGGER `actualizarObra2` 
AFTER UPDATE ON `detalle_obra` 
FOR EACH ROW 
if NEW.estado = 1 THEN 
    UPDATE obra 
        SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos), 
        `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos), 
        `recepcion_t`= obra.recepcion_t + (NEW.recepcion - OLD.recepcion) , 
        `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion), 
        `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision) 
    WHERE obra.codigo = OLD.cod_obra 
    END; # ESTO ESTÁ DE MÁS
    END IF; 
    if NEW.estado = 0 THEN 
        UPDATE obra SET `egresos_t`= obra.egresos_t - OLD.egresos, 
        `ingresos_t` = obra.ingresos_t - OLD.ingresos, 
        `recepcion_t`= obra.recepcion_t - OLD.recepcion, 
        `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion, 
        `combustible_emision_t` = obra.combustible_emision_t - OLD.combustible_emision 
    WHERE obra.codigo = OLD.cod_obra # NO PUEDEN EXISTIR DOS WHERE EN UNA MISMA SENTENCIA!
    WHERE obra.codigo = NEW.cod_obra 
    END; # ESTO ESTÁ DE MÁS
 
END IF;
 

Además, toda rutina (procedure, function, trigger, event) que contenga un conjunto de sentencias debe forzosamente estar contenido en un bloque BEGIN / END, y no lo estás haciendo.

Código MySQL:

CREATE TRIGGER `actualizarObra2` 
AFTER UPDATE ON `detalle_obra` 
FOR EACH ROW 
BEGIN
    IF NEW.estado = 1 THEN 
        UPDATE obra 
            SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos), 
            `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos), 
            `recepcion_t`= obra.recepcion_t + (NEW.recepcion - OLD.recepcion) , 
            `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion), 
            `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision) 
        WHERE obra.codigo = OLD.cod_obra 
    ELSE
        UPDATE obra 
            SET `egresos_t`= obra.egresos_t - OLD.egresos, 
            `ingresos_t` = obra.ingresos_t - OLD.ingresos, 
            `recepcion_t`= obra.recepcion_t - OLD.recepcion, 
            `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion, 
            `combustible_emision_t` = obra.combustible_emision_t - OLD.combustible_emision 
        WHERE obra.codigo = OLD.cod_obra 
    END IF;
END
 

No puedo garantizarte que eso funcione, proque yo le veo algunos problemas de lógica...

[Begotten]

aun no logra funcionar, tengo este error ahora: MySQL ha dicho: #1064 - Erreur de syntaxe près de 'ELSE IF NEW.estado = 0 THEN UPDATE obra ' à la ligne 9

[gnzsoloyo]

Yo no puse ningún "ELSE IF NEW.estado = 0 THEN ". Solo un ELSE....
¿Acaso vas a ingresar más valores que 0 ó 1?

[Begotten]

Modifique un poco el codigo para ir probando pero el error es en esa parte, ademas la variable estado solo recibe cero y uno ya que es para el borrado logico (como mensionaste anteriormente). En cambio si se desea editar realmente, esa variable se mantiene tal cual, las variables que se ingresan son egresos, ingresos, recepcion, combustible recepcion y combustible emision.

PD: No creo que la logica sea el problema ya que el otro trigger que realice funciona bien.

[gnzsoloyo]

Para simplificar tu problema, necesitas un campo que sea fecha_baja, nulable. Y sólo enviar la fecha del día cuando quieras darlo de baja.
Es lo mas usual, y lo mas simple de validar.

[Begotten]

pero aun asi tendria que hacer una comparacion dentro del trigger o me equivoco? creo que el problema no esta en eso.

[gnzsoloyo]

Iniciado por Begotten

pero aun asi tendria que hacer una comparacion dentro del trigger o me equivoco? creo que el problema no esta en eso.

Completamente errado. La comparación se vuelve muchísimo mas simple y eficiente.

Código MySQL:

delimiter $$
CREATE TRIGGER `actualizarObra2` 
AFTER UPDATE ON `detalle_obra` 
FOR EACH ROW 
BEGIN
    IF NEW.fecha_baja IS NULL THEN 
        UPDATE obra 
            SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos), 
            `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos), 
            `recepcion_t`= obra.recepcion_t + (NEW.recepcion - OLD.recepcion) , 
            `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion), 
            `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision) 
        WHERE obra.codigo = OLD.cod_obra;
    ELSE
        UPDATE obra 
            SET `egresos_t` = obra.egresos_t - OLD.egresos, 
            `ingresos_t`    = obra.ingresos_t - OLD.ingresos, 
            `recepcion_t`   = obra.recepcion_t - OLD.recepcion, 
            `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion, 
            `combustible_emision_t`  = obra.combustible_emision_t - OLD.combustible_emision ,
            `fecha_baja` = NEW.fecha_baja
        WHERE obra.codigo = OLD.cod_obra ;
    END IF;
END $$
delimiter ;
 

Además, la existencia de un campo de fecha de baja permite algo que tu no estás considerando: trazabilidad.

Por cierto, había un error severo: faltaban los cierres de las sentencias UPDATE...

[Begotten]

Al fin funciono

Código MySQL:

CREATE TRIGGER `actualizarObra2` AFTER UPDATE ON `detalle_obra`
 FOR EACH ROW BEGIN
    IF NEW.estado = 1 THEN
        UPDATE obra
            SET `egresos_t`= obra.egresos_t + (NEW.egresos - OLD.egresos),
            `ingresos_t` = obra.ingresos_t + (NEW.ingresos - OLD.ingresos),
            `recepcion_t`= obra.recepcion_t + (NEW.recepcion - OLD.recepcion) ,
            `combustible_recepcion_t`= obra.combustible_recepcion_t + (NEW.combustible_recepcion - OLD.combustible_recepcion),
            `combustible_emision_t` = obra.combustible_emision_t + (NEW.combustible_emision - OLD.combustible_emision)
        WHERE obra.codigo = NEW.cod_obra;
    ELSE
        UPDATE obra
            SET `egresos_t`= obra.egresos_t - OLD.egresos,
            `ingresos_t` = obra.ingresos_t - OLD.ingresos,
            `recepcion_t`= obra.recepcion_t - OLD.recepcion,
            `combustible_recepcion_t`= obra.combustible_recepcion_t - OLD.combustible_recepcion,
            `combustible_emision_t` = obra.combustible_emision_t - OLD.combustible_emision
        WHERE obra.codigo = OLD.cod_obra;
    END IF;
END
 

Gracias por la ayuda.

[gnzsoloyo]