Perl::Critic::Policy::BuiltinFunctions::RequireSimpleSortBlock - Sort blocks should have a single
Contents
Affiliation
This Policy is part of the core Perl::Critic distribution.
Configuration
This Policy is not configurable except for the standard options.
Copyright
Copyright (c) 2006-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.40.0 2024-10-28 Perl::Critic::P...SimpleSortBlock(3pm)
Description
Conway advises that sort functions should be simple. Any complicated operations on list elements should
be computed and cached (perhaps via a Schwartzian Transform) before the sort, rather than computed inside
the sort block, because the sort block is called "N log N" times instead of just "N" times.
This policy prohibits the most blatant case of complicated sort blocks: multiple statements. Future
policies may wish to examine the sort block in more detail -- looking for subroutine calls or large
numbers of operations.
Name
Perl::Critic::Policy::BuiltinFunctions::RequireSimpleSortBlock - Sort blocks should have a single
statement.
