Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl: the Markov chain saw
 
PerlMonks  

Re^2: how to speed lookups?

by blazar (Canon)
on Nov 12, 2008 at 19:04 UTC ( #723258=note: print w/ replies, xml ) Need Help??


in reply to Re: how to speed lookups?
in thread how to speed lookups?

Also, using split does more work than you need. Try replacing

I personally believe that people indeed tend to overuse split too much. But at the same time IIRC the latter is optimized if an explicit LIMIT parameter and maybe (I only have a vague memory of this, and I admit I may just have "invented" it...) even simply if a specific number of items are assigned, i.e.:

my ($searchfield) = split /'/, $line;

Update: as far as the last point is concerned, I definitely stand corrected, as per ikegami's remark, which I thoroughly trust.

But at this point one should certainly do a benchmark to be sure, and I don't have the slightest intention of doing so, especially given that the question asked by the OP has bigger issues to the point of not really being understandable at all, as far as I'm concerned...

--
If you can't understand the incipit, then please check the IPB Campaign.


Comment on Re^2: how to speed lookups?
Select or Download Code
Re^3: how to speed lookups?
by ikegami (Pope) on Nov 12, 2008 at 19:39 UTC

    While the OP mentions fetching the first field, you'll notice the OP used $fields[1], not $fields[0]. I believe that means he's using split to parse a single-quoted string. That's not what I'd call an appropriate use of split.

    So in this case, the optimal split would be

    my $searchfield = ( split /'/, $line, 3 )[1];

    Using split requires matching twice, creating three variables and copying the entire string.

    my ($searchfield) = $line =~ /^'([^']*)/;

    Using the match operator requires matching once, creating one variable and copying only the field. Actual performance aside, it's definitely a cleaner process.

    Finally, split will be very bad at handling escaping when the OP discovers the need for it.

    maybe ([...]) even simply if a specific number of items are assigned

    An operand doesn't know what the caller will do with the returned list, so your "maybe" doesn't apply. If it behaves as you think, the following snippet will result in $c being assigned 2 when it should be assigned 3.

    $c = ($f1,$f2) = (4,5,6);

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://723258]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (14)
As of 2014-10-23 13:51 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    For retirement, I am banking on:










    Results (125 votes), past polls