質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

1578閲覧

一対多を用いて自分の登録タスクのみを管理する方法

iwashita

総合スコア3

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2020/06/02 16:53

編集2020/06/03 01:36

前提・実現したいこと

プログラミング初心者です。
タスク管理アプリケーションの作成においてユーザが登録し、自身の登録したタスクのみにアクセスし新たなタスクの登録・削除・編集、自身のタスクのみを閲覧できるようにしたいと思っています。

ユーザが登録しサインイン、サインアウトし、タスクリスト画面において新規タスクの登録・削除・編集できる状態にはなっているのですが、登録されてるユーザ全てにおいて同じタスクを共有しており、一人のユーザが新しくタスクを登録すれば別のユーザが登録した際も同じくタスクリストにて自分のではないタスクが表示される状態となっており、一対多が実装出来ておりません。

「多」側に当たる方にbelongs_toのコードを記載しました。

class Task < ApplicationRecord belongs_to :user validates :status, presence: true, length: { maximum: 10} validates :content, presence: true, length: { maximum: 255 } end
class Tasklist < ApplicationRecord belongs_to :user validates :task, presence: true, length: {maximum: 255 } end

また、「1」側にhas_manyを次のように記載しているのですが…

class User < ApplicationRecord before_save { self.email.downcase! } validates :name, presence: true, length: { maximum: 50 } validates :email, presence: true, length: { maximum: 255 }, format: { with: /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i }, uniqueness: { case_sensitive: false } has_secure_password has_many :tasks end

この状態ではユーザに管理をさせたいstatus及びcontentをユーザと紐づけることが出来ずすべてのユーザで同じタスクリストを管理することになってしまっています。

イメージ説明

status及びcontentをユーザに紐づけて一対多の関係を成り立たせるにはどうすればよいのでしょうか?

未熟な質問で申し訳ございませんがご教授頂けると助かります。

宜しくお願い致します。

頂いたご質問への追加です。

1)Tasklistは間違って作成してしまったモデルファイルかもしれません。
存在しているタスクモデルに追加でユーザとの一対多の関係を保持するカラムを用意する際に間違ってマイグレーション**rails g migration Add-To-**を行ってしまい、その際にモデルファイルの作成もも行ったとは思うのですが、はっきりどのファイルを間違って作成したのかは覚えておらず、rails migrate:resetは実行しているのと、覚えている限りの作成したファイルは削除しています。

2)tasks_controller.rb

class TasksController < ApplicationController before_action :require_user_logged_in before_action :correct_user, only: [:destroy] def index @tasklists = Task.order(id: :desc).page(params[:page]).per(3) end def show end def new @task = Task.new end def create @task = current_user.tasks.build(task_params) if @task.save flash[:success] = "タスクが正常に登録されました" redirect_to root_url else @tasks = current_user.tasks.order(id: :desc).page(params[:page]) flash.now[:danger] = "タスクが登録されませんでした" render :new end end def edit end def destroy @task.destroy flash[:success] = 'タスクを削除しました。' redirect_back(fallback_location: root_path) end private def set_task @task = Task.find(params[:id]) end def task_params params.require(:task).permit(:content, :status) end def correct_user @task = current_user.tasks.find_by(id: params[:id]) unless @task redirect_to root_url end end end

併せてマイグレーションファイル作成後に変更、作成したファイルのコードも載せさせてもらいます。

config/routes.rb

Rails.application.routes.draw do # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html root to: 'tasks#index' get 'login', to: 'sessions#new' post 'login', to: 'sessions#create' delete 'logout', to: 'sessions#destroy' resources :tasks get 'signup', to: 'users#new' resources :users, only: [:index, :show, :new, :create] resources :tasks, only: [:create, :destroy] end

tasks/_tasks.html.erb

<ul class="list-unstyled"> <% tasks.each do |task| %> <li class="media mb-3"> <img class="mr-2 rounded" src=" <%= gravater_url(task.user, { size: 50 }) %> " alt=""> <div class="media-body"> <div> <%= link_to task.user.name, user_path(task.user) %> <span class="text-muted">posted at <%= task.created_at %> </span> </div> <div> <p class="mb=0"> <%= task.content %> </p> </div> <div> <%= if current_user == task.user %> <%= link_to "Delete", task, method: :delete, data: { confirm: "You sure?" }, class: "btn btn-danger btn-sm" %> <% end %> </div> </div> </li> <% end %> <%= paginate tasks %> </ul>

toppages/index.html.erb

<% if logged_in? %> <div class="row"> <aside class="col-sm-4"> <%= form_with(model: @tasklist, local: true) do |f| %> <div class="form-group"> <%= f.text_area :content, class: 'form-control', rows: 5 %> </div> <%= f.submit 'Post', class: 'btn btn-primary btn-block' %> <% end %> </aside> <div class="col-sm-8"> <%= render 'tasklists/tasklists', tasklists: @tasklists %> </div> </div> <div class="center jumbotron"> <div class="text-center"> <hi>Welcome to the Tasklists</hi> <%= link_to 'Sign up now!', signup_path, class: 'btn btn-lg btn-primary' %> </div> </div> <% end %>

宜しくお願い致します。

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

Ruby 5.2.2
Rails 5.2.4.3

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

winterboum

2020/06/02 22:23

1) Tasklist って何のためにありますか? 2) taskの一覧を出すためのcontrollerを載せてください
iwashita

2020/06/03 01:38

ご回答と修正点ありがとうございます。 修正してあげさせてもらいました。
guest

回答1

0

ベストアンサー

model Task の関連定義も載せてもらうのだった。
通常の定義がされているとして

@tasklists = Task.order(id: :desc).page(params[:page]).per(3)

@tasklists = Task.where(user_id: current_user.id).order(id: :desc).page(params[:page]).per(3)

@tasklists = current_user.task.order(id: :desc).page(params[:page]).per(3)

投稿2020/06/03 04:46

winterboum

総合スコア23567

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

iwashita

2020/06/03 06:48

ありがとうございます。 現在こちら-- @tasklists = current_user.tasks--で一対多の関係実装できたので、--order(id: :desc).pafe(params[:page]).per(3)--ご参考にさせてもらいます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問