2

Tengo este codigo, pero no me funciona. Lo que quiero es meter el valor del case en una variable y esa variable tendrá un valor u otro dependiendo del valor del parámetro de entrada

DELIMITER //
CREATE FUNCTION fn_diabreviado (IN diasemana varchar(15)) RETURNS CHAR(1) DETERMINISTIC
BEGIN

DECLARE diaabreviado char(1);

SELECT CASE 
WHEN diasemana='Monday' THEN SELECT 'M' INTO diaabreviado
WHEN diasemana='Tuesday' THEN SELECT 'T' INTO diaabreviado
WHEN diasemana='Wednesday' THEN SELECT 'W' INTO diaabreviado
WHEN diasemana='Thursday' THEN SELECT 'H'  INTO diaabreviado
WHEN diasemana='Friday' THEN SELECT 'F'  INTO diaabreviado
WHEN diasemana='Saturday' THEN SELECT 'A' INTO diaabreviado
WHEN diasemana='Sunday' THEN SELECT 'S' INTO diaabreviado
END;

RETURN diaabreviado;

END //
3
  • has probado con if condiicon then .... elseif condicion then ... ect Commented el 29 abr. 2022 a las 9:19
  • No, pero más que nada porque me habían dicho que un case es más óptimo que usar if elseif
    – Adrián
    Commented el 29 abr. 2022 a las 10:15
  • Si es mas optimo pero menos lógico, luego cuesta comprenderlo Commented el 29 abr. 2022 a las 10:36

2 respuestas 2

3

Varias cosas:

  1. No tienes que poner en cada CASE la instrucción THEN SELECT 'laLetra' INTO diaabreviado indica cada CASE ... WHEN sin más, poniendo el INTO sólo al final.
  2. Si estás declarando una FUNCTION no tienes que indicar el tipo de parámetro, si es IN en este caso. En las funciones los parámetros siempre son reconocidos como del tipo IN, a diferencia de las PROCEDURE, que son las que aceptan diferentes tipos de parámetro. El MR es claro en esto:

Note

Specifying a parameter as IN, OUT, or INOUT is valid only for a PROCEDURE. For a FUNCTION, parameters are always regarded as IN parameters.

De este modo debería funcionar:

DROP FUNCTION IF EXISTS fn_diabreviado;
DELIMITER $$

CREATE FUNCTION  fn_diabreviado (diasemana varchar(15)) RETURNS CHAR(1) 
DETERMINISTIC
BEGIN
    DECLARE diaabreviado char(1);
    SELECT CASE 
        WHEN diasemana='Monday'    THEN 'M' 
        WHEN diasemana='Tuesday'   THEN 'T' 
        WHEN diasemana='Wednesday' THEN 'W' 
        WHEN diasemana='Thursday'  THEN 'H'  
        WHEN diasemana='Friday'    THEN 'F'  
        WHEN diasemana='Saturday'  THEN 'A' 
        WHEN diasemana='Sunday'    THEN 'S' 
    END 
    INTO diaabreviado;

    RETURN diaabreviado;
END$$
DELIMITER ;

Control de nulos

Si quieres evitar los nulos desde la función, puedes poner esto al final de los CASE:

ELSE ''

Algo así:

SELECT CASE 
    WHEN diasemana='Monday'    THEN 'M' 
    WHEN diasemana='Tuesday'   THEN 'T' 
    WHEN diasemana='Wednesday' THEN 'W' 
    WHEN diasemana='Thursday'  THEN 'H'  
    WHEN diasemana='Friday'    THEN 'F'  
    WHEN diasemana='Saturday'  THEN 'A' 
    WHEN diasemana='Sunday'    THEN 'S' 
    ELSE ''
END 
INTO diaabreviado;

Sin el ELSE, cuando pases un día que no esté en los CASE te retornará null y con el ELSE te retornará una cadena vacía. Aplica lo que mejor te convenga según el contexto.

Fiddle

Te dejo un fiddle completo como prueba:

Schema (MySQL v5.7)

DROP FUNCTION IF EXISTS fn_diabreviado;
DELIMITER $$

