Beefy Boxes and Bandwidth Generously Provided by pair Networks
No such thing as a small change
 
PerlMonks  

Re^4: Merge hash into another hash

by linuxer (Deacon)
on Jul 17, 2009 at 09:13 UTC ( #780973=note: print w/ replies, xml ) Need Help??


in reply to Re^3: Merge hash into another hash
in thread Merge hash into another hash

Hi, let me try to explain with my limited knowledge of perl's internal behaviour ;o)

Your solution converts %target and %source into flat lists, combines it to one list and assigns that list to %target...

Grandfather's solution only inserts the key-value pairs from %source into %target, overwriting already existing pairs.

This is less expensive than converting into lists and reassigning these lists.

I tried a benchmark which shows that Grandfather's solution is faster than yours.

I have to admit, that - until this post - I preferred and used the way you presented (mainly because of readability), but now I have to reconsider ;o)

#!/usr/bin/perl # vim: set ts=4 sw=4 et sta: use strict; use warnings; use Benchmark qw( cmpthese ); my %target; my %source; ## prefill @target{'aa'..'zz' } = 'aa' .. 'zz'; @source{'ww'..'yy', 1..100} = ( 'ww'..'yy', 1..100 ); cmpthese( -1, { grandfather => sub { my %t = %target; @t{keys %source} = values %source; }, sleepyjay => sub { my %t = %target; %t = ( %t, %source ); }, }); __END__ Rate sleepyjay grandfather sleepyjay 598/s -- -62% grandfather 1585/s 165% -- Rate sleepyjay grandfather sleepyjay 649/s -- -62% grandfather 1706/s 163% --


Comment on Re^4: Merge hash into another hash
Select or Download Code

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others pondering the Monastery: (8)
As of 2015-07-07 07:07 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (87 votes), past polls