Como escrever uma consulta que encontre todas as referências circulares quando uma tabela faz referência a si mesma?

26

Eu tenho o seguinte esquema (nomes alterados), que não posso alterar:

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

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

Ou seja, cada registro é filho de outro registro. Se um registro ParentIdé igual ao seu Id, então o registro é considerado um nó raiz.

Eu quero executar a consulta que encontrará todas as referências circulares. Por exemplo, com os dados

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

a consulta deve retornar

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

Eu escrevi a seguinte consulta para o SQL Server 2008 R2 e gostaria de saber se esta consulta pode ser melhorada:

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
fonte
Não 0 > 0deve ser considerado um ciclo?
ypercubeᵀᴹ
11
Não, 0 é um nó raiz, pois é ParentIdigual a Id, portanto, não é um ciclo para esse cenário.
cubetwo1729

Respostas:

30

Isso exige uma CTE recursiva:

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;

Veja em ação aqui: SQL Fiddle


Atualizar:

Distância adicionada para poder excluir todos os ciclos próprios (consulte o comentário do 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

Qual deles você deve usar depende de sua exigência.

Sebastian Meine
fonte
Isso deve ser corrigido. Atualmente, ele também mostra 1 ciclo, como 6 > 6, desde que não seja 0 > 0.
ypercubeᵀᴹ
Entendi o OP que apenas o auto ciclo real dos nós raiz deve ser excluído. No entanto, você pode adicionar facilmente esse requisito verificando se R.Path gosta de '%>%' na cláusula where final. Ou você pode adicionar uma coluna de contagem de duração do ciclo dentro da CTE recursiva.
Sebastian Meine
2
Você pode apenas adicionar WHERE Id <> ParentIda primeira parte do CTE.
ypercubeᵀᴹ
AND C.ParentId <> C.Idnão é o suficiente. Se um caminho leva a um círculo mais longo ( A->B, B->C, C->B), você ainda recebe uma recursão infinita pela construção dos caminhos que começam A. Você realmente precisa verificar o caminho inteiro.
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
fonte
11
E como isso funciona? Geralmente é a explicação que dá uma boa resposta. As postagens somente de código são desaprovadas aqui (geralmente, pelo menos).
Dez
2
Parece que responde a uma pergunta semelhante, mas diferente.
ypercubeᵀᴹ