http://www.perlmonks.org?node_id=11125781


in reply to Re: using ref to hash of hash effectively
in thread using ref to hash of hash effectively

Though it's nice to show different ways to do it, I feel obliged to point out that many folks disapprove of using map in void context.

Though map in void context no longer suffers the huge performance penalty it once did, some still consider it poor style (see, for example, the performance-related discussion in this old thread).

I personally applaud the simple stylistic advice summary given in Effective Perl Programming, in the item "Use foreach, map and grep as appropriate", namely:

because I find the code clearer and easier to maintain if everyone in my team follows those four simple rules.

Note that Perl::Critic provides a ProhibitVoidMap policy to allow the code police to prohibit using map in void context.