定義したメソッドをメソッドチェーンとして使用する方法について

Question

###前提・実現したいこと
定義したメソッドをメソッドチェーンとして使用したいと考えております。

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

class Test
	def test1
		arry = ["aaaa", "bbbb", "cccc"]
		arry.each do |i|
			p "起動" if i.nextTH("cc")
		end
	end
	def nextTH(s)
		index(s.to_s)
	end
end
Test.new.test1

サンプルですが、上記のnextTHというメソッドを定義し、eachで分割したiに対して定義したメソッドを実行したいと考えております。

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

undefined method `nextTH' for "aaaa":String (NoMethodError)

メソッドが定義されていないというメッセージが出てしまうのですが、定義はされているはずなのでいまいち納得が出来ません。
今まで定義したメソッドは、メソッド単体で使用してきたのですが、上記のようにメソッドチェーンをして使用するのが今回が始めてでとても戸惑っています。
アドバイスの方、宜しくお願い致します。

Answer 1

メソッドチェーンにするためのは、 自分自身を返すメソッドを定義する必要があります。
質問文のコードを、不自然ではありますが無理やりメソッドチェーンにしてみました。

a.rb

class Test
  def test1
    arry = ['aaaa', 'bbbb', 'cccc']
    arry.each do |str|
      puts str
      puts "  起動" if for_str(str).nextTH("cc")
    end
  end

  def for_str(str)
    @str = str
    self
  end

  def nextTH(s)
    @str.index(s)
  end
end

arry = ["aaaa", "bbbb", "cccc"]
Test.new.test1

実行

$ ruby a.rb
aaaa
bbbb
cccc
  起動

for_str(...) は自分自身を返しているので、 for_str(str1).nextT(str2) のように書くことができています。

Answer 2

nextTHはTestクラスのメソッドであり、iにはいる文字列のメソッドではありません。普通に二つの引数を渡せばいいかと思います。

class Test
    def test1
        arry = ["aaaa", "bbbb", "cccc"]
        arry.each do |i|
            p "起動" if nextTH(i, "cc")
        end
    end
    def nextTH(str, s)
        str.index(s.to_s)
    end
end
Test.new.test1

Comments

[yuki_90453]

回答ありがとうございます。
nextTHがstringのクラスのメソッドでは無いことが原因だったみたいですね。
例えば下記のようにStringクラスにメソッドを追加する方法と、上記のように引数を2つにする方法どちらが適しているのでしょうか？

class String
def nextTH(s)
index(s.to_s)
end
end

[raccy]

そういう書き方もできますが、Stringのような組み込みクラスへのメソッドの安易な追加や変更は思わぬバグを起こす可能性が高くなりますので、推奨できません。

[yuki_90453]

なるほど、わかりました。確かに自分のような初心者がむやみにいじらないほうがよさそうですね。
丁寧な回答ありがとうございました。