I\'m looking for presence of an element in a list.
In Python there is an in
keyword and I would do something like:
if element in list:
This blog post discusses the best answers to this question.
As a short summary, if you can install CPAN modules then the best solutions are:
if any(@ingredients) eq 'flour';
or
if @ingredients->contains('flour');
However, a more usual idiom is:
if @any { $_ eq 'flour' } @ingredients
which i find less clear.
But please don't use the first() function! It doesn't express the intent of your code at all. Don't use the "Smart match" operator: it is broken. And don't use grep() nor the solution with a hash: they iterate through the whole list. While any() will stop as soon as it finds your value.
Check out the blog post for more details.
PS: i'm answering for people who will have the same question in the future.
On perl >= 5.10 the smart match operator is surely the easiest way, as many others have already said.
On older versions of perl, I would instead suggest List::MoreUtils::any.
List::MoreUtils
is not a core module (some say it should be) but it's very popular and it's included in major perl distributions.
It has the following advantages:
in
does) and not the value of the element, as List::Util::first
does (which makes it hard to test, as noted above);grep
, it stops at the first element which passes the test (perl's smart match operator short circuits as well);Here is an example which works with any searched (scalar) value, including undef
:
use List::MoreUtils qw(any);
my $value = 'test'; # or any other scalar
my @array = (1, 2, undef, 'test', 5, 6);
no warnings 'uninitialized';
if ( any { $_ eq $value } @array ) {
print "$value present\n"
}
(In production code it's better to narrow the scope of no warnings 'uninitialized'
).
grep is helpful here
if (grep { $_ eq $element } @list) {
....
}
The smartmatch family of features are now experimental
Smart match, added in v5.10.0 and significantly revised in v5.10.1, has been a regular point of complaint. Although there are a number of ways in which it is useful, it has also proven problematic and confusing for both users and implementors of Perl. There have been a number of proposals on how to best address the problem. It is clear that smartmatch is almost certainly either going to change or go away in the future. Relying on its current behavior is not recommended.
Warnings will now be issued when the parser sees ~~, given, or when.
The smart match ~~ operator.
if( $element ~~ @list ){ ... }
if( $element ~~ [ 1, 2, 3 ] ){ ... }
You could also use the given/when construct. Which uses the smart match functionality internally.
given( $element ){
when( @list ){ ... }
}
You can also use a for
loop as a "topicalizer" ( meaning it sets $_
).
for( @elements ){
when( @list ){ ... }
}
One thing that will come out in Perl 5.12 is the ability to use the post-fix version of when
. Which makes it even more like if
and unless
.
given( $element ){
... when @list;
}
You might think you can get away with using List::Util::first, but there are some edge conditions that make it problematic.
In this example it is fairly obvious that we want to successfully match against 0
. Unfortunately this code will print failure
every time.
use List::Util qw'first';
my $element = 0;
if( first { $element eq $_ } 0..9 ){
print "success\n";
} else {
print "failure\n";
}
You could check the return value of first for defined-ness, but that will fail if we actually want a match against undef
to succeed.
You can safely use grep however.
if( grep { $element eq $_ } 0..9 ){ ... }
This is safe because grep
gets called in a scalar context. Arrays return the number of elements when called in scalar context. So this will continue to work even if we try to match against undef
.
You could use an enclosing for
loop. Just make sure you call last
, to exit out of the loop on a successful match. Otherwise you might end up running your code more than once.
for( @array ){
if( $element eq $_ ){
...
last;
}
}
You could put the for
loop inside the condition of the if
statement ...
if(
do{
my $match = 0;
for( @list ){
if( $element eq $_ ){
$match = 1;
last;
}
}
$match; # the return value of the do block
}
){
...
}
... but it might be more clear to put the for
loop before the if
statement.
my $match = 0;
for( @list ){
if( $_ eq $element ){
$match = 1;
last;
}
}
if( $match ){ ... }
If you're only matching against strings, you could also use a hash. This can speed up your program if @list
is large and, you are going to match against %hash
several times. Especially if @array
doesn't change, because then you only have to load up %hash
once.
my %hash = map { $_, 1 } @array;
if( $hash{ $element } ){ ... }
You could also make your own subroutine. This is one of the cases where it is useful to use prototypes.
sub in(&@){
local $_;
my $code = shift;
for( @_ ){ # sets $_
if( $code->() ){
return 1;
}
}
return 0;
}
if( in { $element eq $_ } @list ){ ... }