インスタンス変数に代入した文字列をつかって、メソッドを生成したい

Question

###実現したいこと
インスタンス変数に代入した文字列をつかって、メソッドを生成したい。

理想の結果

Test.new.docByID("dd") => "doc.getElementByIdee"

現在

メソッド名にインスタンス変数名が入っていないメソッドが生成されている。

Test.new.ByID("ee") => "doc.getElementByIdee"

###発生している問題・エラーメッセージ
Test.new.docByID("dd") => error

エラーメッセージ
undefined method `docByID' for #<Test:0x007f990a03c910 @obj="doc"> (NoMethodError)

###該当のソースコード

class Test
  attr_accessor :obj
  
  def initialize
    @obj = "doc"
  end

  gets = {ByID: "ById", ByClassName: "ByClassName", ByTagName: "ByTagName"}
  gets.each do |by, val|
    define_method "#{@obj}#{by}" do |base|
      p @obj << base
    end
  end
 
end

###補足情報(言語/FW/ツール等のバージョンなど)
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-darwin15]

Comments

[Zuishin]

なぜメソッドを生成しなければなりませんか？ 共通のメソッドに引数を渡すことではいけませんか？

[momijiMac]

必要性は今のところないのですが、メソッドの生成においてできることとできないことを勉強しています。現在の例では、objc ="doc"ですが、必要に応じて、obj="wind"としてメソッドは生成できないのかと思ったしだいです。

Answer 1

gets.each以下はクラス内に直接書いてあるので、@objはクラスインスタンス変数を指してしまい、インスタンス変数を読んではくれません。

そもそも、「インスタンス変数でクラスのメソッドを変えてしまう」という構造自体、本質的におかしい気がしますので、@objをクラスインスタンス変数にしてしまいましょう（それか、生成されるメソッドをインスタンスの特異メソッドにするかのどちらかが自然です）。

なお、getsはカーネルメソッドですので、ローカル変数の名前に使うことはおすすめできません。