SQL Server 2008 déclencheur ne fonctionne pas correctement avec plusieurs insertions

Question

J'ai le déclencheur suivant;

CREATE TRIGGER trFLightAndDestination 
ON checkin_flight 
AFTER INSERT,UPDATE 
AS 
BEGIN 
    IF NOT EXISTS 
    (
        SELECT 1 
      FROM Flight v 
      INNER JOIN Inserted AS i ON i.flightnumber = v.flightnumber 
      INNER JOIN checkin_destination AS ib ON ib.airport = v.airport 
      INNER JOIN checkin_company AS im ON im.company = v.company 
      WHERE i.desk = ib.desk AND i.desk = im.desk 
    ) 
    BEGIN 
     RAISERROR('This combination of of flight and check-in desk is not possible',16,1) 
     ROLLBACK TRAN  
    END 
END 

Ce que je veux que le déclencheur à faire est de vérifier les tables de vol, checkin_destination et checkin_company quand un nouveau record pour checkin_flight est ajouté. Chaque enregistrement de checkin_flight contient un numéro de vol et un numéro de bureau où les passagers doivent s'enregistrer pour cette destination. Les tables checkin_destination et checkin_company contiennent des informations sur les entreprises et les destinations réservées à certains bureaux d'enregistrement. Lors de l'ajout d'un enregistrement à checkin_flight j'ai besoin d'informations de la table de vol pour obtenir la destination et flightcompany avec le numéro de vol inséré. Cette information doit être vérifiée par rapport aux combinaisons d'enregistrement disponibles pour les vols, les destinations et les compagnies.

J'utilise le déclencheur comme indiqué ci-dessus, mais lorsque j'essaie d'insérer une mauvaise combinaison, la gâchette le permet. Qu'est-ce que j'oublie ici?

EDIT 1: J'utilise l'instruction d'insertion multiple suivant

INSERT INTO checkin_flight VALUES (5315,3),(5316,3),(5316,2) 
//5315 is the flightnumber, 3 is the desknumber to checkin for that flight 

EDIT 2: Testés une seule insertion de la ligne qui est impossible, l'erreur est lancée correcte. Donc, c'est l'insertion multiple qui semble donner le problème.

Answer 1

Le problème est que votre logique permet à tout insert incluant au moins un jeu de valeurs valide. Il échouera uniquement si tous les enregistrements des enregistrements insérés ne sont pas valides, au lieu de si des enregistrements insérés ne sont pas valides.

Changez votre "SI N'EXISTE PAS (...)" en une instruction "SI EXISTE (...)" et changez votre instruction SELECT pour retourner les vols invalides.

par exemple:

IF EXISTS 
(
       SELECT 1 
     FROM Flight v 
     INNER JOIN Inserted AS i ON i.flightnumber = v.flightnumber 
     LEFT JOIN checkin_destination AS ib ON ib.airport = v.airport 
      AND i.desk = ib.desk 
     LEFT JOIN checkin_company AS im ON im.company = v.company 
      AND i.desk = im.desk 
     WHERE (im.desk IS NULL OR ib.desk IS NULL) 
) 
BEGIN 
    RAISERROR('This combination of of flight and check-in desk is not possible',16,1) 
    ROLLBACK TRAN  
END 

Answer 2

Je ne suis pas sûr de la logique de votre entreprise, mais vous devez vérifier que la requête fait la bonne chose.

Votre problème est le IF NOT EXISTS, si la condition est vraie pour 1 des 3 lignes dans INSERTED il n'existe pas. Vous devez le convertir pour trouver une ligne de problèmes et utiliser IF EXISTS puis une erreur.

Toutefois, lorsque dans un déclencheur la meilleure façon d'erreur out est:

RAISERROR() 
ROLLBACK TRANSACTION 
RETURN 

Je doute genre de que l'absence d'un RETURN est votre problème, mais il est toujours préférable d'inclure les trois R s en cas d'erreur dans un déclencheur.

Answer 3

Le tableau inséré peut contenir plusieurs lignes et donc toute logique dans un déclencheur DOIT être en mesure d'appliquer à toutes les lignes. Les déclencheurs d'idée doivent tirer une fois par effet de ligne est un déclencheur de WRT commun malentendu. SQL Server aura tendance à fusionner les appels à un déclencheur pour augmenter les performances lorsqu'ils se produisent dans la même transaction.

Pour corriger, vous pourriez commencer avec un compteur () de inséré et de comparer cela avec un compteur () des conditions correspondantes et d'élever une erreur en cas de non-concordance.

Answer 4

Le problème est que la condition sera vraie si un seul des enregistrements insérés est correct. Vous devez vérifier que tous les documents sont corrects, .: par exemple

if (
    (
    select count(*) from inserted 
) = (
    select count(*) from flight v 
    inner join inserted i ... 
) 
) ...