Rails5で変数のスコープ理解が曖昧で呼び出しが出来ない

Question

Rails5で
app/models/event.rb
内のメソッドで変数を定義しました。
それをコントローラー内で下記のように使用したいです。

#app/controllers/events_controller.rb
  def create
    @event = Event.new()
    @event.probe_data_investigate
    send_data
  end

    def send_data
      Spark.new().webex_rooms_get
      entry = Entry.new()
      entry.entry_data_in_txt
      entry.exit_data_in_txt
      entry.webex_send_txt

      entry.entry_exit_situation_update
      EntryLog.new().entry_exit_log_save
    end

上記において、
probe_data_investigateメソッドで色々は変数を定義しております
その定義した変数をsend_dataメソッド内のEntryモデルのメソッドで使用したいのですが、
それが出来ません。
グローバル変数は使わずに済む方法はないでしょうか？

回答から追記

def create
  @apple = new_data
  @orenge = before_data
  @bunana = max_data
　　　・
　　　・
　　　・
end

上記のような@appleなどの変数を下記のメソッド内で使用したいです。

#app/models/entry.rb
def entry_data_in_txt
     $entry_txt = ""
     @apple.each do |mac,time|
       personal_info = []
       personal_info = @orenge[mac].map{|k,v| v.present? ? "┃     ・#{k} : #{v}" : nil}.sort
       if personal_info.empty?
           $entry_txt << "┣━#{mac}さんが入室しました(#{time})\n"
       else
           personal_info.map!{|x| x.split(",").join("\n┃               ")}
           $entry_txt << "┣━#{mac}さんが入室しました(#{time})\n#{personal_info.join("\n")}\n"
       end
     end
end

#app/controllers/events_controller.rb
  def create
    @event = Event.new(data: params[:data])
    $timenow = Time.now

    respond_to do |format|
      if @event.save
        format.html { redirect_to @event, notice: 'Event was successfully created.' }
        format.json { render :show, status: :created, location: @event }
      else
        format.html { render :new }
        format.json { render json: @event.errors, status: :unprocessable_entity }
      end
    end

    Entry.new().probe_data_investigate

    Spark.new().webex_rooms_get
    entry = Entry.new()
    entry.entry_data_in_txt
    entry.exit_data_in_txt
    entry.webex_send_txt

    entry.entry_exit_situation_update
    EntryLog.new().entry_exit_log_save
  end

仮に別モデルの定義したものをすべてmodels/entry.rbに移動させてもコントローラーからモデルメソッド呼び出しなのでインスタンス変数でも出来ませんでした。
（コントローラーから呼び出しだからインスタンス変数では使えないからかな？？）
別モデル内の変数なので、やっぱりグローバル変数が適切なような気がしてきてますがいかがでしょうか？

---

追記　バッチ処理した際の変数をモデルで使用したい

#config/schedule.rb
rails_env = ENV['RAILS_ENV'] || :development
set :output, 'log/crontab.log'
set :environment, rails_env

every 30.minute do
  runner "Tasks::PersonalLogArrange.execute"
end


# webex/lib/tasks/personal_log_arrange.rb
require 'webex/app/models/personal_log'
class Tasks::PersonalLogArrange
  def self.execute
    PersonalLog.new.arrange
  end
end

#app/models/personal_log.rb
    def arrange
      array = Array.new
      personal_log = PersonalLog.all
      personal_log.find_each do |x|
          hash = Hash.new
          begin
             test = JSON.parse(x.decrypted_data)
          rescue => error
             test = JSON.parse(x.data)
          end
          if test.keys.include?("uid")
               test["uid"] = "#{test["uid"]}(#{x.provider})"
          end
          hash[x.macaddr] = test
          array.push(hash)
      end
      @new_data = array.each_with_object(Hash.new{|h,k|h[k] = Hash.new}){|it,memo|
          it.each{|mac,val|
          memo[mac].merge!(val){|_,s,o| s + ',' + o}
          }
      }
      @new_data.transform_values!{|val| val.transform_values{|v| v.to_s.split(',').uniq.join(',')}}
      puts @new_data   
    end
   
    def export
      {new_data: @new_data}
    end

    attr_reader :new_data

#app/controllers/events_controller.rb

    def send_data
      entry = Entry.new()
      event = Event.new
      event.probe_data_investigate
      data = event.export

      #追加したコード
      personal_log = PersonalLog.new
      new_data = personal_log.export
      data.merge!(new_data)

      entry.import(data)
      Spark.new().webex_rooms_get
      entry.entry_data_in_txt
      entry.exit_data_in_txt
      entry.webex_send_txt

      entry.entry_exit_situation_update
      entry.entry_exit_log_save
    end

