全てのページを「マイページ」のような自分が投稿した内容だけの一覧表示にしたい。

Question

こんばんは。

自分の目標等を投稿できるオリジナルアプリを作っています。

実装したい内容について

• indexページについて、いろんなユーザーが投稿した内容が全て表示されるのではなく、カレントユーザーが投稿した内容を、カレントユーザーのみに表示されるようにしたいです。
• 目標設定に限らず、今後作成するページ全てについて他の人が見られないようにしたいです。

現在の状況について

• 現在deviseによるuserモデルと、目標を立てるためのobjectiveモデルを別々に作成し、アソシエーションを組んでいる状況です。

• 下記のように実装して、エラーは消えたのですが、肝心のindexページにカレントユーザーが投稿した内容が反映されない状況です。（DBには保存できています）

• これはもしかしてルーティングにおいてuserに全て入れ子にすることが必要なのでしょうか？

（実装したい内容としては、ユーザー情報を消せば、それに紐づく目標なども全て消えたりする状況にしたいです）

• 今まで、ユーザー登録した人が投稿してみんなが見れるようなアプリは作成してきたのですが、ユーザー登録した人がそれぞれのマイページで投稿した目標を作成・一覧できるような実装方法が調べても分からず教えて欲しいです。

# app/views/objectives/index.html.erbの状況

<div class="contents row">
<%= link_to '投稿する', new_objective_path, class: "login" %>
<% if current_user.id == @objective.user_id %>
  <% @objectives.each do |objective| %>
    <div class="content_post" style="background-image: url(<%= asset_path('note.png') %>);"%>
      <div class="more">
        <span><%= image_tag 'menu.png' %></span>
        <ul class="more_list">
          <li>
            <%= link_to '詳細', objective_path(objective.id), method: :get %>
          </li>
          <li>
            <%= link_to '編集', edit_objective_path(objective.id), method: :get %>
          </li>
          <li>
            <%= link_to '削除', objective_path(objective.id), method: :delete %>
          </li>
        </ul>
      </div> 
      <p><%= objective.title %></p>
      <p><%= objective.category_id %></p>
      <span class="name">
        <%= objective.created_at %>
      </span>
    </div>
  <% end %>
  <% end %>
</div>

# app/controllers/objectives_controller.rbの状況

class ObjectivesController < ApplicationController
  before_action :authenticate_user!
  before_action :set_objective, only: [:edit, :show]

  def index
    @objectives = Objective.includes(:user).order('created_at DESC')
    @objective = Objective.new
  end

  def new
    @objective = Objective.new
  end

  def create
    @objective = Objective.new(objective_params)
    if @objective.save
      redirect_to objectives_path
    else
      render :new
    end
  end

  def destroy
    objective = Objective.find(params[:id])
    objective.destroy
  end

  def edit
  end

  def update
    objective = Objective.find(params[:id])
    objective.update(objective_params)
  end

  def show
  end

  private

  def objective_params
    params.require(:objective).permit(:title, :category_id, :wish, :outcome, :obstacle, :plan).merge(user_id: current_user.id)
  end

  def set_objective
    @objective = Objective.find(params[:id])
  end
end

# config/routes.rbの状況
Rails.application.routes.draw do
  devise_for :users
  root to: 'homes#index'
  get 'homes/index'
  resources :objectives
  resources :troubles
  resources :users
end

# app/models/objective.rbの状況

class Objective < ApplicationRecord
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :user
  belongs_to_active_hash :category

  with_options presence: true do
    validates :title
    validates :category_id
    validates :wish
    validates :outcome
    validates :obstacle
    validates :plan
  end
end

# app/models/user.rbの状況

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :objectives

  with_options presence: true do
    validates :nickname, uniqueness: true, length: { maximum: 15 }
    validates :birthdate
    # @含むこと・存在することはdeviseのデフォルト実装のため省略
    validates :email, uniqueness: true
    # 存在すること・確認用を含めて2回入力はデフォルト実装のため省略 安全性を高めるために15文字以上に設定
    validates :password, length: { minimum: 15 }
    # パスワードが半角英数字（空文字NG）以外の場合には、メッセージを出す
    PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i.freeze
    validates_format_of :password, with: PASSWORD_REGEX, message: 'Include both letters and numbers'
  end
end

どうか教えていただけますと幸いです。
よろしくお願いします。

Comments

[m.ts10806]

実装あんまり見てないですけど、データ取得時にそもそもuser_idで絞ればいいのでは？

[cherry2020]

コメントありがとうございました！！
解決できました！！

[m.ts10806]

https://teratail.com/help#resolve-myself
では↑

Answer 1

コントローラーのindexアクションを以下のように変更したらできました。

m.ts10806さん、初めてコメントいただき嬉しかったです、ありがとうございました。

class ObjectivesController < ApplicationController
  before_action :authenticate_user!
  before_action :set_objective, only: [:edit, :show]

  def index
    @objectives = current_user.objectives.all.order(created_at: :desc)
  end

  def new
    @objective = Objective.new
  end

  def create
    @objective = Objective.new(objective_params)
    if @objective.save
      redirect_to objectives_path
    else
      render :new
    end
  end

  def destroy
    objective = Objective.find(params[:id])
    objective.destroy
  end

  def edit
  end

  def update
    objective = Objective.find(params[:id])
    objective.update(objective_params)
  end

  def show
  end

  private

  def objective_params
    params.require(:objective).permit(:title, :category_id, :wish, :outcome, :obstacle, :plan).merge(user_id: current_user.id)
  end

  def set_objective
    @objective = Objective.find(params[:id])
  end
end