Ruby eachとif文が入れ子になった処理を終える

Question

超初歩的な質問で申し訳無いのですが、以下の様な処理があった時に、cだったらif文とeach文を抜けて、trueを出力するためにはどのような処理を加えればよいですか？
現状ですとif文は抜けていると思うのですが、次のeach文が始まっていしまい結果的にfalseが返ってきてしまいます。

よろしくお願いします m(_ _)m

def method(data)
  data.each do |ele|
    if ele == 'c'
      return true
      break
    else
      return false
    end
  end
end

data = ['a', 'b', 'c', 'd', 'e']

puts method(data)

Answer 1

「cが含まれていたらtrueを返す、そうでなければfalseを返す」のような処理をしたいのでしたら、ロジックがおかしいですね。
なぜなら、最初のaの時点でif ele == 'c'がfalseとなってしまい、その時点でreturn falseしてしまうからです。
また、何を返すかどうかに関係なくreturnした時点でメソッドを抜けてしまうので、それ以降の処理（ここではbreak）は意味がありません。

cが見つかって時点でtrueを返して終了、一度も見つからなければeachループが終了した時点でfalseを返す、のようにすれば目的は達成できると思いますがいかがでしょうか。

• コード例:

def method(data)
  data.each do |ele|
    if ele == 'c'
      return true
    end
  end
  return false
end

Comments

[退会済みユーザー]

丁寧な解説、ありがとうございます！

一つ気になった点があるのですが、私が最初に投稿したコードの場合（break文は無かったと仮定して）、aの時点でreturn falseが該当しますが、そのとき抜けるのはif文であり、each文はまだ抜けていないという理解で正しいですか？

[退会済みユーザー]

あ、恐らく私の認識が間違っている気がします。
each と if が入れ子になっていても、return の時点で全てを（即ちeach と if を含んだメソッド）抜けるというのが正しいですか？

[argius]

> each と if が入れ子になっていても、return の時点で全てを（即ちeach と if を含んだメソッド）抜けるというのが正しいですか？

そうです。
returnは「メソッドを抜ける」なので、メソッドが入れ子になっていない限りはeachなどのループやifのブロックは関係なく抜けます。

breakの場合だと、ループが入れ子になっているときは内側のループのみ抜けます。

[退会済みユーザー]

分かりやすい解説、ありがとうございました！

Answer 2

質問の直接の回答ではありませんが、
配列に 'c' が含まれているかの判定結果を true/false で得るなら、 include?() メソッドが使えます。

def method(data)
  data.include?('c')
end

data = ['a', 'b', 'c', 'd', 'e']
puts method(data)

data = ['a', 'b', 'X', 'd', 'e']
puts method(data)

実行結果

$ ruby c.rb
true
false

Comments

[退会済みユーザー]

includes?メソッドもこのような場合に使えるのですね、ありがとうございます！