[SQL] Problema creazione trigger

[seya]

Salve a tutti, vi scrivo per chiedere aiuto in merito alla creazione di un trigger in SQL Server. La tabella in questione è la seguente:

CREATE TABLE edizione (
  id_edizione int PRIMARY KEY,
  data_inizio date,
  data_fine date,
  corso nchar[5] REFERENCES corso(codice_corso)
)

Vorrei creare un trigger che al momento dell'insert permetta di inserire una edizione di un corso solo se la sua data di inizio è maggiore della data di fine delle altre edizioni (dello stesso corso) eventualmente già inserite.

Ho provato a scriverlo (allego il codice a fine post) ma per qualche ragione che non riesco a comprendere esso impedisce l'inserimento di qualsiasi nuova edizione, anche se non rientra nei parametri specificati sopra.

Prima prova

CREATE TRIGGER TR_date 
   ON  dbo.edizione 
   AFTER INSERT, UPDATE
AS 
DECLARE @data_inizio date, @data_fine date
SELECT @data_inizio = inserted.data_inizio, @data_fine = edizione.data_fine
FROM edizione INNER JOIN inserted
WHERE edizione.corso = inserted.corso

IF @data_fine >= @data_inizio
BEGIN
    -- SET NOCOUNT ON added to prevent extra result sets from
    -- interfering with SELECT statements.
    SET NOCOUNT ON;

    -- Insert statements for trigger here
   ROLLBACK TRAN 
   PRINT 'Errore: una edizione deve iniziare dopo la conclusione dell altra'
END
GO

Seconda Prova

CREATE TRIGGER [dbo].[TR_date] 
   ON  [dbo].[edizione] 
   AFTER INSERT, UPDATE
AS 
DECLARE @data_inizio date, @data_fine date, @corso nchar(5)
SET @data_inizio = (SELECT inserted.[data_inizio] FROM inserted)
 SET @corso = (SELECT inserted.[corso] FROM inserted)
SET @data_fine = (SELECT edizione.[data_fine] FROM edizione WHERE corso = @corso)

IF @data_fine >= @data_inizio
BEGIN
    -- SET NOCOUNT ON added to prevent extra result sets from
     -- interfering with SELECT statements.
    SET NOCOUNT ON;

    -- Insert statements for trigger here
   ROLLBACK TRAN 
   PRINT 'Errore: una edizione deve iniziare dopo la conclusione dell altra'
 END

Grazie in anticipo per l'aiuto!

[cdimauro]

Non conosco l'engine SQL che stai usando, per cui vado a sensazione:
- non stai usando NOW.data_inizio nel confronto
- devi recuperare MAX(edizione.data_fine) e usarlo nel confronto
- il confronto devi farlo con > anziché >=

[gugoXX]

Ti consiglio di non mettere mai statement COMMIT o ROLLBACK dentro alcun trigger. Il controllo della transazione lascialo fare all'esterno. Ci sono problemi che potresti non vedere, come es. transazioni annidate.

E per questo caso ti consiglio di fare trigger "Instead of", nei quali a seconda della condizione piloterai o meno la vera Insert / Update.

[seya]

Originariamente inviato da gugoXX

Ti consiglio di non mettere mai statement COMMIT o ROLLBACK dentro alcun trigger. Il controllo della transazione lascialo fare all'esterno. Ci sono problemi che potresti non vedere, come es. transazioni annidate.

E per questo caso ti consiglio di fare trigger "Instead of", nei quali a seconda della condizione piloterai o meno la vera Insert / Update.

Grazie per il consiglio. In questo caso però il problema rimane, nel senso che inserirei comunque gli stessi controlli e ho timore che il mio errore sia proprio in quella parte.

[gugoXX]

Originariamente inviato da seya

Grazie per il consiglio. In questo caso però il problema rimane, nel senso che inserirei comunque gli stessi controlli e ho timore che il mio errore sia proprio in quella parte.

Sisi', ma per quelli mi sembrava che il consigli di cdimauro fossero corretti.

[seya]

Ho modificato così il codice:

CREATE TRIGGER [dbo].[TR_date]
   ON  [dbo].[edizione]
    INSTEAD OF INSERT

AS
DECLARE @data_inizio date, @data_fine date, @corso nchar(5)
SET @data_inizio = (SELECT inserted.[data_inizio] FROM inserted)
SET @corso = (SELECT inserted.[corso] FROM inserted)
SET @data_fine = (SELECT MAX(edizione.[data_fine]) FROM edizione WHERE corso = @corso)


IF @data_fine > @data_inizio
BEGIN
    -- SET NOCOUNT ON added to prevent extra result sets from
    -- interfering with SELECT statements.
    SET NOCOUNT ON;

    -- Insert statements for trigger here
    INSERT INTO edizione
    SELECT data_inizio, data_fine, corso
    FROM inserted
   
END

Però così facendo non fa nessun controllo e mi lascia eseguire qualsiasi inserimento quindi evidentemente la mia interpretazione non è molto corretta :x

[gugoXX]

Io per intanto vedo questo

.. che mi permetta di inserire una edizione di un corso solo se la sua data di inizio è maggiore della data di fine delle altre edizioni...

IF @data_fine > @data_inizio

peraltro fai attenzione, perche' insert e anche update lavorano per SET, ovvero per insiemi di record, non record singoli.
"inserted" e' quindi un insieme di record, non uno solo.
Devi quindi operare per SET, facendo magari un loop sui vari dati in ingresso (poi magari toglierai questi loop, dato che e ' possibile torglierli quasi sempre e forse sempre).
e per ciascuno dei record dovrai decidere se inserirlo oppure no.