Rails : 日付ごとに投稿をグループ化して表示したい

Question

やりたいこと

「List>Card>Log」のグループを日付ごとに並べたい

・ListごとにTodoを並べられるアプリケーションを作っています
・作成したデータが日付ごとに区切られて表示する、表示方法を考えています
・新しいもの順に並べるのはできそうだが、日付でグループが分かれる形で表示する方法がよくわかっていないです
・SQLのgroup_byを使ってやるのかなと思っているのですが・・・？
↓

---

質問

以下のようなアドバイスをもらっているのですが、内容を理解できず全然見通しがたってないです。。。
（あんまり何度も聞ける相手ではないのでこちらに投稿してます）

この内容でやる場合は「学んだ方が良さそう内容」や「もう少し詳しい実装方針」のアドバイスいただけると嬉しいです。

他のやり方でも構いません！

リストのループの先頭で日付を変数に入れて、変数の日付とリストの日付をifで比較して、異なった場合に表示を切り替えるという処理にすればいけそうな気がする

before_date = ''
@list.each do |list|
  # ループ1回転目のみ必ず日付を入れる
  before_date = list.date if before_date.blank?
  # 前回日付と自身の日付を比較
  if before_date == list.date
  #   日付の切り替わりがなかった時の処理
  else
  #   日付が切り替わった時の処理
  end
  # 自身の日付を前回日付として格納
  before_date = list.date
end

github：コードを載せておきます

ここに現在のものすべて載ってます↓
https://github.com/takuming/focus

現在のView側のコード
スクショのものを表示している、View部分のコードを貼っておきます

<div class="topPage">
  <div class="listWrapper">
    <% @list.each do |list| %>
      <div class="list">
        <div class="list_header">
          <h2 class="list_header_title">
              <%= list.title %>
          </h2>
          <div class="list_header_action">
              <%= link_to content_tag(:i, '削除', class: 'fas fa-trash'), list, method: :delete, data: { confirm: "#{list.title}を削除して大丈夫ですか?" } %>
              <%= link_to(content_tag(:i, '編集', class: 'fas fa-pen'), edit_list_path(list)) %>
          </div>
        </div>

        <div class="cardWrapper">
        <% list.cards.each do |card|%>
            <div class="card">
              <div class="head"> 
                    <h3 class="card_title">
                      <%= card.title %>
                    </h3>
                    <li class="btn_start">
                            <%= form_tag("/cards/#{card.id}/log/start_time") do %>
                              　<input type="hidden" name="card_id">
                                <input type="submit" value="スタート" class="btn_small">
                            <% end %>
                    </li>
                    <li class="btn_start">
                            <%= form_tag("/cards/#{card.id}/log/end_time") do %>
                                <input type="hidden" name="card_id">
                                <input type="submit" value="ストップ" class="btn_small">
                            <% end %>
                    </li>
                </div>

                <div class="section_log">
                  <% card.logs.each do |log|%>
                      <div class='time_log'>
                          <% @t1 = log.created_at.strftime("%H:%M:%S") %>
                          <% @t2 = log.end_time %>
                          <p>計測 → </p><p>開始: <%= @t1 %></p><span>-</span><p>終了: <%= @t2 %></p>
                          <% if @t1.present? && @t2.present? &&  %>
                                <p>合計: <%= log.total_time%> </p>
                          <% else %>
                                <p>何もないよ</p>
                          <% end %>
                      </div>
                  <%end%>
              </div>

              <div class="cardContents_btnArea">
                <%= link_to '編集する', edit_list_card_path(card.list, card), class: "edit_btn" %>
                <%= link_to '削除をする', list_card_path(card.list, card), class: "text-danger delete_btn", method: :delete, data: { confirm: "このカードを削除して大丈夫ですか?" } %>
              </div>
            </div>
        <%end%>

          <div class="addCard">
            <i class="far fa-plus-square"></i>
            <%= link_to "さらにカードを追加", new_list_card_path(list), class: "addCard_link" %>
          </div>
        </div>

      </div>
    <% end %>
  </div>
</div>

Comments

[winterboum]

「日付ごとに」とはどうしたいのか、が良くわかりません。

１） List>Card>Log　のモデルの関連の定義部分を追記して下さい
2) 日付 は List>Card>Log　のどこに入り込むのですか？
　こういう画面にしたい、　というのがどういうものになるのかが判るようにして下さい。

[siruku6]

あとは、viewだけでなく、controllerやmodelのソースも掲載されているとよいと感じます
・modelは、associationの状況を一目で把握するため
・controllerは、instance変数のセット状況を確認するため

また、実際に変更が提案されるとしたらviweだけでなく、controllerやmodel側の変更が含まれると思いますので、そういう意味でも現状を掲載しておくとよさそうです。

[takumikai]

List単位で日付でグループ化/並ぶ

というのがやりたいです！ありがとうございます。

[takumikai]

githubのリンクを追記しました
https://github.com/takuming/focus

[winterboum]

＞　List単位で日付でグループ化/並ぶ
ではイメージが湧きません。cardとlogの関係もわからない。
目に見える形に出来ませんか？

[no1knows]

すいません。今気づいたのですが、タイトルの「ailsRails」→「Rails」の間違いでしょうか？もしそうなら修正をお願いします。

[takumikai]

イメージ追記しました????

[winterboum]

