[SOLUCIONADO] Error en función

[kies89]

Holaa :).

Me ha surgido un problema a la hora de programar una función. El problema creo que se debe a incluir un IF dentro de otro IF, pero no sé cómo se arregla, y puesto que no he encontrado ninguna solución después de googlearlo, lo pongo aquí:

function.sql

Código MySQL:

DELIMITER $$
CREATE FUNCTION animales(animal VARCHAR(15))
MODIFIES SQL DATA
BEGIN
  IF (SELECT COUNT(nombre) FROM animales WHERE nombre=animal) > 0  THEN -- Si el animal ya existe en la tabla,
    IF (SELECT sexo FROM animales WHERE nombre=animal) = "macho" THEN -- Comprobar si es macho o hembra
      RETURN 1; -- Si es macho, devolver 1
    ELSE
      RETURN 0; -- Si es hembra, devolver 0
    END IF;
  ELSE -- Si el animal no existe en la tabla,
    INSERT INTO animales(nombre, sexo) VALUES(animal, "macho"); -- Por defecto será siempre macho
    RETURN animal; -- Devolvemos el nombre del animal registrado
  END IF;
END $$
 

table.sql

Código MySQL:

CREATE TABLE animales
(
  nombre varchar(15),
  sexo varchar(6)
);
 

