Что случилось с моим заявлением 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, так что это может быть очевидным, но я не вижу, все, кажется, в заказывайте мне.

EDIT1:

так это код, я знаю, что есть, модернизирован как предложено:

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 

Но это потом дает мне ошибку folowwing:

«Неверное имя столбца 'пул' "

Только FYI, пул присутствует в таблице Vsp_Pool, а также в temp.

Answer 1

вам нужно указать имена столбцов в инструкции «вставить в».

INSERT INTO temp (poolVal, maxcapacity, maxfreecapacity) 

может помочь дать имена столбцов в некоторых псевдонимах заявления, как в моем примере. #

бассейн ключевого слова использовать что-нибудь другое для вашего имени поля.

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 заявление. Также обратите внимание, что я запрашиваю результат запроса и полностью исключаю таблицу temp. Вынос здесь - начать думать в наборах. Если запрос кажется, что вы проходите, и атомизированное обновление каждой записи индивидуально, а не как набор, вероятно, лучший, оптимизированный SQL способ перезаписи.