我的 INSERT 语句有什么问题？ 44

Question

我正在使用 SQL Server 2008 并希望创建一个过程，该过程只需从一个表中获取数据并将其插入到另一个表中。这是我目前所拥有的：

CREATE PROCEDURE consomVspPools (@start DATE, @end DATE)
AS
CREATE TABLE dbo.temp (pool NVARCHAR (250), endUsedCapacityInKB FLOAT, 
                         startUsedCapacityInKB FLOAT)
WHILE (@end >= @start)
BEGIN
      INSERT INTO temp
      SELECT pool, MAX(capacityInKB), MAX(freeCapacityInKB)
      FROM Vsp_Pool
      WHERE dateCollecte IN (DATEADD(day, -1, @end), @end)
      GROUP BY pool
      SET @end = DATEADD(day, -1, @end)
END
SELECT *
FROM temp
DROP TABLE dbo.temp

我现在知道代码并没有真正做太多，但我必须对其进行大量简化以缩小我的错误（无论如何这对我没有多大帮助）。我得到的错误是：

“列名或提供的值的数量与表定义不匹配”

我是 SQL 新手，所以这可能很明显，但我看不出来，一切对我来说似乎都是为了。

编辑1：

所以这是我知道的代码，按照建议升级：

CREATE PROCEDURE consomVspPools (@start DATE, @end DATE)
AS
CREATE TABLE #dbo.temp (pool NVARCHAR (250), endUsedCapacityInKB FLOAT, 
                      startUsedCapacityInKB FLOAT)
WHILE (@end >= @start)
BEGIN
      INSERT INTO temp (pool, endUsedCapacityInKB, startUsedCapacityInKB)
      SELECT pool, MAX(capacityInKB), MAX(freeCapacityInKB)
      FROM Vsp_Pool
      WHERE dateCollecte IN (DATEADD(day, -1, @end), @end)
      GROUP BY pool
      SET @end = DATEADD(day, -1, @end)
END
SELECT *
FROM temp
DROP TABLE #dbo.temp

但这给了我以下错误：

“无效的列名‘池’”

仅供参考，表 Vsp_Pool 和 temp 中都存在池。

Answer 1

您需要在“插入”语句中包含列名。

INSERT INTO temp (poolVal, maxcapacity, maxfreecapacity)

可能有助于在我的示例中为 select 语句中的列名称提供别名。#

pool 是一个关键字，可以使用其他任何名称作为您的字段名称。

Answer 2

Put if condition because while is in infinite loop:


CREATE PROCEDURE consomVspPools (@start DATE, @end DATE)
AS
CREATE TABLE dbo.temp (pool NVARCHAR (250), endUsedCapacityInKB FLOAT, 
                         startUsedCapacityInKB FLOAT)
if (@end >= @start)
BEGIN
      INSERT INTO temp
      SELECT pool, MAX(capacityInKB), MAX(freeCapacityInKB)
      FROM Vsp_Pool
      WHERE (dateCollecte >= (DATEADD(day, -1, @end)) and  (dateCollecte <= @end)
      GROUP BY pool
      SET @end = DATEADD(day, -1, @end)
END
SELECT *
FROM temp
DROP TABLE dbo.temp

Answer 3

您可能听说过 SQL 是一种声明性语言，而不是过程性语言。这意味着你应该声明如何你想做什么，而不是说如何。 SQL 引擎针对这种行为进行了优化。由于这个原因，通常不使用 WHILE 语句。不知道你的数据库结构，我最好的猜测是，你应该尝试这样的事情：

CREATE PROCEDURE consomVspPools (@start DATE, @end DATE)
AS

BEGIN
    SELECT *
    FROM
        (
        SELECT dateCollecte, pool, MAX(capacityInKB), MAX(freeCapacityInKB)
        FROM Vsp_Pool 

        -- Is "dateCollecte" a typo? Might be your issue.
        WHERE dateCollecte BETWEEN @start AND @end
        GROUP BY dateCollecte, pool
        )
END

查看BETWEEN 声明。另请注意，我正在从查询结果中查询并完全避免使用临时表。这里的要点是开始分组思考。如果一个查询感觉就像您正在遍历并自动更新每个条目而不是作为一个集合，那么可能有一种更好的、经过 SQL 优化的方式来重写它。