Ayuda para crear un procedimiento

[morillortega]

Hola.

Ante todo muchas gracias.

Estoy intentando crear un procedimiento en MySQL 5.0 , pero me devuelve este error:

"Variable or condition declaration after cursor or handler declaration"

Este es el código del procedimiento. A ver si alguien me puede decir algo.



************************************************** ***************
CREATE PROCEDURE `C_MUNICIPIOS`()
NOT DETERMINISTIC
SQL SECURITY DEFINER
COMMENT ''
BEGIN
DECLARE C1 CURSOR FOR SELECT C_PROVINCIA, C_MUNICIPIO FROM MUNICIPIOS;
DECLARE PROVINCIA VARCHAR(5);
DECLARE MUNICIPIO VARCHAR(5);
DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;

OPEN C1;

REPEAT
FETCH C1 INTO PROVINCIA, MUNICIPIO;

IF NOT DONE THEN
UPDATE `municipios`
SET C_MUNICIPIO=PROVINCIA & MUNICIPIO
WHERE C_PROVINCIA=PROVINCIA AND C_MUNICIPIO=MUNICIPIO;
END IF
UNTIL DONE END REPEAT;
END;

************************************************** ***************

Bueno, espero que alguien me diga porque ocurre esto.
Muchas gracias a todos de antemano.

[gnzsoloyo]

Dos errores de sintaxis y uno de proceso:
1. No declaraste la variable DONE. Debe ser declarada e inicializada a cero.
2. El END IF lleva cierre de sentencia porque es una estructura.

El error de proceso se verá cuando lo corras: Si estás haciendo un FETCH de una tabla implica que la estás leyendo, por lo tanto no puedes escribirle nada. Tiene en ese momento un bloqueo de escritura a nivel de tabla, entre otras cosas porque MySQL no puede saber si lo que vas a modificar no va a afectar el filtro que estás usando para seleccionar.
Me explico: Si el WHERE busca el valor X en un campo, obtiene un conjunto de registros, pero si la misma acción lo modifica a ese campo ¿cómo puede estar leyéndolo si ya no cumple con la condición de filtro? ¿En qué estado se encuentra, lo lee o no? Es un problema de consistencia. Por eso no se permiten escrituras de una tabla en proceso de lectura.
Tienes que crear una tabla intermedia y luego usar esa tabla para actualizar la MUNICIPIOS.
El código quedaría:

Código sql:

DELIMITER $$
 
DROP PROCEDURE IF EXISTS `C_MUNICIPIOS` $$
CREATE PROCEDURE `C_MUNICIPIOS`()
NOT DETERMINISTIC
SQL SECURITY DEFINER
COMMENT ''
BEGIN
  DECLARE PROVINCIA VARCHAR(5);
  DECLARE MUNICIPIO VARCHAR(5);
  DECLARE DONE INT DEFAULT 0;
  DECLARE C1 CURSOR FOR SELECT C_PROVINCIA, C_MUNICIPIO FROM MUNICIPIOS;
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
  OPEN C1;
  REPEAT
    FETCH C1 INTO PROVINCIA, MUNICIPIO;
    IF NOT DONE THEN
      UPDATE `municipios`
      SET C_MUNICIPIO=PROVINCIA & MUNICIPIO
      WHERE C_PROVINCIA=PROVINCIA AND C_MUNICIPIO=MUNICIPIO;
    END IF;
  UNTIL DONE END REPEAT;
END $$
 
DELIMITER ;
 

[morillortega]

Muchas gracias gnzsoloyo por contestar.

Con todo esto se demuestra que soy un pringao jejejeje, y que llevo mucho tiempo sin tocar nada de sql porque lo del IF y el DONE por ejemplo se me habia pasado, ahora, lo de actualizar una tabla sobre la misma que estoy haciendo una selección, no lo sabia.

Bueno, has comentado que deberia de hacer una tabla intermedia, pero observo que en tu código eso no lo haces tampoco, asi que: ¿cómo debería de crear dicha tabla?

Una vez más, MIL GRACIAS.

[gnzsoloyo]

No me quería alejar de la explicación, por lo que simplemente modifiqué lo que ya habías hecho.
En realidad, para mantener el ejemplo dentro de lo ya escrito, la cosa sería mas o menos así:

Código sql:

DELIMITER $$
  
