Problème de curseur imbriqué SQL Server

Question

J'ai un problème étrange avec mes curseurs imbriqués et je n'ai aucune idée de quoi il s'agit.

Voici mon code T-SQL:

declare @dbname varchar(50) 
declare @servername varchar(50) 
declare srv cursor for select servername from test.dbo.servers 
declare @str varchar(200) 

truncate table test.dbo.temp 

open srv 
fetch next from srv into @servername 
while @@fetch_status = 0 
begin 
    set @str = 'Data Source='+@servername+';Integrated Security=SSPI' 
    declare db cursor for select name from opendatasource('SQLNCLI', @str).master.dbo.sysdatabases 
    open db 
    fetch next from db into @dbname 
    while @@fetch_status = 0 
    begin 
     insert test.dbo.temp (dbname, servername) values (@dbname, @servername) 
     fetch next from db into @dbname 
    end 
    fetch next from srv into @servername 
    close db 
    deallocate db 
end 
close srv 
deallocate srv 

Il me donne message d'erreur suivant:

syntaxe incorrecte près 'de @str'. [SQLSTATE 42000] (erreur 102)

dirait que le problème est en donnant la variable en tant que paramètre de fonction OpenDataSource. Mais pourquoi? Et comment éviter ce problème?

Answer 1

Vous avez raison: les variables ne peuvent pas être passées à OPENDATASOURCE. Au lieu de cela, vous devez utiliser un littéral à la place. Autant que nous déconseillons l'utilisation de SQL dynamique, il y a des cas où c'est inévitable. Essayez quelque chose comme ceci:

declare @dbname varchar(50) 
declare @servername varchar(50) 
declare srv cursor for select servername from test.dbo.servers 
declare @str varchar(200) 
declare @sql nvarchar(MAX) 

truncate table test.dbo.temp 

open srv 
fetch next from srv into @servername 
while @@fetch_status = 0 
begin 
    SET @sql = N' 
    declare db cursor for select name from opendatasource(''SQLNCLI'', ''Data Source='+@servername+';Integrated Security=SSPI'').master.dbo.sysdatabases 
    open db 
    fetch next from db into @dbname 
    while @@fetch_status = 0 
    begin 
     insert test.dbo.temp (dbname, servername) values (@dbname, @servername) 
     fetch next from db into @dbname 
    end 
    close db 
    deallocate db 
    ' 
    EXEC sp_executesql 
    @sql, 
    N'@dbname  varchar(50), 
     @servername varchar(50)', 
    @dbname, 
    @servername 

    fetch next from srv into @servername 
end 
close srv 
deallocate srv 

Answer 2

Si vous devez utiliser des curseurs imbriqués, vous faites quelque chose de mal. Il y a très peu de raisons d'utiliser un curseur au lieu d'une autre opération basée sur un ensemble, et l'utilisation d'un curseur dans un curseur est comme l'anti-pattern SQL Server ultime.

Pour votre curseur intérieur, vous pouvez changer d'utiliser la fonction non documentée sp_msforeachdb(qui crée apparemment un curseur dans les coulisses):

open srv 
fetch next from srv into @servername 
while @@fetch_status = 0 
begin 
EXEC sp_msforeachdb ' 
Data Source='+@servername+';Integrated Security=SSPI 
insert test.dbo.temp (dbname, servername) values (?, @Servername)' 
fetch next from srv into @servername 
end 
close srv 
deallocate srv 

Vous devrez peut-être joindre la? entre guillemets simples et leur échapper, comme:

EXEC sp_msforeachdb 'insert test.dbo.temp (dbname, servername) values (''?'', @Servername)