views:

1498

answers:

5

I have a perl variable $results that gets returned from a service. The value is supposed to be an array, and $results should be an array reference. However, when the array has only one item in it, $results will be set to that value, and not a referenced array that contains that one item.

I want to do a foreach loop on the expected array. Without checking ref($results) eq 'ARRAY', is there any way to have something equivalent to the following:

foreach my $result (@$results) {
    # Process $result
}

That particular code sample will work for the reference, but will complain for the simple scalar.

EDIT: I should clarify that there is no way for me to change what is returned from the service. The problem is that the value will be a scalar when there is only one value and it will be an array reference when there is more than one value.

A: 

I've just tested this with:

#!/usr/bin/perl -w
use strict;

sub testit {

 my @ret = ();
 if (shift){
   push @ret,1;
   push @ret,2;
   push @ret,3;
}else{
  push @ret,"oneonly";
}

return \@ret;
}

foreach my $r (@{testit(1)}){
  print $r." test1\n";
}
foreach my $r (@{testit()}){
   print $r." test2\n";
}

And it seems to work ok, so I'm thinking it has something to do with the result getting returned from the service? If you have no control over the returning service this might be hard one to crack

svrist
+15  A: 

im not sure there's any other way than:

$result = [ $result ]   if ref($result) ne 'ARRAY';  
foreach .....
svrist
A: 

I would re-factor the code inside the loop and then do

if( ref $results eq 'ARRAY' ){
    my_sub($result) for my $result (@$results);
}else{
    my_sub($results);
}

Of course I would only do that if the code in the loop was non-trivial.

Brad Gilbert
+11  A: 

Another solution would be to wrap the call to the server and have it always return an array to simplify the rest of your life:

sub call_to_service
{
    my $returnValue = service::call();

    if (ref($returnValue) eq "ARRAY")
    {
        return($returnValue);
    }
    else
    {
       return( [$returnValue] );
    }
}

Then you can always know that you will get back a reference to an array, even if it was only one item.

foreach my $item (@{call_to_service()})
{
  ...
}
+1  A: 

Well if you can't do...

for my $result ( ref $results eq 'ARRAY' ? @$results : $results ) {
    # Process result
}

or this...

for my $result ( ! ref $results ? $results : @$results ) {
    # Process result
}

then you might have to try something hairy scary like this!....

for my $result ( eval { @$results }, eval $results ) {
    # Process result
}

and to avoid that dangerous string eval it becomes really ugly fugly!!....

for my $result ( eval { $results->[0] } || $results, eval { @$results[1 .. $#{ $results }] } ) {
    # Process result
}

/I3az/

PS. My preference would be to abstract it away in sub ala call_to_service() example given by reatmon.

draegtun
That's not a string eval. And looping over (some expression involving @$results) is very different from looping over (@$results). The former will copy the array (consuming memory); the latter will alias to it (and allow modifying elements via the loop variable).
ysth
@ysth - There is... see "eval $results". My suggestion was to use call_to_service() example given earlier. My answer was a bit of a "tongue in cheek solution" to the restriction imposed by the poster so yes its good to point out its flaws.
draegtun