[tetroel]

so i have an array

m=Array.new(20)
for i in 1..20
 m[i]=rand(1001)
end
puts m.min

if i try to run it, ruby says:



What's wrong with my code?

[Atli]

You are looping through 1 to 20, assigning each element in the array at those indexes a random number. However arrays are zero-indexed, which means you are skipping the first index: 0. This is causing the min call to fail.

To fix this, loop through 0 to 19 instead.

You can also simplify this by doing:

m=Array.new(20) {|i| rand(1001)}
puts m.min

Each element in the new array will then be initialized with a random number.

[xclite]

So, arrays are indexed starting at 0. You create an array of size 20. Then you initialize the elements at index 1 to 20 to a number. Then you calculate the minimum, which includes the element at index 0...

Your array looks like this:
[nil, rand(1001), rand(1001).... and so on]

You want to change the for i in 1..20 to for i in 0..19

Atli's solution is more idiomatic, while mine just changes the numbers.

This post has been edited by xclite: 24 March 2012 - 01:26 PM

[tetroel]

Thank you guys very much!!!