Я столкнулся с этим немного назад и задавался вопросом, почему "Начинание" и "Конец" являются потребностью привести к правильным значениям. если оператор (операторы) является одиночным элементом и не требует "Начинания" и "Конца" где несколько операторов в, если бы потребовал бы его и, если опущено генерировал бы ошибку выполнения при попытке создать/изменить процедуру.
Какие-либо идеи относительно того, почему это поведение происходит в MS SQL?????
Спасибо, Craig
- Набор результатов 2 Возвращает правильные значения.
SQL.
Declare @Qty DECIMAL(10,2), @UOM VARCHAR(5), @CasePack Numeric(7,1), @CaseQty Numeric(11, 4), @im_weigh_item SmallInt, @rank_wi_ven_ctg Char(1), @po_qty_uom Char(1), @po_Qty float
Select
-- these 2 Params are Const in this process
@im_weigh_item =0, @rank_wi_ven_ctg = 'C',
-- Set Values
@UOM = 'C' , @po_Qty_uom = 'M', @po_Qty = 3, @casepack =6, @Qty = 2
/*
Check and Set vars. accordingly
This Conditional Block Generates no errors, but the results are incorrect
** NO "Begin" & End"
*/
If(@im_weigh_item=1)
If(@rank_wi_ven_ctg='U')
Select @UOM = 'U'
Else
If(@po_Qty_uom != 'C' )
If(@po_Qty!=@casepack)
Select @UOM = 'U', @Qty = @Qty * @po_Qty
-- Debug
Select @po_Qty_uom as po_Qty_uom, @UOM as UOM, @casepack as casepack, @po_Qty as po_Qty, @Qty as Qty
-- Debug
-- reset vars, test 2
Select @UOM = 'C' , @po_Qty_uom = 'M', @po_Qty = 3, @casepack =6, @Qty =2
/*
*** Works *** Calcs Correctly
Check and Set vars. accordingly
*** This Block uses the "Begin" & "End"
*/
If(@im_weigh_item=1)
begin
If(@rank_wi_ven_ctg='U')
Select @UOM = 'U'
end
Else
begin
If(@po_Qty_uom != 'C' )
If(@po_Qty!=@casepack)
Select @UOM = 'U', @Qty = @Qty * @po_Qty
end
-- Debug
Select @po_Qty_uom as po_Qty_uom, @UOM as UOM, @casepack as casepack, @po_Qty as po_Qty, @Qty as Qty
-- Debug
Я думаю, что If без BEGIN и END должно содержать только ОДНУ инструкцию. Я предлагаю добавлять BEGIN .. END к каждому IF, чтобы обеспечить единообразие кодирования.
Добавление явных BEGIN
s и END
s к вашей сломанной версии дает следующее, что не имеет ту же логику, что и ваша рабочая версия:
If(@im_weigh_item=1)
BEGIN
If(@rank_wi_ven_ctg='U')
BEGIN
Select @UOM = 'U'
END
Else -- this "else" is associated with the wrong "if"
BEGIN
If(@po_Qty_uom != 'C' )
BEGIN
If(@po_Qty!=@casepack)
BEGIN
Select @UOM = 'U', @Qty = @Qty * @po_Qty
END
END
END
END
Оператор ELSE всегда связывается с ближайшим предыдущим IF без ELSE, поэтому ваш
If(@im_weigh_item=1)
If(@rank_wi_ven_ctg='U')
Select @UOM = 'U'
Else
If(@po_Qty_uom != 'C' )
If(@po_Qty!=@casepack)
Select @UOM = 'U', @Qty = @Qty * @po_Qty
интерпретируется как:
If(@im_weigh_item=1) begin
If(@rank_wi_ven_ctg='U') begin
Select @UOM = 'U'
end
Else begin
If(@po_Qty_uom != 'C' ) begin
If(@po_Qty!=@casepack) begin
Select @UOM = 'U', @Qty = @Qty * @po_Qty
end
end
end
end