problema SP, con un IN dentro de un cursor

[socialk]

Espero ser lo más claro posible... tengo un SP el cual recibe un parametro por ejemplo

call spTest('10,11,12')

dentro del procedimiento tengo una consulta del tipo select * from tabla where id IN (parametro), al parecer MySQL le agrega caracteres de escape por eso no funciona, si lo hago con CONCAT, funciona, pero no puedo utilizar CONCAT dentro de un Cursor. pongo algo de codigo

Código MySQL:

DELIMITER $$
 
CREATE DEFINER=`root`@`localhost` PROCEDURE `spListPrice`(
        IN idInstrument VARCHAR(500),
        IN quality CHAR(3)
    )
BEGIN
    DECLARE valores varchar(500);
    DECLARE queryt VARCHAR(1000);
    DECLARE _pkIntradayTransaction INTEGER;
    DECLARE done INT DEFAULT 0;
    DECLARE cTag CURSOR FOR
        SELECT 
            max(pkIntradayTransaction)
        FROM
            tbIntradayTransactions
        WHERE
            fkInstrument IN (SELECT 
                pkInstrument
            FROM
                tbInstruments
            WHERE
                fkInstrument IN (valores)) AND datetimeLastPrice <= DATE_SUB(NOW(),INTERVAL 15 MINUTE)
        GROUP BY fkInstrument;
 
        DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
 

luego necesito hacer algo como

Código MySQL:

SELECT 
            max(pkIntradayTransaction)
        FROM
            tbIntradayTransactions
        WHERE
            fkInstrument IN (SELECT 
                pkInstrument
            FROM
                tbInstruments
            WHERE
                fkInstrument IN (valores)) AND datetimeLastPrice <= DATE_SUB(NOW(),INTERVAL 15 MINUTE)
        GROUP BY fkInstrument;
 

lo que sucede aquí es que no devuelve nada, como dije antes, creo que MySQL hace algo como

SELECT * FROM table WHERE id IN (`10,11,12`); al agregar eso, lo toma como si fuera un solo STRING.

ojalá alguien pudiera ayudarme, gracias!

[gnzsoloyo]

Primero: Las cadenas de texto se encierran entre apóstrofes. Los acentos inversos (`) son para los nombres de las tablas, bases, columnas y todo objeto creado por una sentencia DDL.
Segundo: IN() se usa para comparar un valor contra una lista de valores, pero cuando envías esto:

Código MySQL:

all spTest('10,11,12')
 

No estás enviando una lista de valores, sino una cadena de texto que contiene valores separados por comas, pero en esencia es un sólo valor, y no tres.
En tu caso, lo que debes hacer es comparar el valor buscado contra la lista contenida en la cadena, y para eso no sirve el IN(). Se debe usar una función de cadena denominada FIND_IN_SET():

Código MySQL:

DELIMITER $$
 
CREATE DEFINER=`root`@`localhost` PROCEDURE `spListPrice`(
        IN idInstrument VARCHAR(500),
        IN quality CHAR(3)
    )
BEGIN
    DECLARE valores VARCHAR(500);
    DECLARE queryt VARCHAR(1000);
    DECLARE _pkIntradayTransaction INTEGER;
    DECLARE done INT DEFAULT 0;
    DECLARE cTag CURSOR FOR
        SELECT 
            max(pkIntradayTransaction)
        FROM
            tbIntradayTransactions
        WHERE
            fkInstrument IN (SELECT 
                pkInstrument
            FROM
                tbInstruments
            WHERE
                (FIND_IN_SET (fkInstrument , valores)>0) 
              AND datetimeLastPrice <= DATE_SUB(NOW(),INTERVAL 15 MINUTE)
        GROUP BY fkInstrument;
 
        DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
    -- El resto del codigo
END
 
DELIMITER;
 

De todos modos, para el caso específico, es mejor usar sentencias preparadas y no un parámetro de cadena de este modo. Pero sin saber cómo es el resto del proceso, no estoy muy seguro de lo que conviene hacer.

[socialk]

Muchas gracias por responder tan pronto, justamente como dices, es lo que sucede, pero el requerimiento era así...

lo resolví de la siguiente manera,

hice un select con concat dentro del procedimiento, lo que me devuelve lo inserté a una tabla temporal, y cree un cursor en base a esa tabla temporal, tuve que agregar un paso más.

Código MySQL:

DECLARE cTag CURSOR FOR SELECT id FROM tbTransactionsTemp;
DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
 

y dentro del procedimiento...

Código MySQL:

DROP TEMPORARY TABLE IF EXISTS tbTransactionsTemp;
SET @qt = CONCAT("CREATE TEMPORARY TABLE tbTransactionsTemp SELECT max(pkIntradayTransaction) FROM tbIntradayTransactions WHERE fkInstrument IN (SELECT pkInstrument FROM tbInstruments WHERE fkInstrument IN (", idInstrument ,")) AND datetimeLastPrice <= DATE_SUB(NOW(),INTERVAL 15 MINUTE) GROUP BY fkInstrument;");
 
PREPARE stmt FROM @qt;
EXECUTE stmt;
DEALLOCATE PREPARE stmt;
 
OPEN cTag;
 
    REPEAT 
        FETCH cTag INTO _pkIntradayTransaction;
        IF NOT done THEN
           //lo que necesitaba hacer
        END IF;
        UNTIL done
    END REPEAT;
    CLOSE cTag;
 

no sé si sea la mejor forma para lograrlo, pero por ahora hace lo que necesitaba.

una vez más... muchas gracias...

[gnzsoloyo]

Es la alternativa que te mencionaba sobre las sentencias preparadas.
Nunca la había probado en ese contexto, pero si te funciona, adelante.
En cuanto a si es la mejor forma, sin conocer más sobre el requerimiento y el diseño de los datos, es difícil aconsejarte.