Trigger para descontar stock, MSQL

[Eric_Alberto]

Hola muy buenos dias, quería saber si alguien puede ayudarme en este tema ya que soy nuevo en esto de los triggers, lo que quiero que quiero que haga el trigger es que me descuente el stock del producto cuando alguien pide cierta cantidad de dicho producto mi base de datos es la siguiente
-articulos-
fecha_entrada
fecha_salida
id_detalle
id_producto
marca
modelo
nombre_produc
no_requisicion
stock

-pedido-
area
empleado
id_detalle
id_pedido
modelo
producto
solicitud
status
stock

y estas 2 tablas estan relacionadas con

-detalle-
descripcion
id_detalle

el trigger que realize es el siguiente, no se si este bien estructurado, o si me falta por agregarle mas campos

Código MySQL:

delimiter //
CREATE TRIGGER destock AFTER INSERT ON articulos
FOR EACH
ROW
BEGIN
    UPDATE articulos 
    SET stock = stock - stock
    WHERE id_detalle = id_detalle;
END
//
delimiter ;
 

espero puedan ayudarme, de antemano gracias

[gnzsoloyo]

El trigger está mal escrito por varias razones:
1) MySQL no permite que un trigger realice operaciones INSERT/UPDATE/DELETE sobre la misma tabla que está ejecutando el trigger.
2) No debes invocar campos directamente, ni valores, sin usar las pseudovariables NEW y OLD. Por favor, lee el Manual de Referencia para entender como se trabaja con TRIGGERs.
3) Si la operación es un INSERT en la tabla Articulo, es porque el stock no existe. En consecuencia hacer un update del stock no tiene sentido.

[Eric_Alberto]

Iniciado por gnzsoloyo

El trigger está mal escrito por varias razones:
1) MySQL no permite que un trigger realice operaciones INSERT/UPDATE/DELETE sobre la misma tabla que está ejecutando el trigger.
2) No debes invocar campos directamente, ni valores, sin usar las pseudovariables NEW y OLD. Por favor, lee el [URL="https://dev.mysql.com/doc/refman/5.5/en/create-trigger.html"]Manual de Referencia[/URL] para entender como se trabaja con TRIGGERs.
3) Si la operación es un INSERT en la tabla Articulo, es porque el stock no existe. En consecuencia hacer un update del stock no tiene sentido.

ooh ok, podrías ayudarme con código de como es la estructura correcta del trigger para realizar la acción que quiero que haga por favor? si no es mucha molestia claro

[gnzsoloyo]

Es que el problema es que todo lo tienes mal diseñado...
Esas tres tablas están directamente mal hechas. No cumplen un esquema maestro-detalle necesario para los pedidos, y la tabla de articulos no tiene que relacionarse con los pedidos, sino al revés.

La verdad es que tienes que descartar todo el modelo y empezarlo de cero.

ARTICULO
id_producto (PK)
marca
modelo
nombre_produc
stock

PEDIDO
id_pedido (PK)
area
empleado
estado
fecha_entrada
fecha_salida

DETALLE_PEDIDO
id_pedido
nro_item
id_producto
cantidad

Este campo no se entiende:

no_requisicion

El resto de los campos no queda claro para qué se usan.

Describe completamente los detalles del sistema en esta parte y veremos qué está faltando.
Recién entonces tendría sentido ponerse a ver un TRIGGER...

[Eric_Alberto]

Iniciado por gnzsoloyo

Es que el problema es que todo lo tienes mal diseñado...
Esas tres tablas están directamente mal hechas. No cumplen un esquema maestro-detalle necesario para los pedidos, y la tabla de articulos no tiene que relacionarse con los pedidos, sino al revés.

La verdad es que tienes que descartar todo el modelo y empezarlo de cero.






Este campo no se entiende:



El resto de los campos no queda claro para qué se usan.

Describe completamente los detalles del sistema en esta parte y veremos qué está faltando.
Recién entonces tendría sentido ponerse a ver un TRIGGER...

