条件分岐を無くしたい

Question

rubyを使ってあるプログラムを書いています。
ユーザーから入力を受け取り、その入力された値によって、出力を変えるプログラムを書こうとしているのですが、switch文で条件分岐しており、今後条件が増えたときにコードが増えそうで、嫌な感じがします。
各条件分岐で使用するクラスは性質としては似たものなので、共通の親クラスを持たせています。
何かこの共通するクラスをうまく使って、条件分岐を消す方法はないでしょうか？？
ポリモーフィズムを活用し、オブジェクトに振る舞いを持たせる方法も考えたのですが、どのオブジェクトを使用するかによって、結局条件分岐が必要なので、難しいです。

    case input
    when "A" then
      # Aについての処理
    when "B" then
      # Bについての処理
    when "C" then
      # Cについての処理
    end

class Parent
end

class A < Parent
end

class B < Parent
end

class C < Parent
end

Answer 1

他の回答にあるように、表引きで条件分岐を置き換えるのが一般的だと思います。
この回答では、各クラス(またはベースクラス) に条件判定を持たせるという方法で書いてみました。

pr.rb

class BaseCommand
  def check(commnd) = commnd == self.class.name
  def run() = raise('Abstract Class !!')
end

class A < BaseCommand
  def run
    puts 'run by A.'
  end
end

class B < BaseCommand
  def run
    puts 'run by B.'
  end
end

class C < BaseCommand
  def run
    puts 'run by C.'
  end
end

def execute(command)
  ACTORS.each do |cls|
    return cls.run if cls.check(command)
  end
  puts ' --- ignore the command.'
end

# ---- Main ------
ACTORS = [A.new, B.new, C.new].freeze

loop do
  print('input command:')
  x = gets
  break if x.nil?

  execute(x.chomp)
end

実行例

Ctrl-D の入力で終了します。(Ctrl-C で強制終了しても良いですけど)

Answer 2

「についての処理」
はみな同じmethod名 にします。 仮に run

で、Parent では

def run
  raise 適当なError
end

しておきます。
calss A,B,C ではしかるべく定義する。
で、ホワイトリストを作るのが嫌なら
eval(input).run
しかし、Parentを入力したときの問題は回避できますが
Dを入れたときに頓挫。
すると ABCのいずれであるか、を判断することになるので
結局ホワイトリストが必要になる、か eval(input).run に対する例外対策を行う。

なんてより otn さんの FOO={"A"=>A, "B"=>B, "C"=>C} でしょうね。
これに default で Parent を入れておけば、ABC以外はParentになって、raise 適当なError になります。
質問の case文ではABC以外は何もしないでよいようなので、でしたらParentのdef run は空にしておく。

Answer 3

Class.const_get でクラスオブジェクトを取得してインスタンスメソッド(or クラスメソッド)を実行するなど。

class Parent
  def greeting()
    "I'm a abstract method."
  end
end

class A < Parent
  def greeting()
    "Hi, I'm #{self.class.name}, now #{Time.now}."
  end
end

class B < Parent
  def greeting()
    "Hi, I'm #{self.class.name}, rolling dice and got #{rand(1..6)}."
  end
end

class C < Parent
  def greeting()
    "Hi, I'm #{self.class.name}, 1 + 1 = #{1 + 1} // Q.E.D."
  end
end

class D
end

# testing
["A", "B", "C", "D", "Parent", "String", "", 1, true, nil].each{|input|
  output = ((Class.const_get(input) < Parent rescue false) || nil) &&
           Class.const_get(input).new.greeting
  printf("%8p: %p\n", input, output)
}

#
     "A": "Hi, I'm A, now 2021-12-01 03:21:21 +0900."
     "B": "Hi, I'm B, rolling dice and got 2."
     "C": "Hi, I'm C, 1 + 1 = 2 // Q.E.D."
     "D": nil
"Parent": nil
"String": nil
      "": nil
       1: nil
    true: nil
     nil: nil

Comments

[otn]

"Parent"という入力をエラーにしないといけないと思いますが。
また、"String"等の入力だとNoMethodErrorですね。
結局、ホワイトリストを持たないと複雑になるだけではないでしょうか。

Answer 4

具体的なことが書かれてないので、いまいちやりたいことが不明ですが、
文字列を起因として処理を分ける場合に、明示的に分岐しないのなら表を引くくらいでしょうか。
あるいはevalっぽいことをするか。

Comments

[oeiqgfodgfhps]

表を引くとは何でしょうか？？

[otn]

A B Cというクラスが定義されていたとして、
FOO={"A"=>A, "B"=>B, "C"=>C}

if FOO[input]
x = FOO[input].new
else
～～
end

みたいな。分岐を書く代わりに表を書く必要がありますが。