Resultado null de una select en un procedimiento almacenado

cgonzalezc · #1 (**permalink**) 01/03/2011, 03:40

Buenos días.

Tengo un problema con una select que tengo creada en un procedimiento almacenado.

DECLARE PRUEBA INT;
SELECT ANYTRA FROM VENTAS_CLIENTE_REPRE_BACKUP INTO PRUEBA;
RETURN PRUEBA;

Esta select me retorna como valor NULL. La tabla contiene un registro con valores diferentes de NULL.

Pero si utilizo
SELECT COUNT(*) FROM VENTAS_CLIENTE_REPRE_BACKUP
El resultado que me devuelve es correcto, 1.

Si alguien me puede echar una mano, gracias.

Heimish2000 · #2 (**permalink**) 01/03/2011, 04:12

Puede que la tabla tenga un registro cuyo campo ANYTRA sea null, de esa manera la primera consulta devolveria null (porque coges el valor del campo) y la segunda devuelve uno (porque cuentas el número de registros de la tabla)

cgonzalezc · #3 (**permalink**) 01/03/2011, 04:51

Como comento arriba, la tabla sólo tiene un registro y este campo tiene el valor 2011.

Heimish2000 · #4 (**permalink**) 01/03/2011, 04:58

No veo donde lo comentas arriba, la verdad...

¿Que te devuelve si haces SELECT * FROM tabla? ¿Puedes postear aquí el resultado?

cgonzalezc · #5 (**permalink**) 01/03/2011, 05:30

Si hago lo que me comentas funciona correctamente.

Lo que no entiendo como con SELECT * funciona y con SELECT ANYTRA no.

Esto quiere decir que no podría hacer DISTINCTS si me hiciera falta ya que tengo que sacar todas la columnas.

gnzsoloyo · #6 (**permalink**) 01/03/2011, 05:50

Porque no se pueden hacer operaciones con NULL. NULL no es un valor sino un puntero a la nada, a la inexistencia o a un estado indeterminado (según el caso).
¿Puedes contar las indeterminaciones o la nada? No, y tampoco puedes a una variable real pasarle una indeterminación. Lo único que puedes hacer es definir qué hacer si es NULL o establecer que es un NULL.

Lee un poco acerca del tratamiento de los NULL en MySQL: 3.3.4.6. Trabajar con valores NULL

gnzsoloyo · #7 (**permalink**) 01/03/2011, 05:54

Por otro lado, estás diciendo que el asunto está en un procedimiento almacenado, pero RETURN no es una sentencia de procedimientos almacenados sino de funciones almacenadas.

cgonzalezc · #8 (**permalink**) 01/03/2011, 06:03

Perdón, he puesto mal el titulo. Sería una función si.

Pero mi duda está en que la columna no vale NULL. Tiene como valor 2011, pero sólo me lo coge bien con el Select *.

gnzsoloyo · #9 (**permalink**) 01/03/2011, 06:38

Magnífico, pero ¿cómo haces para recuperar el valor de esa función? ¿En qué contexto y con qué consulta la usas?
¿Puedes postear todo el código SQL completo?, es decir tanto el código completo del SF, como la consulta donde la usas

cgonzalezc · #10 (**permalink**) 01/03/2011, 06:49

Aqui al función.

DELIMITER $$

DROP FUNCTION IF EXISTS `CUR` $$
CREATE DEFINER=`root`@`%` FUNCTION `CUR`() RETURNS int(4)
BEGIN
DECLARE PRUEBA INT(4) DEFAULT 999;

DECLARE done INT DEFAULT 0;
DECLARE ANYTRA INT(4);
DECLARE W_ANYTRA INT(4);
DECLARE MESTRA int(2);
DECLARE NOMMES varchar(15);
DECLARE NCUENTA varchar(30);
DECLARE NOMCLIENTE varchar(255);
DECLARE CODREPRE varchar(20);
DECLARE NOMREPRE varchar(50);
DECLARE MES_ACTUAL_IMP float(15,2);
DECLARE MES_ACTUAL_CANT float(15,2);
DECLARE IMPBRU float(11,2);
DECLARE DTOFAM float(11,2);
DECLARE DTOART float(11,2);
DECLARE DTOCAB float(11,2);
DECLARE DTOOTR float(11,2);
DECLARE PRECIO float(11,2);
DECLARE PRECOS float(11,2);
DECLARE VAR_MES_ANT_IMP float(15,2);
DECLARE VAR_MES_ANT_IMP_PCT float(10,2);
DECLARE VAR_MES_ANT_CANT float(15,2);
DECLARE VAR_MES_ANT_CANT_PCT float(10,2);
DECLARE VAR_FIN_ANY_IMP float(15,2);
DECLARE VAR_FIN_ANY_IMP_PCT float(10,2);
DECLARE VAR_FIN_ANY_CANT float(15,2);
DECLARE VAR_FIN_ANY_CANT_PCT float(10,2);
DECLARE VAR_INTERANUAL_IMP float(15,2);
DECLARE VAR_INTERANUAL_IMP_PCT float(10,2);
DECLARE VAR_INTERANUAL_CANT float(15,2);
DECLARE VAR_INTERANUAL_CANT_PCT float(10,2);
DECLARE VAR_PRESUPUESTO_IMP float(15,2);
DECLARE VAR_PRESUPUESTO_IMP_PCT float(10,2);
DECLARE VAR_PRESUPUESTO_CANT float(15,2);
DECLARE VAR_PRESUPUESTO_CANT_PCT float(10,2);

