質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.35%

  • Ruby

    10201questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    9543questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 5

    3766questions

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,646

ohamadayo

score 9

やりたいこと

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

今現在
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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+3

落ち着いてコードとエラーメッセージを見れば解決できます。
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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/31 20:11

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

    キャンセル

checkベストアンサー

+1

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

結論から言うと上記エラーに関しては@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がちゃんと取得できているのであればうまくいくと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/31 20:09

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

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • Ruby

    10201questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    9543questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 5

    3766questions