1:nの関係において、indexで最新のレコードのみ取得する方法

Question

1:nのnでindexを表示しようと考えております。

表示内容はcreated_atなのですが、

  <% post.details.each do |detail| %>
                  <td><%= detail.updated_at %></td>
              <% end%>

でレコード自体取得は問題なく出来ているのですが、n側にて例えば５つ更新を行っていると、５つ全てがindex.html.erbに表示されてしまいます。

最新のレコードのみを表示したいのですが、どうすれば良いでしょうか？

ご教示よろしくお願い致します。

  <% @posts.each do |post| %>
                <tr>
                  <td><%= link_to post.company, post_path(post) %> </td>
                  <td><%= link_to post.owner, post_path(post) %></td>
                  <td><%= post.tel %></td>
              <% post.details.each do |detail| %>
                  <td><%= detail.updated_at %></td>
              <% end%>
                  <td><%= link_to '編集', edit_post_path(post), class: 'command'%>
                  <%= link_to '削除',
                    post_path(post),
                    method: :delete,
                    class: 'command',
                    data: { confirm: '本当に削除しますか？'} %></td>
                    <% end %>
                </tr>
                <tr>
                  <th colspan = "5">
                  前のページ<%= paginate @posts %>次のページ
                  </th>
                </tr>
              </tbody>
            </table>

Answer 1

Answer 2

# == Schema Information
#
# Table name: posts
#
#  id         :integer          not null, primary key
#  company    :string
#  owner      :string
#  tel        :string
#  address    :string
#  people     :string
#  status     :string
#  datetime   :string
#  comment    :string
#  created_at :datetime         not null
#  updated_at :datetime         not null
#

class Post < ApplicationRecord
    has_many :details



   def self.import(file)
        CSV.foreach(file.path, headers: true) do |row|
		# IDが見つかれば、レコードを呼び出し、見つかれなければ、新しく作成
        post = find_by(id: row["id"]) || new
        # CSVからデータを取得し、設定する
        post.attributes = row.to_hash.slice(*updatable_attributes)
        # 保存する
        post.save!
       end
    end


    def self.updatable_attributes
    		["company", "store", "owner","kana", "tel", "tel2","fax", "industry" "address", "mail" , "url ", "people"]
  	end

  	
  def self.search(search) #self.でクラスメソッドとしている
    if search # Controllerから渡されたパラメータが!= nilの場合は、titleカラムを部分一致検索
      Post.where(
                ['company LIKE ?', "%#{company}%"],
                ['store LIKE ?', "%#{store}%"],
                ['owner LIKE ?', "%#{owner}%"],
                ['kana LIKE ?', "%#{kana}%"],
                ['tel LIKE ?', "%#{tel}%"],
                ['tel2 LIKE ?', "%#{tel2}%"],
                ['fax LIKE ?', "%#{fax}%"],
                ['industry LIKE ?', "%#{industry}%"],
                ['mail LIKE ?', "%#{mail}%"],
                ['url LIKE ?', "%#{url}%"],
                ['people LIKE ?', "%#{people}%"],
                ['address LIKE ?', "%#{address}%"],
                ['statu LIKE ?', "%#{statu}%"],
                ['time LIKE ?', "%#{time}%"],
                ['comment LIKE ?', "%#{comment}%"],
                ['created_at BETWEEN ? AND ?', from, to, "%#{created_at}%"],
                )
        
    else
      Post.all #全て表示。
    end
  end

  def next_post
    Post.where("id > ?", id).first
  end

  def prev_post
    Post.where("id < ?", id).last
  end
