On Wed, 2012-09-05 at 21:15 -0400, David Johnston wrote:
> On Sep 5, 2012, at 19:02, Sergio Basurto <sbasu...@soft-gator.com>
> wrote:
> 
> 
> 
> > On Tue, 2012-09-04 at 21:58 -0400, David Johnston wrote: 
> > 
> > > On Sep 4, 2012, at 21:39, Sergio Basurto <sbasu...@soft-gator.com> wrote:
> > > 
> > > > I am using regexp_matches in a function like this
> > > > 
> > > > create or replace function test (v_string       in text) returns 
> > > > varchar as $$
> > > > declare
> > > > i_strings               text[];
> > > > i_string                text[];
> > > >         
> > > > i_strings := 
> > > > regexp_matches(v_string,E'[a-zA-Z0-9:\\s\\-\\.#%]*:[A-Za-z0-9\\s\\-\\.#%]+','g');
> > > 
> > > You can store a single array value into i_strings.  It does not magically 
> > > convert a multi-row result into an array.  You can use ARRAY_AGG to do so 
> > > or execute the query directly as part of the loop while using a "record" 
> > > variable to store the current row's value(s). 
> > > 
> > > >         
> > > > -- Then I use  the results
> > > > foreach i_string slice 1 in array i_strings
> > > > loop
> > > >         raise notice 'row = %',i_string;
> > > > end loop;
> > > >         
> > > > when I run the function like this:
> > > > 
> > > > select test('1:Warehouse1;2:Warehouse2;');
> > > >         
> > > > postgresql complains:
> > > > ERROR:  query "SELECT 
> > > > regexp_matches(v_string,E'[a-zA-Z0-9:\\s\\-\\.#%]*:[A-Za-z0-9\\s\\-\\.#%]+','g')"
> > > >  returned more than one row
> > > > 
> > > > Why postgres is sending the ERROR?
> > > >         
> > > > Off course I am expecting more than one row!, that's why is in a 
> > > > foreach loop in the first place.
> > > >         
> > > > If I run:
> > > > select 
> > > > regexp_matches('1:Warehouse1;2:Warehouse2;',E'[a-zA-Z0-9:\\s\\-\\.#%]*:[A-Za-z0-9\\s\\-\\.#%]+','g');
> > > > regexp_matches 
> > > > ----------------
> > > > {1:Warehouse1}
> > > > {2:Warehouse2}
> > > > (2 rows)
> > > > 
> > > > I am doing something wrong?
> > > 
> > > Note that because you do not use grouping in your expression there is 
> > > only a single array "cell" in each row - but there could be more than one 
> > > in which case your for-each above would effectively loop through each 
> > > sub-component of the match.
> > > 
> > > > 
> > > > Regards,
> > > >        
> > > 
> > > David J.
> > > 
> > 
> > Thanks for your response David, but my doubt arise because if I use
> > this
> > 
> > i_strings text[] := array [[1:Warehouse1],[2:Warehouse2]];
> > 
> > loops without problem. Is not the same thing?
> > 
> > it prints:
> > 
> > NOTICE: row = {1:Warehouse1}
> > NOTICE: row = {2:Warehouse2}
> 
> 
> 
> A 2-dimensional array is not the same as a set of 1-dimensional
> arrays.
> 
> 
> David J.

Thank you David for all your help,

I got it finally thanks your explanation, so the code that works for me
is:

create or replace function test (v_string       in text) returns varchar
as $$
declare
i_strings              refcursor := null;
i_string                text[];
i_query                text;

begin

    i_query := 'select regexp_matches('''||v_string||''',E''[a-zA-Z0-9:\
\s\\-\\.#%]*:[A-Za-z0-9\\s\\-\\.#%]+'',''g'')';
    open i_strings for execute i_query;

    if i_strings is not null then
        loop    fetch i_strings into i_string;
        exit when not found;
                                raise notice 'row =
%',i_string;                      
        end loop;
        close i_strings;
    end if;
    return 0;
end;
$$ LANGUAGE plpgsql;

Thanks again David.

Kind Regards,

Reply via email to