Triggers

Sådan skal din trigger laves...
Er testet på SQL2000 og virker!!! Både når du ændre en eller MANGE records
/Thomas

CREATE TRIGGER ut_main ON [dbo].[Main]
FOR UPDATE
AS
    SET NOCOUNT on

    Declare @@IDnumber as Integer
    Declare @@ForSale as integer

    IF UPDATE(ForSale)
    IF @@ROWCOUNT = 1
    BEGIN
        SELECT @@IDnumber=IDnumber, @@ForSale=ForSale FROM inserted
        if @@ForSale=0 Delete from ItemsForSale where IDnumber=@@IDnumber
    END ELSE
    BEGIN
        DECLARE ut_main_cursor CURSOR FOR
        SELECT IDnumber, ForSale FROM inserted

        OPEN ut_main_cursor

        -- hent første
        FETCH NEXT FROM ut_main_cursor into @@IDnumber, @@ForSale

        -- Check @@FETCH_STATUS
        WHILE @@FETCH_STATUS = 0
        BEGIN
            if @@ForSale=0 Delete from ItemsForSale where IDnumber=@@IDnumber
            FETCH NEXT FROM ut_main_cursor into @@IDnumber, @@ForSale
        END

        CLOSE ut_main_cursor
        DEALLOCATE ut_main_cursor

    END

uhauha da, sådan noget må man aldrig gøre, en cursor i en trigger... Meget meget skamfuldt!

Det eneste der skal til for opfylde tullas behov er:

CREATE TRIGGER delForSale ON dbo.Main
FOR UPDATE
AS
IF NOT UPDATE(ForSale)
RETURN
DELETE ItemsForSale WHERE id = (SELECT id FROM INSERTED)

---------

tulla, tag det som en fortsættelse af den anden tråd. Jeg vil ikke have point, men lov mig at du aldrig bruger en cursor i en trigger medmindre det er eneste udvej (og det er det meget sjældent).

tulla = hulla ... *LOL*

hmm, hvis den sføli kun skal slette når forsale bliver sat til 0, da:

CREATE TRIGGER delForSale ON dbo.Main
FOR UPDATE
AS
IF NOT UPDATE(ForSale)
RETURN
DECLARE @id INT, @fs INT
SELECT @id = id, @fs = ForSale FROM INSERTED
IF @fs = 0
DELETE ItemsForSale WHERE id = @id

Janus_007 du har helt ret i at det ikke er en god ide at bruge cursor i en trigger, derfor er koden lavet så den checkker @@rowcount og kun benytter cursoren HVIS man fx opdaterer FLERE rækker. Din løsning virker IKKE på flere rækker (fx update forsale=0 where kunde='janus_007' ) men kun på EN, men vi man aldring kommer i den sitiuation er dette jo ikke noget problem!

/Thomas

Det ser rigtigt ud,.. men der er lige et lille problem, mit id er ikke en int, men en VarChar, og det er den ganske enkelt nødt til at være. Jeg bruger så selvfølgelig varchar i triggeren,.meeen det virker ikke, ved en af jer noget om det ?

Har jeg ret i at INSERTED tager fat i en række der lige er sat ind ??

sådan her ser min trigger ud nu:

CREATE TRIGGER delForSale ON dbo.Main
FOR UPDATE
AS
IF UPDATE(ForSale)
DECLARE @id varchar, @fs int
SELECT @id = IDnumber, @fs = ForSale FROM INSERTED
IF @fs = 0
DELETE dbo.ItemsForSale WHERE dbo.ItemsForSale.IDnumber = @id

Men den sletter ingenting, og jeg får ingen fejlmeddelelser af nogen art.

Fjerner jeg WHERE klausulen sletter den det hele som forventet, så det virker som om INSERTED ikke er den rigtige,.eller ???

Du mangler en BEGIN END og så synes jeg det vil være en rigtig god ide af sætte størrelse på din varchar
/Thomas

CREATE TRIGGER delForSale ON dbo.Main
FOR UPDATE
AS
IF UPDATE(ForSale)
BEGIN
  DECLARE @id varchar(10), @fs int
  SELECT @id = IDnumber, @fs = ForSale FROM INSERTED
  IF (@fs = 0) DELETE FROM dbo.ItemsForSale WHERE dbo.ItemsForSale.IDnumber = @id
END

Det virker ikke,...

Når jeg fyrer denne her af i min Enterprise manager:

UPDATE    Main
SET      ForSale = 0

sætter den alle ForSale = 0, men sletter ikke noget i ItemsForSale.
Jeg har også prøvet med curser metoden, men den sletter ingenting

Det er fordi du ikke benytte cursor som i mit eksample
Du skal benytte cursor hvis du ønsker at lave ændringer på mere end en record

Jeg har også prøvet med curser metoden, men den sletter ingenting

Update triggere er i sig selv ikke til at opdatere med, de er kun til at få udført et event som fx bremser updaten eller sørger for at der opdateres i andre tabeller.
Update skal ses som, delete + insert.

Jo så virker det techhouse,.. tak for hjælpen, det blev dit jeg brugte :o)

Jeg gentager lige igen... Lad være med at bruge cursor i en trigger! -

Ved updatering af flere værdier, laves blot en SELECT ala...
DELETE ItemsForSale WHERE id IN (SELECT id FROM Main WHERE ForSale = 0)

eller måske, hvis man hellere er ligeglad med 0 eller 1 så bare:
DELETE ItemsForSale WHERE id IN (SELECT id FROM INSERTED)

techhouse, få styr på din sql.... og undgå at bruge cursor løsninger!!
og forøvrigt techhouse, man behøver ikke en BEGIN END ved 1 linjes statement, en varchar skal altid sættes til en størrelse, så det er ikke bare en god idé, det er et krav!!

Så lærte jeg også noget.
Takker

CREATE TRIGGER ut_main ON [dbo].[Main]
FOR UPDATE
AS
SET NOCOUNT on

    IF UPDATE(ForSale)
    Delete from ItemsForSale where IDnumber in
        (select IDnumber from inserted where ForSale=0)

Efter janus_007 gode indlæg og ide kan triggeren ændres til overnævnte
Jeg har til tilført "where ForSale=0" således at vi ikke får formeget access til SQLen.

/Thomas

jamen så lærte jeg også noget, var ikke klar over man kunne sætte en where-clause på inserted!!

Utroligt hvad man kan udrette i fællesskab :O)

Hygge og god dag/ aften....

jeps og det samme til dig...