Рубиновый массив — наибольшее целое число

Проблема в том, что у вас есть оператор puts внутри цикла each, поэтому во время каждой итерации он выводит максимальное число на данный момент. Попробуйте переместить его за пределы цикла each, чтобы получилось следующее:

public

def highest
  highest_number = 0
  each do |number| 
    number = number.to_i
    highest_number = number if number > highest_number
  end
  puts highest_number
end

array = %w{1 2 4 5 3 8 22 929 1000 2}
array.highest

Что дает желаемый результат:

1000

Вы также можете избавить себя от некоторых проблем, используя max_by:

>> a = %w{1 2 4 5 3 8 22 929 1000 2}
=> ["1", "2", "4", "5", "3", "8", "22", "929", "1000", "2"]
>> m = a.max_by { |e| e.to_i }
=> "1000"

Вы также можете использовать другую версию max_by:

m = a.max_by(&:to_i)

чтобы избежать лишнего шума «блока, который просто вызывает метод».

Но это, вероятно, упражнение для изучения блоков Ruby, поэтому использование существующих частей стандартных библиотек не считается. OTOH, полезно знать, что есть в стандартных библиотеках, так что загляните на max_by или max также считается учебным упражнением.

Вы можете сделать это вместо этого и избежать переменной highest_number.

array = %w{1 2 4 5 3 8 22 929 1000 2}

class Array
  def highest
    collect { |x| x.to_i }. \
    sort.                   \
    last.to_i
  end
end

array.highest # 1000

В этом случае collect { |x| x.to_i } также может быть записано как collect(&:to_i).