Как получить несколько столбцов для использования в цикле курсора?

88

Когда я пытаюсь запустить следующий фрагмент SQL внутри цикла курсора,

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

Я получаю следующее сообщение,

Msg 15248, уровень 11, состояние 1, процедура sp_rename, строка 213
Либо параметр @objnameнеоднозначен, либо заявленный @objtype(COLUMN) неверен.

Что не так и как это исправить? Я попытался заключить имя столбца в скобки []и заключить в двойные кавычки, ""как некоторые из предлагаемых результатов поиска.

Изменить 1 -

Вот весь сценарий. Как передать имя таблицы функции переименования? Я не уверен, как это сделать, поскольку имена столбцов находятся в одной из многих таблиц.

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

Изменить 2 - сценарий предназначен для переименования столбцов, имена которых соответствуют шаблону, в данном случае с префиксом «pct». Столбцы встречаются в различных таблицах базы данных. Все имена таблиц имеют префикс «TestData».

Джо
источник
1
эта строка объединяет строку. почему бы вам не распечатать его, чтобы вы могли видеть, каково содержимое строки,
Прит Сангха,
@testимеет форму table.columnилиschema.table.column , не так ли?
GSerg
Если @test содержит полное имя, оно должно быть в апострофах. Если то же предположение верно, right () удалит первые три символа в имени таблицы; если вы хотите заменить последние символы имени столбца, это будет LEFT. Не могли бы вы немного расширить сценарий, добавив set @test = ...?
Никола Марковинович
Ваш код действительно помог мне решить мою проблему - спасибо за это!
Невилл

Ответы:

157

Вот немного измененная версия. Изменения отмечены как комментарии к коду.

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
Никола Марковинович
источник
2
Один из моих любимых ответов в SO.
Рубенс Мариуццо
@RubensMariuzzo Спасибо, вы слишком великодушны :-)
Никола Марковинович
62
TL; DR; FETCH NEXT FROM db_cursor INTO @var1, @ var2
Don Rolling
7
TL; DR обычно используется для обозначения короткой версии более длинной информации. Это означает «Слишком долго не читал». Я предположил, что есть более короткий вариант ответа, который вы дали, и дал его.
Дон Роллинг
1
Приятное прикосновение, не загружаю дважды. :)
Winner_joiner 01