Trigger sobre día de la semana

[ruben_077]

Buenas, llevo dándole vueltas a esto un montón de tiempo y no sé por qué no funciona el siguiente trigger en Oracle SQL Developer:

Código Sql:

CREATE OR REPLACE TRIGGER TR_Dias_Tutorias
AFTER INSERT OR UPDATE OF Fecha ON Tutorias
FOR EACH ROW
DECLARE
    v_Fecha DATE := :NEW.Fecha;
    v_Dia VARCHAR2(20);
BEGIN
    SELECT TO_CHAR(v_Fecha, 'DAY', 'NLS_DATE_LANGUAGE=SPANISH') INTO v_Dia FROM Dual;
    IF v_Fecha LIKE 'SÁBADO' OR v_Fecha LIKE 'DOMINGO' THEN
        RAISE_APPLICATION_ERROR(-20020, 'Los días de la semana en dónde se puede realizar una tutoría deben estar entre lunes y viernes');
    END IF;
END;
 

Lo que quiero concretamente es que salte el mensaje de error si la fecha que se inserta o actualiza es sábado o domingo.

Curiosamente si hago la consulta a parte sí que me devuelve el día de la semana pero a la hora de ejecutar el trigger e intentar insertar un dato o actualizarlo, no sale nunca el error y se inserta o actualiza correctamente.

No sé qué puede estar fallando, he probado varias alternativas y ninguna funciona T_T. Gracias por adelantado!

[gnzsoloyo]

En primer lugar, porque el IF no lo estás haciendo sobre el la variable destino del INTO, sino sobre la variable de entrada, con la fecha...

Código SQL:

IF v_Fecha LIKE 'SÁBADO' OR v_Fecha LIKE 'DOMINGO' THEN
 

Y asi tomas el valor que deseas:

Código SQL:

SELECT TO_CHAR(v_Fecha, 'DAY', 'NLS_DATE_LANGUAGE=SPANISH') INTO v_Dia FROM Dual;
 

No es la misma variable, como ves.


Por otro lado es totalmente innecesario hacer comparaciones con cadenas de texto (las cuales podrían causarte problemas con los acentos), cuando puedes usar valores numéricos:

Código SQL:

CREATE OR REPLACE TRIGGER TR_Dias_Tutorias
AFTER INSERT OR UPDATE OF Fecha ON Tutorias
FOR EACH ROW
DECLARE
    v_Fecha DATE := :NEW.Fecha;
    v_Dia VARCHAR2(20);
BEGIN
    SELECT TO_CHAR(v_Fecha, 'D') INTO v_Dia FROM Dual;
    IF v_Fecha IN (1, 7) THEN
        RAISE_APPLICATION_ERROR(-20020, 'Los días de la semana en dónde se puede realizar una tutoría deben estar entre lunes y viernes');
    END IF;
END;
 

Adicionalmente, LIKE se usa para cadenas de texto en consulta por patrones y no por igualdades. No tiene la misma logica interna en Oracle, no se usan para las mismas cosas, y la performance puede ser pésima siendo usada de modo inadecuado.

[ruben_077]

Vaya fail de despiste lo de la variable x.x. Por otro lado, esto:

Código SQL:

IF v_Fecha IN (6, 7) THEN
 

No crees que es peor que mi otra opción porque dependiendo del territorio en el que estés va a ser [7,1] en vez de [6,7]? Entonces deja de ser un trigger válido universalmente.

[gnzsoloyo]

El valor devuelto por TO_CHAR(fecha, 'D') aplica al sitio donde se encuentre el servidor de la base de datos, y es independiente del formato de fecha local, porque devuelve un número en el rango 1 a 7 dependiendo del día de la semana en cuestión (Lunes = 1, Martes = 2, etc).

Código SQL:

SELECT TO_CHAR (TO_DATE ('30/01/2016', 'dd/mm/rrrr',
     'NLS_DATE_LANGUAGE = Spanish'), 'D') "Formato SP",
       TO_CHAR (TO_DATE ('01/30/2016', 'mm/dd/rrrr',
     'NLS_DATE_LANGUAGE = American'), 'D') "Formato EN",
       TO_CHAR (TO_DATE ('01/30/2016', 'mm/dd/rrrr',
     'NLS_DATE_LANGUAGE = German'), 'D') "Formato GER"
  FROM DUAL;
 

Cualquier duda, consulta el manual y haz pruebas en diferentes servidores de datos.

http://docs.oracle.com/cd/B19306_01/...nctions180.htm


Y... NO. Trabajar con resultados en cadenas de texto no es mejor que hacerlo con valores numéricos.

Haz la prueba y seguimos.