【SQL SERVER】カーソルによる動的クエリの書き方

Question

いつもお世話になってます。

SQLSERVER 2014で動的クエリを作成中にふと疑問がでたので質問させて頂きます。
SELECT文を動的に作成する場合、以下のコードが存在したとします。(UNION前提)

DECLARE @SQL   NVARCHAR(4000), --SQL文格納用変数
        @CD    VARCHAR(2)      --コード格納用変数

DECLARE CUR CURSOR LOCAL FOR
        SELECT CD FROM TESTDB.dbo.TEST_TBL --CDには1から10の文字列が取れると仮定

OPEN CUR
FETCH NEXT FROM CUR INTO @CD

WHILE @FETCH_STATUS = 0
BEGIN
    SET @SQL = 'SELECT ' + @CD + ' UNION '
    FETCH NEXT FROM CUR INTO @CD
END
SET @SQL = SUBSTRING(@SQL,1,LEN(@SQL) - 7) -- 末尾のUNIONを省く

CLOSE CUR
EXECUTE(@SQL)

OPEN CUR
FETCH NEXT FROM CUR INTO @CD

WHILE @FETCH_STATUS = 0
BEGIN
    SET @SQL = 'SELECT ' + @CD
    FETCH NEXT FROM CUR INTO @CD

    IF @FETCH_STATUS = 0
    BEGIN
        SET @SQL = @SQL + ' UNION ' --FETCH_STATUSが0のままであればUNIONを追加する
    END
END
CLOSE CUR
DEALLOCATE CUR
EXECUTE(@SQL)

何れのループも1から10までの数値を出力します。
個人的な見解としてはループ文の中で完結している後者の書き方がよいではないのか？と考えていますが
ループ内で条件をつけるよりもループ後、末尾の文字列を排除すべきなのでは？という意見もあがり
書き方としてどちらが適しているのかがわからなくなりました。

書き方をご教示頂けますと幸いです。
よろしくお願い致します。

Answer 1

こういった処理に限った訳では無いですけど、仕様に関わるものは除き、if文などによる分岐は少なければ良いと思っているので、前者です。

余談ですが、実際の動的SQLでの実装でもそうしてます。
でも、unionで繋ぐような実装はしたことはありません。（正規化により回避してきた）

Comments

[TAKAYASU]

回答ありがとうございます！
やはり分岐が少ないほうがよいのですね。
大変勉強になりました！
ありがとうございます！