en mi tabla pedido vienen los campos stock que es la cantidad que el usuario quiere pedir, nombre del producto y la marca para saber que producto, cantidad y marca para saber que esta pidiendo el empleado. y el campo que comentas que no se entiende el que esta en articulos que es "no_requisicion" es para saber que cantidad se compraron y con el "stock" que esta en la misma es para que valla reduciendo cuando el empleado realize un pedido, el campo que agregaste a la tabla detalle "nro_item" que funcion tendria?

[Eric_Alberto]

Iniciado por gnzsoloyo

Es que el problema es que todo lo tienes mal diseñado...
Esas tres tablas están directamente mal hechas. No cumplen un esquema maestro-detalle necesario para los pedidos, y la tabla de articulos no tiene que relacionarse con los pedidos, sino al revés.

La verdad es que tienes que descartar todo el modelo y empezarlo de cero.






Este campo no se entiende:



El resto de los campos no queda claro para qué se usan.

Describe completamente los detalles del sistema en esta parte y veremos qué está faltando.
Recién entonces tendría sentido ponerse a ver un TRIGGER...

ya hize la BD como me comentaste, pero agrege unos campos que creo yo faltan asi quedo

-articulo-
id_producto PK
marca
modelo
nombre
stock
no_requisicion <-este campo es para saber que cantidad se compro de dicho producto-

-pedidos-
id_pedido PK
area
empleado
estado
fecha_entrada
fecha_salida
stock <-----
producto estos campos los agrege para saber que es lo que pide y cuanto
marca ------>

-detalle-
id_detalle
nro_item <-no se que función tiene, me explicarías
id_producto
cantidad

no se si los campos que agregue a las tablas que yo tenia en la BD anterior afecte al menos los de la tabla "pedidos"

[gnzsoloyo]

-articulo-
id_producto PK
marca
modelo
nombre
stock
no_requisicion <-este campo es para saber que cantidad se compro de dicho producto-

El campo que llamas "no_requisicion" no pertenece a esta entidad. Debería pertenecer a una entidad diferente que represente las compras a los proveedores, y solo tendría impacto SUMANDO STOCK. Por su lado, el stock preexistente se establece como la suma del actual más las ventas realizadas.
Es decir, ese valor se obtiene ñor consulta. No se deben usar campos calculables para almacenar datos.

-pedidos-
id_pedido PK
area
empleado
estado
fecha_entrada
fecha_salida
stock <-----
producto estos campos los agrege para saber que es lo que pide y cuanto
marca ------>

Incorrecto. Totalmente.
El "stock" no es tal sino la cantidad de unidades en un mismo pedido de un mismo producto, y eso es un dato que pertenece al DETALLE_PEDIDO, donde se listan todos los items de un mismo pedido.

Por su lado, la MARCA, no debe ir en el pedido, y no debe ir en ninguna de las tablas del pedido. La "marca" es un atributo propio del PRODUCTO, y debe haber una entrada de producto por cada marca, modelo y presentación (volumen, peso, etc.).
La relación entre el pedido y el producto y su marca es transitiva, a traves del id_producto que aparece en el item del detalle.

-detalle-
id_detalle
nro_item <-no se que función tiene, me explicarías
id_producto
cantidad

Que no entiendas eso resulta bastante extraño...

¿Nunca has comprado algo en un supermercado, o varios artículos en algún comercio?
¿Viste que la factura tiene de 1 a N items, cada uno de los cuales es un producto que compras?
Bueno, en numero de ítem es eso: Cada uno de los items que entraron en la lista, numerados del 1 a N, siendo N el máximo numero posible de items facturados en una sola compra...
En el caso de las tablas de detalle, la PK está formada por el numero de su entidad padre (Pedido en este caso), y el numero del ítem.
¿Se entiende?

En este contexto, cuando haces un ingreso de pedido entrgado, el TRIGGER (que debe estar en la tabla DETALLE_PEDIDO) realiza la correspondiente resta de stock en la tabla Producto.
Seria un trigger terriblemente simple, aunque en el mundo real NO SE USAN para estos fines. Las restas de stock se validan de otra forma...