Query: perl::critic::policy::builtinfunctions::prohibitcomplexmappings
OS: debian
Section: 3pm
Format: Original Unix Latex Style Formatted with HTML and a Horizontal Scroll Bar
Perl::Critic::Policy::BuiltinFunctions::ProhibitComplexMUsernContributed Perl Perl::Critic::Policy::BuiltinFunctions::ProhibitComplexMappings(3pm)NAMEPerl::Critic::Policy::BuiltinFunctions::ProhibitComplexMappings - Map blocks should have a single statement.AFFILIATIONThis Policy is part of the core Perl::Critic distribution.DESCRIPTIONThe map function can be confusing to novices in the best of circumstances. Mappings with multiple statements are even worse. They're also a maintainer's nightmare because any added complexity decreases readability precipitously. Why? Because map is traditionally a one-liner converting one array to another. Trying to cram lots of functionality into a one-liner is a bad idea in general. The best solutions to a complex mapping are: 1) write a subroutine that performs the manipulation and call that from map; 2) rewrite the map as a for loop.CAVEATSThis policy currently misses some compound statements inside of the map. For example, the following code incorrectly does not trigger a violation: map { do { foo(); bar() } } @listCONFIGURATIONBy default this policy flags any mappings with more than one statement. While we do not recommend it, you can increase this limit as follows in a .perlcriticrc file: [BuiltinFunctions::ProhibitComplexMappings] max_statements = 2AUTHORChris Dolan <cdolan@cpan.org>CREDITSInitial development of this policy was supported by a grant from the Perl Foundation.COPYRIGHTCopyright (c) 2007-2011 Chris Dolan. This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. perl v5.14.2 2012-06-07Perl::Critic::Policy::BuiltinFunctions::ProhibitComplexMappings(3pm)