>>>>> "Mathew" == Mathew Snyder <[EMAIL PROTECTED]> writes:

Mathew> Is there an easy way to determine if an item is in an array without 
iterating
Mathew> through the array and comparing each element to the item in question?

It's a *really* good idea if you're subscribed to this list to type "perldoc
perlfaq" at least once a month (maybe once a week) and at least *look* at the
table of contents.  Then, when you're tackling something new in Perl
and you say "there's gotta be a better way", you'll be reminded of some
keyword in the FAQ (like this one was), and save yourself the posting
time to a mailing list.

Here's the FAQ entry for your exact question:

Found in /usr/libdata/perl5/pod/perlfaq4.pod
  How can I tell whether a certain element is contained in a list or array?
    Hearing the word "in" is an *in*dication that you probably should have
    used a hash, not a list or array, to store your data. Hashes are
    designed to answer this question quickly and efficiently. Arrays aren't.

    That being said, there are several ways to approach this. If you are
    going to make this query many times over arbitrary string values, the
    fastest way is probably to invert the original array and maintain a hash
    whose keys are the first array's values.

        @blues = qw/azure cerulean teal turquoise lapis-lazuli/;
        %is_blue = ();
        for (@blues) { $is_blue{$_} = 1 }

    Now you can check whether $is_blue{$some_color}. It might have been a
    good idea to keep the blues all in a hash in the first place.

    If the values are all small integers, you could use a simple indexed
    array. This kind of an array will take up less space:

        @primes = (2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31);
        @is_tiny_prime = ();
        for (@primes) { $is_tiny_prime[$_] = 1 }
        # or simply  @[EMAIL PROTECTED] = (1) x @primes;

    Now you check whether $is_tiny_prime[$some_number].

    If the values in question are integers instead of strings, you can save
    quite a lot of space by using bit strings instead:

        @articles = ( 1..10, 150..2000, 2017 );
        undef $read;
        for (@articles) { vec($read,$_,1) = 1 }

    Now check whether "vec($read,$n,1)" is true for some $n.

    Please do not use

        ($is_there) = grep $_ eq $whatever, @array;

    or worse yet

        ($is_there) = grep /$whatever/, @array;

    These are slow (checks every element even if the first matches),
    inefficient (same reason), and potentially buggy (what if there are
    regex characters in $whatever?). If you're only testing once, then use:

        $is_there = 0;
        foreach $elt (@array) {
            if ($elt eq $elt_to_find) {
                $is_there = 1;
                last;
            }
        }
        if ($is_there) { ... }



-- 
Randal L. Schwartz - Stonehenge Consulting Services, Inc. - +1 503 777 0095
<merlyn@stonehenge.com> <URL:http://www.stonehenge.com/merlyn/>
Perl/Unix/security consulting, Technical writing, Comedy, etc. etc.
See PerlTraining.Stonehenge.com for onsite and open-enrollment Perl training!

-- 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
http://learn.perl.org/


Reply via email to