[SOLUCIONADO] problemas con triggers

[alfa18]

hola amigos,
tengo problemas con este trigger y no veo donde esta el error ¿me echais una mano?

el trigger en cuestion es este:

Código MySQL:

CREATE TRIGGER k2 BEFORE INSERT ON k FOR EACH ROW BEGIN IF District IS NULL OR District= '' THEN SET NEW.District='Unknown';end if;
 

.. y los errores que obtengo son estos:
ERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

ERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'end if' at line 1

[gnzsoloyo]

Lo mas probable es que estés intentado crearlo a mano, en algún front-end como phpmyadmin. Si es así, el problema radica en que el trigger en cuestión es de más de una linea, y para eso se deben cargar de un modo especial.

¿Qué estás usando para intentar compilarlo?

[alfa18]

estoy en el cliente de mysql, nada de pMA

lo que no entiendo es porque me da error en el 'end if', si lo quito no aparece ese error

[gnzsoloyo]

Ok.
Bueno, para cargar CUALQUIER rutina almacenada por consola, como stored procedures, stored functions, trigers o events, lo que debes hacer es cambiar momentaneamente el delimitador de sentencais de MySQL por otra cosa, a fin de que tome el conjunto de lineas como ua sola cosa.
Eso se hace simplemente con el comando DELIMITER>

Código MySQL:

DELIMITER $$
 
CREATE TRIGGER k2 BEFORE INSERT ON k 
FOR EACH ROW 
BEGIN 
    IF District IS NULL OR District= '' THEN 
        SET NEW.District='Unknown';
    END IF;
END$$ 
 
DELIMITER ;
 

En la primera linea se cambia el terminador ";" por "$$", algo que carece de significado en MySQL y solo es para que interprete que termino la sentencia y que ejecute todo.

Luego viene el codigo a cargar (dicho sea de paso, te olvidaste el END final para el BEGIN), cerrado por el "$$".

Finalmente restauras el terminador estandar de nuevo y listo.

[alfa18]

ok, entiendo el problema: cuando hay mas de un ';' en el trigger debemos cambiar el delimitador ¿es asi?

Pero ¿no deberia bastar con meter todos los ';' dentro de un bloque BEGIN-END? asi solo tendriamos/"veria" un solo ';'(el del END de cierre de bloque)

[gnzsoloyo]

Pero ¿no deberia bastar con meter todos los ';' dentro de un bloque BEGIN-END? asi solo tendriamos/"veria" un solo ';'(el del END de cierre de bloque)

Eso no existe en MySQL.

Tal vez exista en otro DBMS, pero no en MySQL y si vas a crear el TRIGGER en MySQL, debes cumplir las reglas de MySQL.
Así de simple.

[alfa18]

abstraer todo el cuerpo del trigger en en una sola instruccion(bloque BEGIN-END) y asi no tener problemas con el delimitador suena lo mas logico, no crees? no conozco otros DBMS............

una vez creado el trigger, sigo teniendo problemas(no ya con el trigger), intento que el trigger se dispare al insertar en la tabla k(la tabla k solo contiene el capo District, de tipo char(8)):

Código MySQL:

insert into k values('');
 

pero obtengo este "misterioso" error: ERROR 1054 (42S22): Unknown column 'District' in 'field list'

[gnzsoloyo]

Porque, como en TODOS los demás DBMS, los campos de un registro que se inserta o se actualiza en un trigger NO se puede invcar solamente por su nombre. Debes usar las peudovariables NEW y OLD, como referencia del registro que estás modificando (cada uno de os que el trigger procesa).
Me parece que necesitas leer de nuevo el manual de referencia, porque este tema está claramente explicado en él.

La cosa es así: Cada vez que insertas un registro, y en la tabla hay un trigger para ese evento, el registro queda en un estado de proceso y asociado a una pseudovariable denominada NEW. En el caso de los UPDATE y DELETE, el registro existente es el OLD.
Entonces, mientras estás dentro del trigger obligatoriamente debes referirte a él con ese nombre, de lo contrario se generará un error de ejecución porque MySQL intenta encontrar una columna o tabla con el nombre de campo que quieres, y no existe.

Vamos de vuelta entonces:

Código MySQL:

DELIMITER $$
DROP TRIGGER IF EXISTS  k $$
 
CREATE TRIGGER k2 BEFORE INSERT ON k 
FOR EACH ROW 
BEGIN 
    IF NEW.District IS NULL OR NEW.District= '' THEN 
        SET NEW.District='Unknown';
    END IF;
END$$
 
 DELIMITER ;
 

[alfa18]

Con la modificacion de ltrigger(NEW.xxx), ahora si:

>insert into k values('');
Query OK, 1 row affected (0.00 sec)

>select * from k;
+----------+
| District |
+----------+
| Unknown |
+----------+
1 row in set (0.00 sec)

mil gracias gonzalo:D

[gnzsoloyo]