use #upto rather than Range.new.each