Procedimiento con un resultado pasado como parámatreo

[bhyktorio6]

Hola,
A ver si podeis ayudarme con este ejercicio.
En la base de datos ‘liga’, crea un procedimiento que rellene los campos ‘pg’ y ‘pp’ de la tabla ‘equipo’ en base a los resultados. Para ello crea antes un procedimiento que
devuelva un ‘1’,’X’ o ‘2’, con un resultado que se pase
como parámetro, según sea victoria, empate o derrota.
He realizado la segunda parte del ejercicio,y no se si es que me he dejado alguna sentencia porque al hacer la llamada al procedimiento el valor de la variable result es null,cuando debería se 1,x o 2 según las condiciones.

Código MySQL:

delimiter $$
drop procedure if exists resultados$$
create procedure resultados(resultado char(7))
begin
 
declare tantos1 integer;
declare tantos2 integer;
declare result char(1);
 
if length(resultado)=7
then
set tantos1=substr(resultado,1,3);
set tantos2=substr(resultado,5,7);
elseif 
tantos1 >  tantos2  
then
set result='1';
elseif
tantos2 > tantos1 
then
set result='2';
elseif
tantos1=tantos2
then
set result='x';
else
select ('error');
end if;
select result as 'resultado';
end;$$
 
call resultados('088-087');$$
 

en cuánto a que se actualicen los partidos ganados y perdidos según el resultado sea 1,x, o 2 sería código dentro del mismo procedimiento?

Un saludo.

[gnzsoloyo]

En MySQL, todo parámetro declarado en el prototipo, donde no se indique si es IN, OUT o INOUT, por defecto es IN, lo que significa que no tu SP no puede devolver resutlados por parámetro...
Delcaralo INOUT.

Código MySQL:

CREATE PROCEDURE resultados(INOUT resultado char(7))
...
 

De esa forma podrás recuperar el parámetro.

Ahora bien, si el tema es que estás invocando este SP programáticamente, dependerá del lenguiaje el cómo se hace (OFF TOPIC en este foro), pero en cualquiera de los casos debes enviar a ejecutar el SP como sio fuera un SELECT común, porque lo que devolverá será una tabla, y no un valor único en un parámetro.

[bhyktorio6]

Hola,
Sigue sin devolver 1,x o 2 con INOUT al pasarle un resultado.
Y otra cosa,¿es correcto el código que señalo para que se rellenen los campos pg,pp según el resultado obtenido?.
Gracias.

[gnzsoloyo]

Bueno, tomé tu SP, lo copie y le di un formato un poco más claro, porque tal como estaba escrito, sin indentaciones, era imposible de entender a simple vista.
Te recomiendo codificar los SP de un modo estructurado, tal que te permita ver claramente lo que sucede.

La cosa es así:

Código MySQL:

delimiter $$
DROP PROCEDURE IF EXISTS resultados$$
CREATE PROCEDURE resultados(INOUT resultado CHAR(7))
BEGIN
 
    DECLARE tantos1 INTEGER;
    DECLARE tantos2 INTEGER;
    DECLARE result CHAR(1);
 
    IF LENGTH(resultado)=7 THEN
        SET tantos1=SUBSTR(resultado,1,3);
        SET tantos2=SUBSTR(resultado,5,7);
    ELSEIF tantos1 > tantos2 THEN
        SET result='1';
    ELSEIF tantos2 > tantos1 THEN
        SET result='2';
    ELSEIF tantos1=tantos2 THEN
            SET result='x';
    ELSE
        SELECT ('error') resultado;
    END IF;
    SELECT result resultado;
END$$
 
DELIMITER ;
 

Con eso a la vista, el problema es harto evidente: Nunca entrará en ningún ELSEIF ni ELSE, porque al momento de iniciar el IF, el valor de ambas variables es CERO.
Le estás asignando el valor dentro del IF inicial, y todos los ELSEIF operan después.
Un ELSEIF devuelve TRUE y ejecuta su código si y sólo si el IF inicial devuelve FALSE, que no sería el caso. Solo sucedería si el CALL fuese algo como por ejemplo:

