undefined method `save'エラーで詰まってしまいました。。

Question

やりたいこと

保有ポイントから購入分の値段が引かれるという処理を考えています。

今現在
Userモデル(pointカラム)があり
orderモデル(priceカラム)

コントローラーのorderアクション内ででPointカラム(数値データ)からPriceカラム(数値データ)を引き、それをPointカラムに代入する？処理になるのかなと思い次のようなコードを書いているのですがうまく動かず詰まってしまいました。。

どなたかご教授願えませんでしょうか。。よろしくお願いいたします。。

エラー内容
NoMethodError in OrdersController#create
undefined method `save' for 796:Integer

コード

def order
@user = current_user
@tpoint = @user.point.to_i
@tpoint = @user.point -= @order.price.to_i
@user.point = @tpoint
@user.point.save
end

def create
@order = Order.new(order_params)
if @order.save
order
else
render 'root'
end
end

private
def order_params
params.require(:order).permit(:service, :quantity, :link, :price ,:agreement, :user_id)
end

Answer 1

落ち着いてコードとエラーメッセージを見れば解決できます。
saveメソッドはなんでもかんでも使えるわけではなく、ActiveRecordを継承しているモデルクラスのみで使用できます。
(Rails5では、ApplicationRecordを継承していますが、その上にActiveRecordがあります)。

まず、このエラーをよく見てください。
Integerクラスにsaveメソッドなんてないぞという意味です。

NoMethodError in OrdersController#create
undefined method `save' for 796:Integer

Rubyでは10とか20などの整数値もオブジェクトであり、Integerクラスのインスタンスです。
ポイントの値が10だとして、10.save と書いてエラーになったようなものです。

では、整数(integer)に対してsaveと書いてしまっているのはどこか？と見て行くと、

def order
  ...
  @user.point.save
end

明らかにここですね。
ここまで来れば、あとはどう直せば良いか分かりますよね？

Comments

[ohamadayo]

ご回答ありがとうございます！！
saveメソッドはモデルクラスに使うものなんですね！
rubyの基礎復習してみます！！！おかげさまで無事実装の方出来ました！ありがとうございました！

Answer 2

うーん、あまりよくわからないですがポイントを使って買い物をしているみたいな処理でしょうか...？

結論から言うと上記エラーに関しては@user.saveで解決だと思います。
saveメソッドはカラムに対して使うのではなくレコードに対して使用します。

ちょっと突っ込みたいところがあるので一応記述しておきます...。

def order
@user = current_user # @userはここ以外で使うの？ 使わないのであればインスタンスにするべきではない...
@tpoint = @user.point.to_i # 現在の残りポイント？ @tpointはインスタンスにするべきではない...
@tpoint = @user.point -= @order.price.to_i
@user.point = @tpoint

数値を扱うのであればカラムの属性をintegerにしましょう。そうすれば、@user.point -= @order.price だけで上の3行と同じ働きができますよ。

@user.point.save # 間違い

@user.save

end

あとは`current_user`と`@order`がちゃんと取得できているのであればうまくいくと思います。

Comments

[ohamadayo]

ご回答ありがとうございます！
インスタンス変数の使い方、saveメソッドの意義などかなりガバガバな点が多々ありました；；

おかげさまで無事実装の方出来ました！！改めてありがとうございました！！