DECLARE CUR_VENTAS_CLIENTE_REPRE CURSOR FOR
SELECT *
FROM VENTAS_CLIENTE_REPRE_BACKUP;
DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;

OPEN CUR_VENTAS_CLIENTE_REPRE;

REPEAT
FETCH CUR_VENTAS_CLIENTE_REPRE INTO ANYTRA,MESTRA,NOMMES,NCUENTA,NOMCLIENTE,CODREPRE,N OMREPRE,MES_ACTUAL_IMP,MES_ACTUAL_CANT,IMPBRU,DTOF AM,DTOART,DTOCAB,DTOOTR,PRECIO,PRECOS,VAR_MES_ANT_ IMP,vAR_MES_ANT_IMP_PCT,VAR_MES_ANT_CANT,VAR_MES_A NT_CANT_PCT,VAR_FIN_ANY_IMP,VAR_FIN_ANY_IMP_PCT,VA R_FIN_ANY_CANT,VAR_FIN_ANY_CANT_PCT,VAR_INTERANUAL _IMP,VAR_INTERANUAL_IMP_PCT,VAR_INTERANUAL_CANT,VA R_INTERANUAL_CANT_PCT,VAR_PRESUPUESTO_IMP,VAR_PRES UPUESTO_IMP_PCT,VAR_PRESUPUESTO_CANT,VAR_PRESUPUES TO_CANT_PCT;

IF NOT done THEN
/* Inserción de los registros de referencia agrupados por CLIENTE y REPRE */
INSERT INTO VENTAS_CLIENTE_REPRE_BACKUP VALUES (ANYTRA,MESTRA,NULL,NCUENTA,NOMCLIENTE,CODREPRE,NO MREPRE,MES_ACTUAL_IMP,MES_ACTUAL_CANT,IMPBRU,DTOFA M,DTOART,DTOCAB,DTOOTR,PRECIO,PRECOS,VAR_MES_ANT_I MP,vAR_MES_ANT_IMP_PCT,VAR_MES_ANT_CANT,VAR_MES_AN T_CANT_PCT,VAR_FIN_ANY_IMP,VAR_FIN_ANY_IMP_PCT,VAR _FIN_ANY_CANT,VAR_FIN_ANY_CANT_PCT,VAR_INTERANUAL_ IMP,VAR_INTERANUAL_IMP_PCT,VAR_INTERANUAL_CANT,VAR _INTERANUAL_CANT_PCT,VAR_PRESUPUESTO_IMP,VAR_PRESU PUESTO_IMP_PCT,VAR_PRESUPUESTO_CANT,VAR_PRESUPUEST O_CANT_PCT);
UPDATE VENTAS_CLIENTE_REPRE_BACKUP SET NOMMES=InformarNombreMes(MESTRA);
END IF;
UNTIL done END REPEAT;

CLOSE CUR_VENTAS_CLIENTE_REPRE;

RETURN ANYTRA;

END $$

DELIMITER ;

Y la llamada.

select cur();

El resultado me lo devuelve como si fuera una columna de una tabla.

El problema que veo con esta solución a parte de no poder hacer DISTINCT es el siguiente, utilizando el SELECT *, si la tabla sobre la que lo hago me cambia (+ o -columnas), entonces tendría que cambiar todas las Funciones donde estubiera esta tabla, para añadirle las variables de destino en le INTO.

De ahi viene mi pregunta de saber porque razón con el SELECT * funciona pero con el SELECT ANYTRA no funciona.

gnzsoloyo · #11 (**permalink**) 01/03/2011, 07:08

