Início / coding / Perguntas / 78042905
Accepted
Val
Asked: 2024-02-23 01:14:52 +0800 CST

Loop SQL inválido porque não há variável escalar?

  • 772

Estou tentando preencher minha tabela de banco de dados com alguns dados fictícios e escrevi um loop para isso. Estou usando o DBeaver para conectar-me ao meu banco de dados SQL Server e executo o código nele.

DECLARE @counter INT = 1;

WHILE @counter <= 1000
BEGIN
    INSERT INTO dbo.Articles
        (ArticleCode, ArticleName)
    VALUES
        ('ARTCODE' + CAST(@counter AS VARCHAR), 'Name' + CAST(@counter AS VARCHAR))
    ;
    
    SET @counter = @counter + 1;
END;

Por algum motivo, recebo um erro na linha 3 (no loop WHILE).

Erro SQL [137] [S0002]: Deve declarar a variável escalar "@counter"

O que eu fiz errado?

sql

3 respostas

  1. Best Answer

    O problema é o DBeaver, e não o seu SQL. DBeaver, quando usado com T-SQL, vê um ponto e vírgula ( ;) como um separador de lote e não como um terminador de instrução , apesar de ser definido como um "terminador de instrução" nas configurações.

    Como resultado deste comportamento indesejado (bug), isso significa que mesmo um lote simples como o seguinte irá falhar:

    DECLARE @test int;

SET @test = 1;

    No SSMS, seria como escrever o seguinte:

    DECLARE @test int
GO
GO
SET @test = 1
GO

    (2 GOs propositalmente, pois o DBeaver também trata uma linha em branco como um separador de lote de instruções .)

    Em vez disso, você precisará alterar suas preferências em Editores>Editor SQL>Processamento SQL. O método que usei para obter o comportamento normal do T-SQL foi:

    • Delimitadores > Delimitador de instrução:GO
    • Delimitadores > Ignorar delimitador nativo: ativado
    • Delimitadores > Linha em branco é delimitador de instrução: Desabilitado
    • Delimitadores > Remover delimitador de consulta final: ativado (não tenho certeza se isso é relevante)

    Isso significa que o script acima funcionou e ocorreu um erro em um segundo lote que @testnão foi definido, e o "intelisense" na tela também pareceu apresentar o comportamento correto. Captura de tela

    • 1

  2. Incluir exemplos de DDL e DML tornará isso muito mais fácil de resolver. Considerar: DECLARE @Table TABLE (Column1 INT, Column2 DATE, ...); INSERT INTO @Table (Column1, Column2, ...) VALUES (1, '2024-01-01', ...), (2, '2024-02-01', ...);

    Além disso, conforme mencionado nos comentários, é uma prática inadequada usar [N]Varchar sem comprimento.

    Não consegui reproduzir seu problema, mas considere:

    DECLARE @TempTable TABLE(ArticleCode VARCHAR(30), ArticleName VARCHAR(50));

;WITH n AS (
SELECT 1 AS n
UNION ALL
SELECT n+1
 FROM n
 WHERE n < 1000
)

INSERT INTO @TempTable
SELECT 'ARTCODE' + CAST(n AS VARCHAR(20)), 'Name' + CAST(n AS VARCHAR(45))
 FROM n
 OPTION (MAXRECURSION 1000);

    Isso gera uma expressão de tabela comum com 1.000 linhas e, em seguida, seleciona a partir dela com sua concatenação de strings para inserir em uma tabela fictícia.

    SQL não funciona bem com loops e é melhor evitá-los sempre que possível. A regra geral é que se você estiver usando um cursor, provavelmente não está fazendo isso da melhor maneira.

    • 0

  3. Você pode simplesmente selecionar em uma tabela do sistema, por exemplo, syscommentse usar rownumber()para simplificar bastante e fazer uma única inserção. Acredito que concatfoi introduzido em 2012 também.

    insert into dbo.Articles(ArticleCode, ArticleName)
select Concat('ARTCODE', rn), Concat('Name', rn)
  from (
    select top(1000) Row_Number() over (order by (select null)) rn
    from master.dbo.syscomments
)t;
    • 0

relate perguntas