Createアクションの条件分岐方法についてご教授願います。

Question

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

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

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

####制作物仕様：　

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

####現状：　
個人、グループ、どちらもタスクの作成画面までは、出力できている。
グループでタスク作成できる（createアクションが機能している）が、個人でタスクの作成ができない。
####目標：　
グループに所属していなくても、タスクを作成できる様にしたい。
createアクション内の記述方法次第かと考えてます。

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

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

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

該当のソースコード

createアクションについては、試行錯誤しております。見にくく申し訳ありません。
task_controller.rb

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(@group)
    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アクションについて
こちらでご対応いただき、newアクションについては目標物に近づけることができました。ありがとうございました。

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

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

#追記１
###grop の場合と 個人の場合で createに渡ってくる情報が各々どうなのか、というデータ
以下、task_controller内のcreateアクションにてbinding.pryを追記しました。
group_idの有無で条件分岐を試みてます。

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
  binding.pry
    if @group = Group.find_by(id: params[:group_id])
      @task = @group.tasks.new(task_params)
    else
      @task = Task.new(task_params)
    end
    if @task.save
      redirect_to group_task_index_path(@group)
    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

###個人でタスクを新規作成
実際のビュー
binding.pry後、ターミナルの画面

###グループでタスクを新規作成
実際のビュー
binding.pry後、ターミナルの画面

###タスクの新規作成のコード(個人とグループでviewが異なる様に、条件分岐を用いております。）

<% - if @group.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 %>
aaa
  <%= form_with model: @tasks, 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

[winterboum]

grop の場合と 個人の場合で createに渡ってくる情報が各々どうなのか、というデータを載せてください
何を条件に分岐したら良いのか、がわからない

[raigakun]

winterboum様
ご教授頂きありがとうございます。
Createアクションでbinding.pryを行い、個人の場合とグループの場合のタスク新規作成画面からどのような情報が出ているかをターミナル上で確認します。少々お待ち頂けますと幸いです。

何卒よろしくお願いします。

Answer 1

多分でしかないが

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

追記

      if @group
        redirect_to group_task_index_path(@group)
      else
        redirect_to :root
      end

で出来るとは思いますが
分岐して、共通処理して、また同じ条件で分岐するというのはあまりよくないですね。
後からコードを読む人が共通処理によって条件が変化する可能性を考慮する必要があるのが駄目です。

なので、次善としては

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

def create
  if @group = Group.find_by(id: params[:group_id])
    group_create(@group, task_params)
  else
    private_create(task_params)
  end
end

def group_create(group, param)
  @task = group.tasks.new(param)
  if @task.save
    redirect_to group_task_index_path(group)
  else
    render :new
  end
end
def private_create(param)
  @task = Task.new(task_params)
  if @task.save
    redirect_to :root
  else
    render :new
  end
end

もしくは

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

ですかね

Comments

[raigakun]

asm　様
ご教授いただきありがとうございます。
早速、試してみたところ、下記のリンク先のエラーが出てしまいました。

https://gyazo.com/97006ed8a16ce6fd499940f9253e1b7a

params内も変更が必要なのでしょうか。

何卒宜しく御願い致します。

[asm]

> <%= form_with model: @tasks, url: { action: :create }, class: "new_task", local: true do |f| %>

@taskの間違いではないでしょうか?
それとも、なにか違いがあり@tasksにしなければいけない理由があるのですか？

[raigakun]

asm　様
ご教授いただきありがとうございます。
ご指摘の通り、@taskです。特段理由もありませんでした。申し訳ありません。

[raigakun]

asm　様
重ねて、ご教授・ご助言頂けますと幸いです。
ご指摘いただいた通り、@tasks → @task　へ訂正しました。その結果、以下のエラーが出ました。
https://gyazo.com/9328d5ac29c83bcd4f91f364692d6f16

仮説では、個人でcreateしたタスクの反映先はgroup_idが必要な場所（グループがタスクを作成した場合に反映する場所）になっているためのエラーと思っております。
ですので、反映先を変更すれば、解決すると思われます。（例えば、redirect_to :root)
しかし、その記述に変更した場合、グループで作成したタスクも反映先が変わってしまします。

反映先も条件分岐できるのでしょうか。
わかりづらい文章で申し訳ありませんが、何卒宜しく御願い致します。

[raigakun]

asm　様
お忙しい中、ご教授いただきありがとうございます。
ご助言いただいた通り、記載致しましたところ、目標通りの動きができました。ありがとうございます。
asm　様がおっしゃる通り、第三者のことを考えたコードを書ける様に日々学習いたします。
とても勉強になりました。またご教授頂くことがあるかと思いますが、その際は何卒宜しく御願い致します。
ありがとうございました。