光标陷入无限循环

Question

第一次尝试使用光标，所以放轻松 =P 光标应该抓取全部位于伞组下的公司 ID 列表。然后定位特定公司，将其工作流程记录复制到光标中的公司。

它无限地将这些工作流程记录插入到所有公司中......这里有什么问题？

n00b错误在哪里？

DECLARE @GroupId int = 36;
DECLARE @CompanyToCopy int = 190
DECLARE @NextId int;
Declare @Companies CURSOR;

SET @Companies = CURSOR FOR 
SELECT CompanyId
FROM Company C 
    INNER JOIN [Group] G 
        ON C.GroupID = G.GroupID
WHERE   C.CompanyID != 190 
        AND
        G.GroupId = @GroupId
        AND
        C.CompanyID != 0

OPEN @Companies
FETCH NEXT
FROM @Companies INTO @NextId

WHILE (@@FETCH_STATUS = 0)
BEGIN

    INSERT INTO COI.Workflow(CompanyID, EndOfWorkflowAction, LetterType, Name)
    (SELECT 
            @NextId,
            W.EndOfWorkflowAction,
            W.LetterType,
            W.Name
     FROM COI.Workflow W)

    FETCH NEXT
    FROM @Companies INTO @NextId
END
CLOSE @Companies;
DEALLOCATE @Companies;

编辑：

我决定尝试将这个集合作为基础，只是因为在被告知这样做之后......我意识到我并没有真正找到关于如何将其作为基于集合的查询的答案。

感谢大家的帮助。我将发布基于集合的版本供后代使用。

INSERT INTO COI.Workflow(CompanyID, EndOfWorkflowAction, LetterType, Name)
(
SELECT 
    CG.CompanyId,
    W.EndOfWorkflowAction,
    W.LetterType,
    W.Name
FROM COI.Workflow W
     CROSS JOIN (SELECT C.CompanyID
                 FROM Company C 
                    INNER JOIN [Group] G
                        ON G.GroupID = C.GroupID
                 WHERE  C.CompanyID != 190 
                        AND 
                        C.CompanyID != 0
                        AND 
                        G.GroupID = 36
                 ) AS CG
 WHERE W.CompanyID = 190
 )

Answer 1

你对此没有 WHERE 条件：

SELECT 
            @NextId,
            W.EndOfWorkflowAction,
            W.LetterType,
            W.Name
     FROM COI.Workflow W
     -- WHERE CompanyID = @CompanyToCopy -- This should be here

所以你得到了一种加倍效果。

initial state, company 190, seed row (0)

pass one, company 2, copy of seed row (1)
now 2 rows

pass two, company 3, copy of seed row (0) - call this (2)
pass two, company 3, copy of copy of seed row (1) - call this (3)
now 4 rows

then 8 rows, etc

Answer 2

您将在每次迭代的工作流表中插入所有工作流记录的新副本，因此每次大小都会增加一倍。例如，如果您的光标中有 30 个项目，您最终会得到一个工作流程表，其中的记录数量比以前多 1073741824 倍。

Answer 3

我相信你的逻辑是错误的（由于使用了光标，它有些隐藏！）。

您发布的代码正在尝试将一行插入到 COI.Workflow 中，其中 COI.Workflow 中的每一行乘以与您第一个选择的条件匹配的公司数量。 （请注意插入的 SELECT 语句没有条件：您正在选择整个表）。每次循环时，COI.Workflow 中的行数都会加倍，

因此，它不是无限的，但很可能非常非常长！

我建议你重写为基于集合的语句，逻辑会变得更清晰。

Answer 4

第一次使用游标是可以的，所有问题都在INSERT ... SELECT逻辑中。
我无法理解您需要将什么插入到 COI.Workflow 表中。
我同意之前的评论员的观点，即您当前的 WHERE 条件将记录加倍，但我无法相信您每次都想为每个公司插入全倍记录。
所以，我认为您需要类似的东西

INSERT INTO COI.Workflow(CompanyID, EndOfWorkflowAction, LetterType, Name)
(SELECT TOP 1
        @NextId,
        W.EndOfWorkflowAction,
        W.LetterType,
        W.Name
 FROM COI.Workflow W)

，或者，我们需要更多地了解您插入记录的逻辑。