Одиночный, если для оператора требуется начало и конец в блоке кода - PullRequest
5 голосов
/ 07 июля 2010

Я столкнулся с этим немного назад, и мне было интересно, почему «Начало» и «Конец» должны приводить к правильным значениям. оператор if является одноэлементным и не требует «Begin» и «End», где несколько операторов в if требуют его и если он опущен, это приведет к ошибке выполнения при попытке создать / изменить процедуру.

Любые идеи относительно того, почему это происходит в MS SQL ?????

Спасибо, Крейг

- Набор результатов 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 

Ответы [ 3 ]

4 голосов
/ 07 июля 2010

Добавление явных 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
4 голосов
/ 07 июля 2010

Инструкция 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
4 голосов
/ 07 июля 2010

Я думаю, что If без BEGIN и END должно содержать только одну инструкцию.Я предлагаю добавлять BEGIN .. END к каждому IF, чтобы обеспечить согласованность кода.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...