/usr/local/perl/lib/site_perl/5.8.5/Perl/Critic/Policy/CodeLayout/ProhibitQuotedWordLists.pm |
Perl::Critic::Policy::CodeLayout::ProhibitQuotedWordLists
Conway doesn't mention this, but I think qw()
is an underused
feature of Perl. Whenever you need to declare a list of one-word
literals, the qw()
operator is wonderfully concise, and makes
it easy to add to the list in the future.
@list = ('foo', 'bar', 'baz'); #not ok @list = qw(foo bar baz); #ok
This policy can be configured to only pay attention to word lists with
at least a particular number of elements. By default, this value is 2,
which means that lists containing zero or one elements are ignored.
The minimum list size to be looked at can be specified by giving a
value for min_elements
in .perlcriticrc like this:
[CodeLayout::ProhibitQuotedWordLists] min_elements = 4
This would cause this policy to only complain about lists containing four or more words.
In the PPI parlance, a ``list'' is almost anything with parens. I've
tried to make this Policy smart by targeting only ``lists'' that could
be sensibly expressed with qw()
. However, there may be some edge
cases that I haven't covered. If you find one, send me a note.
This policy was formerly called RequireQuotedWords
which seemed a
little counter-intuitive. If you get lots of ``Cannot load policy
module'' errors, then you probably need to change RequireQuotedWords
to ProhibitQuotedWordLists
in your .perlcriticrc file.
Jeffrey Ryan Thalhammer <thaljef@cpan.org>
Copyright (c) 2005-2007 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.
/usr/local/perl/lib/site_perl/5.8.5/Perl/Critic/Policy/CodeLayout/ProhibitQuotedWordLists.pm |