Bueno, ahora me queda amás claro (aunque no termino de entender todo este procedimiento para algo que puedes obtener con dos o tres consultas directamente).
Paso a explicar: ANYTRA toma el valor del campo correspondiente en la tabla origen en cada FETCH, pero el último FETCH, el que pone done en 1, trata de acceder a un registro que no existe y es precisamente eso lo que controla el HANDLER. Como el puntero trata de ir más allá de los registros de la tabla, y estos no existen, el valor devuelto por ese último FETCH es NULL...
Este es un tema que a veces no se conoce si no lo has leido en profundidad, no te preocupes. El punto esencial es que como FETCH devuelve NULL en ese momento, todos los campos del INTO se vuelve NULL... simplemente.
De hecho, si en lugar de RETURN ANYTRA se pudiese hacer un SELECT a todos los campos, comprobarías que todos los valores están en NULL.
El error consiste en pretender obtener el último valor real que tuvo ANYTRA en esta forma. No lo puedes hacer así. Yo en tu caso usaría otra variable INT, la cual sólo llenaría con el valor de ANYTRA dentro del IF y la usaría para el RETURN.
De todos modos, a mi entender, esto se puede hacer sin un CURSOR, y con menos complicaciones.

cgonzalezc · #12 (**permalink**) 01/03/2011, 09:43

Gracias por adelantado, pero creo que quizas no me he acabado de explicar del todo.

Mi idea original es agrupar todos los registros de la tabla original por Año y Mes y sumarizar unas cantidades que tengo en la tabla.
Y luego hacer unos Updates con cantidades de años anteriores.

El cursor que he enviado es una primera prueba de lo que queria hacer.
Como no me funcionaba pues he ido simplificando la query.
Lo que supongo que al enviar toda la copia lo he liado más.

Creo que todavía no he sido capaz de explicar el problema que tengo.
Centrandonos en el caso del cursor, si la definición del mismo es como "Select *" devuelve bien los datos, pero si es como "Select ANYTRA" el resultado que me devuelve es NULL.
Si ejecuto el cursor lo que me hace es insertarme los n registros de la tabla origen en la tabla destino pero con el valor a NULL.

Vuelvo a enviar el procedimiento con el problema original, pero solo con la Select.

DELIMITER $$

DROP FUNCTION IF EXISTS `CUR` $$
CREATE DEFINER=`root`@`%` FUNCTION `CUR`() RETURNS int(4)
BEGIN
DECLARE PRUEBA INT(4) DEFAULT 999;

DECLARE done INT DEFAULT 0;
DECLARE ANYTRA INT(4);
DECLARE W_ANYTRA INT(4);
DECLARE MESTRA int(2);
DECLARE NOMMES varchar(15);
DECLARE NCUENTA varchar(30);
DECLARE NOMCLIENTE varchar(255);
DECLARE CODREPRE varchar(20);
DECLARE NOMREPRE varchar(50);
DECLARE MES_ACTUAL_IMP float(15,2);
DECLARE MES_ACTUAL_CANT float(15,2);
DECLARE IMPBRU float(11,2);
DECLARE DTOFAM float(11,2);
DECLARE DTOART float(11,2);
DECLARE DTOCAB float(11,2);
DECLARE DTOOTR float(11,2);
DECLARE PRECIO float(11,2);
DECLARE PRECOS float(11,2);
DECLARE VAR_MES_ANT_IMP float(15,2);
DECLARE VAR_MES_ANT_IMP_PCT float(10,2);
DECLARE VAR_MES_ANT_CANT float(15,2);
DECLARE VAR_MES_ANT_CANT_PCT float(10,2);
DECLARE VAR_FIN_ANY_IMP float(15,2);
DECLARE VAR_FIN_ANY_IMP_PCT float(10,2);
DECLARE VAR_FIN_ANY_CANT float(15,2);
DECLARE VAR_FIN_ANY_CANT_PCT float(10,2);
DECLARE VAR_INTERANUAL_IMP float(15,2);
DECLARE VAR_INTERANUAL_IMP_PCT float(10,2);
DECLARE VAR_INTERANUAL_CANT float(15,2);
DECLARE VAR_INTERANUAL_CANT_PCT float(10,2);
DECLARE VAR_PRESUPUESTO_IMP float(15,2);
DECLARE VAR_PRESUPUESTO_IMP_PCT float(10,2);
DECLARE VAR_PRESUPUESTO_CANT float(15,2);
DECLARE VAR_PRESUPUESTO_CANT_PCT float(10,2);

SELECT * INTO ANYTRA,MESTRA,NOMMES,NCUENTA,NOMCLIENTE,CODREPRE,N OMREPRE,MES_ACTUAL_IMP,MES_ACTUAL_CANT,IMPBRU,DTOF AM,DTOART,DTOCAB,DTOOTR,PRECIO,PRECOS,VAR_MES_ANT_ IMP,vAR_MES_ANT_IMP_PCT,VAR_MES_ANT_CANT,VAR_MES_A NT_CANT_PCT,VAR_FIN_ANY_IMP,VAR_FIN_ANY_IMP_PCT,VA R_FIN_ANY_CANT,VAR_FIN_ANY_CANT_PCT,VAR_INTERANUAL _IMP,VAR_INTERANUAL_IMP_PCT,VAR_INTERANUAL_CANT,VA R_INTERANUAL_CANT_PCT,VAR_PRESUPUESTO_IMP,VAR_PRES UPUESTO_IMP_PCT,VAR_PRESUPUESTO_CANT,VAR_PRESUPUES TO_CANT_PCT from VENTAS_CLIENTE_REPRE_backup;

