Stored Procedure

[[gluglu]]

Hola de nuevo !

.... no lo entiendo ....

Intento crear un Stored Procedure complejo, del cual las primeras sentencias son las que pongo a continuación :

Código Delphi [-]
CREATE OR ALTER PROCEDURE NEW_PROCEDURE2 (
    in_random decimal(12,0))
returns (
    out_random decimal(12,0))
as
  declare variable aux_rand decimal(12,0);
  declare variable aux_true integer;
begin
 
  AUX_TRUE  = 0;
 
  while (:"AUX_TRUE" = 0) do begin

    Select RANDOMNO from AVAIL_CALC_TEMP
      where RANDOMNO = :"IN_RANDOM"
      rows 1 into AUX_RAND;

    if (:"AUX_RAND" is null) then begin
      Leave;
    end
    else begin
      IN_RANDOM = :"IN_RANDOM" + 1;
      /* También he probado IN_RANDOM = IN_RANDOM +1; */
      if (:"IN_RANDOM" > 200) then Leave;
    end

  end
 
  OUT_RANDOM = IN_RANDOM;
 
  suspend;
 
end

Se trata de detectar si un valor de entrada ya existe en AVAIL_CALC_TEMP y si existiera, incrementar el valor hasta que obtenga un valor nuevo que no exista. Ese valor se pasa después como resultado de salida.

Supongamos que la tabla AVAIL_CALC_TEMP contiene registros con el campo RANDOMNO = 1. Entonces si llamo al procedimiento con

Código Delphi [-]
Select * from NEW_PROCEDURE2(1)

me debería devolver el valor 2 (asumiendo que 2 no existe en la tabla). Pero resulta que he añadido la sentencia comprobando que el bucle no cree valores mayores de 200 y siempre me devuelve el valor 201.

Si por el contrario ejecuto

Código Delphi [-]
Select * from NEW_PROCEDURE2(2)

me devuelve correctamente el valor 2.

No sé qué estoy haciendo mal !

He probado todo, añadiendo variables propias del procedimiento, etc. Pero no consigo saber por qué no funcionan estas líneas.

Gracias como siempre por vuestra inestimable colaboración.

[Casimiro Notevi]

Para empezar, prueba a quitar las comillas a esas variables que has declarado sin comillas. Sin comillas puedes usarla como quieras, mayúscula o minúscula, pero si lo pones entre comillas entonces la busca exactamente como está puesto entre las comillas, así que supongo que aux_true es una variable distinta a "AUX_TRUE". Al menos, con los campos, es así, por eso nunca uso las comillas.

[[gluglu]]

Gracias Casimiro.

Creo que no tiene nada que ver con las comillas, porque después de tu indicación lo he probado también sin comillas, y sigue igual.

El hecho es que entra en el bucle 'while', pero no sale correctamente de él. Si el Select encuentra un valor (p.ej. el 1), Aux_Rand (mayúscula o minúscula, da igual ahora ...) no es null e intento entonces incrementar el valor de entrada para encontrar uno que no exista.

Al pasar como valor de entrada el 1, me devuelve 201, lo que significa que sin duda alguna entra en el bucle de la suma para finalmente, al ser mayor de 200, devolverme 201.

En cambio si haga la llamada con el valor de entrada 2, entonces me devuelve correctamente 2, ya que el 2 no existe, y entonces Aux_Rand sí que es null y ejecuta el primer 'Leave'.

Por lo tanto tengo que deducir que el error está en el Select y el valor que le pasa 'into Aux_Rand'. No sé por qué rara razón, si en la primera vez que se ejecuta el Select, el valor no existe entonces Aux_Rand es null y sale del While. Si en cambio a la primera el valor existe, y ya ejecuta después el incremento + 1, entonces ya Aux_Rand nunca vuelve a ser null.

[Casimiro Notevi]

¿Puedes poner un metadata simplificado con algunos datos para probar?

Un script con su create table y los mínimos campos necesarios y algunos datos extraidos con ibexpert, por ejemplo, o el que uses.

[[gluglu]]

Voy a cambiar la tabla de nombre para simplicarlo todo. Introducimos un único registro con un único valor = 1. Creamos el Stored Procedure.

Código Delphi [-]
CREATE TABLE PRUEBA(
    RANDOMNO DECIMAL(12,0)
);
 
INSERT INTO PRUEBA (RANDOMNO)
            VALUES (1);
COMMIT WORK;
 
SET TERM ^ ;
CREATE OR ALTER PROCEDURE NEW_PROCEDURE2 (
    in_random decimal(12,0))
returns (
    out_random decimal(12,0))
as
 declare variable aux_rand decimal(12,0);
 declare variable aux_rand2 decimal(12,0);
 declare variable aux_true integer;
begin
 
  AUX_TRUE  = 0;
 
  while (:"AUX_TRUE" = 0) do begin

    Select RANDOMNO from PRUEBA
      where RANDOMNO = :"IN_RANDOM"
      rows 1 into AUX_RAND;

    if (:"AUX_RAND" is null) then begin
      Leave;
    end
    else begin
      IN_RANDOM = IN_RANDOM + 1;
      if (IN_RANDOM > 200) then Leave;
    end

  end
 
  OUT_RANDOM = IN_RANDOM;
 
  suspend;
 
end
^
SET TERM ; ^

Y por último ejecutamos :

Código Delphi [-]
Select * from NEW_PROCEDURE2(1)

Esta sentencia me devuelve el valor 201.

Esta otra

Código Delphi [-]
Select * from NEW_PROCEDURE2(2)

me devuelve el valor 2.

Si pruebo con lo siguiente :

Código Delphi [-]
SET TERM ^ ;
CREATE OR ALTER PROCEDURE NEW_PROCEDURE2 (
    in_random decimal(12,0))
returns (
    out_random decimal(12,0))
as
 declare variable aux_rand decimal(12,0);
 declare variable aux_rand2 decimal(12,0);
 declare variable aux_true integer;
begin
 
  AUX_TRUE  = 0;
 
  while (:"AUX_TRUE" = 0) do begin

    Select RANDOMNO from PRUEBA
      where RANDOMNO = :"IN_RANDOM"
      rows 1 into AUX_RAND;

    if (:"AUX_RAND" is null) then begin
      Leave;
    end
    else begin
      IN_RANDOM = IN_RANDOM + 1;
      if (IN_RANDOM > 200) then Leave;
    end

  end
 
  OUT_RANDOM = AUX_RAND;
 
  suspend;
 
end
^
SET TERM ; ^

para ver qué valor queda en Aux_Rand, entonces

Código Delphi [-]
Select * from NEW_PROCEDURE2(1)

me devuelve el valor 1
y

Código Delphi [-]
Select * from NEW_PROCEDURE2(2)

me devuelve null.

P.D. La versión de Firebird es la 2.1

[[gluglu]]

... parece como si una vez que Aux_Rand no ha tomado el valor null, ya no lo volviera a tomar más en el bucle y en su comprobación correspondiente If (:"Aux_Rand" is null) porque no hace el Leave adecuado, pero en cambio, al comprobar el valor con OUT_RANDOM, entonces sí que aparece el null.

... no lo entiendo ...

[Casimiro Notevi]

Me parece que está claro:

Si lo llamas con el valor 1, que existe, entonces como aux_rand no es null aumenta in_random en 1 hasta que llega a 200+1 y sale, devolviendo 201
Sin embargo, cuando lo llamas con 2, que no existe, no lo encuentra, entonces aux_rand es null y sale con el valor pasado, el 2 en este caso.