メソッド 切り分け方

Question

メソッドの切り分け方
card = Card.find_by(user_id: current_user.id)は他のactionでも使用してるのでメソッドに切り分けましょうと言われたのですがやり方が分かりません　　
ご教授お願いします

class PurchaseController < ApplicationController

  require 'payjp'

  def index
    card = Card.find_by(user_id: current_user.id)

    if card.blank?

    else
      Payjp.api_key = ENV["PAYJP_PRIVATE_KEY"]

      customer = Payjp::Customer.retrieve(card.customer_id)

      @default_card_information = customer.cards.retrieve(card.card_id)
    end
  end

  
  def pay
    card = Card.find_by(user_id: current_user.id)
    Payjp.api_key = ENV['PAYJP_PRIVATE_KEY']
    Payjp::Charge.create(
    :amount => 10000, 
    :customer => card.customer_id, 
    :currency => 'jpy', 
  )
  redirect_to action: 'done'
  end

  def done
    card = current_user.cards.first
    redirect_to controller: "card", action: "new" if card.blank?
   end
 end

Answer 1

こんな短いのを切り出す必要もないかとおもうのですが、敢えてやるなら
before_action :set_card,only: [:index, :pay]を最初の方に書いて

def set_card
  @card = Card.find_by(user_id: current_user.id)
end

でしょうか

Comments

[yakumo02]

回答ありがとうございます
追加するだけで記述の変更はなしでいいでしょうか？

[winterboum]

card が　@card に変わっているので、その修正はして下さい
例えば　if card.blank?　は　if @card.blank?