триггер на обновление

[don_zaresh]

Здравствуйте!
Помогите, пожалуйста, с триггером.

Есть две таблицы: product - в ней содержатся данные о товаре и category - данные о категориях товаров

В таблице product есть два поля: cost_price - себестоимость и retail_price - цена продажи, которая рассчитывается как произведение: cost_price * category_mark_up (наценка из таблицы category).

Код триггера на вставку отрабатывает нормально, retail_price считается верно. Вот сам код:

CREATE TRIGGER i_trigger_retail_price
    ON product
    AFTER INSERT
AS
BEGIN
    DECLARE @rt INT
    DECLARE @cp INT
    DECLARE @cmu FLOAT
 
SELECT @cp = cost_price
FROM product
WHERE id_product = (SELECT id_product FROM inserted)
 
SELECT @cmu = category_mark_up
FROM category 
WHERE id_category = (SELECT id_category FROM inserted)
 
SET @rt = @cp*@cmu
 
UPDATE product SET retail_price = @rt WHERE id_product = (SELECT id_product FROM inserted)
 
END

Проблема с триггером на обновление. Я думал, что по сути он работает также, то есть код практически тот же:

CREATE TRIGGER u_trigger_retail_price
    ON product
    AFTER UPDATE
AS
BEGIN
    DECLARE @rt INT
    DECLARE @cp INT
    DECLARE @cmu FLOAT
 
SELECT @cp = cost_price
FROM product
WHERE id_product = (SELECT id_product FROM inserted)
 
SELECT @cmu = category_mark_up
FROM category 
WHERE id_category = (SELECT id_category WHERE category.id_category = (product.id_category FROM inserted))
 
SET @rt = @cp*@cmu
 
UPDATE product SET retail_price = @rt WHERE id_product = (SELECT id_product FROM inserted)
 
END

На деле выходит, что retail_price реагирует на изменение cost_price. Но совсем не реагирует на изменение category_mark_up.

Подскажите, пожалуйста, что надо подправить в коде.

Пробовал с реализацией через старые и новые значения (inserted / deleted), но совсем не пошло

[Пепел Феникса]

а почему триггер на таблице product должен реагировать на изменение в таблице category?

[don_zaresh]

Ну с триггером на вставку мне понятно. Я объявил две переменные, присвоил им значения из двух таблиц, перемножил.

А вот с триггером на обновление не понимаю, как сделать. Хотя логика вроде та же. Если меняется наценка category_mark_up для данного id_product, то цена retail_price должна пересчитываться.

Я уже понял, что в моем триггере обновление срабатывает лишь в том, случае, когда меняется cost_price.

Что делать? Соединять таблицы через JOIN?

А ведь одна из переменных может и не меняться. Тогда как? Проверить сначала, изменилось ли поле?

[Пепел Феникса]

наценка на категории обновляется в другой таблице.
вы обратите внимание на какую таблицу триггер то висит.

Что делать? Соединять таблицы через JOIN?

JOIN это для селектов, на триггеры это не влияет.
вешать триггер на обновление в таблицу category, и там(если изменилось) пересчитывать все продукты этой категории(один апдейт всего)

А ведь одна из переменных может и не меняться. Тогда как? Проверить сначала, изменилось ли поле?

обязательно.

[don_zaresh]

Вашу мысль то я понял. Но допустим я создам триггер на обновление наценки category_mark_up и навешу его на таблицу category.

Но потом же как-то надо увязать новые значения? Формула расчета у меня в первом триггере. То есть все равно придется объявлять в первом триггере переменную, принимающую значение наценки.

Вот этот момент я никак не могу сообразить. Может, подскажете типовое решение для такого случая?


Я правильно понимаю, что:
1) пишем второй триггер на обновление наценки и вешаем его на таблицу category
2) код первого триггера по сути не меняется, то есть AS INSERT, UPDATE

[don_zaresh]

Не увидел ваше отредактированное сообщение.

А как может выглядеть код - условие на проверку изменения поля category_mark_up?

[don_zaresh]

Пепел Феникса, дайте, пожалуйста, ссылку на пример, как это делается. Или поделитесь кодом. Честное слово, уже мыслей нет, как это надо реализовать )

Я уже и через представления пробовал, такой вариант, наверно, тоже подойдет. Но хочется разобраться с триггерами

[Аватар]

Примерно так, писано на коленке

//Тут еще будет в пустую вызов триггера на обновление
CREATE TRIGGER product_insert ON product AFTER INSERT AS
BEGIN
  UPDATE product
    SET retail_price = I.cost_price*C.category_mark_up
    FROM product P,inserted I,category C
    WHERE P.id_product=I.id_product AND
          C.id_category=I.id_category
END
GO

//Тут рекурсии не будет, если только не установлен RECURSIVE_TRIGGERS
CREATE TRIGGER product_update ON product AFTER UPDATE AS
BEGIN
  UPDATE product
    SET retail_price = I.cost_price*C.category_mark_up
    FROM product P,inserted I,category C,deleted D
    WHERE P.id_product=I.id_product AND
          C.id_category=I.id_category AND
          I.id_product=D.id_product AND
          I.cost_price<>D.cost_price
END
GO

//Тут еще будет в пустую вызов триггера на обновление product
CREATE TRIGGER category_update ON category AFTER UPDATE AS
BEGIN
  UPDATE product
    SET retail_price = P.cost_price*I.category_mark_up
    FROM product P,inserted I,deleted D
    WHERE P.id_category=I.id_category AND
          I.id_category=D.id_category AND
          I.category_mark_up<>D.category_mark_up
END
GO

[don_zaresh]

Аватар, спасибо! Сейчас буду разбираться. А что значит, "не будет рекурсии"?

[Аватар]

что значит, "не будет рекурсии

внутри триггера обновление таблицы, для которой есть триггер на обновление. В том случае он не будет вызываться