How can I divide two strings in Ruby with gets.chomp?

Question

I try to divide two strings. Here's the code:

puts "Enter a weight in lbs: "
lbs = gets.chomp
stconversion = 14
stone = lbs / stconversion
puts "That is #{stone} stone"

I keep getting this error:

/home/ubuntu/workspace/lbs to stones.rb:4:in `<main>': undefined method `/' for "14\n":String (NoMethodError)

Always say what error you get, never say "I got an error". Read the error message. sometimes it is helpful. — Meier, Oct 10 '15 at 21:11

score 3 · Accepted Answer · edited Oct 12 '15 at 21:33

3

The command gets stands for "get a string". You are trying to divide a string by a number.

Change the line

lbs = gets.chomp

to

lbs = gets.chomp.to_i

to convert the string to an integer, or use to_f if you prefer using floats.

edited Oct 12 '15 at 21:33

Marcos Dimitrio

answered Oct 11 '15 at 09:33

0r4cl3

This is over kill don't need to use `gets.chomp.to_i` can use `gets.to_i` – Casey Robinson Oct 12 '15 at 18:48

Pedro Lobito · Answer 2 · 2015-10-10T19:40:13.633

2

You cannot divide a string, you need to convert it to an int, i.e.:

stone = lbs.to_i / stconversion.to_i

Or convert a string to a float:

stone = lbs.to_f / stconversion.to_f

edited Oct 10 '15 at 19:40

answered Oct 10 '15 at 19:34

Pedro Lobito

2

int ? What language is this? Also, this does integer division in ruby, 13/14 results in 0. – steenslag Oct 10 '15 at 19:36
You use `str.to_i`in ruby – pppery Oct 10 '15 at 19:37
3

@ppperry in this case `to_f`may be a better choice. – steenslag Oct 10 '15 at 19:38
1

Unless integer division is specifically requested, assume that a float is required, which you might then convert. Dependiing on requirements, that might be `lbs.to_f/stconversion`, `(lbs.to_f/stconversion).round(n)`, `(lbs.to_f/stconversion).floor` (or `to_i`) or `(lbs.to_f/stconversion).ceil`. – Cary Swoveland Oct 10 '15 at 20:31
@steenslag You're right. the reason I used `.to_i` is because the OP used the python `int` function instead of the `float` function. – pppery Oct 11 '15 at 13:30

score 0 · Answer 3 · answered Oct 12 '15 at 18:46

0

Its over kill to utilize lbs = gets.chomp.to_i when you can simply utilize lbs = gets.to_i This is the proper way of handling it.

puts "Enter a weight in lbs: " lbs = gets.to_i stconversion = 14 stone = lbs / stconversion puts "That is #{stone} stone"

answered Oct 12 '15 at 18:46

Casey Robinson

3 Answers3