Código MySQL:

CALL resultados('088087');
 

Es decir, debería tener menos o más de 7 caracteres para ser FALSE y recién alli entraría en los ELSEIF.
Ahora bien, si pasara eso, el valor de tantos1 y tantos2 sería NULL, porque están declaradas pero no incializadas. Y en todo caso podrían tener valor cero, de poseer una condición de default.
Pero como no estás evaluando el NULL, y NULL no funciona con operadores lógicos comunes en MySQL, la consecuencia sería que saliese por ELSE, devolviendo la cadena "error". Si el valor por default de las variabls fuese cero, entonces podría salir con valor "X".
Pero son las unicas alternativas.

Yo veo dos cosas principales:
1) No entiendo por qué quieres usar una única variable de entrada, cuando siendo una comparación lo óptimo sería hacerlo con dos numéricas.

Código MySQL:

delimiter $$
DROP PROCEDURE IF EXISTS resultados$$
CREATE PROCEDURE resultados(IN valor1 INT, IN valor2 INT, OUT resultado CHAR(10))
BEGIN
    IF valor1 > valor2  THEN
        SET resultado = '1';
    ELSEIF tantos2 < tantos1 THEN
        SET resultado = '2';
    ELSE 
            SET resultado = 'x';
    END IF;
    SELECT resultado;
END$$
 
delimiter ;
 
CALL resultados(88, 087, result);
 

2) Estás complicando innecesariamente el script y anidando incorrectamente las validaciones, además de validar en un mismo IF cosas que no están relacionadas.
Esto sería lo tuyo, limpio:

Código MySQL:

delimiter $$
DROP PROCEDURE IF EXISTS resultados$$
CREATE PROCEDURE resultados(IN resultado CHAR(7))
BEGIN
 
    DECLARE tantos1 INTEGER;
    DECLARE tantos2 INTEGER;
    DECLARE result CHAR(7);
    IF LENGTH(TRIM(resultado)) =7 THEN
        SET tantos1=SUBSTR(resultado,1,3);
        SET tantos2=SUBSTR(resultado,5,7);
        IF tantos1 > tantos2 THEN
            SET result='1';
        ELSEIF tantos1 < tantos2 THEN
            # No cambies la posicion de las variables, genera errores de comprension a primera vista
            SET result='2';
        ELSE 
            # Si tantos1 no es mayor ni menor a tantos2, sólo puede ser igual... 
            SET result='x';
        END IF;
    ELSE
        # No tiene sentido evaluarlo dentro de las condiciones de igualdad.
        # Es una condicion del parametro de entrada. No mezcles logicas incompatibles
        SELECT ('error') resultado;
    END IF;
    # No tiene sentido usar variables de salida, ni tampoco 
    # dos salidas, si el SP sólo devolvera un unico resultado.
    # alcanza con que luego en la aplciacion evalues la respuesta.
    SELECT result resultado;
END$$
 
DELIMITER ;
 

Una nota final: Los SP no se hacen para hacer procesos de este tipo, sino para procesos donde realizará consultas a la base de datos. En todo caso, si devolverá sólo una cadena de texto siempre, debería ser una stored function, y no un stored procedure.

Este ejercicio no tiene sentido en MySQL, es propio de programación en algún lenguaje y carece de utilidad en una base de datos. Si te lo dieron como ejercitación, está muy mal pensado.

Nota 2: Cambie los parametros a IN, porque si estás enviando una constante, entonces el resultado se pierde. Para que te funcione deberás invocar el SP en MYSQL, o bien deberás hacerlo por lenguaje de programación de la misma forma en que realizas una consulta a una tabla, ya que ese SP devolverá una tabla.

[bhyktorio6]

Muy buena explicación ,creo que ya lo he entendido.Perdón por la mala indexación,y si,es un ejercicio del profesor de mysql de la plataforma a distancia.Solo puse un parámetro en el SP porque quería pasarle el resultado en formato 'xxx-xxx'.
Saludos!