Ошибка синтаксиса примерное положение begin

CREATE FUNCTION RETURNONE(DATE)
BEGIN
  RETURN 1;
END

ERROR:  syntax error at or near "BEGIN"
LINE 2: BEGIN
        ^
END

3

PREVENT YOUR SERVER FROM CRASHING!

Never again lose customers to poor server speed! Let us help you.

Our server experts will monitor & maintain your server 24/7 so that it remains lightning fast and secure.

GET STARTED

licensed under cc by-sa 3.0 with attribution.

Я пытаюсь создать функцию, которая суммирует результат всех значений запроса и сравнивает ее с рядом других простых запросов.

Это то, что у меня есть, однако я получаю синтаксическую ошибку около начала (2-я строка):

CREATE FUNCTION trigf1(sbno integer, scid numeric(4,0)) RETURNS integer
BEGIN
declare sum int default 0;
declare max as SELECT totvoters FROM ballotbox WHERE cid=scid AND bno=sbno;

for r as
SELECT nofvotes FROM votes WHERE cid=scid AND bno=sbno;
do
set sum = sum + r.nofvotes;
end for

if sum > max
then return(0);
else
return(1);
END

Это приводит к:

Ошибка синтаксиса рядом с ‘BEGIN’

Я использую postgreSQL и pgadminIII (на всякий случай это актуально).

Я понятия не имею, почему я получаю эту ошибку, все, кажется, точно так же, как определяется учебник. (Это текстовая книга, которую я использую: http://digilib.usu.ac.id/buku/107859/Database-systems-concepts,-6th-ed.html)

Я не знаю, какой “учебник” вы использовали, но если все, что вы написали, точно так же, как в этой книге, эта книга совершенно неверна:

CREATE FUNCTION trigf1(sbno integer, scid numeric(4,0)) 
    RETURNS integer
AS         -- error #1: no AS keyword
$body$     -- error #2: use dollar quoting to specify the function body as a string
DECLARE    -- error #3: the declare block comes before the actual code
   sum_ integer := 0; -- error #5: you can't use a reserved keyword as a variable
   max_ integer;      -- error #6:  you can't initialize a variable with a select,
   r   record;   -- you need to declare the record for the cursor loop
BEGIN
   select totvoters
     into max_
   from ballotbox 
   WHERE cid=scid AND bno=sbno;

    -- error #7: the syntax for a loop uses IN not AS
    -- error #8: you need to declare R before you can use it
    -- error #9: the SELECT for a cursor loop must NOT be terminated with a ;
    FOR r IN SELECT nofvotes FROM votes WHERE cid=scid AND bno=sbno
    loop  -- error #10: you need to use LOOP, not DO

        sum_ := sum_ + r.nofvotes;  -- error #11: you need to use := for an assignment, not SET
    end loop; -- error #12: it END LOOP
              -- error #13: you need to terminate the statement with a ;

    if sum_ > max_ then 
       return 0;
    else
       return 1;
    end if; -- error #14: an END if is required
END;
$body$
language plpgsql; -- error #14: you need to specify the language

В руководстве указано все это:

• ошибка № 1, № 2: http://www.postgresql.org/docs/current/static/sql-createfunction.html
• ошибка № 3: http://www.postgresql.org/docs/current/static/plpgsql-structure.html
• ошибка № 6: http://www.postgresql.org/docs/current/static/plpgsql-declarations.html
• ошибка # 7, # 8, # 9, # 10, # 12: http://www.postgresql.org/docs/current/static/plpgsql-control-structures.html#PLPGSQL-RECORDS-ITERATING
• ошибка № 11: http://www.postgresql.org/docs/current/static/plpgsql-statements.html#PLPGSQL-STATEMENTS-ASSIGNMENT
• ошибка № 14: http://www.postgresql.org/docs/current/static/plpgsql-control-structures.html#PLPGSQL-CONDITIONALS

Весь цикл FOR не нужен и крайне неэффективен. Его можно заменить на:

SELECT sum(nofvotes)
  into sum_
FROM votes 
WHERE cid=scid AND bno=sbno;

Postgres имеет собственный булевский тип, лучше использовать это вместо целых чисел. Если вы объявите функцию как returns boolean, последняя строка может быть упрощена до

return max_ > sum_;

Эта часть:

 select totvoters
   into max_
 from ballotbox 
 WHERE cid=scid AND bno=sbno;

будет работать только в том случае, если cid, bno уникален в камере голосования. В противном случае вы можете получить ошибку во время выполнения, если выбор возвращает более одной строки.

Предполагая, что выбор на ballotbox использует первичный (или уникальный) ключ, вся функция может быть упрощена до небольшого выражения SQL:

create function trigf1(sbno integer, scid numeric(4,0))
  returns boolean
as
$body$
  return (select totvoters from ballotbox WHERE cid=scid AND bno=sbno) > 
         (SELECT sum(nofvotes) FROM votes WHERE cid=scid AND bno=sbno);
$body$
language sql;

Я не человек postgresSQL, но я бы подумал

declare max as SELECT totvoters FROM ballotbox WHERE cid=scid AND bno=sbno;

должно быть

declare max := SELECT totvoters FROM ballotbox WHERE cid=scid AND bno=sbno;

Тело функции должно быть строкой после ключевого слова as, то есть as 'code...'. Обычно используется строка с кавычками в долларах:

CREATE FUNCTION trigf1(sbno integer, scid numeric(4,0)) RETURNS integer
AS $$
BEGIN
declare sum int default 0;
declare max as SELECT totvoters FROM ballotbox WHERE cid=scid AND bno=sbno;

for r as
SELECT nofvotes FROM votes WHERE cid=scid AND bno=sbno;
do
set sum = sum + r.nofvotes;
end for

if sum > max
then return(0);
else
return(1);
END
$$