end

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# Note that this schema.rb definition is the authoritative source for your
# database schema. If you need to create the application database on another
# system, you should be using db:schema:load, not running all the migrations
# from scratch. The latter is a flawed and unsustainable approach (the more migrations
# you'll amass, the slower it'll run and the greater likelihood for issues).
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2018_11_22_055616) do

  create_table "comments", force: :cascade do |t|
    t.string "responsible"
    t.string "product"
    t.string "statu"
    t.datetime "time"
    t.string "detail"
    t.integer "sfa_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["sfa_id"], name: "index_comments_on_sfa_id"
  end

  create_table "details", force: :cascade do |t|
    t.string "statu"
    t.datetime "time"
    t.string "comment"
    t.integer "post_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["post_id"], name: "index_details_on_post_id"
  end

  create_table "posts", force: :cascade do |t|
    t.string "company"
    t.string "store"
    t.string "owner"
    t.string "kana"
    t.string "tel"
    t.string "tel2"
    t.string "fax"
    t.string "industry"
    t.string "mail"
    t.string "url"
    t.string "people"
    t.string "address"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "sfas", force: :cascade do |t|
    t.string "company"
    t.string "store"
    t.string "owner"
    t.string "kana"
    t.string "person"
    t.string "p_kana"
    t.string "tel"
    t.string "tel2"
    t.string "fax"
    t.string "industry"
    t.string "mail"
    t.string "url"
    t.string "people"
    t.string "post_number"
    t.string "address"
    t.string "employment"
    t.string "social"
    t.string "appointer"
    t.string "sales_staff"
    t.string "sales_day"
    t.string "status"
    t.string "target"
    t.string "sales_forecast"
    t.string "impression"
    t.string "repeat_sales"
    t.string "next_sales"
    t.string "prospect"
    t.string "ditails"
    t.string "examination"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "users", force: :cascade do |t|
    t.string "email", default: "", null: false
    t.string "encrypted_password", default: "", null: false
    t.string "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.integer "sign_in_count", default: 0, null: false
    t.datetime "current_sign_in_at"
    t.datetime "last_sign_in_at"
    t.string "current_sign_in_ip"
    t.string "last_sign_in_ip"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["email"], name: "index_users_on_email", unique: true
    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
  end

end

Answer 3

<% post.details.each do |detail| %>
  <td><%= detail.updated_at %></td>
<% end %>

を

<td><%= post.details.order('updated_at DESC').first.updated_at %></td>

とすれば、関連テーブル先の更新された最新データ一件についての更新日時を取得できると思います。

Comments

[退会済みユーザー]

ありがとう御座います。
上記だと、エラーが発生してしまうようです。
controller設定等が必要なのでしょうか？

[退会済みユーザー]

<% post.details.each do |detail| %>
<td><%= post.details.order('updated_at DESC').first.updated_at %></td>
<% end %>

上記の場合、エラーは発生せずに表示出来ますが、当初の写真通り、updatedの全てが表示されてしまいます・

[troch]

↑で一応、出力されるのに、
<td><%= post.details.order('updated_at DESC').first.updated_at %></td>
だとエラーになる、というのは変ですね…。

エラー文を見せていただけますでしょうか？

[troch]

それから、app/models/post.rbと、db/schema.rbの内容も見せていただけませんでしょうか。

[退会済みユーザー]

ご確認よろしくお願い致します。

[troch]

エラー画像 確認しました。
post.detailsで取得した値がnilなのでエラーになっているのかもしれません…

確認なのですが、すべてのpostは必ずdetailsレコードを持つのでしょうか？
detailsを持たないpostもあるのでしょうか？
KentaOkuyamaさんが質問欄にアップしてくださいました当初の写真を見ますと、最終更新日の列にデータが入っておらず、[編集 削除] ボタンが入っている行もありますが…
（つまりその行はdetailsを持たないpostなのではないでしょうか？）

[troch]

・detailsを持たないpostレコードもある
・その場合、空白の列として表示できればいい
という条件でいいのであれば、
<td><%= post.details.order('updated_at DESC').first.try(:updated_at) %></td>
のコードに書き換えればおそらく動くと思います。

[退会済みユーザー]

ありがとう御座います。解決しました。もっと自分で勉強してみます。