RETURN ANYTRA;

END $$

DELIMITER ;

En este caso me funciona bien porque he puesto "Select *", si esta select la cambio y pongo los nombres de las columnas entonces es cuando me devuelve NULL.

gnzsoloyo · #13 (**permalink**) 01/03/2011, 10:17

Me parece que no estás entendiendo la idea:
El CURSOR lo usas para leer uno a uno un conjunto de registros dado, luego, con cada registro si y sólo si no has superado el límite (SQLSTATE '02000'), haces un INSERT. Pero si has llegado al final, el último FETCH devuelve NULL y pone done = 1, por lo que no entra en el IF que contiene el INSERT, pero sí realiza la asignación de NULL a todas las variables.
¿Se entiende ahora?
A partir de ese último FETCH, ANYTRA contiene un NULL.
Para que te devuelva el valor de ANYTRA la idea sería:

Código MySQL:

Ver originalDELIMITER $$
 
DROP FUNCTION IF EXISTS `CUR` $$
CREATE FUNCTION `CUR`() RETURNS int(4)
BEGIN
  DECLARE PRUEBA INT(4) DEFAULT 999;
  DECLARE done INT DEFAULT 0;
  /* ------------------------------------------------------------------------ */
  DECLARE T_ANYTRA INT(4);
  /* ------------------------------------------------------------------------ */
  DECLARE ANYTRA INT(4);
  DECLARE W_ANYTRA INT(4);
  DECLARE MESTRA int(2);
  DECLARE NOMMES varchar(15);
  DECLARE NCUENTA varchar(30);
  DECLARE NOMCLIENTE varchar(255);
  DECLARE CODREPRE varchar(20);
  DECLARE NOMREPRE varchar(50);
  DECLARE MES_ACTUAL_IMP float(15,2);
  DECLARE MES_ACTUAL_CANT float(15,2);
  DECLARE IMPBRU float(11,2);
  DECLARE DTOFAM float(11,2);
  DECLARE DTOART float(11,2);
  DECLARE DTOCAB float(11,2);
  DECLARE DTOOTR float(11,2);
  DECLARE PRECIO float(11,2);
  DECLARE PRECOS float(11,2);
  DECLARE VAR_MES_ANT_IMP float(15,2);
  DECLARE VAR_MES_ANT_IMP_PCT float(10,2);
  DECLARE VAR_MES_ANT_CANT float(15,2);
  DECLARE VAR_MES_ANT_CANT_PCT float(10,2);
  DECLARE VAR_FIN_ANY_IMP float(15,2);
  DECLARE VAR_FIN_ANY_IMP_PCT float(10,2);
  DECLARE VAR_FIN_ANY_CANT float(15,2);
  DECLARE VAR_FIN_ANY_CANT_PCT float(10,2);
  DECLARE VAR_INTERANUAL_IMP float(15,2);
  DECLARE VAR_INTERANUAL_IMP_PCT float(10,2);
  DECLARE VAR_INTERANUAL_CANT float(15,2);
  DECLARE VAR_INTERANUAL_CANT_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_IMP float(15,2);
  DECLARE VAR_PRESUPUESTO_IMP_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_CANT float(15,2);
  DECLARE VAR_PRESUPUESTO_CANT_PCT float(10,2);
 
  DECLARE CUR_VENTAS_CLIENTE_REPRE CURSOR FOR
  SELECT * FROM VENTAS_CLIENTE_REPRE_BACKUP;
 
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
 
  OPEN CUR_VENTAS_CLIENTE_REPRE;
 
  REPEAT
    FETCH CUR_VENTAS_CLIENTE_REPRE INTO
      ANYTRA,MESTRA,NOMMES,NCUENTA,NOMCLIENTE,CODREPRE,N OMREPRE,
      MES_ACTUAL_IMP,MES_ACTUAL_CANT,IMPBRU,DTOF AM,DTOART,DTOCAB,
      DTOOTR,PRECIO,PRECOS,VAR_MES_ANT_ IMP,vAR_MES_ANT_IMP_PCT,VAR_MES_ANT_CANT,
      VAR_MES_A NT_CANT_PCT,VAR_FIN_ANY_IMP,VAR_FIN_ANY_IMP_PCT,
      VAR_FIN_ANY_CANT,VAR_FIN_ANY_CANT_PCT,VAR_INTERANUAL _IMP,
      VAR_INTERANUAL_IMP_PCT,VAR_INTERANUAL_CANT,VA R_INTERANUAL_CANT_PCT,
      VAR_PRESUPUESTO_IMP,VAR_PRES UPUESTO_IMP_PCT,VAR_PRESUPUESTO_CANT,
      VAR_PRESUPUESTO_CANT_PCT;
    IF NOT done THEN
      /* Inserción de los registros de referencia agrupados por CLIENTE y REPRE */
      INSERT INTO VENTAS_CLIENTE_REPRE_BACKUP
      VALUES(ANYTRA, MESTRA, NULL, NCUENTA, NOMCLIENTE, CODREPRE, NOMREPRE,
      MES_ACTUAL_IMP, MES_ACTUAL_CANT, IMPBRU, DTOFAM,
      DTOART,DTOCAB,DTOOTR,PRECIO,PRECOS,
      VAR_MES_ANT_I MP,vAR_MES_ANT_IMP_PCT,VAR_MES_ANT_CANT,
      VAR_MES_AN T_CANT_PCT,VAR_FIN_ANY_IMP,VAR_FIN_ANY_IMP_PCT,
      VAR _FIN_ANY_CANT,VAR_FIN_ANY_CANT_PCT,VAR_INTERANUAL_IMP,
      VAR_INTERANUAL_IMP_PCT,VAR_INTERANUAL_CANT,VAR_INTERANUAL_CANT_PCT,
      VAR_PRESUPUESTO_IMP,VAR_PRESU PUESTO_IMP_PCT,
      VAR_PRESUPUESTO_CANT,VAR_PRESUPUEST O_CANT_PCT);
 
      UPDATE VENTAS_CLIENTE_REPRE_BACKUP SET NOMMES=InformarNombreMes(MESTRA);
      /* ------------------------------------------------------------------------ */
      SET T_ANYTRA = ANYTRA;
      /* ------------------------------------------------------------------------ */
    END IF;
  UNTIL done END REPEAT;
 
  CLOSE CUR_VENTAS_CLIENTE_REPRE;
 
  RETURN T_ANYTRA;
