Railsでeachメソッドを用いて合計金額の計算

Question

前提・実現したいこと

railsにてeachメソッドを用いて取り出した要素の合計

該当のソースコード

<tbody>
<h1>カートの中身</h1>
<% @cart.each do |item| %>
<% @sum = 0 %>
<tr>
<td>商品番号: <%= item.id %></td>
<td>価格: <%= item.price %>円</td><br>
<% @sum += item.price %>
<% end %><br>
合計金額 <%= @sum %>

</tr>
</tbody>

###発生している問題

商品番号: 3 価格: 3000円
商品番号: 1 価格: 2500円
商品番号: 4 価格: 1000円

合計金額 1000

となり、計算が出来ていない。

試したこと

sumのままではundefined methodエラーが帰って来るので@をつけてインスタンス変数に変更
Rubyではこの記述で問題無いはずが、なぜRailsでは機能しないのか理解出来ません。

補足情報（FW/ツールのバージョンなど）

Rails 5.2.0

Answer 1

よくわからないときは、ループの中で@sumをログに吐いたり（あくまでデバッグの際なら）画面に出したりしてみるとよいですよ。

で、確認すると分かるかと思いますが、<% @sum = 0 %>は@cart.each ...の行よりも前に書く必要があります。現状は毎回初期化しているようです。

sumだとエラーになる、というのは重要なヒントで、ローカル変数のスコープの問題かと思います。上記修正を行えばsumでも動くんじゃないでしょうか。

Comments

[shirou]

丁寧な解説ありがとうございます！
sumでダメなら@を付けてしまえと、短絡的発想でしたが仰る通りです。
ループ内で用いる変数とはいえ、初期値の指定は外に出してあげなければいけませんでした..！

Answer 2

こんにちは

以下の２行

<% @cart.each do |item| %>
<% @sum = 0 %>

を入れ替えて、

<% @sum = 0 %>
<% @cart.each do |item| %>

としてみるといかがでしょうか？

Comments

[shirou]

回答ありがとうございます。
全くその通りです、ループ内に変数の初期値を記述していたのでは当然の結果でした...！

[jun68ykt]

大丈夫です。落ち着いていきましょう。先は長いです。

Answer 3

前提のRubyでは正しい記述の認識が違うのでは無いかと思い、適当にエディタで試してみたら果たして動きませんでした。
回答に指摘されている通り、sum=0がループ内に入っていることが問題です。
質問を投稿した直後に気づいて削除要請を出したのですが、お恥ずかしい。