Проблема с суммированием массива в ванильном рубине

Question

У меня возникли проблемы с суммированием массива. Вот мой существующий код:

pageArray = Array.new 
puts "How many pages long is the book you're reading?" 
pageArray << gets.chomp 
puts "Are you reading any other books right now?" 
yn = gets.chomp 
while yn != "no" do  
    puts "How many pages long is your next book?" 
    pageArray << gets.chomp 
    puts "Are you reading any other books right now?" 
    yn = gets.chomp 
end 
pageSum = pageArray.reduce(:+) 
puts pageSum 

Когда я прохожу и введите значения 100 и 50, окончательное возвращение «10050», а не «150» Мне что-то не хватает?

Answer 1

Код может быть написан более ясно и более рубин подобное. Вот некоторые непроверенный код, который более идиоматический:

page_array = [] 

puts "How many pages long is the book you're reading?" 
page_array << gets.chomp 

loop do 
    puts 'Are you reading any other books right now?' 
    yn = gets.chomp.downcase 
    break if yn == 'no' 

    puts 'How many pages long is your next book?' 
    page_array << gets.chomp.to_i 
end 

page_sum = page_array.reduce(:+) 
puts page_sum 

Примечание:

• переменных в Ruby, в snake_case, notInCamelCase.
• loop do будет работать навсегда. Просто break, когда вы получили значение разрыва.
• вы должны сложить случай значения, используемого как разрыв, чтобы уловить вариации в 'y' и 'Y'.
• найдите время, чтобы отформатировать код, чтобы его можно было легко прочитать, даже для быстрых тестов. Удивительно, как часто код, который, по нашему мнению, просто тест, фактически вводится в производство, так что делайте это правильно в первый раз.

Answer 2

Вы вводите строки и конкатенируете их.

Использование gets.chomp.to_i

Answer 3

Вам нужно изменить строку, которую вы получаете от gets к междунар.

pageArray << gets.chomp.to_i