在更新语句中传递给 RIGHT 函数的长度参数无效

Question

如果我

尝试同时设置两列，有人知道为什么我的更新不起作用吗？

  UPDATE mytable
    SET [Customer] = RIGHT([Customer], CHARINDEX('#', REVERSE([Customer])) -1) ,
        [Segment] = RIGHT([Segment], CHARINDEX('#', REVERSE([Segment])) -1)
    WHERE CHARINDEX('#', [Customer]) > 0 OR
              CHARINDEX('#', [Segment]) > 0

如果我只更新一列来运行查询，那么它就会按预期工作。如果我同时运行两者，则会出现以下错误：

Invalid length parameter passed to the RIGHT function. The statement has been terminated.

我知道当 CHARINDEX 返回 0 时可能会发生这种情况，但我正试图使用 WHERE 子句来控制它。

Answer 1

您在WHERE子句中有一个OR。 如果您真的想控制它，请将其替换为AND：

UPDATE mytable
    SET [Customer] = RIGHT([Customer], CHARINDEX('#', REVERSE([Customer])) -1) ,
        [Segment] = RIGHT([Segment], CHARINDEX('#', REVERSE([Segment])) -1)
    WHERE CHARINDEX('#', [Customer]) > 0 AND
          CHARINDEX('#', [Segment]) > 0

或者：

UPDATE mytable
    SET [Customer] = (CASE WHEN Customer LIKE '%#%'
                           THEN RIGHT([Customer], CHARINDEX('#', REVERSE([Customer])) -1)
                           ELSE Customer
                      END)
        [Segment] = (CASE WHEN Segment LIKE '%#%'
                          THEN RIGHT([Segment], CHARINDEX('#', REVERSE([Segment])) -1)
                          ELSE Segment
                     END)
    WHERE Customer LIKE '%#%' OR Segment LIKE '%#%';

Answer 2

错误消息的原因是将负值传递给RIGHT函数。从波尔：

integer_expression 是一个正整数，它指定将返回多少character_expression字符。如果integer_expression为负数，则返回错误。

您的WHERE条件包含一个OR，这将允许在"其他"列包含"#"的情况下包含没有"#"的行。

尝试将您的OR修改为AND：

UPDATE mytable
    SET [Customer] = RIGHT([Customer], CHARINDEX('#', REVERSE([Customer])) -1) ,
        [Segment] = RIGHT([Segment], CHARINDEX('#', REVERSE([Segment])) -1)
    WHERE CHARINDEX('#', [Customer]) > 0 AND
              CHARINDEX('#', [Segment]) > 0

Answer 3

如果你想同时更新两者，有时一个或另一个会有"#"，那么你可以使用：

UPDATE MyTable
SET
    Customer = CASE
                   WHEN CHARINDEX('#', Customer) > 0 THEN RIGHT(Customer, CHARINDEX('#', REVERSE(Customer)) -1)
                   ELSE Customer
               END,
    Segment = CASE
                   WHEN CHARINDEX('#', Segment) > 0 THEN RIGHT(Segment, CHARINDEX('#', REVERSE(Segment)) -1)
                   ELSE Segment
               END
WHERE
    CHARINDEX('#', Customer) > 0 OR
    CHARINDEX('#', Segment) > 0

当然，或者你可以把它分解成两个UPDATE语句，但我认为这个语句会运行得更快，因为它只需要通过表格进行一次传递。

Answer 4

也许是这样的：

    UPDATE mytable SET 
    [Customer] = Case When CHARINDEX('#', [Customer]) > 0 Then RIGHT([Customer], CHARINDEX('#', REVERSE([Customer])) -1) Else [Customer] End ,
    [Segment] = Case When CHARINDEX('#', [Segment]) > 0 Then RIGHT([Segment], CHARINDEX('#', REVERSE([Segment])) -1) Else [Segment] End
    WHERE
    CHARINDEX('#', [Customer]) > 0 OR
    CHARINDEX('#', [Segment]) > 0