Jak napisać zapytanie, które znajdzie wszystkie odwołania cykliczne, gdy tabela odwołuje się do siebie?

26

Mam następujący schemat (nazwy zmienione), których nie mogę zmienić:

CREATE TABLE MyTable (
    Id INT NOT NULL PRIMARY KEY,
    ParentId INT NOT NULL
);

ALTER TABLE MyTable ADD FOREIGN KEY (ParentId) REFERENCES MyTable(Id);

Oznacza to, że każdy rekord jest dzieckiem innego rekordu. Jeśli rekord ParentIdjest równy Id, rekord jest uważany za węzeł główny.

Chcę uruchomić zapytanie, które znajdzie wszystkie odwołania cykliczne. Na przykład z danymi

INSERT INTO MyTable (Id, ParentId) VALUES
    (0, 0),
    (1, 0),
    (2, 4),
    (3, 2),
    (4, 3);

zapytanie powinno zostać zwrócone

Id | Cycle
2  | 2 < 4 < 3 < 2
3  | 3 < 2 < 4 < 3
4  | 4 < 3 < 2 < 4

Napisałem następujące zapytanie do SQL Server 2008 R2 i zastanawiam się, czy to zapytanie można ulepszyć:

IF OBJECT_ID(N'tempdb..#Results') IS NOT NULL DROP TABLE #Results;
CREATE TABLE #Results (Id INT, HasParentalCycle BIT, Cycle VARCHAR(MAX));

DECLARE @i INT,
    @j INT,
    @flag BIT,
    @isRoot BIT,
    @ids VARCHAR(MAX);

DECLARE MyCursor CURSOR FAST_FORWARD FOR
    SELECT Id
    FROM MyTable;

OPEN MyCursor;
FETCH NEXT FROM MyCursor INTO @i;
WHILE @@FETCH_STATUS = 0
BEGIN
    IF OBJECT_ID(N'tempdb..#Parents') IS NOT NULL DROP TABLE #Parents;
    CREATE TABLE #Parents (Id INT);

    SET @ids = NULL;
    SET @isRoot = 0;
    SET @flag = 0;
    SET @j = @i;
    INSERT INTO #Parents (Id) VALUES (@j);

    WHILE (1=1)
    BEGIN
        SELECT
            @j = ParentId,
            @isRoot = CASE WHEN ParentId = Id THEN 1 ELSE 0 END
        FROM MyTable
        WHERE Id = @j;

        IF (@isRoot = 1)
        BEGIN
            SET @flag = 0;
            BREAK;
        END        

        IF EXISTS (SELECT 1 FROM #Parents WHERE Id = @j)
        BEGIN
            INSERT INTO #Parents (Id) VALUES (@j);
            SET @flag = 1;
            SELECT @ids = COALESCE(@ids + ' < ', '') + CAST(Id AS VARCHAR) FROM #Parents;
            BREAK;
        END
        ELSE
        BEGIN
            INSERT INTO #Parents (Id) VALUES (@j);
        END        
    END

    INSERT INTO #Results (Id, HasParentalCycle, Cycle) VALUES (@i, @flag, @ids);

    FETCH NEXT FROM MyCursor INTO @i;
END
CLOSE MyCursor;
DEALLOCATE MyCursor;

SELECT Id, Cycle
FROM #Results
WHERE HasParentalCycle = 1;
cubetwo1729
źródło
Nie 0 > 0należy tego uważać za cykl?
ypercubeᵀᴹ
1
Nie, 0 jest węzłem głównym, ponieważ ParentIdjest równy jego Id, więc nie jest to cykl dla tego scenariusza.
cubetwo1729

Odpowiedzi:

30

Wymaga to rekurencyjnego CTE:

WITH FindRoot AS
(
    SELECT Id,ParentId, CAST(Id AS NVARCHAR(MAX)) Path
    FROM dbo.MyTable

    UNION ALL

    SELECT C.Id, P.ParentId, C.Path + N' > ' + CAST(P.Id AS NVARCHAR(MAX))
    FROM dbo.MyTable P
    JOIN FindRoot C
    ON C.ParentId = P.Id AND P.ParentId <> P.Id AND C.ParentId <> C.Id
 )
SELECT *
FROM FindRoot R
WHERE R.Id = R.ParentId 
  AND R.ParentId <> 0;

Zobacz to w akcji tutaj: SQL Fiddle


Aktualizacja:

Dodano odległość, aby móc wykluczyć wszystkie cykle samodzielne (patrz komentarz ypercube):

WITH FindRoot AS
(
    SELECT Id,ParentId, CAST(Id AS NVARCHAR(MAX)) Path, 0 Distance
    FROM dbo.MyTable

    UNION ALL

    SELECT C.Id, P.ParentId, C.Path + N' > ' + CAST(P.Id AS NVARCHAR(MAX)), C.Distance + 1
    FROM dbo.MyTable P
    JOIN FindRoot C
    ON C.ParentId = P.Id AND P.ParentId <> P.Id AND C.ParentId <> C.Id
 )
SELECT *
FROM FindRoot R
WHERE R.Id = R.ParentId 
  AND R.ParentId <> 0
  AND R.Distance > 0;

SQL Fiddle

To, którego należy użyć, zależy od wymagań.

Sebastian Meine
źródło
To powinno zostać poprawione. Obecnie pokazuje również 1-cykli, tak 6 > 6długo, jak to nie jest 0 > 0.
ypercubeᵀᴹ
Zrozumiałem PO, że należy wykluczyć tylko rzeczywisty cykl własny węzłów głównych. Możesz jednak z łatwością dodać ten wymóg, sprawdzając, czy R.Path jak „%>%” w końcowej klauzuli where. Lub możesz dodać kolumnę zliczania długości cyklu w rekurencyjnym CTE.
Sebastian Meine
2
Możesz po prostu dodać WHERE Id <> ParentIdpierwszą część CTE.
ypercubeᵀᴹ
AND C.ParentId <> C.Idnie wystarcza. Ścieżka prowadzi do dłuższego okręgu ( A->B, B->C, C->B), wciąż można uzyskać nieskończoną rekurencję do konstruowania ścieżek zaczynających się od A. Naprawdę musisz sprawdzić całą ścieżkę.
Bergi
2
SELECT RC.CONSTRAINT_NAME FK_Name
, KF.TABLE_SCHEMA FK_Schema
, KF.TABLE_NAME FK_Table
, KF.COLUMN_NAME FK_Column
, RC.UNIQUE_CONSTRAINT_NAME PK_Name
, KP.TABLE_SCHEMA PK_Schema
, KP.TABLE_NAME PK_Table
, KP.COLUMN_NAME PK_Column
, RC.MATCH_OPTION MatchOption
, RC.UPDATE_RULE UpdateRule
, RC.DELETE_RULE DeleteRule
FROM INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS RC
JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE KF ON RC.CONSTRAINT_NAME = KF.CONSTRAINT_NAME
JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE KP ON RC.UNIQUE_CONSTRAINT_NAME = KP.CONSTRAINT_NAME
WHERE KF.TABLE_NAME = KP.TABLE_NAME
StuntThumper
źródło
1
Jak to działa? Zwykle to wyjaśnienie stanowi dobrą odpowiedź. Wpisy zawierające tylko kod są niezadowolone z tego powodu (zwykle przynajmniej).
dezso
2
Wygląda na to, że odpowiada na podobne, ale inne pytanie.
ypercubeᵀᴹ