rails メソッドのインスタンス変数を渡す

Question

メソッドac_warrantの@ac_warrantの値がnewとcreateに渡らずnilとなります。ac_warrantでは正しい値が取れています。

どうしたら値を他のメソッドに正しく渡せるでしょうか

  #対象のワラント
  def  ac_warrant
    #アクセスしたitem_idを取得
    @ac_item_take_params_id = take_params.values.pop.to_i

    #アイテムのワラントレコードを取得
    @ac_warrant = Warrant.find_by(item_id: @ac_item_take_params_id)
  end



 def new
    ac_warrant
    @opportunity = @ac_warrant.opportunities.build
  end



  def create
    @opportunity = @ac_warrant.opportunities.build(opportunity_params)

    respond_to do |format|
      if @opportunity.save
        format.html { redirect_to @opportunity, notice: 'オポチュしました。' }
        format.json { render :show, status: :created, location: @opportunity }
      else
        format.html { render :new }
        format.json { render json: @opportunity.errors, status: :unprocessable_entity }
      end
    end
  end

Answer 1

@foo の変数をつかいすぎるのはよくありません。
また、メソッド名と変数名が一致していると コードを読む際に混乱しやすくなります。

def ac_warrant　は名前とその実装を次のように変えてみては如何ですか？

def get_ac_warrant
  # アクセスした item_id を取得
  id = take_params.values.pop.to_i

  # アイテムのワラントレコードを取得
  Warrant.find_by(item_id: id)
end

def new は次のように書き換えることが必要になります。

def new
  @ac_warrant = get_ac_warrant
  @opportunity = @ac_warrant.opportunities.build
end

def create も同様の書き換えが必要です。(ここでは提示しませんが)

Comments

[smith]

いつもありがとう御座います。