Re: Problem with Create Domain example - Mailing list pgsql-patches

From David Fetter
Subject Re: Problem with Create Domain example
Date
Msg-id 20050501024528.GA29151@fetter.org
Whole thread Raw
In response to Re: Problem with Create Domain example  (Robert Treat <xzilla@users.sourceforge.net>)
Responses Re: Problem with Create Domain example  (Alvaro Herrera <alvherre@dcc.uchile.cl>)
List pgsql-patches
On Sat, Apr 30, 2005 at 10:08:32PM -0400, Robert Treat wrote:
> On Saturday 30 April 2005 21:14, Alvaro Herrera wrote:
> > On Sat, Apr 30, 2005 at 06:12:27PM -0400, Robert Treat wrote:
> > > --- 167,174 ----
> > >   <programlisting>
> > >   CREATE DOMAIN us_postal_code AS TEXT
> > >   CHECK(
> > > !    VALUE ~ '^\\d{5}$'
> > > ! OR VALUE ~ '^\\d{5}-\\d{4}$'
> > >   );
> >
> > Huh, why not
> >
> > VALUE ~ '^\\d{5}(-\\d{4})?$'
> >
> > ?
>
> Not sure what your driving at here... my point is that the \
> escaping is incorrect in the current examples.
>
> If you want to argue that we could make the check constraint simpler
> (or is that more advanced) that seems like another issue.  IMHO
> having the OR'd checks is better because it shows that you can
> "stack" constraints inside a domain if you want.

That was pretty much my thought.  It's extremely easy to turn regexes
into unmaintainable garbage, whereas the original code is maintainable :)

Cheers,
D
--
David Fetter david@fetter.org http://fetter.org/
phone: +1 510 893 6100   mobile: +1 415 235 3778

Remember to vote!

pgsql-patches by date:

Previous
From: Robert Treat
Date:
Subject: Re: Problem with Create Domain example
Next
From: Alvaro Herrera
Date:
Subject: Re: Problem with Create Domain example