PostgreSQL sql状态:SELECT INTO上的42601

Question

好吧，我试图创建一个简单的过程，检查是否有这样的登录名的用户，如果没有，则在users表中添加新行。但遇到了意想不到的问题。

CREATE OR REPLACE FUNCTION register_user(character varying, character varying, character varying,character varying,character varying)
  RETURNS bigint AS
$BODY$
DECLARE
    new_user_login ALIAS FOR $1;
    new_user_password ALIAS FOR $2;
    new_user_email ALIAS FOR $3;
    new_user_first_name ALIAS FOR $4;
    new_user_last_name ALIAS FOR $5;
    login_exist bigint;
    new_user_id bigint;
    emails_array character varying array; --yep, it's array of emails
BEGIN       
    SELECT INTO login_exist count(login) FROM users WHERE users.login = new_user_login;
    IF (login_exist = 0) THEN
        SELECT array_append(emails_array, new_user_email);
        INSERT INTO users (login,password,emails,first_name,last_name) 
        VALUES (new_user_login,new_user_password,emails_array,new_user_first_name,new_user_last_name)
        RETURNING id INTO new_user_id;
        RETURN new_user_id;
    ELSE
        RETURN 0;
    END IF;
END
$BODY$
  LANGUAGE plpgsql VOLATILE
  COST 100;

它在SELECT INTO上返回sql状态：42601。但如果计数为0。当存在登录时，它正确地返回0；问题出在哪里？我甚至不知道这是什么。thx寻求帮助；

Answer 1

此指令：

SELECT array_append(emails_array, new_user_email);

应该失败，因为array_append返回修改后的数组，并且不允许忽略选择的结果。

如果您想附加到源数组中，这应该是：

SELECT array_append(emails_array, new_user_email) INTO emails_array;

然而，这甚至没有必要。您可以将您的函数体简化为：

BEGIN
INSERT INTO users (login,password,emails,first_name,last_name) 
SELECT new_user_login,new_user_password,array[new_user_email],new_user_first_name,new_user_last_name
WHERE NOT EXISTS (select 1 FROM users WHERE users.login = new_user_login)
RETURNING id INTO new_user_id;
RETURN coalesce(new_user_id,0);
END;

Answer 2

除了@Daniel已经回答的问题之外，不要使用过时的ALIAS来命名函数参数。手册：

最好只将其用于覆盖预定名称的目的。

改为使用参数名称：

CREATE OR REPLACE FUNCTION register_user(_login      text
                                       , _password   text
                                       , _email      text
                                       , _first_name text
                                       , _last_name  text)
  RETURNS bigint
  LANGUAGE plpgsql AS
$func$
DECLARE
   _id  bigint;
BEGIN
   INSERT INTO users (login, password, emails, first_name, last_name) 
   SELECT _login, _password, ARRAY[_email], _first_name, _last_name
   WHERE  NOT EXISTS (SELECT 1 FROM users WHERE login = _login)
   RETURNING id INTO _id;
   RETURN COALESCE(_id, 0);
END
$func$;