END $$
 
DELIMITER ;

cgonzalezc · #14 (**permalink**) 01/03/2011, 11:17

Creo que sigo sin saber explicar el origen del problema. Quiza he juntado demasiados temas.

Digamos que quiero hacer dos cosas en la función.
Primera: devolver como parámetro el valor de ANYTRA.
Segunda: insertar los registros de la tabla origen a la tabla destino.

Primera:
Lo del cursor lo entiendo, en la ultima pasada del FETCH está todo a NULL, con lo que al intentar devolverlo como parámetro de la función me da NULL. Correcto.

Segunda:
El caso es que si monto el cursor con "Select *" inserta los registros correctamente.
Si el cursor lo monto con "Select ANYTRA", a la hora de insertar los registros lo que hace es insertarlos todos a NULL.
Tanto si es el primero como si es el último (contando como último el que tiene datos, ya que si no, no pasa por el INSERT)

Y aparte del tema del cursor.
La prueba simple que he hecho, es una "SELECT ANYTRA" a pelo, sin cursores ni nada, esta SELECT me devuelve NULL, no esta dentro de un cursor ni nada, o sea, no es un FETCH, es una SELECT pura y dura.
Si la hago la "SELECT *", me funciona correctamente.

Mi duda reside ahí en que me devuelve NULL la columna de la tabla, no en si es el primero o último registro. Ya que al ser una SELECT sólo me devuelve un registro (porque mi tabla solo tiene un registro)

gnzsoloyo · #15 (**permalink**) 01/03/2011, 12:57

Cita:

Segunda:
El caso es que si monto el cursor con "Select *" inserta los registros correctamente.
Si el cursor lo monto con "Select ANYTRA", a la hora de insertar los registros lo que hace es insertarlos todos a NULL.
Tanto si es el primero como si es el último (contando como último el que tiene datos, ya que si no, no pasa por el INSERT)

Pregunta del millón: ¿ANYTRA es el nombre de la columna que almacena ese valor y además es el nombre de la variable?

cgonzalezc · #16 (**permalink**) 02/03/2011, 03:18

Si, es el mismo.

He probado a cambiar el nombre de la Variable para que no coincidiera pero ocurre lo mismo. Y tambien he probado con las variables que empiezan con la @, pero igual.

cgonzalezc · #17 (**permalink**) 03/03/2011, 08:00

Hola gnzsoloyo.

Me podrías confirmar algo de este tema, si te suena que puede estar sucediendo o no se te ocurre nada.

Gracias de antemano.

gnzsoloyo · #18 (**permalink**) 03/03/2011, 08:32

