updateアクションが発動するとcurrent_userが毎回追加されてしまう

Question

下記のコントローラーにあるupdateアクション内にある「@group.users << current_user」の位置が悪いせいで毎回追加されているのはわかるのですが、
「もし加入済みならこの記述は無視する」みたいな記述を付け加えればいいのだとは思うのですがどこにどのように書けばいいのかわからないです。

class GroupsController < ApplicationController
  before_action :set_group, only: [:edit, :update]
  
  def index
  end

  def new
    @group = Group.new
  end

  def create
    @group = Group.new(group_params)
    @group.users << current_user 
    if @group.save
      redirect_to root_path, notice: 'グループを作成しました'
    else
      render :new
    end
  end

  def edit
  end
  
  def update
    if @group.update(group_params)
      @group.users << current_user
      redirect_to group_messages_path(@group), notice: 'グループを編集しました'
    else
      render :edit
    end
  end

  private

  def group_params
    params.require(:group).permit(:name, { :user_ids => [] })
  end

  def set_group
    @group = Group.find(params[:id])
  end
end

Answer 1

updateアクションの @group.users << current_userを
「@groupeのusersにcurrent_userが存在しなければ」という条件
を付け加えると、うまくいきそうですね。

例えば、以下のような感じかと！

if @group.users.find_by(id: current_user).nil?
 @group.users << current_user
end

Comments

[nemutai]

回答ありがとうございます！！
自分がやりたいことがわかりやすく書いてあり感動です。
やってみます！！！