Re^4: Can this script be Optimized?

No, just pointing out that map (and grep) are basically equivalent to for. But like many other features of Perl, they can make some things easier to code and easier to read.

Your point is correct. One additional detail is that a new array (actually list) is always created.

So, it would really be more like:

{
    my $newarray;
    for (@array)
    {
        push @newarray, expr;
    }
    @newarray;
}
[download]

Comment on Re^4: Can this script be Optimized? Download Code

Replies are listed 'Best First'.
Re^5: Can this script be Optimized? by kcott (Archbishop) on May 01, 2014 at 18:51 UTC
"One additional detail is that a new array (actually list) is always created." That sounds like you're referring to map; if so, that changed in `5.8.1` (over a decade ago). From perl581delta: Miscellaneous Enhancements: "map is now context aware, and will not construct a list if called in void context." I don't follow what you're trying to show with that code in the anonymous block. What does the "it" in "So, it would really be more like:" refer to? Is the anonymous block supposed to be a `do {...}` block? And `$newarray` is probably a typo. -- Ken	[reply] [d/l] [select]

Replies are listed 'Best First'.

Re^5: Can this script be Optimized?
by kcott (Archbishop) on May 01, 2014 at 18:51 UTC

"One additional detail is that a new array (actually list) is always created."

That sounds like you're referring to map; if so, that changed in 5.8.1 (over a decade ago).

From perl581delta: Miscellaneous Enhancements:

"map is now context aware, and will not construct a list if called in void context."

I don't follow what you're trying to show with that code in the anonymous block. What does the "it" in "So, it would really be more like:" refer to? Is the anonymous block supposed to be a do {...} block? And $newarray is probably a typo.

-- Ken

[reply]
[d/l]
[select]