Francamente me gustaría hacer pruebas directas para terminar de analizar por qué te da ese resultado. Sería bueno contar con una copia de la base para probar, pero bueno...
Respecto a los nombres, MySQL trabaja en forma jerárquica con los objetos de base de datos, por eso no es buena práctica usar los mismos nombres entre objetos distintos, como es el caso de mismo nombre para tablas, bases, columnas o variables. Eso puede hacer confundir al parser.
Uno de los resultados que he visto en ocasiones es que si pones el mismo nombre a una variable que a una tabla, la sentencia SET variable = variable devuelve NULL, porque no se puede asignar una tabla a una variable en MySQL. Esto pasa porque el objeto de mayor jerarquía pisa al de menor, y el parser interpreta que te refieres a la tabla...
Además de eso, si usas variables de usuario (las que llevan "@" como prefijo), deben ser inicializadas antes de ser usadas, porque su valor inicial es siempre NULL. Por eso un

Código MySQL:

Ver originalSELECT A FROM tabla INTO @B

devuelve un NULL.
Para usarla lo primero es incializarla con un valor que le defina el tipo (que tampoco tienen), y además que sea un valor sin significado:

Código MySQL:

Ver originalSET @v_anytra = 0;
SET @v_anytra = T_ANYTRA;

cgonzalezc · #19 (**permalink**) 03/03/2011, 10:53

Gracias.

Aun a malas te puedo enviar un backup en txt de la base de datos, pero claro, por aqui no se puede.

La verdad que es un caso muy raro, que de una forma funcione y de la otra no, y claro con el * no me sirve porque necesito hacer GROUP BY y Sumatorios de columnas.

gnzsoloyo · #20 (**permalink**) 03/03/2011, 11:06

Fíjate de comprimirlo y subirlo a algún servicio de archivos en web. NO recuerdo en este momento el link de ninguno, pero debe ser simple ubicarlos.

cgonzalezc · #21 (**permalink**) 04/03/2011, 02:59

Aqui te envío el link

http://www.filedropper.com/testbackup201103040953

La función es la CUR()

Carlos

reivaj01 · #22 (**permalink**) 04/03/2011, 03:35

Hola Carlos

Lo primero que te aconsejo es que borres ese fichero si los datos son de producción, ya que has mandado un backup completo.

Continuando con el post, y revisando el backup enviado, tu problema está aquí:

Código:

INSERT INTO `ventas_cliente_repre_backup` (`ANYTRA`,`MESTRA`,`NOMMES`,`NCUENTA`,`NOMCLIENTE`,`CODREPRE`,`NOMREPRE`,`MES_ACTUAL_IMP`,`MES_ACTUAL_CANT`,`IMPBRU`,`DTOFAM`,`DTOART`,`DTOCAB`,`DTOOTR`,`PRECIO`,`PRECOS`,`VAR_MES_ANT_IMP`,`VAR_MES_ANT_IMP_PCT`,`VAR_MES_ANT_CANT`,`VAR_MES_ANT_CANT_PCT`,`VAR_FIN_ANY_IMP`,`VAR_FIN_ANY_IMP_PCT`,`VAR_FIN_ANY_CANT`,`VAR_FIN_ANY_CANT_PCT`,`VAR_INTERANUAL_IMP`,`VAR_INTERANUAL_IMP_PCT`,`VAR_INTERANUAL_CANT`,`VAR_INTERANUAL_CANT_PCT`,`VAR_PRESUPUESTO_IMP`,`VAR_PRESUPUESTO_IMP_PCT`,`VAR_PRESUPUESTO_CANT`,`VAR_PRESUPUESTO_CANT_PCT`) VALUES 
 (NULL,NULL,'NO INFORMADO',NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL),
 (NULL,NULL,'NO INFORMADO',NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL),
 (NULL,NULL,'NO INFORMADO',NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL);

Como puedes ver, la columna ANYTRA no tiene valor en ninguna de las 4 filas.

cgonzalezc · #23 (**permalink**) 04/03/2011, 03:47

No son datos de producción. Por eso lo he enviado.

Esta sería la tabla de destino, con lo que el resultado que haya ahí no es el problema.

No es problema de datos. El problema está en la función.

Gracias de todos modos.

Carlos

gnzsoloyo · #24 (**permalink**) 04/03/2011, 06:48

Gracias por darme acceso al backup. Si lo quieres retirar ya, no hay problema.
Estuve probando y revisando todo luego de cargar la base de prueba y hay algunos detalles que resolver:
- La tabla "VENTAS_CLIENTE_REPRE_BACKUP" carece de PK, por lo que puede llegar a registrarse más de un backup conteniendo los mismos datos. Eso puede ser eventualmente un error, que favorece que se llene de "basura". La misma tabla tampoco evita que sus valores sea null, lo que también ayuda a guardar datos "basura".
- El backup muestra que se han almacenado datos "basura", lo que te dará problemas al momento de realizar pruebas.
- Un error que debes estar recibiendo es que la consulta devmuelve más de una fila. Eso sucede en esta parte:

Código MySQL:

