Foros Club Delphi

Foros Club Delphi (https://www.clubdelphi.com/foros/index.php)
-   SQL (https://www.clubdelphi.com/foros/forumdisplay.php?f=6)
-   -   Optimizacion De Rendimiento (https://www.clubdelphi.com/foros/showthread.php?t=75368)

fjcg02 21-12-2011 19:07:19

Hola,
sería interesante que pusieras todo el SP o la vista. Además, la estructura de las tablas y los índices.

He estado revisando, y ahora no se si utilizas la condición "Like :CALLED"

Si siempre miras por :CALLED+'XXX' o CALLED+'%', te convendría añadir un campo con esa información para evitar utilizar like, que es terriblemente ineficiente. Si no puedes, sería mejor utilizar left(PLAT.DESTINO,LONGITUDCALCULADA) en la condición que el like.

Ya nos contarás.

Saludos

juanlito 22-12-2011 10:13:39

De primera voy a poner la vista, este codigo solo como vista me funciona bien si meto un valor que pertenece a la tabla lo muestra una sola vez y si meto uno que no pues no lo muestra.

Código SQL [-]
CREATE OR ALTER VIEW VISUALIZA(
    CALLINGNUMBER)
AS
SELECT FIRST 1 (CALLINGNUMBER) FROM GUAN WHERE ((CALLINGNUMBER) = '956303030')
;

Ahora os pongo el codigo del SP

Código SQL [-]
 FOR SELECT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
 WHERE (PLAT.FECHA_HORA_INICIO > :F_INICIO) AND (PLAT.FECHA_HORA_INICIO < :F_FIN)
 INTO :LINEA, :DESTINO, :HORA_LLAMADA    DO
 BEGIN
  SELECT FIRST 1 (GUAN.CALLINGNUMBER) FROM GUAN WHERE ((GUAN.CALLINGNUMBER) = :LINEA)
  INTO :VECE;
   if (:VECE IS NOT NULL ) then
/*   SELECT count (GUAN.CALLINGNUMBER) FROM GUAN WHERE (GUAN.CALLINGNUMBER = :LINEA)
  INTO :VECES;
   if (:VECES <> 0 ) then*/
   BEGIN
       lon =  CHAR_LENGTH (:DESTINO);
       DEST = :DESTINO;
       if (:lon > 9) then
       begin
         DEST = left(:DEST,:lon-4);
       end
       DEST = :DEST ||'%';
       SELECT COUNT (GUAN.CALLINGNUMBER) FROM GUAN WHERE (((GUAN.CALLINGNUMBER) = :LINEA) AND ((GUAN.CALLEDNUMBER) like :DEST) AND ((GUAN.CALLTIME) >DateAdd(hour,-2,:HORA_LLAMADA) And (GUAN.CALLTIME)hour,2,:HORA_LLAMADA)))
       INTO :VECES ;
       if (:VECES = 0 ) then
       begin
          suspend;
       end
 END
 END

Los parametros de entrada son F_INICIO y F_FIN que son de tipo TIMESTAMP.

Los parametros de salida son LINEA,DESTINO y HORA_LLAMADA, que son de tipo VARCHAR(10), VARCHAR(20) y TIMESTAMP respectivamente.

Las variables son VECES, LON con tipo INTEGER, y VECE y DEST que son VARCHAR(40) y VARCHAR(20).

Usando el select que aparece entre /* */ el SP me funciona correctamente pero he comprobado que en ese select recorro toda la tabla cuando en realidad necesito solo que me compruebe si el parametro esta, con lo que cuando lo encontrase por primera vez deberia de parar, porque la tabla ira creciendo con miles de filas.


Aqui os pongo la estructura de las tablas, los campos VARCHAR(255) se crearon asi porque la tabla se creo automaticamente asi al importar los datos, no creo que eso me genere el problema porque realmente los datos que gestiono entran en los
VARCHAR(10)..VARCHAR(20).. que yo uso

Código SQL [-]
 
CREATE TABLE GUAN (
    COLSORT              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    PHONEID              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLINGNUMBER        VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLEDNUMBER         VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLTIME             TIMESTAMP,
    CALLDURATION         VARCHAR(255) CHARACTER SET UNICODE_FSS,
);
 

/******************************************************************************/
/*                                  Indices                                   */
/******************************************************************************/
CREATE INDEX GUAN_CALLINGNUMBER_NEW ON GUAN (CALLINGNUMBER);
CREATE INDEX GUAN_CALLTIME_NEW ON GUAN (CALLTIME);
CREATE INDEX GUAN_PHONEID_NEW ON GUAN (PHONEID);


y la de la otra tabla:

Código SQL [-]

CREATE TABLE PLAT (
    ID_LLAMADA           INTEGER NOT NULL,
    LINEA                VARCHAR(255) CHARACTER SET UNICODE_FSS,
    FECHA_HORA_INICIO    TIMESTAMP,
    DESTINO              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    ID_DESTINO           VARCHAR(255) CHARACTER SET UNICODE_FSS,
    DURACION             INTEGER,
);
 

/******************************************************************************/
/*                                Primary Keys                                */
/******************************************************************************/
ALTER TABLE PLAT ADD PRIMARY KEY (ID_LLAMADA);

/******************************************************************************/
/*                                  Indices                                   */
/******************************************************************************/
CREATE INDEX PLAT_LINEA ON PLAT (LINEA);
CREATE INDEX PLAT_TIEMPO ON PLAT (FECHA_HORA_INICIO);

Espero que con esta informacion podamos ver donde esta el problema, porque no entiendo porque en el VIEW me funciona el select y en el SP no.

Un saludo

Gracias

juanlito 22-12-2011 10:14:40

De primera voy a poner la vista, este codigo solo como vista me funciona bien si meto un valor que pertenece a la tabla lo muestra una sola vez y si meto uno que no pues no lo muestra.


Código SQL [-]
CREATE OR ALTER VIEW VISUALIZA(
CALLINGNUMBER)
AS
SELECT FIRST 1 (CALLINGNUMBER) FROM GUAN WHERE ((CALLINGNUMBER) = '956303030')
;




Ahora os pongo el codigo del SP


<DIV class=frame_codigo_sql>Código SQL [-]<DIV id=sql_div_4ef2f4c3b3b6d class=texto_codigo_sql>
FOR SELECT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
WHERE (PLAT.FECHA_HORA_INICIO > :F_INICIO) AND (PLAT.FECHA_HORA_INICIO < :F_FIN)
INTO :LINEA, :DESTINO, :HORA_LLAMADA DO
BEGIN
SELECT FIRST 1 (GUAN.CALLINGNUMBER) FROM GUAN WHERE ((GUAN.CALLINGNUMBER) = :LINEA)
INTO :VECE;
if (:VECE IS NOT NULL ) then
/* SELECT count (GUAN.CALLINGNUMBER) FROM GUAN WHERE (GUAN.CALLINGNUMBER = :LINEA)
INTO :VECES;
if (:VECES <> 0 ) then*/
BEGIN
lon = CHAR_LENGTH (:DESTINO);
DEST = :DESTINO;
if (:lon > 9) then
begin
DEST = left(:DEST,:lon-4);
end
DEST = :DEST ||'%';

SELECT COUNT (GUAN.CALLINGNUMBER) FROM GUAN WHERE (((GUAN.CALLINGNUMBER) = :LINEA) AND ((GUAN.CALLEDNUMBER) like :DEST) AND ((GUAN.CALLTIME) >DateAdd(hour,-2,:HORA_LLAMADA) And (GUAN.CALLTIME)

juanlito 22-12-2011 10:15:55

Se duplico la respuesta, lo siento, no se como puedo eliminar una de ellas.

Un saludo y lo siento.

Casimiro Noteví 22-12-2011 20:21:19

No olvides poner etiquetas al código, ejemplo:



.

fjcg02 23-12-2011 00:03:48

Prueba con esto

Código SQL [-]
 SELECT DISTINCT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
 INNER JOIN GUAN ON (PLAT.LINEA=GUAN.CALLINGNUMBER)
 WHERE 
 (PLAT.FECHA_HORA_INICIO BETWEN :F_INICIO AND :F_FIN) AND
 ( GUAN.CALLTIME BETWEN DATEADD(HOUR,-2,:F_INICIO) AND DATEADD(HOUR,2,:F_FIN)

Te quitas las dos primeras select y una parte de la tercera. Es decir, cruzamos las tablas PLAT y GUAN, con destino iguales, con fecha_hora_inicio entre el rango y con CALLTIME dos horas antes o despues.

Pongo DISTINCT para que sólo me saque un registro por combinación de los tres campos, obviando los duplicados.

Esto tiene que ir rápido, tienes indices en los campos por los que cruzamos.


Ahora nos queda el tema de DESTINO, que si es de longitud mayor que 9 te quedas con el campo menos los 4 ultimos caracteres, y si no, con todos.

Tendrías que probar algo similar a esto
Código SQL [-]
 
 AND (CASE 
         WHEN LENGTH(PLAT.DESTINO) > 9
        THEN
           CASE
             WHEN 
                LEFT(PLAT.DESTINO,LENGTH(PLAT.DESTINO)-4)=LEFT(CALLEDNUMBER,LENGTH(PLAT_DESTINO)-4) THEN
               1
             ELSE
                0
            ENDCASE
        ELSE
           CASE
             WHEN 
               LEFT(PLAT.DESTINO,LENGTH(PLAT.DESTINO))=LEFT(CALLEDNUMBER,LENGTH(PLAT_DESTINO)) THEN
              1
            ELSE
              0
            ENDCASE
      ENDCASE ) = 1

en las condiciones. No se si tragará, pero es cuestion de probarlo. Si no funciona sería cuestión de ponerlo en un campo calculado, y en la where preguntar por ese campo.

La ultima parte es un poco liosa, pero creo que puede entenderse. No estoy seguro de lo que quieres filtrar, si pones un juego de ensayo, podríamos ayudarte a cambiar la consulta para darle otea orientación.

Un saludo


La franja horaria es GMT +2. Ahora son las 14:49:40.

Powered by vBulletin® Version 3.6.8
Copyright ©2000 - 2026, Jelsoft Enterprises Ltd.
Traducción al castellano por el equipo de moderadores del Club Delphi