Ruby FizzBuzz問題 文字列も数値で返ってくる

Question

前提・実現したいこと

FizzBuzzで、
a.３で割り切れる→Fizz
b.５で割り切れる→Buzz
c.３でも５でも割り切れる→FizzBuzz
d.それ以外の数値→数値をそのまま返す
e.数値以外の入力→エラーメッセージ

このようなプログラムを組みたいです。
入力できる数値に制限は設けないこととします。

発生している問題・エラーメッセージ

dまでは正しく返ってくるプログラムを組めました。
しかし、”e.数値以外の入力”がすべてFizzBuzzで返ってきてしまいます。
例）こんにちは→BuzzFizzで返ってきてしまう

該当のソースコード

puts "数字を入力してください"

number = gets.to_i



if  number % 3 == 0 && number % 5 == 0
    puts "FizzBuzz"

elsif number % 5 == 0
    puts "Buzz"

elsif number % 3 == 0
    puts "Fizz"

elsif number
    puts number

else
    puts "Error/Type number"

end

試したこと

入力時にto_iを使用しているからだと推測はできるのですが、
このサイトの質問を遡ったり、自分なりに調べてもうまくいきませんでした。

ご教授お願いします。

Answer 1

def number?(str)
  str =~ /\A\s*[+\-]?[0-9]+\s*\z/
end

puts '数字を入力してください'
str = gets

message = ''
if number?(str)
  number = str.to_i
  message += 'Fizz' if (number % 3).zero?
  message += 'Buzz' if (number % 5).zero?
  message += number.to_s if message.length.zero?
else
  message += 'Error/Type number'
end

puts(message)

0, 1, 2, 3, +1, -1, 1.2 や 前後に SPACE を入れた場合などを試してみてください。

Answer 2

数値の型をしていない文字列に対してはto_iは0を返すので、getsをそのままto_iしては駄目です。
元が何だったか分からなくなってしまう。

案1：
正規表現で数字だけかチェックする。普通のやり方。
number = gets
if /\A[0-9]+\z/ =~ number

案2：
to_i したものを to_s して、元と一致するか確認する。整数であっても一致しないことも。
number = gets
if number == number.to_i.to_s

案3：
Integer（整数でない場合は例外） と rescue を使う。あまり普通でない気がする。
number = (Integer(gets) rescue nil)
if number

Comments

[退会済みユーザー]

一番具体的に回答していただいたので、もう少しお尋ねしたいです。

案１、案２は
number = get.to_iの箇所と置き換えればよいということでしょうか？

[otn]

コードの意味を理解した上で、適用してください。

Answer 3

入力時にto_iを使用しているからだと推測はできるのですが、

そのとおりです。String#to_iは、数値としての解釈に失敗した場合、0を返します（るりま）。

Answer 4

入力時にto_iを使用しているからだと推測はできるのですが、

数値に変換できなければ0と返すのが仕様のようですね。

puts "a".to_i
puts "あいうえお".to_i
puts "abc001".to_i
puts "8792137".to_i

となれば、いきなりto_iするのではなく、正規表現などで「渡された文字列が全て数値かどうか」を先に判断しては？

Comments

[m.ts10806]

to_i だと小数点以下切り上げするようなので、to_iが本当にいいかどうかは要件次第