22 respuestas en este tema

[eduarcol]

tengo la siguiente instruccion para la  actualización de la base de datos

sql

UPDATE productos SET PrecioSinIva1 = (PrecioVenta1) / (1 + (AlicuotaVenta / 100)),
PrecioSinIva2 = (PrecioVenta2 / (1 + (AlicuotaVenta / 100)), 
PrecioSinIva3 = (PrecioVenta3) / (1 + (AlicuotaVenta / 100)), 
PrecioSinIva4 = (PrecioVenta4) / (1 + (AlicuotaVenta / 100))

el asunto es que con unos 500 o 600 registros la consulta se torna insoportablemente lenta.... 

[Delphius]

Hola eduardo,
¿Desde donde estás lanzando la instrucción SQL? ¿Desde tu aplicación, o de forma interna a la DB como ser desde un trigger o SP?
¿Indices? ¿Haces otros cálculos, cuantos y que tan complicados son?
¿Que tienes en la condición WHERE?

Disculpa mi bombardeo de preguntas, nomás quiero asegurarme de por donde puede venir la lentitud.

Saludos,

[eduarcol]

Aunque sea dificil tratare de contestarte en secuencia

Hola eduardo,

Hola Delphius :D

¿Desde donde estás lanzando la instrucción SQL? ¿Desde tu aplicación, o de forma interna a la DB como ser desde un trigger o SP?

Desde la aplicacion, en un TZQuery con la propiedad requestlive en true y el TZconnection la propiedad autocommit en true

¿Indices?

la clave primaria, el codigo del producto

¿Haces otros cálculos, cuantos y que tan complicados son?

nop, es una instruccion sencilla y no se dispara ningun trigger

¿Que tienes en la condición WHERE?

nada, tengo que actualizarlos todos, pero solo son unos 500 o 600

Disculpa mi bombardeo de preguntas, nomás quiero asegurarme de por donde puede venir la lentitud.

Saludos,

yo tambien tengo ganas de saber lo mismo :D

[Delphius]

Pues si que es raro.

Hice una prueba, desde el IbExpert (es el que tengo abierto y para hacer una prueba rápida) contra una tabla que tiene casi el doble de registros, con unas operaciones más sencillas que tu (no da para muchas operaciones los campos que tengo) y el resultado es inmediato. E intuyo que si lo hago desde una aplicación de prueba el resultado será bastante similar.

¿Cuantos campos debe actualizarse? ¿Hasta cuantos "PrecioSinIVA" tienes?

No manejo Zeos amigo... en algunos lados se dice que en ocasiones Zeos es lento en Firebird, yo no puedo asegurar el rumor porque no los probé. ¿No habrá algo entre las propiedades que afecte a la velocidad?

¿No habrá otras tareas ejecutándose como otras consultas, transacciones, etc?

Saludos,

[egostar]

Hola.

Genera esa misma instrucción desde IBX para verificar si el problema es realmente Zeos, si es la misma lentitud ya podremos ver por otro lado... por ejemplo, esta tu base en red?????

Salud OS

[eduarcol]

esta en local, voy a probar en ibexpert a ver...

[egostar]

esta en local, voy a probar en ibexpert a ver...

Mas bien quise decir con el componente IBX de Delphi

Salud OS

[eduarcol]

ya se por donde viene la cosa, y no es en esa instruccion, es en esta:

sql

UPDATE productos SET PorcUtilidad1 = ((PrecioSinIva1 - CostoUnitario) / PrecioSinIva1) * 100, 
PorcUtilidad2 = ((PrecioSinIva2 - CostoUnitario) / PrecioSinIva2) * 100, 
PorcUtilidad3 = ((PrecioSinIva3 - CostoUnitario) / PrecioSinIva3) * 100, 
PorcUtilidad4 = ((PrecioSinIva4 - CostoUnitario) / PrecioSinIva4) * 100

Cuando uno de los precios esta en 0, me da error de division por zero, pero no se lanza la excepcion.

hay una forma de colocar un condicional, que si uno de los precios es cero no se actualize ese precio en especifico pero el resto si...


Ahora lo que me extraña es que la primera instruccion a pesar de haber precios en cero no salte ese error.

[egostar]

Hola

Puedes usar la función Coalesce(Campo,valor_en_error)

sql

UPDATE productos SET PorcUtilidad1 = ((PrecioSinIva1 - CostoUnitario) / COALESCE(PrecioSinIva1,1) ) * 100, 
PorcUtilidad2 = ((PrecioSinIva2 - CostoUnitario) / COALESCE(PrecioSinIva2,1) ) * 100, 
PorcUtilidad3 = ((PrecioSinIva3 - CostoUnitario) / COALESCE(PrecioSinIva3,1) ) * 100, 
PorcUtilidad4 = ((PrecioSinIva4 - CostoUnitario) / COALESCE(PrecioSinIva4,1) ) * 100

Nota, le ponemos al valor_en_error "1" para que sea una division entre 1 en caso de error.

Salud OS

[Delphius]

Hola Eduardo,
Ummm ... estoy pensando...
Tal vez empleando IIF, Coalesce o alguna otra... pero creo que eso ya termina complicando más la instrucción SQL.

Yo en todo caso optarí­a por disponer de un SP que lance el UPDATE de los registros, previa consulta del valor de los campos y mediante el condicional hacer una u otra cosa.
Es decir que por cada registro vas viendo si es cero los campos PrecioSinIVA y en base a ello calculas el nuevo valor, en otro caso asumes que es cero.

Luego actualizas dicho campo.

No se, es una idea... a lo mejor hay otras.

Saludos,

[egostar]

Hola Eduardo,
Ummm ... estoy pensando...
Tal vez empleando IIF, Coalesce o alguna otra... pero creo que eso ya termina complicando más la instrucción SQL.

Yo en todo caso optarí­a por disponer de un SP que lance el UPDATE de los registros, previa consulta del valor de los campos y mediante el condicional hacer una u otra cosa.
Es decir que por cada registro vas viendo si es cero los campos PrecioSinIVA y en base a ello calculas el nuevo valor, en otro caso asumes que es cero.

Luego actualizas dicho campo.

No se, es una idea... a lo mejor hay otras.

Saludos,

Hola

Yo creo que no es necesario hacer todo lo que dices amigo Delphius, pienso que esto solo lo quiere hacer una sola vez.

Salud OS

[eduarcol]

Una vez, siempre que Chavez amanezca de ganas de modificar el IVA 


La pega que le veo a tu solucion Delpius es que tendria que hacer el proceso campo por campo, cosa que a mi entender volveria pesada la consulta, pero a fin de cuentas tampoco es que entienda mucho de esto jejeje...


Voy a probar con coalesce, pero creo que es solo para nulos... de todas formas mañana los actualizo.,..

[Delphius]

Por eso digo, que es una sola vez, y se usa Coalesce()

Jejeje.

Pues entonces amigo mejor no intentes mi idea, Porque no es del todo práctica. No he probado el código de Eliseo pero creo que va a andar.
En realidad no es campo a campo, sino registro a registro.

La idea que me imagino es:

sql

CREATE PROCEDURE ActualizarPorcentajeUtilidad
...
AS
DECLARE variable PrecioIVA1 NUMERIC(15,2); -- Poner el tipo indicado
...
DECLARE variable PrecioIVA4 NUMERIC(15,2);
BEGIN
-- ejecutamos la consulta SQL select con los valores...
FOR excecute statement
    'select ID, PrecioSinIVA1, ... PrecioSinIVA4, 
              CostoUnitario' INTO :_ID :PrecioIVA1, ... :PrecioIVA4, :CostoU
do
   -- ¿el precio del iva puede ser cero?
   IF (PrecioIVA1 = 0) OR (PrecioIVA IS NULL) THEN PrecioIVA = 1;
   ...
   IF (PrecioIVA4 = 0) OR (PrecioIVA IS NULL) THEN PrecioIVA = 1;
  
  -- actualizamos registro a registro. ¿deberí­a ir el filtro where?
  UPDATE productos SET 
    PorcUtilidad1 = ((PrecioIVA1 - CostoUnitario) / PrecioIVA1) * 100,
    PorcUtilidad2 = ((PrecioIVA2 - CostoUnitario) / PrecioIVA2) * 100,
    PorcUtilidad3 = ((PrecioIVA3 - CostoUnitario) / PrecioIVA3) * 100,
    PorcUtilidad4 = ((PrecioIVA4 - CostoUnitario) / PrecioIVA4) * 100
    WHERE ID = _ID;
  suspend;
END

No he probado mi SP, lo más probable que tenga alguna fallita

Saludos,

[Delphius]

Eduardo espero que no te moleste, edité tus mensajes para ajustar las consultas. Estaban un tanto horizontal

Saludos,

[eduarcol]

claro que pruebo tu idea amigo, lo anterior no fue una afirmacion, solo una duda que tenia....

[eduarcol]

que les parece esto:

sql

UPDATE productos SET PorcUtilidad1 = ((PrecioSinIva1 - CostoUnitario) / iif(PrecioSinIva1= 0, 1, PrecioSinIva1)) * 100, 
PorcUtilidad2 = ((PrecioSinIva2 - CostoUnitario) / iif(PrecioSinIva2= 0, 1, PrecioSinIva2)) * 100, 
PorcUtilidad3 = ((PrecioSinIva3 - CostoUnitario) / iif(PrecioSinIva3= 0, 1, PrecioSinIva3)) * 100, 
PorcUtilidad4 = ((PrecioSinIva4 - CostoUnitario) / iif(PrecioSinIva4= 0, 1, PrecioSinIva4)) * 100

El codigo de Delphius aun no lo he probado y Coalesce no sirve...

[Rolphy Reyes]

Saludos.

Creo que para obtener el resultado deseado, deberás de usar Coalesce ligado con IIF, así­ Coalesce se lleva los nulos por ceros y IIF les pone uno en caso de ser cero.

[eduarcol]

buena idea Rolphy, no habia pensado en los nulos..

[joseme]

eduarcol, me parece que lo podrí­as poner así­:

set PorcUtilidadX=iif(coalesce(Preciox,0)=0,0,(Preciox-Costox)/Preciox*100)

[eduarcol]

esta buena esa, me ahorraria una ejecucion en caso de que sea cero...