Въз основа на първите две изисквания няма нищо лошо във вашия тригер сам по себе си, но можете значително да го опростите:
CREATE OR REPLACE FUNCTION timelog() RETURNS trigger AS $BODY$
DECLARE
t_ix real;
BEGIN
-- First check if you need to change NEW at all
IF (NEW.time_type = 'Start') OR (NEW.time_type = 'Lap') THEN
-- Now perform the expensive lookup for either of 'Start' or 'Lap'
SELECT time_index INTO t_ix
FROM table_ebscb_spa_log04
WHERE fn_name = NEW.fn_name
AND (time_type = 'Start' OR time_type = 'Lap')
ORDER BY stmtserial DESC LIMIT 1;
IF NOT FOUND THEN
-- Nothing found, so NEW.time_index := 1
NEW.time_index := 1;
ELSIF NEW.time_type = 'Start' THEN
-- Start new index for fn_name, discard any fractional part, then increment
NEW.time_index := floor(t_ix) + 1;
ELSE
-- Continue the lap, increment NEW.time_index
NEW.time_index := t_ix + 0.1;
END IF;
END IF;
RETURN NEW;
END; $BODY$ LANGUAGE plpgsql;
Има обаче много по-лесен начин, който също така ще изпълни третото изискване без проблем. Вместо да гледате стойностите на „time_index“, трябва да погледнете стойността на „time“, защото това е, на което се основава „time_index“:
CREATE OR REPLACE FUNCTION timelog() RETURNS trigger AS $BODY$
DECLARE
t_ix real;
BEGIN
-- Find the most recent entry for the same "fn_name" as the new record
SELECT time_index INTO t_ix
FROM table_ebscb_spa_log04
WHERE fn_name = NEW.fn_name
ORDER BY time DESC LIMIT 1;
-- Nothing found, so NEW.time_index := 1
IF NOT FOUND THEN
NEW.time_index := 1;
RETURN NEW;
END IF;
-- Some record exists, so update "time_index" based on previous record
CASE NEW.time_type
WHEN 'Start' THEN
-- Start new index for fn_name, discard any fractional part, then increment
NEW.time_index := floor(t_ix) + 1;
WHEN 'Lap' THEN
-- Continue the lap, increment NEW.time_index
NEW.time_index := t_ix + 0.1;
ELSE
-- Break, find previous break or start, increment by 0.1
SELECT time_index + 0.1 INTO NEW.time_index
FROM table_ebscb_spa_log04
WHERE fn_name = NEW.fn_name
AND (time_type = 'Start' OR time_type = 'Break')
ORDER BY time DESC LIMIT 1;
END CASE;
RETURN NEW;
END; $BODY$ LANGUAGE plpgsql;
Това прилага вашата логика, но имайте предвид, че има някои потенциални клопки:
- Ами ако вмъкнете „Lap“ или „Break“ преди „Start“?
- Ами ако имате повече от 9 събития „fn_name“ след „Начало“ (дробната част „time_index“ ще се прехвърли към следващото цяло число)?
Можете, разбира се, да забравите за полето "time_index" и тригера изобщо и да ги генерирате в движение в изглед, ако вашият модел на данни го позволява (същото с "time_elapse").