[SOLUCIONADO] Fallo al crear procedimiento

[M_Bosch]

Hola buenas, estoy realizando una tarea en la que me piden crear una rutina que sume todos los tantos marcados por los jugadores de cada clase, muestre el resultado con la suma y el nombre del grupo ordenado de mayor a menor y luego tengo que sumar 2 puntos al primer grupo y 1 punto al segundo. El código aun no esta terminado por lo que aún faltan (o sobran) cosas, pero los selects los he probado individualmente y funcionan correctamente. El problema que tengo es al crear el procedimiento, si lo dejo tal y como esta en el código me dice que falta un END en el primer select, y si lo comento, en el END final me dice que falta una sentencia IF. Alguien sabe porque pasa esto? Y ya que aún no lo he podido probar, pensáis que el cursor y el IF están bien?

Código SQL:

DELIMITER $
 
CREATE PROCEDURE ejercicio1 ()
BEGIN
 
    DECLARE SUM INT;
    DECLARE cla CHAR;
    DECLARE punt INT;
    DECLARE var1 INT DEFAULT 2;
    
    SELECT SUM(tantos_marcados) AS tantos, clase
    FROM jugadores
    GROUP BY clase
    ORDER BY tantos DESC;
    
    DECLARE cursor1 CURSOR FOR
    SELECT SUM(tantos_marcados) AS tantos, j.clase, c.puntuacion
    FROM jugadores AS j INNER JOIN clases AS c ON j.clase = c.codigo
    GROUP BY j.clase
    ORDER BY tantos DESC
    LIMIT 2;
    
   /* DECLARE CONTINUE HANDLER FOR NOT FOUND SET @var1 = 0; */
    
    OPEN cursor1;
    
    IF var1 = 2 THEN
    SET var1 = 1;
    FETCH cursor1 INTO SUM, cla, punt;
    SET punt = punt + 2;
    UPDATE clases SET puntuacion = punt WHERE codigo = cla;
    ELSE IF var1 = 1 THEN 
    SET var1 = 0;
    FETCH cursor1 INTO SUM, cla, punt;
    SET punt = punt + 1;
    UPDATE clases SET puntuacion = punt WHERE codigo = cla;
    END IF;
    
    CLOSE cursor1;
   
END$
 
DELIMITER ;
 

[gnzsoloyo]

Hay cuestiones de orden, por ejemplo, los cursores van despues de los DECLARE de las variables y antes del HANDLER.
El HANDLER siempre va después de los cursores, y no se declara ninguna variable despues de él.
El END, tenía un error. No es lo mismo poner ELSE IF, que ELSEIF
El primero determina un IF anidado, el otro es un ELSE condicional. No es lo mismo.

Código MySQL:

DELIMITER $$
 
CREATE PROCEDURE ejercicio1 ()
BEGIN
 
    DECLARE SUM INT;
    DECLARE cla CHAR;
    DECLARE punt INT;
    DECLARE var1 INT DEFAULT 2;
    
    DECLARE cursor1 CURSOR FOR
    SELECT SUM(tantos_marcados) AS tantos, j.clase, c.puntuacion
    FROM jugadores AS j INNER JOIN clases AS c ON j.clase = c.codigo
    GROUP BY j.clase
    ORDER BY tantos DESC
    LIMIT 2;
    /*  Un HANDLER solo tiene sentido si vas a hacer una iteración, de lo 
         contrario la condición no tiene uso.
 
         Ademas, "var" y "@var" no son la misma variable...
   */    
   DECLARE CONTINUE HANDLER FOR NOT FOUND SET @var1 = 0;
 
    /* Esto esta de mas, porque no puede haber un SELECT sin INTO 
       antes de un grupo de sentencias con condicionales de flujo. 
 
    SELECT SUM(tantos_marcados) AS tantos, clase
    FROM jugadores
    GROUP BY clase
    ORDER BY tantos DESC;
 
    */
    OPEN cursor1;
    
    IF var1 = 2 THEN
        SET var1 = 1;
        FETCH cursor1 INTO SUM, cla, punt;
        SET punt = punt + 2;
        UPDATE clases SET puntuacion = punt WHERE codigo = cla;
    ELSEIF var1 = 1 THEN 
        SET var1 = 0;
        FETCH cursor1 INTO SUM, cla, punt;
        SET punt = punt + 1;
        UPDATE clases SET puntuacion = punt WHERE codigo = cla;
    END IF;
    
    CLOSE cursor1;
   
END$$
 
DELIMITER ;
 

[M_Bosch]

Muchísimas gracias por tu respuesta tan rápida gnzsoloyo, me has aclarado bastantes dudas, pero si dices que el primer select sobra, como debería hacerlo para que se muestre su resultado?

[gnzsoloyo]

El SELECT debe ser la última sentencia de un SP, si quieres obtener ese resultado desde la aplicación.
En el interior generan errores de salida, y en ciertos lenguajes de programación no dan buenos resultados, ya que las queries no se espera que devuelvan N tablas resultado.

[M_Bosch]

Gracias por las respuestas, ya he logrado solucionarlo.