3

Hola quiero hacer un trigger que cuando el stock de mi inventario se menor o igual a 0 ya no me permita vender mas producto pero me los sigue restando igualmente

create trigger NoProductos0a on InventarioProducto for update 
as 
declare @cantidad int
select @cantidad = count(InventarioProducto.Cantidad) from InventarioProducto
if (@cantidad<0)
begin
print 'No se puede realizar la venta, supera la cantidad existente del producto.' 
end

La estructura de la tabla es la siguiente

InventarioProducto
IdProducto int identity(1,1)
Nom_Producto varchar(50)
Precio_Uni decimal(18,2)
Cantidad int

El valor de Cantidad es 5 y trato que al ejecutar este comando no me de resultado -1

update InventarioProducto set Cantidad-=6 where IdProducto=1 
3
  • 1
    Seria bueno que pusieras la estructura de tu tabla y el codigo del update que estas tratando de hacer. Creo que lo que intentas se podria hacer con un INSTEAD OF trigger, pero no estoy seguro que esa sea la mejor forma. Talvez seria mejor verificar la cantidad antes de hacer el update, o crear un CHECK constraint que no permita que la cantidad sea menor que 0
    – alanfcm
    Commented el 10 may. 2018 a las 18:09
  • Ya puse la estructura de la tabla Commented el 10 may. 2018 a las 18:39
  • Hola llegaste a resolver ese problema si es asi podrias ayudarme te paso mi fb: facebook.com/fernando.mimbelaaranda Commented el 15 oct. 2018 a las 5:55

2 respuestas 2

2

Algunas observaciones

  • Para efectivamente detener la actualización, no alcanza con hacer un print debes "emitir" un error. Esto se logra con RAISEERROR, en tu caso deberías el print por: RAISERROR ('No se puede realizar la venta, supera la cantidad existente del producto.' ,10,1), pero si además, tu aplicación no esta controlando el error y haciendo un ROLLBACK explícito habría que agregar un ROLLBACK TRANSACTION.

  • No entiendo muy bien está lógica: select @cantidad = count(InventarioProducto.Cantidad) from InventarioProducto, ¿Por que el COUNT?¿Estás actualizando uno o más registros?, Te recomiendo además ir directamente a ver el valor en INSERTED, esta es una vista que se tiene en los triggers y que contiene los registros luego de la actualización, por lo que es mejor hacer SELECT @cantidad = count(Cantidad) FROM INSERTED o SELECT @cantidad = Cantidad FROM INSERTED, este último si solo estás actualizando una fila.

  • Por último, no está de más recordar, que los triggers en SQLServer son por lote, es decir se ejecutan una única vez independientemente si tu UPDATE es de un registro o de varios, esto te lo comento por que el trigger pareciera asumir que solo actualizarán una única fila. Tenlo en cuenta, por que la lógica actual no parece estar preparada para un UPDATE masivo de la tabla, que siempre podremos hacer por otros medios más allá de la aplicación, si no se quiere contemplar esto, lo que se suele hacer es verificar cuantas filas se están actualizando y emitir otro error si son más de las que la lógica del trigger está en condiciones de manejar.

  • Y ahora sí, lo último de lo último, ¿sabes que está verificación que parece bien sencilla: que una determinada columna no tenga un valor menor a 0, podrías eventualmente hacerla por medio de un CHECK, sin necesidad de programar un trigger?

Edita tu trigger con el siguiente código:

IF (SELECT Cantidad FROM INSERTED) < 0 BEGIN
    RAISERROR ('No se puede realizar la venta, supera la cantidad existente del producto.',10,1)
    ROLLBACK TRANSACTION
END
10
  • Si solo quiero que se actualice una fila y ya quite el COUNT pero igual me los resta, me sale el mensaje de error pero siempre me lo resta Commented el 10 may. 2018 a las 17:47
  • @EduardoNoyola, si te está mostrando el mensaje de error del RAISEERROR (no el del print), la transacción debiera volver para atrás, es decir no puede estar haciendote el UPDATE de InventarioProducto . Agrega a tu pregunta la sentencia de UPDATE que estás realizando. Commented el 10 may. 2018 a las 17:54
  • @EduardoNoyola, te pasé en limpio el código del trigger, pruebalo y cualquier cosa me dices Commented el 10 may. 2018 a las 17:59
  • Asi tengo el codigo ahorita create trigger NoProductos0t on InventarioProducto for update as IF (SELECT Cantidad FROM INSERTED) < 0 BEGIN RAISERROR ('No se puede realizar la venta, supera la cantidad existente del producto.',10,1) END Commented el 10 may. 2018 a las 18:45
  • Y ejecuto esto: update InventarioProducto set Cantidad-=6 where IdProducto=1 Commented el 10 may. 2018 a las 18:48
-1

Es que a menos que le mandes un número negativo te va a seguir restando por que le dices que si cantidad es menor a 0 te mande el mensaje, haz esto:

create trigger NoProductos0a on InventarioProducto for update 
as 
declare @cantidad int
select @cantidad = count(InventarioProducto.Cantidad) from InventarioProducto
if (@cantidad<=0)
begin
print 'No se puede realizar la venta, supera la cantidad existente del producto.' 
end

Cambia el if por un menor o igual a 0

7
  • Me lo sigue restando Commented el 10 may. 2018 a las 17:28
  • Que te da de resultado en todo caso el @cantidad? Commented el 10 may. 2018 a las 17:29
  • En esa tabla llamada InventarioProducto tengo un campo Cantidad que es el stock con valor = 5 Y cuando ejecuto el comando update InventarioProducto set Cantidad-=6 where IdProducto=1 me lo resta y me aparece como -1 Commented el 10 may. 2018 a las 17:36
  • Creo que ya vi el detalle, para que haces el count? se supone que el campo InventarioProducto.Cantidad ya trae el valor de tu stock no? si es así quitale el count Commented el 10 may. 2018 a las 17:39
  • Ya quite el count y me imprime el error pero siempre me lo sigue restando Commented el 10 may. 2018 a las 17:49

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.