【Rails5】method間で変数を共有する方法

Question

eventが取れるコード

悠長な書き方になってしまいます。
rubyっぽい書き方がありましたら、教えてください。

def save
      if hoge1?
        event = hoge!
      else
        raise 'Unauthorized!'
      end
end

def hoge!
    Event.create!(
      start_date: start_date,
      end_date: end_date,
    )
end

eventが取れないコード

def save
      if hoge1?
        hoge!
      else
        raise 'Unauthorized!'
      end

      binding.pry
　　# ここでeventを扱いたいです。
end

def hoge!
    event = Event.create!(
      start_date: start_date,
      end_date: end_date,
    )
end

Answer 1

こうじゃダメなのでしょうか？

def save
  raise 'Unauthorized!' unless hoge1?
  event = hoge!
  binding.pry
end

どうしても、変数を経由したい場合はインスタンス変数でどうにかならないか考えます。

---

追記

class Hoge
  def save
    raise 'Unauthorized!' unless hoge1?
    p @event
    binding.pry
  end
  def hoge!
    @event = Eventなんちゃら
  end
end

h = Hoge.new
h.hoge!
h.save

ただ、これだとsave呼び出す前に(もしくはsave内で)hoge!を呼び出さないとエラーになります。
なので

class Hoge
  def save
    raise 'Unauthorized!' unless hoge1?
    # @eventが未定義(もしくはnilやfalseの場合)初期化する
    @event ||= hoge!
    p @event
    binding.pry
  end
  def hoge!
    @event = Eventなんちゃら
  end
end

こんな感じにすることが多いですね

再追記
Rubyはif文ではなくif式なので結果を返します。

if 式は、条件が成立した節(あるいは else 節)の最後に評価し た式の結果を返します。else 節がなくいずれの条件も成り立たなけれ ば nil を返します。

def save
  event = if hoge1?
            hoge!
          elsif hoge2?
            hoge2!
          else
            raise 'Unauthorized!'
          end
  p event
  binding.pry
end

Comments

[ssk]

ありがとうございます。
インスタンス変数を経由する場合はどのように書けば良いでしょうか？

[ssk]

ありがとうございます。

elsifで実行するメソッドを分岐することになると思うので、インスタンス変数で処理した方が綺麗に書けそうでした。

[ssk]

ありがとうございます。
なるほど！より、スッキリ書くことができました。

助かりました。