メソッドをオブジェクト化して利用したい

Question

メソッドをオブジェクト化して利用したい

Rubyを勉強中の者です。よくわからなくなってきました。「メソッドをオブジェクト化して引数にできる」ということを知って、実験していました。しかし、どうにもうまく実行できていません。原因がよくわかりません。

メソッドの定義①file名：my_global_methods.rb

1.メソッド
入力された値に対して、その値が不適切であれば、もう一度入力を促すメソッド

2.変数
object　➡　入力対象
condition　➡　条件（引数1つのメソッド）
input_value　➡　入力値

def input_repeat(object,condition)
  puts object + "を入力してください："
  input_value = gets.chomp.to_i
  until condition.call(input_value) do
    puts "入力が適切ではないです。もう一度。"
    puts object + "を入力してください："
    input_value = gets.chomp.to_i
  end
  return input_value
end

メソッドの定義②file名:my_math_methods.rb

メソッド
最小の逆元を返すメソッド

<逆元の説明>
例
「法」と呼ばれる数字を7とします。
23を法で割った余りは2になります。
「23になにを掛けたら、法で割って余り1になるか」←この問の答えを「（7を法とする）23の逆元」という。
（答え　最小の逆元は、4です）

変数
mod　➡　法
number　➡　逆元を求めたい値

def mod_inverse(mod,number)

  if mod <= 1 then
    return 0
  else
    a = number % mod
    for index in 1..mod do
      if a*index % mod == 1 then
        return index
      end
    end
  end

  return "none"

end

メソッドの定義③file名:my_math_methods.rb

整数か否かを判定

変数
input_value　➡　入力値

def integer_number(input_value)
  if input_value%1==0 then
    return true
  else
    return false
  end
end

メソッドの定義④file名:my_math_methods.rb

自然数か否かを判定

変数
input_value　➡　入力値

def natural_number(input_value)
  if integer_number(input_value)==true && input_value>=1 then
    return true
  else
    return false
  end
end

ソースコード

require "./my_global_methods"
require './my_math_methods'

sentense1 = "法（自然数）"
condition1 = method(:natural_number)
sentense2 = "逆数を求めたい数（整数）"
condition2 = method(:integer_number)

mod = input_repeat(sentense1,condition1)
number = input_repeat(sentense2,condition2)

if mod_inverse(mod,number) == 0
  puts "法は自然数で入力してください"
elsif mod_inverse(mod,number) == "none"
  puts "逆元はありませんでした"
else
  puts "逆元は#{mod_inverse(mod,number)}でした。"
end

b = gets

条件に沿った入力値であれば、ちゃんと答えを返してくれます。
しかし、条件に沿わない入力をしたときに、繰り返し入力を促してくれません。

ご教授お願いします。

Answer 1

入力メソッド群を書き換えてみました。
整数判定を 正規表現で行うようにしました。
判定メソッドは　true/false を返すのではなく、
正当な入力なら その評価値を、 不正な入力なら nil を返す
ようにしました。

a.rb

# frozen_string_literal: true

def input_with_retry(name, condition)
  message = "#{name}を入力してください："
  loop do
    puts message
    val = condition.call(gets.chomp)
    return val if val

    puts '入力が適切ではないです。もう一度。'
  end
  nil
end

def integer_number(str)
  return str.to_i if /\A[+-]?\d+\z/ =~ str

  nil
end

def natural_number(str)
  val = integer_number(str)
  return val if val&.positive?

  nil
end

a = input_with_retry('整数', method(:integer_number))
puts "a: #{a}"
b = input_with_retry('自然数', method(:natural_number))
puts "b: #{b}"

実行例

Comments

[Sho.ik]

回答ありがとうございます！

まだ正規表現は勉強していませんが、かなりすっきり記述できるんだとわかりました！

Answer 2

gets.chomp.to_iと整数にしているので、"3.5"と入力しても3になるからでは？

Comments

[Sho.ik]

ありがとうございます！解決できました！

入力の型を決めないといけないのですね！