Ver originalSELECT ANYTRA, MESTRA, NOMMES FROM VENTAS_CLIENTE_REPRE_BACKUP
INTO ANYTRA, MESTRA, NOMMES;

Como la tabla contiene más de un registro, no puedes volcarlo a variables sin antes restringir la cantidad de registros devueltos a uno solo.
- Lo que comprobé es que los valores no se insertan en la tabla en cuestión cuando se usan los nombres de las columnas como nombres de variables. Deben ser diferentes.

Lo que quisiera que me expliques es el sentido de este último segmento, luego de cerrar el cursor. ¿Por qué pones esos NULL?:

Código MySQL:

Ver originalSELECT MAX(ANYTRA), MAX(MESTRA), NOMMES FROM VENTAS_CLIENTE_REPRE_BACKUP LIMIT 1
  INTO VANYTRA, VMESTRA, VNOMMES;
 
  INSERT INTO VENTAS_CLIENTE_REPRE_BACKUP VALUES
(VANYTRA, 1, 'ENERO', 45, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL);

gnzsoloyo · #25 (**permalink**) 04/03/2011, 07:27

Bien, finalmente encontré los problemas que generaban el error:
1) La variable "done" estaba creada, pero no inicializada, por lo que su valor real era NULL y en ese caso todo el proceso del IF falla. No se puede hacer un IF contra un NULL, sino contra una función para NULLs.
2) Resuelto eso, la cantidad de valores a ingresar en esa tabla no se corresponde con la cantidad de columnas de la tabla. En ese caso lo que corresponde hacer es o bien poner ,todos los valores necesarios, o indicar los valores que se usarán.
Quedó finalmente así:

Código MySQL:

Ver originalDELIMITER $$
 
DROP PROCEDURE IF EXISTS `test`.`CUR` $$
CREATE DEFINER=`root`@`%` PROCEDURE `CUR`()
BEGIN
  DECLARE done INT DEFAULT 0;
  DECLARE VANYTRA int(4);
  DECLARE VMESTRA int(2);
  DECLARE VNOMMES varchar(15);
  DECLARE VNCUENTA varchar(30);
  DECLARE VNOMCLIENTE varchar(255);
  DECLARE VCODREPRE varchar(20);
  DECLARE VNOMREPRE varchar(50);
  DECLARE VMES_ACTUAL_IMP float(15,2);
  DECLARE VMES_ACTUAL_CANT float(15,2);
  DECLARE VIMPBRU float(11,2);
  DECLARE VDTOFAM float(11,2);
  DECLARE VDTOART float(11,2);
  DECLARE VDTOCAB float(11,2);
  DECLARE VDTOOTR float(11,2);
  DECLARE VPRECIO float(11,2);
  DECLARE VPRECOS float(11,2);
  DECLARE VAR_MES_ANT_IMP float(15,2);
  DECLARE VAR_MES_ANT_IMP_PCT float(10,2);
  DECLARE VAR_MES_ANT_CANT float(15,2);
  DECLARE VAR_MES_ANT_CANT_PCT float(10,2);
  DECLARE VAR_FIN_ANY_IMP float(15,2);
  DECLARE VAR_FIN_ANY_IMP_PCT float(10,2);
  DECLARE VAR_FIN_ANY_CANT float(15,2);
  DECLARE VAR_FIN_ANY_CANT_PCT float(10,2);
  DECLARE VAR_INTERANUAL_IMP float(15,2);
  DECLARE VAR_INTERANUAL_IMP_PCT float(10,2);
  DECLARE VAR_INTERANUAL_CANT float(15,2);
  DECLARE VAR_INTERANUAL_CANT_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_IMP float(15,2);
  DECLARE VAR_PRESUPUESTO_IMP_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_CANT float(15,2);
  DECLARE VAR_PRESUPUESTO_CANT_PCT float(10,2);
  /* ------------------------------------------------------------------------------- */
  DECLARE CUR_VENTAS_CLIENTE_REPRE CURSOR FOR
          SELECT ANYTRA,MESTRA,InformarNombreMes(MESTRA) AS NOMMES,NCUENTA,NOMCLIENTE,REPRE AS CODREPRE,NOMREPRE,SUM(IMPNET) AS MES_ACTUAL_IMP,COUNT(*) AS MES_ACTUAL_CANT,
                 SUM(IMPBRU) AS IMPBRU,SUM(DTOFAM) AS DTOFAM,SUM(DTOART) AS DTOART,SUM(DTOCAB) AS DTOCAB,SUM(DTOOTR) AS DTOOTR,SUM(PRECIO)/COUNT(*) AS PRECIO,SUM(PRECOS) AS PRECOS
          FROM DADESVENDES
          WHERE NCUENTA IS NOT NULL
          GROUP BY ANYTRA,MESTRA,NCUENTA,NOMCLIENTE,CODREPRE,NOMREPRE;
  /* ------------------------------------------------------------------------------- */
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
  /* ------------------------------------------------------------------------------- */
  OPEN CUR_VENTAS_CLIENTE_REPRE;
  /* ------------------------------------------------------------------------------- */
  REPEAT
    FETCH CUR_VENTAS_CLIENTE_REPRE INTO
            VANYTRA,
            VMESTRA,
            VNOMMES,
            VNCUENTA,
            VNOMCLIENTE,
            VCODREPRE,
            VNOMREPRE,
            VMES_ACTUAL_IMP,
            VMES_ACTUAL_CANT,
            VIMPBRU,
            VDTOFAM,
            VDTOART,
            VDTOCAB,
            VDTOOTR,
            VPRECIO,
            VPRECOS;
    IF NOT done THEN
      /* ------------------------------------------------------------------------------- */
      /* Inserción de los registros de referencia agrupados por CLIENTE y REPRE */
      /* ------------------------------------------------------------------------------- */
      INSERT INTO VENTAS_CLIENTE_REPRE_BACKUP(
        ANYTRA,
        MESTRA,
        NOMMES,
        NCUENTA,
        NOMCLIENTE,
        CODREPRE,
        NOMREPRE,
        MES_ACTUAL_IMP,
        MES_ACTUAL_CANT,
        IMPBRU,
        DTOFAM,
        DTOART,
        DTOCAB,
        DTOOTR,
        PRECIO,
        PRECOS)
    VALUES
        (VANYTRA,
        VMESTRA,
        VNOMMES,
        VNCUENTA,
        VNOMCLIENTE,
        VCODREPRE,
        VNOMREPRE,
        VMES_ACTUAL_IMP,
        VMES_ACTUAL_CANT,
        VIMPBRU,
        VDTOFAM,
        VDTOART,
        VDTOCAB,
        VDTOOTR,
        VPRECIO,
        VPRECOS);
    END IF;
  UNTIL done END REPEAT;
  /* ------------------------------------------------------------------------------- */
  CLOSE CUR_VENTAS_CLIENTE_REPRE;
  /* ------------------------------------------------------------------------------- */
  SELECT MAX(ANYTRA), MAX(MESTRA), NOMMES FROM VENTAS_CLIENTE_REPRE_BACKUP LIMIT 1
  INTO VANYTRA, VMESTRA, VNOMMES;
  /* ------------------------------------------------------------------------------- */
  SELECT VANYTRA, VMESTRA, VNOMMES;
