change (and update) values of hash

[Jason Lillywhite]

how would I do stuff to each value of a hash and return a modified hash?
h = {"a"=> 3, "b"=> 5}
multiply each value by 2 and return
h = {"a"=> 6, "b"=> 10}

this does what I need but does not modify the hash

h.each {|k,v| h[k]*2}

 

[Jason Lillywhite]

Is this the best solution:

h = {"a"=> 3, "b"=> 5}

x = {}

h.each {|k,v| x[k] = v*2}

=> {"a"=>3, "b"=>5}

h.replace(x)

=> {"a"=>6, "b"=>10}


?

 

[lasitha]

how would I do stuff to each value of a hash and return a modified hash?
h = {"a"=> 3, "b"=> 5}
multiply each value by 2 and return
h = {"a"=> 6, "b"=> 10}

One way (among several):
$: irb
01> h = { 'a' => 3, 'b' => 5 }
--> {"a"=>3, "b"=>5}
02> h.inject(h) {|h, (k, v)| h[k] = v * 2; h }
--> {"a"=>6, "b"=>10}

solidarity,
lasitha.

 

[lasitha]

how would I do stuff to each value of a hash and return a modified hash?
h = {"a"=> 3, "b"=> 5}
multiply each value by 2 and return
h = {"a"=> 6, "b"=> 10}

One way (among several):
$: irb
01> h = { 'a' => 3, 'b' => 5 }
--> {"a"=>3, "b"=>5}
02> h.inject(h) {|h, (k, v)| h[k] = v * 2; h }
--> {"a"=>6, "b"=>10}

I'm being silly (its fun being silly with inject .
This is much easier:
h.each {|k, v| h[k] = v * 2 }

lasitha.

 

[Jason Lillywhite]

This is much easier:
h.each {|k, v| h[k] = v * 2 }

thank you.

I was told it could be bad to change values during iteration. That is
why I proposed doing replace. what do you think?

 

[lasitha]

This is much easier:
h.each {|k, v| h[k] = v * 2 }

thank you.

I was told it could be bad to change values during iteration. That is
why I proposed doing replace. what do you think?

There was a recent thread about this:
http://www.nabble.com/Iterating-a-changing-Hash-under-1.9.1-td22023417.html

I believe assigning values to existing keys is safe.

If in doubt, iterate over the keys instead:
h.keys.each {|k| h[k] = h[v] * 2 }

solidarity,
lasitha.

 

[lasitha]

If in doubt, iterate over the keys instead:
h.keys.each {|k| h[k] = h[v] * 2 }

~~~~
oops. should be h[k] = h[k] * 2
lasitha.

 

[Rob Biedenharn]

If in doubt, iterate over the keys instead:
h.keys.each {|k| h[k] = h[v] * 2 }

~~~~
oops. should be h[k] = h[k] * 2
lasitha.

Or use a little sugar:

h.keys.each {|k| h[k] *= 2 }


Rob Biedenharn http://agileconsultingllc.com
(e-mail address removed)