оператор обновления не выполняется - PullRequest
Новая
       1

оператор обновления не выполняется

0 голосов
/ 01 октября 2011

Я пытаюсь написать запрос UPDATE, который может проверять нулевое значение в одном из полей. У меня три условия, два обязательных, из трех полей. itemCode и itemCheckDigit будут существовать всегда, itemSuffix возможно равен нулю.

Может быть несколько записей, где itemCode и itemCheckDigit вместе равны другим записям, itemSuffix является идентификатором unquie, однако будет существовать только один экземпляр, где itemSuffix равен Null и никогда не будет дублироваться. 

UPDATE item
SET          itemImageFileName = ''' + @fileName + '''
WHERE        (itemCode = ''' + @itemCode5 + ''') AND (itemCheckDigit = ''' + @itemCkDigit + ''') AND (itemSuffix IS NULL);

Это то, что я хотел бы сделать, но это не работает.

1 Ответ

2 голосов
/ 01 октября 2011

Ваша проблема в том, что вы оборачиваете метки вокруг своих параметров в своем выражении, поэтому, когда оно оценивается, оно ищет вещи из таблицы элементов, где itemCode - «игрушка» (обратите внимание на одинарные кавычки)

конкатенация, которую вы делаете, заключается в том, как плохо добавлять параметры в их динамические запросы.Вместо этого снимите галочки примерно так: 

UPDATE
    item
SET
    itemImageFileName = @fileName
WHERE
    (itemCode = @itemCode5 ) 
    AND (itemCheckDigit = @itemCkDigit) 
    AND (itemSuffix IS NULL);

Для обработки необязательных параметров поиска эта статья Билла Грациано превосходна: Использование динамического SQL в хранимых процедурах .Я считаю, что это хороший баланс между тем, чтобы избежать перекомпиляции запросов при установке опции перекомпиляции и избегать сканирования таблиц.

Пожалуйста, наслаждайтесь этим кодом.Он создает временную таблицу для имитации вашей фактической таблицы элементов и загружает ее с 8 строками данных.Я объявляю некоторые параметры, которые вам, скорее всего, не понадобятся, поскольку библиотека ado.net сделает для вас эту магию.

На основе значений, предоставленных для первых 3 параметров, вы получите эквивалентсоответствует строке в таблице и обновит значение имени файла.В моем примере вы увидите, что для строки со всеми NULL имя файла будет изменено с f07.bar на f07.bar.updated.

Оператор print не требуется, но я поместил его туда, чтобы вы могли видетьзапрос, созданный для помощи в понимании шаблона. 

IF NOT EXISTS (SELECT * FROM tempdb.sys.tables T WHERE T.name like '%#item%')
BEGIN
    CREATE TABLE 
        #item
    (
        itemid int identity(1,1) NOT NULL PRIMARY KEY
    ,   itemCode varchar(10) NULL
    ,   itemCheckDigit varchar(10) NULL
    ,   itemSuffx varchar(10) NULL
    ,   itemImageFileName varchar(50) 
    )
    INSERT INTO
        #item
    -- 2008+
    --table value constructor (VALUES allows for anonymous table declaration) {2008}
    --http://technet.microsoft.com/en-us/library/dd776382.aspx
    VALUES 
        ('abc', 'X', 'cba', 'f00.bar')
    ,   ('ac', NULL, 'ca', 'f01.bar')
    ,   ('ab', 'x', NULL, 'f02.bar')
    ,   ('a', NULL, NULL, 'f03.bar')

    ,   (NULL, 'X', 'cba', 'f04.bar')
    ,   (NULL, NULL, 'ca', 'f05.bar')
    ,   (NULL, 'x', NULL, 'f06.bar')
    ,   (NULL, NULL, NULL, 'f07.bar')
END

SELECT * 
FROM #item I;

-- These correspond to your parameters
DECLARE
    @itemCode5 varchar(10)
,   @itemCkDigit varchar(10)
,   @itemSuffx varchar(10)
,   @fileName varchar(50)

-- Using the above table, populate these as
-- you see fit to verify it's behaving as expected
-- This example is for all NULLs 
SELECT
    @itemCode5 = NULL
,   @itemCkDigit = NULL
,   @itemSuffx = NULL
,   @fileName = 'f07.bar.updated'


DECLARE
    @query nvarchar(max)

SET
    @query = N'
UPDATE
    I
SET
    itemImageFileName = @fileName
FROM
    #item I
WHERE
    1=1
' ;


IF @itemCode5 IS NOT NULL
BEGIN
    SET @query += '    AND I.itemCode = @itemCode5 ' + char(13)
END
ELSE
BEGIN
    -- These else clauses may not be neccessary depending on 
    -- what your data looks like and your intentions
    SET @query += '    AND I.itemCode IS NULL ' + char(13)
END

IF @itemCkDigit IS NOT NULL 
BEGIN
    SET @query += '    AND I.itemCheckDigit = @itemCkDigit ' + char(13)
END
ELSE
BEGIN
    SET @query += '    AND I.itemCheckDigit IS NULL ' + char(13)
END

IF @itemSuffx IS NOT NULL 
BEGIN
    SET @query += '    AND I.itemSuffx = @itemSuffx ' + char(13)
END
ELSE
BEGIN
    SET @query += '    AND I.itemSuffx IS NULL ' + char(13)
END

PRINT @query

EXECUTE sp_executeSQL @query   
,   N'@itemCode5 varchar(10), @itemCkDigit varchar(10), @itemSuffx varchar(10), @fileName varchar(50)'
,   @itemCode5 = @itemCode5
,   @itemCkDigit = @itemCkDigit
,   @itemSuffx = @itemSuffx
,   @fileName = @fileName;

-- observe that all null row is now displaying
-- f07.bar.updated instead of f07.bar
SELECT * 
FROM #item I;

до 

itemid  itemCode   itemCheckDigit   itemSuffx    itemImageFileName
1       abc        X                cba          f00.bar
2       ac         NULL             ca           f01.bar
3       ab         x                NULL         f02.bar
4       a          NULL             NULL         f03.bar
5       NULL       X                cba          f04.bar
6       NULL       NULL             ca           f05.bar
7       NULL       x                NULL         f06.bar
8       NULL       NULL             NULL         f07.bar

после 

itemid  itemCode   itemCheckDigit   itemSuffx    itemImageFileName
1       abc        X                cba          f00.bar
2       ac         NULL             ca           f01.bar
3       ab         x                NULL         f02.bar
4       a          NULL             NULL         f03.bar
5       NULL       X                cba          f04.bar
6       NULL       NULL             ca           f05.bar
7       NULL       x                NULL         f06.bar
8       NULL       NULL             NULL         f07.bar.updated
...