Perl::Critic::Policy::BuiltinFunctions::ProhibitVoidGrep.3pm

Langue: en

Version: 2008-03-08 (mandriva - 01/05/08)

Section: 3 (Bibliothèques de fonctions)

NAME

Perl::Critic::Policy::BuiltinFunctions::ProhibitVoidGrep

DESCRIPTION

"map" and "grep" are intended to be pure functions, not mutators. If you want to iterate with side-effects, then you should use a proper "for" or "foreach" loop.
   grep{ print frobulate($_) } @list;           #not ok
   print map{ frobulate($_) } @list;            #ok
 
   grep{ $_ = lc $_ } @list;                    #not ok
   for( @list ){ $_ = lc $_  };                 #ok
 
   map{ push @frobbed, frobulate($_) } @list;   #not ok
   @frobbed = map { frobulate($_) } @list;      #ok
 
 

AUTHOR

Jeffrey Ryan Thalhammer <thaljef@cpan.org> Copyright (c) 2005-2008 Jeffrey Ryan Thalhammer. All rights reserved.

This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module.