Оператор SQL CASE, один результат кажется недостижимым

Question

Я пытаюсь разобраться с большим набором хранимых процедур базы данных, которые я поддерживаю, и я пришел к чему-то запутанному. Я не могу сказать, содержит ли он избыточный/бесполезный код, или если мне не хватает какого-то нюанса, поскольку я все еще только промежуточный на языке SQL. Вот часть кода вопроса:

UPDATE #NewValues 
SET ValidNumber = CASE WHEN SP.ModelNumber IS NULL THEN 0 
          ELSE 1 
         END 
FROM #NewValues NV 
LEFT OUTER JOIN Stage.Products SP 
    ON NV.SKU = SP.SKU 
    AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
WHERE NV.SKU IN (
    SELECT NV.SKU 
    FROM #NewValues NV 
    INNER JOIN Stage.Products SP 
     ON NV.SKU = SP.SKU 
     AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
    WHERE NV.ValidNumber = 1 
) 

От работы через это, мне кажется, как будто нет ни одного случая, в котором SP.ModelNumber будет NULL, так как вложенная SELECT заявление никогда не вернется SKUs с SP.ModelNumber NULL результатов , Так что единственное, что может когда-либо сделать, это SET ValidNumber = 1. Это верно? Опять же, я не уверен, что здесь происходит что-то еще, что я не могу вставить в логику, но похоже, что вложенный оператор SELECT будет всем, что вам нужно будет использовать для достижения тех же результатов. Из-за того, как этот код позиционируется в гораздо большем запросе, у меня очень сложное время, пытаясь настроить ситуацию, которая будет соответствовать нормальному использованию, чтобы проверить это, тем более, что я не уверен, что значения ValidNumber будут до этот раздел кода работает.

Answer 1

Это зависит от характера данных. Если один Sku может иметь несколько записей в таблице newvalues, тогда может понадобиться outer join, и было бы возможно установить validnumber = 0.

---

Вот некоторые примеры данных, чтобы попытаться показать (скрипку ниже):

create table newvalues (validnumber int, sku int, modelnumber int, upc int); 
create table products (sku int, modelnumber int); 

insert into newvalues values (1, 1, 1, 1), (0, 1, 1, 1), (null, 1, 2, 2); 
insert into products values (1, 1); 

Как вы можете видеть, есть несколько результатов в таблице newvalues ​​с sku = 1.

---

И тогда update заявление:

UPDATE NewValues 
SET validnumber = CASE WHEN SP.ModelNumber IS NULL THEN 0 
          ELSE 1 
         END 
FROM NewValues NV 
LEFT OUTER JOIN Products SP 
    ON NV.SKU = SP.SKU 
    AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
WHERE NV.SKU IN (
    SELECT NV.SKU 
    FROM NewValues NV 
    INNER JOIN Products SP 
     ON NV.SKU = SP.SKU 
     AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
    WHERE NV.ValidNumber = 1 
); 

• SQL Fiddle Demo

Это обновляет все строки, некоторые до 0, а некоторые до 1, потому что по крайней мере один из тех, кто существовал в подзапрос с inner join. Это не будет отрицать outer join, поскольку существует несколько строк для одного и того же sku, поэтому все они обновляются (даже те, где sp.modelnumber is null)

Answer 2

Оригинальный кодер просто безопасен. Существует вероятность того, что в #NewValues, который не существует в Stage.Product, может существовать элемент, и если это так, LEFT JOIN приведет к тому, что все поля в SP будут NULL, где нет совпадения.

Answer 3

Возможно, я ошибаюсь, но для чего вы используете первый WHERE?

WHERE NV.SKU IN (
    SELECT NV.SKU 
    FROM #NewValues NV 
    INNER JOIN Stage.Products SP 
     ON NV.SKU = SP.SKU 
     AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
    WHERE NV.ValidNumber = 1 
) 

Вы делаете то же самое JOIN здесь, как и в верхней.

Я хотел бы пойти на:

UPDATE #NewValues 
SET ValidNumber = CASE WHEN SP.ModelNumber IS NULL THEN 0 
         ELSE 1 
        END 
FROM #NewValues NV 
INNER JOIN Stage.Products SP 
    ON NV.SKU = SP.SKU 
    AND (NV.ModelNumber = SP.ModelNumber OR NV.UPC = SP.ModelNumber) 
WHERE NV.ValidNumber = 1