liczyć Aktualizacja kolumny z danych w innej tabeli

Question

W moim DB mam dwie tabele egzemplarze (Id, ..., ToatlViews int) i ItemViews (id, ItemId, Datownik)

W tabeli ItemViews przechowywać wszystkie widoki przedmiot, gdy przychodzą na stronę. Od czasu do czasu chcę wywołać procedurę przechowywaną, aby zaktualizować pole Items.ToatlViews. Próbowałem zrobić to SP za pomocą kursora ... ale instrukcja aktualizacji jest niepoprawna. Czy możesz mi pomóc to poprawić? Czy mogę to zrobić bez kursora?

CREATE PROCEDURE UpdateItemsViews 
AS 
BEGIN 
    -- SET NOCOUNT ON added to prevent extra result sets from 
    -- interfering with SELECT statements. 
    SET NOCOUNT ON; 

    DECLARE @currentItemId int 
    DECLARE @currentItemCursor CURSOR 
    SET @currentItemCursor = CURSOR FOR SELECT Id FROM dbo.Items 

    OPEN @currentItemCursor 
    FETCH NEXT FROM @currentItemCursor INTO @currentItemId 
    WHILE @@FETCH_STATUS = 0 
    BEGIN 
     Update dbo.Items set TotalViews = count(*) 
       from dbo.ItemViews where ItemId=@currentItemId 
     FETCH NEXT FROM @currentItemCursor INTO @currentItemId 
    END 
END 
GO 

Answer 1

Można użyć bezpośredniego UPDATE

update Items set TotalViews = 
    (select COUNT(id) from ItemViews where ItemViews.ItemId = Items.Id) 

Może chcesz przetestować działanie na różne sposoby, aby to zrobić , jeśli to ważne.

Answer 2

Można użyć update ... from zamiast kursora:

update i 
set  TotalViews = iv.cnt 
from dbo.Item i 
join (
     select ItemId 
     ,  count(*) as cnt 
     from dbo.ItemViews 
     group by 
       ItemId 
     ) iv 
on  i.Id = iv.ItemId 

Answer 3

;WITH x AS 
(
    SELECT ItemID, c = COUNT(*) 
    FROM dbo.ItemViews 
    GROUP BY ItemID 
) 
UPDATE i 
SET TotalViews = x.c 
FROM dbo.Items AS i 
INNER JOIN x 
ON x.ItemID = i.ItemID; 

Ale dlaczego chcesz zachować tę wartość, gdy zawsze możesz uzyskać liczenie w czasie wykonywania? Będziesz musiał uruchomić tę instrukcję aktualizacji za każdym razem, gdy dotkniesz tabeli ItemViews w jakikolwiek sposób, w przeciwnym razie licznik przechowywany z Elementami będzie niepoprawny.

Co można rozważyć robi zamiast jest utworzenie indeksowany widok:

CREATE VIEW dbo.ItemViewCount 
WITH SCHEMABINDING 
AS 
    SELECT ItemID, ItemCount = COUNT_BIG(*) 
     FROM dbo.ItemViews 
     GROUP BY ItemID; 
GO 
CREATE UNIQUE CLUSTERED INDEX x ON dbo.ItemViewCount(ItemID); 

Teraz możesz dołączyć do widoku w zapytaniach i wiem, że licznik jest zawsze na bieżąco (bez płacenia kary skanowanie pod kątem liczby sztuk każdego przedmiotu). Wadą indeksowanego widoku jest to, że płacisz ten koszt narastająco, gdy istnieją inserty/aktualizacje/usunięcia do tabeli ItemViews.

Answer 4

Znalazłem to pytanie/odpowiedź rok po napisaniu i odebraniu odpowiedzi. odpowiedź była w porządku, ale szukałem czegoś bardziej automatycznego. Skończyłem na pisaniu spustu, aby automatycznie przeliczyć kolumnę, kiedy odpowiedni wiersz w drugiej tabeli został wstawiony, usunięty lub zaktualizowany.

myślę, że to lepsze rozwiązanie niż bieganie coś ręcznie wykonać przeliczenie jak nie ma żadnej możliwości ktoś zapominając uruchomić kod:

CREATE TRIGGER [dbo].[TriggerItemTotalViews] 
    ON [dbo].[ItemViews] 
    AFTER INSERT, DELETE, UPDATE 
AS 
BEGIN 
SET NOCOUNT ON; 

UPDATE [Items] 
SET [TotalViews] = 
    (
    SELECT COUNT(id) 
    FROM [ItemViews] 
    WHERE [ItemViews].[ItemId] = [Items].[ItemId] 
    ) 
WHERE [Items].[ItemId] IN 
    (
    SELECT [ItemId] FROM [INSERTED] 
    UNION 
    SELECT [ItemId] FROM [DELETED] 
    ) 
END 

Answer 5

samo, ale inaczej:

declare @productId int = 24; 
declare @classificationTypeId int = 86; 

update s 
set CounterByProductAndClassificationType = row_num 
from Samples s 
join 
(
    select row_number() over (order by (select Id)) row_num, Id 
    from Samples 
    where 
     ProductId = @productId and 
     ClassificationTypeId = @classificationTypeId 
) s_row on s.Id = s_row.Id