CREATE FUNCTION  fn_diabreviado (diasemana varchar(15)) RETURNS CHAR(1) 
DETERMINISTIC
BEGIN
    DECLARE diaabreviado char(1);
    SELECT CASE 
        WHEN diasemana='Monday'    THEN 'M' 
        WHEN diasemana='Tuesday'   THEN 'T' 
        WHEN diasemana='Wednesday' THEN 'W' 
        WHEN diasemana='Thursday'  THEN 'H'  
        WHEN diasemana='Friday'    THEN 'F'  
        WHEN diasemana='Saturday'  THEN 'A' 
        WHEN diasemana='Sunday'    THEN 'S' 
    END 
    INTO diaabreviado;

    RETURN diaabreviado;
END$$
DELIMITER ;

Query #1

SELECT fn_diabreviado('Monday');
fn_diabreviado('Monday')
M

Query #2

SELECT fn_diabreviado('Tuesday');
fn_diabreviado('Tuesday')
T

View on DB Fiddle

5
  • muy buena respuesta +1 yo esta como tenia lo mismo pero me falta el INTO diaabreviado; Commented el 29 abr. 2022 a las 9:37
  • Gracias @JavierG.Raya. Saludos.
    – A. Cedano
    Commented el 29 abr. 2022 a las 9:47
  • En un rato añadire otra respuesta con if elseif @A.Cedano Commented el 29 abr. 2022 a las 9:51
  • A mi me funciona su codigo @Adrián Commented el 29 abr. 2022 a las 10:20
  • Si si totalmente. Fallo mio
    – Adrián
    Commented el 29 abr. 2022 a las 10:22
1

Yo en lugar de usar CASE lo haría es la estructura IF ELSEIF el cual te puede resultar mas fácil de entender y realizar :

1. Añado el DROP FUNCION IF EXISTS por si acaso

2. Guardo en la variable diaabreviado la letra con un SET el cal es mas común

3. Te faltaba poner un delimitador al final DELIMITER ;

DELIMITER $$
DROP FUNCTION IF EXISTS fn_diabreviado $$
CREATE FUNCTION fn_diabreviado (diasemana varchar(15)) 
RETURNS CHAR(1) 
BEGIN
    DECLARE diaabreviado CHAR(1) DEFAULT '';

    IF (diasemana='Monday') THEN
        SET diaabreviado = 'M';
    ELSEIF (diasemana='Tuesday') THEN
        SET diaabreviado = 'T';
    ELSEIF (diasemana='Wednesday') THEN
        SET diaabreviado = 'W';
    ELSEIF (diasemana='Thursday') THEN
        SET diaabreviado = 'H';
    ELSEIF (diasemana='Friday') THEN
        SET diaabreviado = 'F';
    ELSEIF (diasemana='Saturday') THEN
        SET diaabreviado = 'A';
    ELSEIF (diasemana='Sunday') THEN
        SET diaabreviado = 'S';
    END IF;

    RETURN diaabreviado;

END $$
DELIMITER ;

Por ejemplo si ejecutamos la función :

SELECT fn_diabreviado('Sunday');

nos devolverá S

Ejecutar Ejemplo

6
  • Gracias por la aportación :))
    – Adrián
    Commented el 29 abr. 2022 a las 10:21
  • de nada amigo :) Commented el 29 abr. 2022 a las 10:23
  • Javier, en realidad CASE ... WHEN es menos verboso, más claro y más sencillo, además es más conforme al estándar SQL.
    – A. Cedano
    Commented el 29 abr. 2022 a las 16:37
  • Vale muchas gracias @A.Cedano pero prefiero esta estructura porque nunca he usado un CASE en mysql Commented el 29 abr. 2022 a las 16:41
  • Entiendo, pero ten en cuenta que CASE ... WHEN es parte del estándar SQL y que IF puede no funcionar en algunos SGBD, mientras que CASE ... WHEN sí funcionará en todos. Esto es importante, sobre todo a la hora de migrar de un SGBD a otro.
    – A. Cedano
    Commented el 29 abr. 2022 a las 16:47

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.