存储过程反复插入同一条记录，而不是循环遍历SELECT

如何解决存储过程反复插入同一条记录，而不是循环遍历SELECT

我在编写程序方面很新（超出了基础）

我正试图编写一个基于循环遍历结果列表（dbo.billing_batch）的select语句插入表（@DealerID FROM dbo.vehicle_info）中的存储过程。

SELECT DISTINCT...语句本身可以完美运行，并返回54条记录的列表。

SELECT语句的结果是动态的，并且每周都会变化，因此我不能每次依靠54条记录。

我正在尝试使用WHILE @DealerID IS NOT NULL遍历INSERT例程。

该循环应该更新dbo.billing_batch，但是它会一遍又一遍地插入相同的第一条记录（BillingBatchRosterID,DealerID）到无穷大。

我知道我一定做错了事（我从未编写过循环的存储过程）。

任何帮助将不胜感激！

这是存储过程代码：

ALTER PROCEDURE [dbo].[sp_billing_batch_set]
    @varBillingBatchRosterID int
AS 
    SET NOCOUNT ON;
BEGIN
    DECLARE @DealerID int

    SELECT DISTINCT @DealerID = vi.DealerID
    FROM dbo.vehicle_info vi
    LEFT JOIN dbo.dealer_info di ON di.DealerID = vi.DealerID
    WHERE di.DealerActive = 1 
      AND (vi.ItemStatusID < 4 OR vi.ItemStatusID = 5 OR vi.ItemStatusID = 8)
END
 
WHILE @DealerID IS NOT NULL
BEGIN TRY
    INSERT INTO dbo.billing_batch (BillingBatchRosterID,DealerID)
    VALUES(@varBillingBatchRosterID,-- BillingBatchRosterID - int
           @DealerID)                   -- DealerID - int
END TRY
BEGIN CATCH
    SELECT  ' There was an error: '  + error_message() AS ErrorDescription
END CATCH

解决方法

您遇到的问题与此处的另一篇最新文章相同：Iterate over a table with a non-int id value

1. 为什么要循环？只需将其作为单个SQL语句即可完成
2. 如果必须使用循环，则每次运行时都需要更新@Dealer值（例如，更新为下一个DealerId），否则将使用相同的DealerID值进行无限循环
3. 不要循环。

这是一个不需要循环的示例。

ALTER PROCEDURE [dbo].[P_billing_batch_set]
    @varBillingBatchRosterID int
AS 
BEGIN
SET NOCOUNT ON;

BEGIN TRY

INSERT INTO dbo.billing_batch (DealerID,BillingBatchRosterID)
    SELECT DISTINCT vi.DealerID,@varBillingBatchRosterID
    FROM dbo.vehicle_info vi
        INNER JOIN dbo.dealer_info di ON di.DealerID = vi.DealerID
    WHERE di.DealerActive = 1 
        AND (vi.ItemStatusID < 4 
            OR vi.ItemStatusID = 5
            OR vi.ItemStatusID = 8
            );

END TRY
BEGIN CATCH
    SELECT  ' There was an error: '  + error_message() AS ErrorDescription;
END CATCH;

END;

注意我

• 将LEFT JOIN更改为INNER JOIN，因为您的WHERE子句需要该记录存在于Dealer_info表中
• 将SET NOCOUNT ON;移到了BEGIN-END部分
• 将END移动到最后
• 根据@marc_s（关于问题本身）的出色注释，重命名了存储过程