Looks to me like you aren't really that interested in the order of the values in
@owner_grouprights and instead just want to be able to easily check whether specific values are present or not. This is not a job for an array. You want a hash:
#!/usr/bin/env perl
use strict;
use warnings;
use 5.010;
# Don't really need the qw, since these are numbers, but the original
# code explicitly made them strings, so whatever.
my @owner_grouprights = qw( 11 14 );
my %rights_hash;
$rights_hash{$_}++ for @owner_grouprights;
if (exists $rights_hash{11}) {
say "test3";
}
for (4 .. 12) {
if (exists $rights_hash{$_}) {
say "test2";
last; # avoid duplicate messages if more than one is present
}
}
# Or, alternately:
if (grep /4|5|6|7|8|9|10|11|12/, keys %rights_hash) {
say "test2 alt";
}
# Note, though, that the two alternatives are not equivalent. The
# first (if exists) only returns exact matches, while the second
# (grep) also finds substring matches. So, e.g., 114 would match the
# 'grep' version, but not the 'if exists' version. Adding begin/
# end-of-string anchors (^/$ or \A/\Z) to the 'grep' version's regex
# will disallow partial matches if you want that, but don't want the
# explicit 'for' loop.
my $owner_grouprights = join( ' ', sort keys %rights_hash);
my @testarr = (4 .. 12);
for my $tst (@testarr) {
if (exists $rights_hash{$tst}) {
say "$tst hit!";
} else {
say "$tst miss";
}
}