DROP PROCEDURE IF EXISTS `C_MUNICIPIOS` $$
CREATE PROCEDURE `C_MUNICIPIOS`()
NOT DETERMINISTIC
SQL SECURITY DEFINER
COMMENT ''
BEGIN
  DECLARE PROVINCIA VARCHAR(5);
  DECLARE MUNICIPIO VARCHAR(5);
  DECLARE DONE INT DEFAULT 0;
  DECLARE C1 CURSOR FOR SELECT C_PROVINCIA, C_MUNICIPIO FROM MUNICIPIOS_TEMP;
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
 
  /*Borra la tabla temporal si existe*/
  DROP TABLE IF EXISTS MUNICIPIOS_TEMP;
 
  /* Crea la tabla nuevamente con el contenido necesario */
  CREATE TABLE MUNICIPIOS_TEMP AS 
  SELECT C_PROVINCIA, C_MUNICIPIO FROM MUNICIPIOS;
  OPEN C1;
  REPEAT
    FETCH C1 INTO PROVINCIA, MUNICIPIO;
    IF NOT DONE THEN
    UPDATE `municipios`
      SET C_MUNICIPIO=CONCAT(PROVINCIA,MUNICIPIO)
      WHERE C_PROVINCIA=PROVINCIA AND C_MUNICIPIO=MUNICIPIO;
    END IF;
  UNTIL DONE END REPEAT;
  CLOSE C1;
  /*Borra la tabla temporal*/
  DROP TABLE IF EXISTS MUNICIPIOS_TEMP;
  
END $$
  
DELIMITER ;
 

Eso debería ser suficiente. Como el cursor pasa a ser sinónimo de la sentencia, y comienza a operar recién al ser abierto, ya no está leyendo la tabla origen, sino la recién creada.

Notas:
1. No te olvides de cerrar el cursor para poder borrar la tabla. Los cursores no deben permanecer abiertos si no los necesitas más.

2. La sentencia DELIMITER sirve para cambiar momentáneamente el fin del sentencia para poder cargar el SP por conosola, o bien en una interfaz del tipo MySQL Query Browser.

[morillortega]

Buenas.

Realicé el procedimiento como me sugirió "gnzsoloyo" pero me daba el mismo error: "Variable or condition declaration after cursor or handler declaration", y no consigo saber porque me da este error. Asi que me puse a pensar para poder hacer la misma operación pero de forma más sencialla sin tener que crear una tabla temporal intermedia, pero me continua devolviendo el mismo error.

el codigo que tengo actualmente es:


CREATE PROCEDURE `new_proc`()
NOT DETERMINISTIC
SQL SECURITY DEFINER
COMMENT ''
BEGIN
DECLARE DONE INT DEFAULT O;
DECLARE C1 CURSOR FOR SELECT C_PROVINCIA, C_MUNICIPIO, C_POSTAL FROM MUNICIPIOS;
DECLARE PROVINCIA VARCHAR(5);
DECLARE MUNICIPIO VARCHAR(5);
DECLARE CP VARCHAR(5);

DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;


OPEN C1;

REPEAT
FETCH C1 INTO PROVINCIA, MUNICIPIO, CP ;

IF NOT DONE THEN

UPDATE `municipios`
SET C_MUNICIPIO=CONCAT(PROV,MUNI)
WHERE C_PROVINCIA=PROV AND C_POSTAL=CP
END IF
UNTIL DONE END REPEAT;

END ;


¿Alguien sabe por qué da el error: "Variable or condition declaration after cursor or handler declaration"?


Un saludo.

[gnzsoloyo]

Porque los cursores se declaran después de todas las variables y los handlers después de todas las declaraciones. No se pueden crear variable luego de declarar un HANDLER. Nunca.

Código sql:

CREATE PROCEDURE `new_proc`()
NOT DETERMINISTIC
SQL SECURITY DEFINER
COMMENT ''
BEGIN
   DECLARE DONE INT DEFAULT O;
   DECLARE PROVINCIA VARCHAR(5);
   DECLARE MUNICIPIO VARCHAR(5);
   DECLARE CP VARCHAR(5);
   DECLARE C1 CURSOR FOR SELECT C_PROVINCIA, C_MUNICIPIO, C_POSTAL FROM MUNICIPIOS;
   DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
 
 
   OPEN C1;
 
   REPEAT
   FETCH C1 INTO PROVINCIA, MUNICIPIO, CP ;
      IF NOT DONE THEN
         UPDATE `municipios`
         SET C_MUNICIPIO=CONCAT(PROV,MUNI)
         WHERE C_PROVINCIA=PROV AND C_POSTAL=CP;
      END IF;
   UNTIL DONE END REPEAT;
 
END ;
 

[morillortega]

Muchas gracias gnzsoloyo, eso era lo que me pasaba, no sabia lo del orden de las declaraciones.

Ya se puede daro por cerrado este hilo.

De nuevo mil gracia gnzsoloyo y un saludo a todos.