Re: using next in a nested foreach()

[gryphon]

Greetings sandrider,

In addition to using labels as two other monks have suggested already, you could use last to exit the inner loop, thus nexting to the outer loop.

foreach my $one (@one) {
    $one =~ m/\(.\)(.+)\(.\)/;
    foreach my $two (@two) {
        if (($1) and ($two eq $1)) {
            push @result, $one;
            last;
        }
    }
}

TMTOWTDI.

Update: I moved the m// outside the inner foreach due to japhy's comment. No idea why I didn't see that before. Doh.

[Codon]

This solution, of course, doesn't help with existing deeply nested loops. That's where the labels come in really handy. There is the caveat that the misuse of labels can cause very odd behavior (like the sudden termination of you script) if you are not careful (typos).

Also, to further optimize (typing optimization; not necessarily execution optimization) based on japhy's comment, you could try this instead:

push @results, grep {
    grep { $1 eq $_ } @two if ( $_ =~ /\(.\)(.+)\(.\)/ )
} @one;

You get to use a nested grep, but you check all values of @two and don't stop when you find a match.

Update: Since grep will localize $_ and I don't need the outer $_ in the inner grep, I optimized away reassigning the outer $_;