コントローラーのアクション内での分岐について

Question

#前提・実現したいこと
初質問させていただきます。
情報不足もあるかと思いますが、何卒宜しく御願い致します。

####制作物：　
タスク管理アプリ

####環境：　
macOS Catalina ver10.15.3　、　rails (~> 5.2.3)　、　　ruby 2.6.3p62

####制作物仕様：　

• 個人で個人情報の編集、削除可能。
• グループの作成可能。
• グループでタスク作成可能。　　　　　　

####現状：　
グループでタスク作成できるが、個人でタスクの作成ができない。
####目標：　
グループに所属していなくても、タスクを作成できる様にしたい。

発生している問題・エラーメッセージ

taskのコントローラーの記述方法次第で、目標達成できると仮説しております。

個人でタスク作成する画面でのエラーメッセージ

該当のソースコード

class TaskController < ApplicationController

  before_action :set_task, only: [:edit, :update, :destroy]

  def index
    @group = Group.find_by(id: params[:group_id])
  end

  def new
    @group = Group.find(params[:group_id])
    @task = Task.new
    # if @task.include(:group_id)
      
    # @tasks = @group.tasks.includes(:user)
    # end
  end

  def create
    @group = Group.find(params[:group_id])
    @task = @group.tasks.new(task_params)
    if @task.save
      redirect_to group_task_index_path
    else
      render :new
    end
  end

  def edit
    @group = Group.find_by(id: params[:group_id])
  end

  def update
    if @task.update_attributes(task_params)
      redirect_to :root
    else
      render :edit
    end
  end

  def destroy
    @task.destroy
    redirect_to :root
  end



  private
    def task_params
      params.require(:task).permit(:title).merge(user_id: current_user.id)
    end

    def set_task
      @task = Task.find_by(id: params[:id])
    end
end

試したこと

newアクションとcreateアクションの条件分岐次第かと仮説しております。
（group_idが含まれている場合と含まれていない場合）

最後に

必要な情報等ございましたら、ご教授頂けますと幸いです。

以上、お手数ですが、何卒ご教授の程宜しく御願い致します。

#追記

####groupsモデルのコード

class Group < ApplicationRecord
  has_many :group_users, dependent: :destroy
  has_many :users, through: :group_users, dependent: :destroy
  has_many :tasks, dependent: :destroy
  
  validates :name, presence: true, uniqueness: true
end

###Taskモデルのコード

class Task < ApplicationRecord
  # belongs_to :group
  belongs_to :user
  # belongs_to :groups
  has_many :comments, dependent: :destroy

  # validates :title, length: {in: 1..255}
end

###個人モデルのコード

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  #usernameを必須・一意とする
 
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :group_users, dependent: :destroy
  has_many :groups, through: :group_users, dependent: :destroy
  has_many :tasks, dependent: :destroy

  validates :username, presence: true, length: { maximum: 20 }
  #usernameを利用してログインするようにオーバーライド
  def self.find_first_by_auth_conditions(warden_conditions)
    conditions = warden_conditions.dup
    if login = conditions.delete(:login)
      #認証の条件式を変更する
      where(conditions).where(["username = :value", { :value => username }]).first
    else
      where(conditions).first
    end
  end

  #emailを不要とする
  def email_required?
    false
  end

  def email_changed?
    false
  end

  def will_save_change_to_email?
    false
  end

end

###変更：グループで作成したタスクの表示画面のビュー
変更前：タスクの新規作成画面のビュー（個人とグループ）

<header class="header2">
  <nav class="nav2">
    <ul class="header_menu2">
      <li class="nav-link">
        <%= @group.name %>
      </li>
      <ul class="header_menu2_inner2">
        <li><%= link_to "Create Group Tasks", new_group_task_path(@group), class: "nav-link listNew2" %></li>
        <li><%= link_to "Edit Group",edit_group_path(@group), class: "nav-link listNew3", method: :get %></li>
      </ul>
    </ul>
  </nav>
</header>

<div class="topPage">
  <div class="taskWrapper">
    <% @group.tasks.each do |task| %>
      <div class="task">
        <div class="task_header">
          <h2 class="task_header_title"><%= task.title %></h2>
          <div class="task_header_action">
             <%= link_to content_tag(:i, '', class: 'fas fa-trash'), task, method: :delete, data: { confirm: "#{task.title}を本当に削除しますか？" } %>
            <%= link_to(content_tag(:i, '', class: 'fas fa-pen'), edit_task_path(task)) %>
          </div>
        </div>
        <div class="commentWrapper">
          <% task.comments.each do |comment| %>
            <%= link_to task_comment_path(task, comment), class:"commentDetail_link" do %>
              <div class="comment">
                <h3 class="comment_title"><%= comment.title %></h3>
                <% if comment.memo.size > 0 %>
                  <div class="comment_detail is-exist"><i class="fas fa-bars"></i></div>
                <% end %>
              </div>
            <% end %>
          <% end %>
          <div class="addComment">
            <i class="far fa-plus-square"></i>
            <%= link_to "さらにコメントを追加", new_task_comment_path(task), class: "addComment_link" %>
          </div>
        </div>
      </div>
    <% end %>
  </div>
</div>

###groups, tasks, 個人テーブルの定義

