Search Postgresql Archives

Re: Calculating a moving average (Coding style)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I personally use 3 seperate triggers on most occasions, depending on how
different the action for each seperate action is, it's just easier for me and my
people to logically distinguish the functions that way, but the example in the
7.4 documentation for triggers is given using the form that i wrote this
function in, and is useful when the trigger procedures for all 3 actions do
roughly the same thing, it also makes editing the code a little easier in this
situation.

regards,
matt

Quoting Russell Smith <mr-russ@xxxxxxxxxx>:

> On Mon, 24 Jan 2005 08:32 pm, Alban Hertroys wrote:
> > mstory@xxxxxxxxxxxx wrote:
> > > CREATE OR REPLACE FUNCTION get_bar_avg() RETURNS TRIGGER AS '
> > > DECLARE
> > > bar_record RECORD;
> > > x INTEGER;
> > > y DOUBLE PRECISION := 0;
> > > BEGIN
> > >      IF TG_OP = ''INSERT'' THEN
> > >           y := y + NEW.bar;
> > ...
> > >           RETURN NEW;
> > >      ELSIF TG_OP = ''DELETE'' THEN
> > >            x := 0;
> > ...
> > >            RETURN OLD;
> > >       ELSE
> > >            y := y + NEW.bar;
> > ...
> > >            RETURN NEW;
> > >       END IF;
> > > END;
> > > ' LANGUAGE plpgsql;
> > 
> > I see people do this from time to time. Just out of curiosity, is this 
> > considered good coding style, or is it considered "lazyness"? I'm not 
> > sure what to think of it.
> > 
> > If I would have written this, there would have been 3 triggers w/o the 
> > check on TG_OP. Is there an important drawback to doing so? Is there any 
> > document on "preferred" coding style in PL/PGSQL?
> > 
> > Yes, I'm a bit of a purist...
> > 
> Given you have to define a function for each trigger, my view is why write
> more functions.
> 
> Along with this.  As a C programmer, I would do a few more IF tests in a
> function, rather than
> write another one.  I find that triggers like this are one functional block
> and all go together.
> Then when you update the function, it's all in one place.
> 
> Others may have "better" reasons for why they do it the way they do.  But
> they are mine.
> 
> Regards
> 
> Russell Smith.
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 7: don't forget to increase your free space map settings
> 



---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Postgresql Jobs]     [Postgresql Admin]     [Postgresql Performance]     [Linux Clusters]     [PHP Home]     [PHP on Windows]     [Kernel Newbies]     [PHP Classes]     [PHP Books]     [PHP Databases]     [Postgresql & PHP]     [Yosemite]
  Powered by Linux