Странная проблема с триггером, когда я делаю INSERT в таблицу

Почему бы не изменить триггер для работы с несколькими строками? Курсор или циклы не нужны: в этом весь смысл SQL...

UPDATE
    dbo.SomeTable
SET
    LastModifiedOn = GETDATE()
WHERE
    EXIST (SELECT * FROM INSERTED I WHERE I.[ID] = dbo.SomeTable.[ID]

Редактировать: Что-то вроде...

INSERT @ATableVariable
    (ContentId, ContentTypeId, UniqueSubject)
SELECT 
    ContentId, ContentTypeId, [dbo].[ToUriCleanText]([Subject])
FROM
    INSERTED

UPDATE
    [dbo].[Contents]
SET
    UniqueSubject + CAST(NumberFound AS NVARCHAR(10))
FROM
    --Your original COUNT feels wrong and/or trivial
    --Do you expect 0, 1 or many rows.
    --Edit2: I assume 0 or 1 because of original WHERE so COUNT(*) will suffice
    -- .. although, this implies an EXISTS could be used but let's keep it closer to OP post
    (
    SELECT ContentId, UniqueSubject, COUNT(*) AS NumberFound
    FROM @ATableVariable
    GROUP BY ContentId, UniqueSubject
    HAVING COUNT(*) > 0
    ) foo
    JOIN
    [dbo].[Contents] C ON C.ContentId = foo.ContentId AND C.UniqueSubject = foo.UniqueSubject

Редактировать 2: и снова с РЕЙТИНГОМ

UPDATE
    C
SET
    UniqueSubject + CAST(foo.Ranking - 1 AS NVARCHAR(10))
FROM
    (
    SELECT
        ContentId, --not needed? UniqueSubject,
        ROW_NUMBER() OVER (PARTITION BY ContentId ORDER BY UniqueSubject) AS Ranking
    FROM
        @ATableVariable
    ) foo
JOIN
    dbo.Contents C ON C.ContentId = foo.ContentId 
    /* not needed? AND C.UniqueSubject = foo.UniqueSubject */
WHERE
foo.Ranking > 1

Триггер будет запущен только один раз для запроса INSERT INTO. Таблица INSERTED будет содержать несколько строк.

Ладно, ребята, я думаю, что разобрался сам. Вдохновленный предыдущими ответами и комментариями, я сделал следующее. (Можете ли вы, ребята, быстро просмотреть, чтобы увидеть, не переделал ли я этого ребенка?)

.1. Создал индексированное представление, представляющее поле «Тема», которое необходимо очистить. Это поле должно быть уникальным... но прежде чем мы сможем сделать его уникальным, нам нужно сгруппировать по нему.

-- Create the view.
CREATE VIEW ContentsCleanSubjectView with SCHEMABINDING AS
SELECT ContentId, ContentTypeId, 
    [dbo].[ToUriCleanText]([Subject]) AS CleanedSubject
FROM [dbo].[Contents]
GO

-- Index the view with three index's. Custered PK and a non-clustered, 
-- which is where most of the joins will be done against.
-- Last one is because the execution plan reakons i was missing statistics
-- against one of the fields, so i added that index and the stats got gen'd.
CREATE UNIQUE CLUSTERED INDEX PK_ContentsCleanSubjectView ON 
    ContentsCleanSubjectView(ContentId)
CREATE NONCLUSTERED INDEX IX_BlahBlahSnipSnip_A ON 
    ContentsCleanSubjectView(ContentTypeId, CleanedSubject)
CREATE INDEX IX_BlahBlahSnipSnip_B ON
    ContentsCleanSubjectView(CleanedSubject)

.2. Создайте код триггера, который теперь
а) захватывает все элементы, "измененные" (ничего нового/сложного в этом)
б) упорядочивает все вставленные строки, строки нумеруются с разбиением по чистому субъекту
c) обновить единственную строку, которую мы делаем, в основном предложении обновления.

вот код...

ALTER TRIGGER [dbo].[UpdateUniqueSubjectAfterInsertUpdate]
   ON  [dbo].[Contents]
   AFTER INSERT,UPDATE
AS
BEGIN
    SET NOCOUNT ON
    
    DECLARE @InsertRows TABLE (ContentId INTEGER PRIMARY KEY,
        ContentTypeId TINYINT,
        CleanedSubject NVARCHAR(300))

    DECLARE @UniqueSubjectRows TABLE (ContentId INTEGER PRIMARY KEY,
    UniqueSubject NVARCHAR(350))
    
DECLARE @UniqueSubjectRows TABLE (ContentId INTEGER PRIMARY KEY,
        UniqueSubject NVARCHAR(350))
        
    -- Grab all the records that have been updated/inserted.
    INSERT INTO @InsertRows(ContentId, ContentTypeId, CleanedSubject)
    SELECT ContentId, ContentTypeId, [dbo].[ToUriCleanText]([Subject])
    FROM INSERTED
    
    
    -- Determine the correct unique subject by using ROW_NUMBER partitioning.
    INSERT INTO @UniqueSubjectRows
    SELECT SubResult.ContentId, UniqueSubject = CASE SubResult.RowNumber 
        WHEN 1 THEN SubResult.CleanedSubject 
        ELSE SubResult.CleanedSubject + CAST(SubResult.RowNumber - 1 AS NVARCHAR(5)) END
    FROM (
        -- Order all the cleaned subjects, partitioned by the cleaned subject.
        SELECT a.ContentId, a.CleanedSubject, ROW_NUMBER() OVER (PARTITION BY a.CleanedSubject ORDER BY a.ContentId) AS RowNumber
        FROM ContentsCleanSubjectView a 
            INNER JOIN @InsertRows b ON a.ContentTypeId = b.ContentTypeId AND a.CleanedSubject = b.CleanedSubject
        GROUP BY a.contentId, a.cleanedSubject
    ) SubResult
    INNER JOIN [dbo].[Contents] c ON c.ContentId = SubResult.ContentId
    INNER JOIN @InsertRows d ON c.ContentId = d.ContentId
    
    -- Now update all the effected rows.
    UPDATE a
    SET a.UniqueSubject = b.UniqueSubject
    FROM [dbo].[Contents] a INNER JOIN @UniqueSubjectRows b ON a.ContentId = b.ContentId
END  

Теперь подзапрос корректно возвращает все вычищенные темы, правильно разбитые на разделы и правильно пронумерованные. Я никогда не знал о команде «РАЗДЕЛ», так что этот трюк был здесь большим ответом :)

Затем я просто присоединился к подзапросу со строкой, которая обновляется в родительском запросе. Номер строки правильный, так что теперь я просто делаю случай. если очищенная тема существует впервые (например, row_number = 1), не изменяйте ее. в противном случае добавьте row_number минус один. Это означает, что 2-й экземпляр одной и той же темы будет иметь уникальную тему => cleansubject + '1'.

Причина, по которой я считаю, что мне нужно иметь индексированное представление, заключается в том, что если у меня есть два очень похожих предмета, то, когда вы удалили (т.е. очистили) все плохие символы (которые, как я определил, плохие).. вполне возможно, что два чистых объекта одинаковы. Таким образом, мне нужно выполнять все соединения с очищенным субъектом, а не с субъектом. Теперь, для огромного количества строк, которые у меня есть, это дерьмо для производительности, когда у меня нет представления. :)

Так .. это перепроектировано?

Редактировать 1:

Переработан код триггера, чтобы он стал более производительным.