MySQL函数有条件的始终评估为true

Question

我有一个SQL函数，根据用户激活是否成功，该函数应该返回0或1。 我需要与以下两个表进行交互：

users {user_id, unique email, ...}
user_activation {activation_hash, unique email, ...}

该函数应该评估：

1. 输入的哈希值是否与user_activation中的一行匹配？
2. 用户表中是否还没有相应的电子邮件？
3. 然后将新用户插入用户并删除激活行并返回1，否则返回0

这是我的功能：

delimiter #
create function activate_user
(
    p_activation_hash char(32)
)
returns int
deterministic
begin

    if not exists (
        select 1 from users u
        inner join (
            select email from user_activation where activation_hash = p_activation_hash
        ) ua
        on u.email = ua.email
    )
  then

    -- hash exists but email doesnt so add
    insert into users (email, password_hash, first_name, last_name, company_name)
      select email, password_hash, first_name, last_name, company_name
      from user_activation
      where activation_hash = p_activation_hash
      and expiry_date > now();

    -- delete the activation row(s)
    delete low_priority from user_activation where activation_hash = p_activation_hash;

    return 1;

  end if;

  return 0;

end #
delimiter ;

我的问题是条件条件的值始终为true（尽管即使没有unique关键字，用户表中也只能插入1行）。

谢谢。

Answer 1

尝试将定义从DETERMINISTIC更改为NOT DETERMINISTIC （或将其删除，因为默认是NOT ），因为对于给定的输入，该函数的结果每次都不相同。

一旦使用了（有效）哈希，该函数将为该哈希返回一个不同的值。 您可能会一遍又一遍地看到相同的结果，因为第一次调用该函数返回1而现在每次调用它都将返回相同的值，即使激活记录不再存在。 哈希的输入可能一时无效，而下一时有效（但是不太可能）。

有关更多详细信息，请参见CREATE PROCEDURE语法 。

Answer 2

我今天再次探讨了这个问题，发现一个非常方便的NOT IN查询可以解决问题：

delimiter #
create procedure activate_user
(
    in p_activation_hash char(32),
    inout status int
)
proc_main:begin

    set status = 0;

    if exists (
        select 1 from user_activation
            where activation_hash = p_activation_hash
            and email not in (select email from users)
    )
    then

    -- hash exists but email doesnt so add
    insert into users (email, password_hash, first_name, last_name, company_name)
      select email, password_hash, first_name, last_name, company_name
      from user_activation
      where activation_hash = p_activation_hash
      and expiry_date > now();

    -- delete the activation row(s)
    delete low_priority from user_activation where activation_hash = p_activation_hash;

    set status = 1;

  end if;

end proc_main #
delimiter ;