データベースの値を更新して保存したいです。（Rails)

Question

Railsのインプットの学習が終わったので、アウトプットの練習に、
簡単なwebアプリを作成しています。

Vegetableテーブルに、string型で、countカラムに、野菜の個数を数字で保存しており、
editアクションで、その数字からいくつ取り出すか、を入力してもらい、
updateアクションで、入力値をparamsで受け取り、@edit_countに代入。入力値の値によってif
文で条件分岐して、

①@edit_count > @vegetable.countの時はeditアクションに戻す。
②@edit_count と　@vegetable.countが同じ時はデータを消去する。
③それ以外の時（0 < @edit_count < @vegetable.count）の時は、@vegetable.countから@edit_countの値を引いて、データを更新して保存したいのですが、

③の部分を試してみると、”-”メソッドが見つからない、という内容のエラーが出てしまいます。

<div class="edit_wrapper">
    <h1 class="edit_title">ぬか床から取り出す</h1>
    <p>【選択中のお野菜】</p>
    <div class="select_date">
        <a><%= @vegetable.month %>月</a>
        <a><%= @vegetable.day %>日</a>
        <a><%= @vegetable.time %>時</a>
        <a>に漬けた</a>
    </div>
    <div class="select_content">
        <a class="select_content_name"><%= @vegetable.name %></a>
        <a class="select_content_count"><%= @vegetable.count %>個</a>
    </div>
    <div class="count_form">
        <p>【取り出す個数を入力してください】</p>
        <div class="edit_error_message">
            <%= @edit_error_message %>
        </div>
        <%= form_tag("/vegetables/#{@vegetable.id}/update")do %>
            <input type="number" class="edit_count" name="edit_count" min="1" max="99"><a> 個</a>
            <p class="edit_caution">一度ぬか床から取り出したら戻せません。</p>
            <p class="edit_caution">本当にお野菜を取り出しますか？</p>
            <input class="update_btn" type="submit" value="ＯK！">
            <%= link_to("やめる", "/users/#{@vegetable.user_id}/mypage", {:class => "cancel_btn"}) %>
        <% end %>
    </div>
</div>

def update
    @vegetable = Vegetable.find_by(id: params[:id])
    @edit_count = params[:edit_count]
    if @edit_count > @vegetable.count
      @edit_error_message = "正しい数値を入力してください"
      render("vegetables/edit")
    elsif @edit_count == @vegetable.count
      @vegetable.destroy
      redirect_to("/users/#{@current_user.id}/mypage")
    else
      @vegetable.count = @vegetable.count - @edit_count
      @vegetable.save
      redirect_to("/users/#{@current_user.id}/mypage")
    end
  end

NoMethodError in VegetablesController#update
undefined method `-' for "3":String Did you mean? -@
Extracted source (around line #37):
35
36
37
38
39
40
              
      redirect_to("/users/#{@current_user.id}/mypage")
    else
      @vegetable.count = @vegetable.count - @edit_count
      @vegetable.save
      redirect_to("/users/#{@current_user.id}/mypage")
    end

試してみたこと↓

paramsで受け取っているため、文字列だから引き算ができないのかと思い、.to_iメソッドを使ってみましたが、うまくいかず。

独学の部分も多いので、常識はずれのコードがあるかもしれません。
見辛かったら申し訳ないです・・・。

Comments

[hoshi-takanori]

Vegetable テーブルの count カラムが　string 型ってことでしょうか？

[nn-901]

はい、そうです！

[FKM]

to_iメソッドを使った計算式の書き方はどうやりましたか？

[nn-901]

def update
@vegetable = Vegetable.find_by(id: params[:id])
@edit_count = params[:edit_count].to_i
@vegetable.count = @vegetable.count.to_i
if @edit_count > @vegetable.count
@edit_error_message = "正しい数値を入力してください"
render("vegetables/edit")
elsif @edit_count == @vegetable.count
@vegetable.destroy
redirect_to("/users/#{@current_user.id}/mypage")
else
@vegetable.count = @vegetable.count - @edit_count
@vegetable.save
redirect_to("/users/#{@current_user.id}/mypage")
end
end

上から３、４行目が修正したコードです。
これでもエラーがでてしまいました。正しい書き方があったら教えていただきたいです・・・！

[FKM]

params[:edit_count]はtype=numberで取得しているので、ここはto_iに変える必要ないような気もします。

[nn-901]

そうなんですね！
paramsを使って取得した時点で全て文字列になってしまうんだと思っていました・・・。

@vegetable.countも、ユーザーから入力してもらうときに、input type =numberで取得してデータベースに保存しているので、
この場合は@vegetable.countも　@edit_countも、どちらも数値なのでしょうか・・・？

[FKM]

array.countはオブジェクト、配列の個数を数えるためのメソッドなので数値を返します。
そもそもedit_countとbagetable.countにちゃんと数値が入っているかlogger.debugで確認してみては？

[nn-901]

承知しました、ありがとうございます！
恥ずかしながらlogger.debugというものの使い方を知らないので調べて使ってみます！

Answer 1

色々試していたら自己解決しました。
hoshi-takanoriさん、FKMさん、アドバイスありがとうございました！

修正後のコードを以下に添付します。

  def update
    @vegetable = Vegetable.find_by(id: params[:id])
    @edit_count = params[:edit_count].to_i
    if @edit_count > @vegetable.count.to_i
      @edit_error_message = "正しい数値を入力してください"
      render("vegetables/edit")
    elsif @edit_count == @vegetable.count.to_i
      @vegetable.destroy
      flash[:notice] = "ぬか床から取り出しました"
      redirect_to("/users/#{@current_user.id}/mypage")
    else
      @vegetable.count = @vegetable.count.to_i - @edit_count
      @vegetable.save
      flash[:notice] ="ぬか床から取り出しました"
      redirect_to("/users/#{@current_user.id}/mypage")
    end
  end

エラーコードの中に、”文字列と数値の比較でエラー”というような記載があったため、.to_iメソッドの使い方を変えてみたところ、このコードで意図する動きをしました。

if文の中で、一度.to_iメソッドを使用していれば、それ以下は使用しなくてもいいのかと思いましたが、そうではなかったようです・・・。

まだ理解度が低いので、paramsの特性など調べて学んでいこうと思います！
助かりました、ありがとうございました！