[SOLUCIONADO] procedimientos almacenados error

[shake2427]

buenas tardes

tengo el siguiente codigo

Código MySQL:

-- --------------------------------------------------------------------------------
-- Routine DDL
-- Note: comments before and after the routine body will not be stored by the server
-- --------------------------------------------------------------------------------
DELIMITER $$
 
CREATE PROCEDURE `movistarmoviles_dev`.`recargas_detallado_agente` (
in p_iniciolimit int,
in p_finlimit int,
IN p_fechainicial datetime,
IN p_fechafinal datetime
)
BEGIN
 
SET @qryrecdene = concat('SELECT a.fecha_venta,
                                                b.cod_vendedor as codigo_vendedor,
                                                b.nom_vendedor as Nombre_vendedor,
                                                a.num_iden_cliente as Identificacion,
                                                a.nombre_Cliente as Cliente,
                                                a.numero_icc as numero_icc,
                                                a.celular as numero_celular,        
                                                b.Valor_recarga as valor_recarga,
                                                (select nombre 
                                                   from distribuidores
                                                  where id_distribuidores = (select id_distribuidor  
                                                                               from distribxnegocio 
                                                                              where id_radicaciones = a.id_radicaciones)) as Distribuidor,
                                                 (select nombre 
                                                     from negocios
                                                    where id_negocio in (select id_negocio from distribxnegocio where id_radicaciones = a.id_radicaciones) ) as Negocio,
                                                b.fecha_recarga,
                                                b.fecha_Actualizacion
                                        
                                          FROM radicaciones a, recargas b, distribxnegocio c
                                         where a.Celular = b.num_celular
                                           AND a.producto ="PREPAGO"
                                           and a.id_radicaciones = c.id_radicaciones
                                           and a.numero_icc = (select numero_icc from simcard where id_simcard = c.id_simcard)
' );
 
if ((p_fechainicial != 0) and (p_fechafinal != 0)) then
    SET @qryrecdene = concat('AND b.fecha_recarga between ', p_fechainicial ,' AND ', p_fechafinal);
end if;
 
SET @qryrecdene = concat('order by Negocio');
SET @qryrecdene = concat('LIMIT ', p_iniciolimit ,',', p_finlimit);
 
PREPARE qryexnede FROM @qryrecdene;
EXECUTE qryexnede;
 
END
 

pero al tratar de ejecutarlo me muestra un error de sintaxis, le he buscado una y mil formas de buscar la solucion, pero no he podido.

por favor si me pueden ayudar o darme alguna guia.

gracias de antemano.

[gnzsoloyo]

Estás pisando lo que contiene la variable.
Lo único que termina quedando es esto:

Código MySQL:

SET @qryrecdene = CONCAT('LIMIT ', p_iniciolimit ,',', p_finlimit);
 

Debes encadenar lo que ya tiene la variable con lo que le agregas.

[shake2427]

Iniciado por gnzsoloyo

Estás pisando lo que contiene la variable.
Lo único que termina quedando es esto:

Código MySQL:

SET @qryrecdene = CONCAT('LIMIT ', p_iniciolimit ,',', p_finlimit);
 

Debes encadenar lo que ya tiene la variable con lo que le agregas.

ya logre que me uniera toda la consulta el codigo es el siguiente

Código SQL:

-- --------------------------------------------------------------------------------
-- Routine DDL
-- Note: comments before and after the routine body will not be stored by the server
-- --------------------------------------------------------------------------------
DELIMITER $$
 
CREATE PROCEDURE `movistarmoviles_dev`.`recargas_detallado_agente` (
IN p_iniciolimit INT,
IN p_finlimit INT,
IN p_fechainicial datetime,
IN p_fechafinal datetime
)
BEGIN
 
SET @qryrecdene = concat('SELECT a.fecha_venta,
                                                b.cod_vendedor as codigo_vendedor,
                                                b.nom_vendedor as Nombre_vendedor,
                                                a.num_iden_cliente as Identificacion,
                                                a.nombre_Cliente as Cliente,
                                                a.numero_icc as numero_icc,
                                                a.celular as numero_celular,        
                                                b.Valor_recarga as valor_recarga,
                                                (select nombre 
                                                   from distribuidores
                                                  where id_distribuidores = (select id_distribuidor  
                                                                               from distribxnegocio 
                                                                              where id_radicaciones = a.id_radicaciones)) as Distribuidor,
                                                 (select nombre 
                                                     from negocios
                                                    where id_negocio in (select id_negocio from distribxnegocio where id_radicaciones = a.id_radicaciones) ) as Negocio,
                                                b.fecha_recarga,
                                                b.fecha_Actualizacion
                                        
                                          FROM radicaciones a, recargas b, distribxnegocio c
                                         where a.Celular = b.num_celular
                                           AND a.producto ="PREPAGO"
                                           and a.id_radicaciones = c.id_radicaciones
                                           and a.numero_icc = (select numero_icc from simcard where id_simcard = c.id_simcard)
' );
 
IF ((p_fechainicial != 0) AND (p_fechafinal != 0)) THEN
    SET @qryrecdene. = concat('AND b.fecha_recarga between ', p_fechainicial ,' AND ', p_fechafinal);
END IF;
 
SET @qryrecdene. = concat('order by Negocio');
SET @qryrecdene. = concat('LIMIT ', p_iniciolimit ,',', p_finlimit);
 
PREPARE qryexnede FROM @qryrecdene;
EXECUTE qryexnede;
 
END
 

quiero saber si la condicion del IF esta bien realizada es que estoy llamando el procedimiento de la siguiente forma

call recargas_detallado_agente(1,1000,'2012-11-01','2012-11-13')

y ahi deveria entrar al if y poner el filtro en la consulta pero no me lo pone.

[gnzsoloyo]

Estás mezclando C con MySQL. Has puesto cosas que nunca vi, y no estoy segudo de que funcionen.
Además, la consulta a mi entender es innecesariamente ineficiente.

Yo la armaría así:

Código MySQL:

-- --------------------------------------------------------------------------------
-- Routine DDL
-- Note: comments before and after the routine body will not be stored by the server
-- --------------------------------------------------------------------------------
DELIMITER $$
 
CREATE PROCEDURE `recargas_detallado_agente` (
IN p_iniciolimit int,
IN p_finlimit int,
IN p_fechainicial datetime,
IN p_fechafinal datetime
)
BEGIN
 
SET @qryrecdene = concat('SELECT 
     a . fecha_venta ,
     b . cod_vendedor        codigo_vendedor ,
     b . nom_vendedor        Nombre_vendedor ,
     a . num_iden_cliente    Identificacion ,
     a . nombre_Cliente      Cliente ,
     a . numero_icc          numero_icc ,
     a . celular             numero_celular ,
     b . Valor_recarga       valor_recarga ,
     D.nombre               Distribuidor,
     N.nombre               Negocio,
     b . fecha_recarga ,
     b . fecha_Actualizacion 
FROM
     radicaciones a 
     INNER JOIN  recargas   b ON a.Celular  =  b.num_celular
     INNER JOIN distribxnegocio c ON a.id_radicaciones  =  c.id_radicaciones  
     INNER JOIN distribuidores D ON  D.id_distribuidores = c.id_distribuidor 
     INNER JOIN negocios N ON N.id_negocio = c.id_negocio
     INNER JOIN simcard s ON s.id_simcard  =  c.id_simcard 
WHERE
       a.producto  = ''PREPAGO''');
 
IF ((p_fechainicial != 0) AND (p_fechafinal != 0)) then
    SET @qryrecdene = concat(@qryrecdene, ' AND b.fecha_recarga between ''', p_fechainicial ,''' AND ''', p_fechafinal, '''');
END IF;
 
SET @qryrecdene = concat(@qryrecdene, ' order by Negocio');
SET @qryrecdene. = concat(@qryrecdene, 'LIMIT ', p_iniciolimit ,',', p_finlimit);
 
PREPARE qryexnede FROM @qryrecdene;
EXECUTE qryexnede;
 
END
 

[shake2427]

Iniciado por gnzsoloyo

Estás mezclando C con MySQL. Has puesto cosas que nunca vi, y no estoy segudo de que funcionen.
Además, la consulta a mi entender es innecesariamente ineficiente.

Yo la armaría así:

Código MySQL:

-- --------------------------------------------------------------------------------
-- Routine DDL
-- Note: comments before and after the routine body will not be stored by the server
-- --------------------------------------------------------------------------------
DELIMITER $$
 
CREATE PROCEDURE `recargas_detallado_agente` (
IN p_iniciolimit int,
IN p_finlimit int,
IN p_fechainicial datetime,
IN p_fechafinal datetime
)
BEGIN
 
SET @qryrecdene = concat('SELECT 
     a . fecha_venta ,
     b . cod_vendedor        codigo_vendedor ,
     b . nom_vendedor        Nombre_vendedor ,
     a . num_iden_cliente    Identificacion ,
     a . nombre_Cliente      Cliente ,
     a . numero_icc          numero_icc ,
     a . celular             numero_celular ,
     b . Valor_recarga       valor_recarga ,
     D.nombre               Distribuidor,
     N.nombre               Negocio,
     b . fecha_recarga ,
     b . fecha_Actualizacion 
FROM
     radicaciones a 
     INNER JOIN  recargas   b ON a.Celular  =  b.num_celular
     INNER JOIN distribxnegocio c ON a.id_radicaciones  =  c.id_radicaciones  
     INNER JOIN distribuidores D ON  D.id_distribuidores = c.id_distribuidor 
     INNER JOIN negocios N ON N.id_negocio = c.id_negocio
     INNER JOIN simcard s ON s.id_simcard  =  c.id_simcard 
WHERE
       a.producto  = ''PREPAGO''');
 
IF ((p_fechainicial != 0) AND (p_fechafinal != 0)) then
    SET @qryrecdene = concat(@qryrecdene, ' AND b.fecha_recarga between ''', p_fechainicial ,''' AND ''', p_fechafinal, '''');
END IF;
 
SET @qryrecdene = concat(@qryrecdene, ' order by Negocio');
SET @qryrecdene. = concat(@qryrecdene, 'LIMIT ', p_iniciolimit ,',', p_finlimit);
 
PREPARE qryexnede FROM @qryrecdene;
EXECUTE qryexnede;
 
END
 

a que te refieres con cosas que nunca habias visto?

la verdad apenas estoy empezando en esto de los procedimientos almacenados, y me gustaria que me orientaras un poco en este sentido.

gracias de antemano

[gnzsoloyo]

".=" no existe en MySQL como operador de concatenación. Por eso te pisa el contenido de la variable.
Ese operador, que yo sepa, es de PHP, no de MySQL.

[shake2427]

Código MySQL:

SET @qryrecdene. = concat(@qryrecdene, 'LIMIT ', p_iniciolimit ,',', p_finlimit);
 
END
 

[/QUOTE]

entonces por que en esta parte le pones el punto, que significa eso?

y en cuanto al query dices que es ineficiente por todas las subconsultas que hay en la consulta?

[gnzsoloyo]

El punto no va como concatenador de contenidos, Simplemente lo pones como te propongo, ya que la única forma de concatenar cadenas es con CONCAT(), al menos en MySQL.

En cuando a la ineficiencia, ten en cuenta que una subconsulta en el SELECT se ejecuta una vez por cada registro devuelto por la consulta principal. Imaginate con algunos miles de registros, cuál es el efecto de semejante sobrecarga.
Todo dato que peuda ser obtenido en el FROM es siempre mejor. Especialmente cuando se trata de campos FK, ya que existen índices que los administran, los cuales son usados por MySQL al momento de ejecutarla.
Son funcionales las subconsultas contra tablas que devuelven muy pocos registros (alta selectividad), por los parámetros dados, pero sin conocer tu base únicamente puedo hacer suposiciones teóricas.
De todos modos, por lo que puedo ver de las relaciones indicadas en tu query base, todo parece ser resoluble con INNER JOIN, por lo que no termino de comprender para qué o por qué razón quieres hacerlo con subconsultas en las columnas del SELECT.

[shake2427]

Iniciado por gnzsoloyo

El punto no va como concatenador de contenidos, Simplemente lo pones como te propongo, ya que la única forma de concatenar cadenas es con CONCAT(), al menos en MySQL.

En cuando a la ineficiencia, ten en cuenta que una subconsulta en el SELECT se ejecuta una vez por cada registro devuelto por la consulta principal. Imaginate con algunos miles de registros, cuál es el efecto de semejante sobrecarga.
Todo dato que peuda ser obtenido en el FROM es siempre mejor. Especialmente cuando se trata de campos FK, ya que existen índices que los administran, los cuales son usados por MySQL al momento de ejecutarla.
Son funcionales las subconsultas contra tablas que devuelven muy pocos registros (alta selectividad), por los parámetros dados, pero sin conocer tu base únicamente puedo hacer suposiciones teóricas.
De todos modos, por lo que puedo ver de las relaciones indicadas en tu query base, todo parece ser resoluble con INNER JOIN, por lo que no termino de comprender para qué o por qué razón quieres hacerlo con subconsultas en las columnas del SELECT.

gracias por tu ayuda.