Jak pobrać wiele kolumn do użycia w pętli kursora?

88

Kiedy próbuję uruchomić następujący fragment kodu SQL w pętli kursora,

set @cmd = N'exec sp_rename ' + @test + N',' +
           RIGHT(@test,LEN(@test)-3) + '_Pct' + N',''COLUMN'''

Otrzymuję następującą wiadomość,

Msg 15248, poziom 11, stan 1, procedura sp_rename, wiersz 213
Albo parametr @objnamejest niejednoznaczny, albo żądany @objtype(KOLUMNA) jest nieprawidłowy.

Co jest nie tak i jak to naprawić? Próbowałem zawinąć nazwę kolumny w nawiasy []i podwójne cudzysłowy, ""jak niektóre sugerowane wyniki wyszukiwania.

Edycja 1 -

Oto cały skrypt. Jak mogę przekazać nazwę tabeli do nazwy sp? Nie jestem pewien, jak to zrobić, ponieważ nazwy kolumn znajdują się w jednej z wielu tabel.

BEGIN TRANSACTION

declare @cnt int
declare @test nvarchar(128)
declare @cmd nvarchar(500) 
declare Tests cursor for
SELECT COLUMN_NAME FROM INFORMATION_SCHEMA.COLUMNS WHERE COLUMN_NAME LIKE 'pct%' AND TABLE_NAME LIKE 'TestData%'

open Tests
fetch next from Tests into @test
while @@fetch_status = 0
BEGIN
  set @cmd = N'exec sp_rename ' + @test + N',' + RIGHT(@test,LEN(@test)-3) + '_Pct' + N', column' 

  print @cmd

  EXEC sp_executeSQL @cmd

  fetch next from Tests into @test
END

close Tests 
deallocate Tests


ROLLBACK TRANSACTION
--COMMIT TRANSACTION

Edycja 2 - skrypt służy do zmiany nazw kolumn, których nazwy pasują do wzorca, w tym przypadku z prefiksem „pct”. Kolumny występują w różnych tabelach w bazie danych. Wszystkie nazwy tabel są poprzedzone przedrostkiem „TestData”.

Joe
źródło
1
ta linia łączy ciąg. dlaczego go nie wydrukujesz, żeby zobaczyć, jaka jest zawartość łańcucha,
Preet Sangha,
Jeśli @test zawiera kwalifikowaną nazwę, musi być w apostrofach. Jeśli zachodzi to samo założenie, funkcja right () usunie pierwsze trzy znaki z nazwy tabeli; w chciałbyś zamienić ostatnie znaki nazwy kolumny to byłoby LEFT. Czy mógłbyś trochę rozszerzyć skrypt, dodając set @test = ...?
Nikola Markovinović
Twój kod pomógł mi rozwiązać problem - dzięki za to!
Neville

Odpowiedzi:

157

Oto nieco zmodyfikowana wersja. Zmiany są odnotowywane jako komentarz do kodu.

BEGIN TRANSACTION

declare @cnt int
declare @test nvarchar(128)
-- variable to hold table name
declare @tableName nvarchar(255)
declare @cmd nvarchar(500) 
-- local means the cursor name is private to this code
-- fast_forward enables some speed optimizations
declare Tests cursor local fast_forward for
 SELECT COLUMN_NAME, TABLE_NAME
   FROM INFORMATION_SCHEMA.COLUMNS 
  WHERE COLUMN_NAME LIKE 'pct%' 
    AND TABLE_NAME LIKE 'TestData%'

open Tests
-- Instead of fetching twice, I rather set up no-exit loop
while 1 = 1
BEGIN
  -- And then fetch
  fetch next from Tests into @test, @tableName
  -- And then, if no row is fetched, exit the loop
  if @@fetch_status <> 0
  begin
     break
  end
  -- Quotename is needed if you ever use special characters
  -- in table/column names. Spaces, reserved words etc.
  -- Other changes add apostrophes at right places.
  set @cmd = N'exec sp_rename ''' 
           + quotename(@tableName) 
           + '.' 
           + quotename(@test) 
           + N''',''' 
           + RIGHT(@test,LEN(@test)-3) 
           + '_Pct''' 
           + N', ''column''' 

  print @cmd

  EXEC sp_executeSQL @cmd
END

close Tests 
deallocate Tests

ROLLBACK TRANSACTION
--COMMIT TRANSACTION
Nikola Markovinović
źródło
2
Jedna z moich ulubionych odpowiedzi w SO.
Rubens Mariuzzo
@RubensMariuzzo Dzięki, jesteś zbyt hojny :-)
Nikola Markovinović
62
TLDR; FETCH NEXT FROM db_cursor INTO @var1, @ var2
Don Rolling
7
TLDR jest powszechnie używany do wskazania krótkiej wersji dłuższej informacji. To oznacza zbyt długo nie czytałem. Sugerowałem, że istnieje krótsza wersja odpowiedzi, której udzieliłeś i ja jej udzieliłem.
Don Rolling
1
Niezły dotyk, nie ściągam dwa razy. :)
zwycięzca_joiner