Rails カウント

Question

前提・実現したいこと

お世話になっております。
現在転職活動のためのrailsにてPFを作成しています。
その中で今月の出勤日数を数える機能を実装しようとしているのですが
うまく計算ができず、
本当に根本的なことだとは思うのですが、１日考えても上手くできなく
ご助力いただけないでしょうか。

該当のソースコード

<% @count_thismonth = 0 %>
<% if Time.new.month == attendance_record.start_time.month %>
<% @count_thismonth = @count_thismonth + 1 %>
<% end %>

<%= @count_thismonth %>

内容

①each文内で@count_thismonthを0に定義
②今月と出席した日の月が同じな場合は@count_thismonth + 1する

ととてもシンプルな内容なのですが
数値が１以上にならず、困っています。

よろしくお願い致します。

また初めての質問のため、この書き方がよくないなどもご指摘もありましたら、お願い致します。

Answer 1

こういう計算はviewではなくmodel に def count_month(month)でも作ったほうが(PFなら余計）良いです。せめてcontrollerに、、、、
で、
eachのloopまで書いてないのではっきりしませんが
「each文内で@count_thismonthを0に定義」したら ゼロですね。足しても足してもゼロに戻される。
eachの外でゼロにしてeachで加える。

PFなら

1. Array（というかEnumerable)のinjectを勉強してください
2. <% if Time.new.month == attendance_record.start_time.month %>で選ぶのではなく、where で 足すべきattendance を選び出す。でcount

ほうが良いです

Comments

[ari66]

お忙しい中、ご回答いただきありがとうございます。
お恥ずかしながら、modelに定義するということを考えもしませんでした。
ご指摘の通り自分で０に定義し続けておりました。
とりあえず、当初に質問させていただいた問題は解決できたのですが
より良い物を作れるようにご指摘いただいたArrayのinjectとwhereでcountを勉強し、コードを書き直そうと思います。PFという点を考慮していただいた、親切なご指摘をいただきありがとうございます。
また機会があればよろしくお願い致します。

[winterboum]

見なおしたら合計ではなく数なんですね、この場合はincectは適さないです。
where().count　か　map{ }.compact.size ですね。( ), { } の中は楽しんでください

Answer 2

each文内で@count_thismonthを0に定義

eachで配列の中身の要素の数だけ繰り返し実行しているっていうのはご存知でしょうか。
繰り返し実行する処理の中でcount_thismonthを毎回0にしていたら当然１以上にはならないと思います

コメント主さんのコードはこんなことになっちゃってるんですかね

["b", "b"].each do
  a = 0
  a = a + 1
end
puts a
# 1

a = 0
["b", "b"].each do
  a = a + 1
end
puts a
# 2

Comments

[ari66]

お忙しい中、ご回答いただきありがとうございます。
とてもわかりやすい例文を記入いただき、ありがとうございます。
記入していただいた例文を読んですぐに、自分のミスに気づくことができました。
自分で何度も０で定義をし続けていたということですね。
今一度Rubyの基礎から勉強しようと思います。
ありがとうございます。また機会があればよろしくお願い致します。

[ari66]

ほとんど同じタイミングでご回答いただき
本来なら御二方ともベストアンサーにさせていただきたいのですが
質問した以外の点でもご指摘いただいた方をベストアンサーにさせていただきました。
ご不快な思いをさせてしまいましたら申し訳ありません。
また機会があればよろしくお願い致します。