error_producido.txt

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 'MODIFIES SQL DATA
BEGIN
IF (SELECT COUNT(nombre) FROM animales WHERE nombre=an' at line 2

Gracias de antemano.

[gnzsoloyo]

No le estas poniendo en el header que valor devuelve ka funcion

Código MySQL:

...returns  int
 

Vuelve a leer el manual.

[kies89]

Perdón xD, se me olvidó esa parte en esta función.

Este es el nuevo código de la función:

Código MySQL:

DELIMITER $$
CREATE FUNCTION web.animales(animal VARCHAR(15) RETURNS TINYINT(1))
MODIFIES SQL DATA
BEGIN
  IF (SELECT COUNT(nombre) FROM animales WHERE nombre=animal) > 0  THEN -- Si el animal ya existe en la tabla,
    IF (SELECT sexo FROM animales WHERE nombre=animal) = "macho" THEN -- Comprobar si es macho o hembra
      RETURN 1; -- Si es macho, devolver 1
    ELSE
      RETURN 0; -- Si es hembra, devolver 0
    END IF;
  ELSE -- Si el animal no existe en la tabla,
    INSERT INTO animales(nombre, sexo) VALUES(animal, "macho"); -- Por defecto será siempre macho
    RETURN 2; -- Devolvemos 2
  END IF;
END $$
 

Y este es el error que sigue dando:

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 'RETURNS TINYINT(1))
MODIFIES SQL DATA
BEGIN
IF (SELECT COUNT(nombre) FROM anim' at line 1

[gnzsoloyo]

El header sigue mal escrito... LEE EL MANUAL DE REFERENCIA. Estas en la web.

[kies89]

Listo, este es el código:

Código MySQL:

DELIMITER $$
CREATE FUNCTION web.animales(animal VARCHAR(15)) RETURNS TINYINT(1)
MODIFIES SQL DATA
BEGIN
  IF (SELECT COUNT(nombre) FROM animales WHERE nombre=animal) > 0  THEN -- Si el animal ya existe en la tabla,
    IF (SELECT sexo FROM animales WHERE nombre=animal) = "macho" THEN -- Comprobar si es macho o hembra
      RETURN 1; -- Si es macho, devolver 1
    ELSE
      RETURN 0; -- Si es hembra, devolver 0
    END IF;
  ELSE -- Si el animal no existe en la tabla,
    INSERT INTO animales(nombre, sexo) VALUES(animal, "macho"); -- Por defecto será siempre macho
    RETURN 2; -- Devolvemos 2
  END IF;
END $$
 

Mensaje de error:

ERROR 1418 (HY000): This function has none of DETERMINISTIC, NO SQL, or READS SQL DATA in its declaration and binary logging is enabled (you *might* want to use the less safe log_bin_trust_function_creators variable)

Google:
Debes especificar qué hará la función, si DETERMINISTIC, si va a modificar datos de tablas...
También me dice: ejecuta SET GLOBAL log_bin_trust_function_creators = 1;

Entonces, si en la función pueden haber distintos casos (como ocurre en esta función), que o hacemos SELECT o hacemos un INSERT, y no sabemos previamente si modificaremos o solamente haremos SELECT, ¿qué hago?

[gnzsoloyo]

Código MySQL:

create function nombre() returns tipo
 

[kies89]

Iniciado por gnzsoloyo

Código MySQL:

create function nombre() returns tipo
 

¿? Eso es lo que tengo puesto.
Pero me da otro error que no tiene nada que ver. Lo acabo de poner.

Saludos.

[gnzsoloyo]

Estaba posteando desde el movil, por eso se colgó un poco y no refresca rápido.

Código MySQL:

DELIMITER $$
CREATE FUNCTION web.animales(animal VARCHAR(15)) RETURNS TINYINT(1)
DETERMINISTIC
BEGIN
    IF (SELECT COUNT(nombre) FROM animales WHERE nombre=animal) > 0  THEN -- Si el animal ya existe en la tabla,
       IF (SELECT sexo FROM animales WHERE nombre=animal) = "macho" THEN -- Comprobar si es macho o hembra
         RETURN 1; -- Si es macho, devolver 1
       ELSE
         RETURN 0; -- Si es hembra, devolver 0
       END IF;
    ELSE -- Si el animal no existe en la tabla,
        INSERT INTO animales(nombre, sexo) VALUES(animal, "macho"); -- Por defecto será siempre macho
        RETURN 2; -- Devolvemos 2
    END IF;
END $$
 

[kies89]

Ok, muchísimas gracias, ya funciona :).

Una última cosa,

DETERMINISTIC y NOT DETERMINISTIC indican si una rutina siempre produce el mismo resultado para entradas dadas. Por defecto es NOT DETERMINISTIC si no se da ninguna característica, así que debe especificar DETERMINISTIC explícitamente para declarar que la rutina es determinista.

¿Entonces si no sé qué va a pasar pongo DETERMINISTIC y si siempre va a pasar lo mismo pongo NOT DETERMINISTIC? :S.

Saludos.

[gnzsoloyo]

Lee este detalle del manual:

Un procedimiento o función se considera “determinista” si siempre produce el mismo resultado para los mismos parámetros de entrada, y “no determinista” en cualquier otro caso. Si no se da ni DETERMINISTIC ni NOT DETERMINISTIC por defecto es NOT DETERMINISTIC.

En tu caso, estás metiendo siempre un mismo tipo de parámetro, y siempre devuelve un entero, dentro de un mismo rango dado.
¿Se entiende ahora?
De todos modos, deberías haber usado un SP y no una SF.
Te hubieses ahorrado dolores de cabeza.

[kies89]

Ok, muchas gracias :D.

De todas formas, las SP las veo igual que las SF, aunque por ahí dicen que las SP son más rápidas...

[gnzsoloyo]

No es el tema de rapidez, sino de uso.
El sentido de una SF es obtener un dato, un valor único.
Un SP es para realizar tareas complejas, que impliquen uno o muchos impactos sobre una o muchas tablas, y/u obtener datos de retorno por parámetro o resultados en tabla.
No se trata de las mismas cosas.

[kies89]

Hola de nuevo! Ya tengo creada la SP, pero lo veo un tanto peligroso, con miles de clientes haciendo consultas simultaneamente y guardado el valor de la consulta en una variable global...

Código MySQL:

CALL animales('caballo', @number);
SELECT @number; -- esto lo veo peligroso, puesto que podría cambiar y darle otro valor a otro cliente.
 

¿Qué opinas?

[gnzsoloyo]

Las variables no son globales en el sentido que aplican los lenguajes de programación.
Las variables globales de MySQL lo son únicamente para cada sesión de conexión, pero no son compartidas entre sesiones distintas ni aunque uses el mismo usuario de MySQL.

Lo que yo no encuentro sentido siquiera es que para obtener un único dato uses un stored procedure. Deberías usar una stored function.

Usando una stored function ni siquiera necesitas la variable, en tanto no la precises para otra tarea adicional, dentro de la misma sesión y sólo a nivel de SQL.

[kies89]

¿Stored Function? Es que googleé y encontré: CREATE FUNCTION y CREATE PROCEDURE, no encuentro lo de las stored functions. Las Stored Functions me salen como "CREATE FUNCTION"... Bueno, hice lo que me dijiste en diciembre, y por eso la creé...

[gnzsoloyo]

Precisamente eso son las stored function. En el argot de informática uno habitualmente usa determinados términos técnicos en inglés, que debes aprender a reconocer y dominar. Es parte del lenguaje técnico.
Una stored function o SF, es una "función almacenada", y que para ser creada se usa habitualmente en todos los DBMS la sentencia de clase DDL que se invoca como "CREATE FUNCTION"...
De igual forma, un stored procedure o "SP" es un "procedimiento almacenado", que se crea con la sentencia DDL "CREATE PROCEDURE".
En cuanto a lo que yo te sugerí no era usar un SP, sino una SF.
Prueba de ello es que el codigo que te postee es este:

Código MySQL:

DELIMITER $$
CREATE FUNCTION web.animales(animal VARCHAR(15)) RETURNS TINYINT(1)
DETERMINISTIC
BEGIN
    IF (SELECT COUNT(nombre) FROM animales WHERE nombre=animal) > 0  THEN -- Si el animal ya existe en la tabla,
       IF (SELECT sexo FROM animales WHERE nombre=animal) = "macho" THEN -- Comprobar si es macho o hembra
         RETURN 1; -- Si es macho, devolver 1
       ELSE
         RETURN 0; -- Si es hembra, devolver 0
       END IF;
    ELSE -- Si el animal no existe en la tabla,
        INSERT INTO animales(nombre, sexo) VALUES(animal, "macho"); -- Por defecto será siempre macho
        RETURN 2; -- Devolvemos 2
    END IF;
END $$
 

Usando eso, tu llamada sería simplemente:

Código MySQL:

SELECT animales('caballo') animal_id;
 

Siendo "animal_id" el alias de salida de la consulta.

¿Está más claro así?

[kies89]

Ok, muchísimas gracias por todo :).

Tema solucionado.