なるほど、見えてきました。
念の為、List,Card,Log のモデルの関連の定義部分を質問に追記して下さい。
関連の定義とは has_* と belongs_to です

[takumikai]

ありがとうございます！後でやりますね！（githubのURL張っているので、お手間でなければそちらにあります
https://github.com/takuming/focus

[winterboum]

あれ。。。　listにdateが有るんですか？
以下の様にするのかと思っていたのですが違うのかな
1)複数のListがある
2)おのおののListには複数のcardがある
3)各々のcardには複数のlogがある。
そこで
logの日付毎に表示する。
つまり、
10/1のlogのあるCard達 （とそのList)
10/2のlogのあるCard達 （とそのList)
10/3のlogのあるCard達 （とそのList)

List にdateが有るということは
10/2のList達とそれにぶら下がるCard と Log
10/3のList達とそれにぶら下がるCard と Log

どちらでしょう

[winterboum]

github見ました。
List には dateなくて、Log に end_time があります。これで分類ですね？

[takumikai]

ありがとうございます！

Listにも Created_at,Update_atがあるので、その考え方だと可能なのかなと思ってます。
あと、Listごとに判別できた方が他の設計を考えるときに良さそうだなと思ってます！
（次の日付になったら、デフォルトでListを作って表示しておく。などやりたいなど考えてます）

[winterboum]

プログラムてきにも、孫のLogの日付でまとめるより、親のListの日付でまとめたほうが楽です。
が、
それが目的に合うかどうか。
「何のために何を日付で分けるのか」
それによって考え方が変わります。
Listのupdated_atは CardやLogを触っても変わりませんよ、それで目的に合いますか？
目的は何でしょう。

[takumikai]

・日付ごとにLIst / Card / Logの塊を並べたい
　ーListはあるが、カードがない場合も想定、
　ーList,Cardはあるが、Logはない場合も想定
しているので、直感的にListでやるべきなのかなと思ってます。

逆に、ListとLogの日付情報を使うことの違いを認識できてないので、そちらを提示してもらえるとより判断はできるのかなと思います。
逆にLogの方が良いと思われたらそのパターンで紹介いただいてもかなり勉強になるので、そちらでも！

Answer 1

これまでのteratailでの質問している内容から、考え方の質問だと推察しました。

グループ化として考えると難しくなるので

リストのループの先頭で日付を変数に入れて、変数の日付とリストの日付をifで比較して、異なった場合に表示を切り替えるという処理にすればいけそうな気がする

これを下記のように考えるとわかりやすいかと思います。

前のレコードと同じ日付の場合は、日付を表示しない。
前のレコードと異なる日付の場合は、日付を表示する。

	日付	やること
①	10/1	Railsを勉強
②	10/1	phpを勉強
③	10/2	Rubyを勉強
④	10/3	jQueryを勉強

　　　　↓

	日付	やること
①	10/1	Railsを勉強
②		phpを勉強	←前のレコードと同じ日付なので、日付を表示しない
③	10/2	Rubyを勉強
④	10/3	jQueryを勉強

---

このように考えるとポイントは下記の2つになると思います。

・前日の日付をループの最後で登録しておく（before_date）
・ループの一番最初の日付を表示するコードが例外として必要となる（アドバイスに記載のない部分）

Comments

[takumikai]

ありがとうございます！

>ループの一番最初の日付を表示するコードが例外として必要となる
これがわかってないのですが "before_date = list.date if before_date.blank?" が相当するのではと思ってるんですが違う？のですかね
・before_dateを定義する
・もし before_dateの中身がなかったら
・list.date(listデータんの日付)から持ってくる
→1番はじめはbefore_dateの値がないので、適用されて、1番最新の日付が入る

と理解してます

[takumikai]

ちなみに書いていただいた概念の理解はだいたいできていたのですが
具体的に何をしたら良いのか、全くわかってない状態です。

「こういう知識が必要」というものがあれば教えていただけると勉強できるのですが、もしよかったらご教授ください・・。
（レベルとしてはもらったアドバイスの書き方自体もちんぷんかんぷんです。ifの基本構造はもちろんわかりますが、「xxx.date→こんなクラスある？」とか、最後endの後ろの「before_date = list.date → なんで過去の日付になるの？」など

[no1knows]

もし僕なら一度立ち止まって、Railsの仕組みを学ぶことをおすすめします。

まずプロゲートのRailsとRubyを受講（1ヶ月980円で受け放題）。
そのあとRailsチュートリアルを実施。

たぶんプロゲートだけでだいたいやりたいことが理解できるかと思います。
ただ気をつけてほしいのは、プロゲートはまだRails6に対応していないため、Rails5だという認識で受講いただくことになります。

それでもやりたいことの全貌がつかめるような気がします。

Answer 2

List,Card,Logの役割、何のために日付順に並べるのか、が分からないので、Listベースで良いのか不安だがとりあえず簡単な　List で並べる方を
方法は２つあって、一つは no1knowsさんの方法、もうひとつの方を。

１）表示すべきList達を得る
lists = List.order(:updated_at).where(なにがし)

1. それを日付ごとにまとめる

@lists_by_date = lists.group_by{|list| list.updated_at.to_date}

1. それを順に表示する

<% @lists_by_date.each do |date,lists| %>
<%= date %>
<% lists.each do |list| %>
.....
<% end %>
<% end %>

Comments

[takumikai]

ありがとうございます！????