カーソルループで使用するために複数の列をフェッチするにはどうすればよいですか?


88

カーソルループ内で次のSQLスニペットを実行しようとすると、

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

次のメッセージが表示されます。

メッセージ15248、レベル11、状態1、プロシージャsp_rename、行213
パラメータ@objnameがあいまいであるか、要求された@objtype(COLUMN)が間違っています。

何が問題で、どうすれば修正できますか?列名を角かっこで囲ん[]でみました""が、検索結果の一部のように二重引用符で囲んでみました。

編集1-

これがスクリプト全体です。テーブル名を名前変更spに渡すにはどうすればよいですか?列名は多くのテーブルの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
この行は文字列を連結します。文字列の内容を確認できるように印刷してみませんか
Preet Sangha 2012


@testに修飾名が含まれている場合は、アポストロフィである必要があります。同じ仮定が当てはまる場合、right()はテーブル名の最初の3文字を削除します。列名の最後の文字を置き換えたい場合、これはになりますLEFT。set @test = ...を追加して、スクリプトを少し拡張してください。
ニコラマルコビノビッチ2012年

あなたのコードは実際に私の問題を解決するのに役立ちました-ありがとう!
ネビル

回答:


156

こちらが少し変更されたバージョンです。変更はコード解説として記載されています。

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での私のお気に入りの答えの1つ。
ルーベンスマリウッゾ2012

@RubensMariuzzoのおかげで、あなたはあまりにも寛大されている:-)
ニコラMarkovinović

61
TLDR; 次のデータベースをdb_cursorから@var1, @ var2にフェッチ
Don Rolling

7
TLDRは通常、長い情報の短いバージョンを示すために使用されます。Too Long Did n'tReadの略です。私はあなたが与えた答えのより短いバージョンがあることを提案していました、そして私はそれを与えました。
ドンローリング

1
ナイスタッチ、2回フェッチしない。:)
winer_joiner 2015
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.