ActiveRecord::Schema.define(version: 2020_02_25_061850) do

  create_table "comments", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "title", null: false
    t.text "memo"
    t.bigint "task_id", null: false
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["task_id"], name: "index_comments_on_task_id"
  end

  create_table "group_users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.bigint "group_id"
    t.bigint "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["group_id"], name: "index_group_users_on_group_id"
    t.index ["user_id"], name: "index_group_users_on_user_id"
  end

  create_table "groups", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "name", null: false
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["name"], name: "index_groups_on_name", unique: true
  end

  create_table "tasks", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "title", null: false
    t.bigint "user_id", null: false
    t.bigint "group_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["group_id"], name: "index_tasks_on_group_id"
    t.index ["user_id"], name: "index_tasks_on_user_id"
  end

  create_table "users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "encrypted_password", default: "", null: false
    t.string "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "username", default: "", null: false
    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
  end

  add_foreign_key "group_users", "groups"
  add_foreign_key "group_users", "users"
  add_foreign_key "tasks", "groups"
  add_foreign_key "tasks", "users"
end

#追記２

###タスク作成画面の表示画面
####個人でタスク作成の場合
個人でのタスク作成の現状

• ozroro 様にご教授頂いた通り、上記task.controller.rb内に対して、Group.find_by(id: params[:group_id])へ変更し、新規作成画面を表示することができました！
• しかし、下記「タスクの新規作成画面のビュー（個人とグループ）」（app/views/task/index.html.erb）

の条件分岐では、elseの方にいかず、groupの方を読み込んでしまいます。

• 動画上では、メリーと表示されているheaderはありますが、目標はそのheaderは表示されないこと。

###グループでタスク作成の画面
グループでのタスク作成の現状

• グループではタスクの新規作成、表示はできる。しかし、バイキンマンと表示されているheaderが、タスク新規作成画面になるとメリーになってしまう。（おそらく、DBのgroupテーブルのidの1を読み込んでいる。

仮説も含めてみました。お手数ですが、何卒ご教授の程宜しく御願い致します。

#追記３
##追記１の上記の「タスクの新規作成画面のビュー（個人とグループ）」は「グループで作成したタスクの表示画面のビュー」でした。申し訳ありませんでした。
###「タスクの新規作成画面のビュー（個人とグループ）」

• ifとelseで条件分岐しております。

<% - if @group_id.present?%>
  <header class="header2">
    <nav class="nav2">
      <ul class="header_menu2">
          <li class="nav-link">
              <%= @group.name %>
          </li>
          <ul class="header_menu2_inner2">
            <li><%= link_to "Create Group Tasks", new_group_task_path(@group), class: "nav-link listNew2", method: :get %></li>
            <li><%= link_to "Edit Group",edit_group_path(@group), class: "nav-link listNew3", method: :get %></li>
          </ul>
        </ul>
    </nav>
  </header>
  <div class="tasknewPage">
    <div class="container">
      <%= form_with model: @task, url: { action: :create }, class: "new_task", local: true do |f| %>
        <%= f.label :title %>
        <%= f.text_field :title, autofocus: true, class: "form-control taskName", placeholder: "タスク名" %>
        <div class="text-center"><%= f.submit "作成", class: "submitBtn" %></div>
      <% end %>

<%  else %>  
  <%= form_with model: @task, url: { action: :create }, class: "new_task", local: true do |f| %>
    <%= f.label :title %>
    <%= f.text_field :title, autofocus: true, class: "form-control taskName", placeholder: "タスク名" %>
    <div class="text-center"><%= f.submit "作成", class: "submitBtn" %></div>
  <% end %>
<% end %>

Comments

[shinoharat]

以下の情報を質問文に追記してください。

【1】Group モデルのコード
【2】Task モデルのコード
【3】個人 モデルのコード
【4】新規画面の view
【5】groups, tasks, 個人テーブルの定義（db/schema.rb から該当部分をコピペすればOKです）

[raigakun]

@shinoharat 様
ありがとうございます。
追記いたします。

Answer 1

恐らくそのviewは作成でなく表示画面ではないですか？

該当部分のエラーなら単純に

 Group.find_by(id: params[:group_id])

とすればエラーはでないです。
findは探してるidがない場合エラーをあげますが
find_byならnilが返ってきます。

あとコメントアウトしてる下記の部分ですが

    # if @task.include(:group_id)

@taskインスタンスのgroup_idが入ってるかどうか調べるなら

 if @task.group_id.present?
 

でいいと思います。

Comments

[raigakun]

@ozroro 様
ご教授いただきありがとうございます。
ご教授いただいたとおり、

```
Group.find_by(id: params[:group_id])
```
に変更いたしましたところ、エラーは出なくなりました。

しかし、次の問題が出ておりまして・・・お手数ですが、ご教授頂けますと幸いです。
詳細は追記いたします。

ありがとうございました。

Answer 2

皆様、ご助言いただきありがとうございました。
個人とグループでの別々の投稿画面は出力することができました。ありがとうございます。

#追加したコード

class TaskController < ApplicationController

  before_action :set_task, only: [:edit, :update, :destroy]

  def index
    @group = Group.find_by(id: params[:group_id])
  end

  def new
    @group = Group.find_by(id: params[:group_id])
    @task = Task.new
  end

  def create
    @group = Group.find_by(id: params[:group_id])
    @task = @group&.tasks.new(task_params)
    @task = Task.new(task_params)
    # if @group_id.present?
    #    @group = Group.find_by(id: params[:group_id])
    #    @group.tasks.save
    #   redirect_to group_task_index_path(@group)
    # elsif 
    if @task.save
      redirect_to group_task_index_path
    elsif @tasks.save
      redirect_to :root
    else
      render :new
    end
  end

  def edit
    @group = Group.find_by(id: params[:group_id])
  end

  def update
    if @task.update_attributes(task_params)
      redirect_to :root
    else
      render :edit
    end
  end

  def destroy
    @task.destroy
    redirect_to :root
  end



  private
    def task_params
      params.require(:task).permit(:title).merge(user_id: current_user.id)
    end

    def set_task
      @task = Task.find_by(id: params[:id])
    end
end

次はcreateアクションで詰まってしまっておりますので、別途質問させて頂きます。
何卒宜しく御願い致します。