Ruby array inject

Question

I am trying to log the average running time of 10 threads by using the inject method but it's giving me this error:

undefined method `+' for #<Thread:0x10b211590 dead> (NoMethodError)
    from /System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/1.8/open-uri.rb:301:in `inject'
    from client_test.rb:13:in `each'
    from client_test.rb:13:in `inject'
    from client_test.rb:13

Here's the code:

require 'open-uri'
program_start_time = Time.now
threads = 10.times.map do
  Thread.new do
    time = Time.now
    open('http://ca.yahoo.com/?p=us').read.length
    Time.now-time
  end
end

threads.map &:join
puts threads.inject() { |sum, e| sum + e.value}.to_f / threads.size
puts Time.now - program_start_time

DarkDust · Accepted Answer · 2012-01-12 03:04:04Z

3

You need to provide an initial value for inject in this case, since if you don't, the initial value is simply the first element in the array:

puts threads.inject(0) { |sum, e| sum + e.value}.to_f / threads.size

answered Jan 12, 2012 at 3:04

DarkDust

92.7k19 gold badges199 silver badges231 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

rampion · Accepted Answer · 2012-01-12 03:04:38Z

2

You didn't provide an initial value for sum in

threads.inject() { |sum, e| sum + e.value}.to_f / threads.size

Fix it with

threads.inject(0) { |sum, e| sum + e.value}.to_f / threads.size

answered Jan 12, 2012 at 3:04

rampion

89.5k49 gold badges208 silver badges321 bronze badges

Collectives™ on Stack Overflow

Ruby array inject

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related