else expressionがうまく動作しない場合

Question

@ accid2が@accidと等しくない場合、比較を実行したい場合は、アクションをロールバックし、そうでない場合はinsertを実行します。

このトリガーの結果は、一致していなくても、私のテーブルに挿入されます。ここ

は私のコードです：

ALTER TRIGGER [dbo].[TG_checkacctypehtl] 
ON [dbo].[Accommodation_Hotel] INSTEAD OF INSERT 
AS 
DECLARE @accid NVARCHAR(50), @accid2 NVARCHAR(50),@hid NVARCHAR(50),@fsp NVARCHAR(50), @fc NVARCHAR(50), @sr NVARCHAR(50); 
SELECT @hid = i.hotel_id FROM INSERTED i; 
SELECT @fsp = i.facillities_swimming_pool FROM INSERTED i; 
SELECT @fc = i.facillities_catering FROM INSERTED i; 
SELECT @sr = i.star_rating FROM INSERTED i; 
SELECT @accid2 = i.accommodation_id FROM INSERTED i; 
SELECT @accid = accommodation_id FROM [dbo].[Accommodation] WHERE accommodation_type= 'hotel' AND accommodation_id=@accid2; 
BEGIN 
BEGIN TRAN 
SET NOCOUNT ON 
PRINT @accid2 
PRINT @accid 
IF(@accid2 != @accid) 
BEGIN 
RAISERROR('Record Not Inserted, Accommodation ID is not a Hotel Id',16,1); ROLLBACK; END 
ElSE BEGIN 
INSERT INTO [dbo].[accommodation_hotel] (hotel_id,facillities_swimming_pool,facillities_catering,star_rating,accommodation_id) 
     VALUES (@hid,@fsp,@fc,@sr,@accid2);COMMIT; 

END 
END 

*プリントは、チェックのために私が取得値です。

私のロジックエラーまたは私の構文エラーですか？

Answer 1

は、私はこのような全体のトリガーに何かを書き換えるでしょう

Answer 2

（@accid @ accid2 <>）のIF ...

ALTER TRIGGER [dbo].[TG_checkacctypehtl] 
ON [dbo].[Accommodation_Hotel] 
INSTEAD OF INSERT 
AS 
BEGIN 
    SET NOCOUNT ON; 
INSERT INTO [dbo].[accommodation_hotel] (hotel_id,facillities_swimming_pool,facillities_catering,star_rating,accommodation_id) 
SELECT i.hotel_id 
     ,i.facillities_swimming_pool 
     ,i.facillities_catering 
     ,i.star_rating 
     ,i.accommodation_id 
FROM inserted i 
WHERE EXISTS (SELECT 1 
       FROM [dbo].[Accommodation] a 
       WHERE a.accommodation_type= 'hotel' 
       AND a.accommodation_id = i.accommodation_id) 

IF EXISTS (SELECT 1 FROM inserted i 
      WHERE NOT EXISTS (SELECT 1 
           FROM [dbo].[Accommodation] a 
           WHERE a.accommodation_type= 'hotel' 
           AND a.accommodation_id = i.accommodation_id) 
      ) 
    BEGIN 
    RAISERROR('Records with invalid Accommodation ID is not a Hotel Id not inserted',16,1); 
    END 
END 

が有効宿泊IDを持つ行を挿入した場合にエラーを発生させることべき無効なホテルIDを持つ行があります。これらの変数もすべて必要ありません。

また、トランザクションごとにトリガーされ、各行でトリガーされません。あなたのコードでは、一度に1つの行だけがテーブルに挿入されると仮定しています。