У меня есть следующая схема (имена изменены), которую я не могу изменить:
CREATE TABLE MyTable (
Id INT NOT NULL PRIMARY KEY,
ParentId INT NOT NULL
);
ALTER TABLE MyTable ADD FOREIGN KEY (ParentId) REFERENCES MyTable(Id);
То есть каждая запись является дочерней по отношению к другой записи. Если запись ParentId
равна ее Id
, то запись считается корневым узлом.
Я хочу запустить запрос, который найдет все циклические ссылки. Например, с данными
INSERT INTO MyTable (Id, ParentId) VALUES
(0, 0),
(1, 0),
(2, 4),
(3, 2),
(4, 3);
запрос должен вернуть
Id | Cycle
2 | 2 < 4 < 3 < 2
3 | 3 < 2 < 4 < 3
4 | 4 < 3 < 2 < 4
Я написал следующий запрос для SQL Server 2008 R2, и мне интересно, можно ли улучшить этот запрос:
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;
sql-server
sql-server-2008-r2
foreign-key
cubetwo1729
источник
источник
0 > 0
Не следует рассматривать как цикл?ParentId
равен своемуId
, поэтому для этого сценария это не цикл.Ответы:
Это требует рекурсивного CTE:
Смотрите это в действии здесь: SQL Fiddle
Обновить:
Добавлена дистанция, чтобы можно было исключить все циклы (см. Комментарий ypercube):
SQL Fiddle
Какой из них вы должны использовать, зависит от ваших требований.
источник
6 > 6
, как долго это не так0 > 0
.WHERE Id <> ParentId
в первую часть CTE.AND C.ParentId <> C.Id
недостаточно. Если путь ведет в более длинный круг (A->B, B->C, C->B
), вы все равно получите бесконечную рекурсию для построения путей, начинающихся сA
. Вам действительно нужно проверить весь путь.источник