END $$
 
DELIMITER ;

La última parte es una adaptación de lo que pusiste para que devuelva algo.

Queda pendiente resolver lo de las PK de las tablas, ya que no se está controlando la unicidad de valores.
Tampoco hay un buen uso de la programación de SP y SF en algunos casos, en los que se están haciendo operaciones innecesariamente complicadas, como por ejemplo en la función `InformarNombreMes`, que yo escribiría así:

Código MySQL:

Ver originalDELIMITER $$
 
DROP FUNCTION IF EXISTS `InformarNombreMes` $$
CREATE FUNCTION `InformarNombreMes`(COD INT) RETURNS varchar(15) CHARSET latin1
BEGIN
  CASE COD
    WHEN 1 THEN
      RETURN 'ENE';
    WHEN 2 THEN
      RETURN 'FEB';
    WHEN 3 THEN
      RETURN 'MAR';
    WHEN 4 THEN
      RETURN 'ABR';
    WHEN 5 THEN
      RETURN 'MAY';
    WHEN 6 THEN
      RETURN 'JUN';
    WHEN 7 THEN
      RETURN 'JUL';
    WHEN 8 THEN
      RETURN 'AGO';
    WHEN 9 THEN
      RETURN 'SEP';
    WHEN 10 THEN
      RETURN 'OCT';
    WHEN 11 THEN
      RETURN 'NOV';
    WHEN 12 THEN
      RETURN 'DIC';
    ELSE
      RETURN 'NO INFORMADO';
  END CASE;
END $$
 
DELIMITER ;

Es un poco más limpio y al parser le resulta más eficiente.

cgonzalezc · #26 (**permalink**) 08/03/2011, 03:07

Buenos dias.

Muchas gracias, lo he probado y funciona, lo he aplicado a mi función para entenderlo todo bien. Y ya funciona.

Lo que me comentas de los NULL, esta puesto así porque estaba haciendo pruebas.

Gracias por la solución de la otra función, yo estube probando con el CASE y como no me salía por eso lo puse en una SELECT, ya sabía que no era la mejor solución pero fue lo único que se me ocurrió.

Gracias.

gnzsoloyo · #27 (**permalink**) 08/03/2011, 06:51

Me alegro que te haya servido todo el tema.
Saludos

:adio: