Sure, grep will simplify that, with a minor change in behavior - maybe for the better.
sub lsearch {
my $item = shift;
grep { $item eq $_[$_] } 0 .. $#_;
}
That will return the empty list for false, and otherwise a list of each index that succeeds
Usage becomes, for your example,
for (reverse lsearch $item, @list) {
splice @list, $_, 1
}
# or
splice @list, $_, 1 for reverse lsearch $item, @list;
so
all instances of $item are removed from the array. The
reverse call is needed to preserve the indexes from being shifted out of sync by
splice.
An alternative is to define a winnow() function to do it all:
sub winnow {
my $item = shift;
grep { not $item eq $_ } @_;
}
@list = winnow $item, @list;
That is maybe less flexible, but I think cleaner.
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.