rubyでreturnしたい処理をメソッド化すると後続処理が走ってしまうので修正したい

Question

前提・実現したいこと

元々Java開発をしていましたが、現在Rails開発に携わっています。

rubyにおいてですが、
returnすることによって、後続処理が実行されなくなると思います。
しかし、その処理をメソッド化するとreturnの箇所処理も実行されてしまいます。

メソッド化（リファクタリング）する際はどのようにすればいいのでしょうか？

試したこと

例えば以下のコードです

if [true, false].sample
  if [true, false].sample
    p 'a'
    return p 'ですよ'
  end

  p '真です'
else
  if [true, false].sample
    p 'a'
    return p 'ですよ'
  end

  p '偽です'
end

上記は、以下のパターンの期待値です。
「a ですよ」「真です」「偽です」

以下の箇所が重複しているので

p 'a'
return p 'ですよ'

これを以下のようにメソッド化すると

def adesuyo
  p 'a'
  return p 'ですよ'
end


if [true, false].sample
  if [true, false].sample
    adesuyo
  end

  p '真です'
else
  if [true, false].sample
    adesuyo
  end

  p '偽です'
end

returnがメソッドの返却値の意味を持ち、期待値は以下のパターンです。
「a ですよ 真です」「a ですよ 偽です」「真です」「偽です」

期待値が変わってしまいます。
メソッド化前の期待値のまま、リファクタリングすることはできますでしょうか？
returnをメソッド内に含ませると意味が変わるので、リファクタリングする余地はないでしょうか。

Answer 1

これでいいのか

def adesuyo
  p 'a'
  throw :symbol, (p 'ですよ')
end

catch(:symbol) do
  if [true, false].sample
    if [true, false].sample
      adesuyo
    end

    p '真です'
  else
    if [true, false].sample
      adesuyo
    end

    p '偽です'
  end
end

Comments

[k499778]

[MEMO]
throwしたらcatchに入ってendまでスキップする

Answer 2

adesuyoというメソッドを定義して、p a; return p "ですよ"をadesuyoの呼び出しに書き換えたということでしょうか？
adesuyoの中にあるreturnはadesuyoを終了させるだけです。これはJavaなどでも同じですね。

元のメソッドを途中で終了させたいなら、そのメソッドの中でreturnを書く必要があります。

一応、呼び出し先から呼び出し元の戻り先を変える大域脱出の機能はあります。
throw :xでbar以降のendまでをスキップする。

def foo(a)
    catch :x do
        if a
            bar
        end
        return p "引数は偽です"
    end
end

def bar
    p "a"
    throw :x, (p "ですよ")
end

p "foo(真) の返り値 => #{foo(true)}"
p "foo(偽) の返り値 => #{foo(false)}"

Comments

[k499778]

ありがとうございます！確認します！

[k499778]

def adesuyo
p 'a'
throw :x, (p "ですよ")
end

if [true, false].sample
catch :x do
if [true, false].sample
adesuyo
end
end

p '真です'
else
catch :x do
if [true, false].sample
adesuyo
end
end

p '偽です'
end

[k499778]

上記のように書き換えてみたのですが、結果が変わらないような気がして（"a"
"ですよ"
"真です"と表示されるthrow以降も実行されてしまうパターン）

なにか認識が違いそうでしょうか????

また、あまり実装で見かけないような気がしたのですが、railsの場合、また違う方法があったりしますでしょうか？????

[otn]

元のメソッドを終了したいのであれば、処理全体を囲まないとだめです。
ということが、理解できなかったのであれば、当面、使うのは無理ですね。

def メソッド
catch :x do
～～～
if [true, false].sample
adesuyo
end

p '真です'
else
if [true, false].sample
adesuyo
end

p '偽です'
end
～～～
end # of catch
end # of def メソッド

> あまり実装で見かけないような気がしたのですが

はい。ロジックの流れがわかりにくいから、ピタリはまるケース以外ではあまり使わないと思います。

> 元のメソッドを途中で終了させたいなら、そのメソッドの中でreturnを書く必要があります。
が普通でしょう。

[k499778]

ご提示頂いたコードも実行したのですが

"a"
"ですよ"
"foo(真) の返り値 => ですよ"
"引数は偽です"
"foo(偽) の返り値 => 引数は偽です"

と出力されました。
真のときも「引数は偽です」ですが呼び出されているかもしれません????

[otn]

わかりにくければ、
p "foo(真) の返り値 => #{foo(true)}"
p "===================="
p "foo(偽) の返り値 => #{foo(false)}"
と書いてみてはどうでしょうか。

[k499778]

ありがとうございます！理解しました????