バッチ処理の際にインスタンス変数を渡しようにすればよいと思うのですがわかりますでしょうか？

Answer 1

果たして、

    @event.probe_data_investigate
    send_data

から、send_dataを呼ぶ前には
@event.probe_data_investigateを呼ぶ必要があるなんて制約をこのコードから読み取れますか？

1週間後・1ヶ月後の自分に「読み解け」と言えますか？

多分3日後あたりで既に怪しいと思いますよ。

---

とりあえず、このコードの部分に手を加えずにやるならばEntryクラスのクラス変数を利用する案があります。
上述したように全くオススメはしませんので解説はしません。

一般的には

  def create
    @event = Event.new()
    data = @event.probe_data_investigate
    send_data data
  end

  def send_data(data)
    Spark.new().webex_rooms_get
    entry = Entry.new()
    ## ここらでなんらかの方法でdataを渡す ##
    # 例: entry.push data
    # 例: entry = Entry.new(data)
    entry.entry_data_in_txt
    entry.exit_data_in_txt
    entry.webex_send_txt

    entry.entry_exit_situation_update
    EntryLog.new().entry_exit_log_save
  end

のようにprobe_data_investigateメソッドに戻り値を持たせます。

def probe_data_investigate
  return {name: "namae", hoge: [:foo, :bar]}
end

のように、ハッシュで返せば複数の値を返す事ができます。

---

とりあえずシンプルな例にしてみたつもり

class Event
  def create
    @apple = "りんご"
  end
  # データ取り出しメソッド
  def export
    {apple: @apple}
  end

  # @appleを外部から取り出せるようにする
  attr_reader :apple
end

class Entry
  # 受け取り側
  def push(data)
    puts data[:apple]
  end
  def push_model(evt)
    puts evt.apple
  end
end

# 以下、コントローラーとして
event = Event.new
event.create
entry = Entry.new

# データ取り出しメソッドを作る
data = event.export
entry.push(data)

# いっそモデルのインスタンスごと渡してしまうのも手
entry.push_model(event)

Comments

[kozica]

制約をコードから読み取れるか？という所はものすごく納得しました！
自分はまだコードの可読性を高めるためにどのようにコーディングすればよいかまだしっかりわかっていないのでこの指摘はすごく勉強になります。
回答で、なんらかの方法でdataを渡すとありますが、ここがわからないです、、、
記載している通りmodels/event.rbのメソッド内で定義した変数をmodels/entry.rbで使用したいです。
上記では単純にdataとなってますが、実際には投稿追記のように@apple,@orenge,@bunanaなどいくつものインスタンス変数を使いたいです。

[asm]

Eventのメソッドから
return {apple: @apple, orenge: @orenge, bunana: @bunana}
な感じで必要なインスタンス変数をハッシュに格納して返します。

コントローラーは受け取ったものをローカル変数に格納してEntryモデルのメソッドに渡します。

[kozica]

やってみた所、インスタンス変数の中身が空になっているようです。
本来returnで返したら、空にはならないものなのでしょうか？

[kozica]

非常にわかりやすく書いて頂きありがとうございます
実行したところエラーがでてしまいました。
おそらく、この状態だとEntryクラスのメソッド内で、puts @appleが実行できない。つまり@appleは使えないようです。
puts @appleができるようにすることは可能でしょうか？

[kozica]

現状ですと、
puts @apple
をするためには
puts @data[:apple]
とpush(data)メソッド内で@data = dataと定義して上記のように書けばうまくいきますが、インスタンス全部で@data[:xxx]とするのも見栄えがよくないので改善策はないでしょうか？

[asm]

手っ取り早いのは
def push(data)
@apple = data[:apple]
end
こんなん

大量にあってめんどくさい場合は
def import(data)
data.each{|name, val| instance_variable_set "@#{name}", val }
end

[asm]

後者の場合は、exportの方を
{ "@apple": @apple}
みたいにして
def import(data)
data.each{|name, val| instance_variable_set name, val }
end
でもよい

[kozica]

後者で
puts @apple
をしましたが、空になってました、、、
これは後者のを実装しても puts @appleで使用できるようになるということではないということでしょうか？

[kozica]

あ、すいません。
勘違いでちゃんとできてました！！

[kozica]

ちなみになんですが、今バッチ処理をしていてそのインスタンス変数もEntryモデル内で使用したい場合はどのように書けばいいでしょうか？
同じ用に書いてやってみましたが、うまくいきません。
記述内容は投稿に追記しております。

[asm]

どこからどこに渡したいのか皆目見当がつきません。