【Ruby on Rails】チャットアプリで、既成のグループがある場合新たにグループ作成されないようにしたい

Question

前提・実現したいこと

今、Ruby on Railsでアプリを作っており、その中でユーザー同士のチャット機能しています。
ユーザーの詳細ページ（user#show）に行くと「メッセージを送る」というボタンがあり、
そのボタンをクリックするとそのユーザーとのチャットルーム（message#index）に飛ぶようにしています。

ここでは、メッセージを送る対象のユーザーとログイン中のユーザーとの関係をgroupと定義しています。
ログイン中のユーザーが「メッセージを送る」ボタンをクリックしたとき、
そのユーザーとのグループが作成されたことがない場合はgroup#createを経由してmessage#indexに飛び、
すでにグループが作成されている場合は直接message#indexに飛ぶように設定したいです。

今その実装がなかなか上手く行かず、すでにグループが作成されている場合でも
同じユーザーとのグループが複数作成されてしまうので困っています。

解決法を教えていただけますと幸いです。

また、users_controller.rbのshowアクション内のよりシンプルな書き方があれば
それも合わせて教えていただけると大変助かります。

よろしくお願いいたします。

該当のソースコード

▼users_controller.rb

class UsersController < ApplicationController

  def show
    @user = User.find(params[:id])
    group_ids = @user.groups.pluck(:id)

    group_users = GroupUser.where(group_id: group_ids)
    group_user = group_users.where(user_id: current_user.id)

    @group_id = group_user.pluck(:group_id).first
  end

end

▼groups_controller.rb

class GroupsController < ApplicationController
  before_action :authenticate_user!

  def index
    groups = current_user.groups.includes(:messages)
    @groups = groups.joins(:messages).order("messages.created_at DESC")
  end

  def create
    group = Group.create
    group_user = GroupUser.create(group_id: group.id, user_id: current_user.id)
    @user = User.find(params[:id])
    group_user = GroupUser.create(group_id: group.id, user_id: @user.id)
    @group_id = group_user.group_id
    redirect_to group_messages_path(@group_id, user_id: @user.id)
  end

  private
  def group_params
    params.permit(:id, user_ids: [])
  end

end

▼views/users/show.html.haml

.Title
  = "#{@user.name}さんのページ"
.Profile
  - if @user.profile_image?
    = image_tag @user.profile_image.url, class: "Profile__Image"
  - else
    = image_tag "no-image.jpg", class: "Profile__Image"
  .Profile__Content
    .Profile__Content__Upper
      .Name
        = @user.name
      .Info_right
        - if user_signed_in? && @user.id == current_user.id
          = link_to "プロフィールを編集", edit_user_path(current_user), class: "Info_right__Btn"
        - else
          - if @group_id.nil?
            = link_to "メッセージを送る", "/groups/#{@user.id}/create", class: "Info_right__Btn"
          - else
            = link_to "メッセージを送る", group_messages_path(@group_id, user_id: @user.id), class: "Info_right__Btn"

▼models/user.rb

class User < ApplicationRecord
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  validates :name, presence: true, length: {maximum: 10}

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

  has_many :messages, dependent: :destroy

end

▼models/group.rb

class Group < ApplicationRecord
  has_many :group_users, dependent: :destroy
  has_many :users, through: :group_users
  has_many :messages, dependent: :destroy
end

試したこと

users_controller.rbのshowアクション内に
@is_exists_user_group = current_user.groups.pluck(:user_id).include?(@user.id)
と記述し、show.html.hamlにif @is_exists_user_groupと条件分岐で書いてみたりもしましたが、
状況は変わらず、やはり複数同じグループが作られてしまいました。

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

▼rails routesの結果

group_messages GET    /groups/:group_id/messages(.:format)          messages#index
  　　　　　  　 GET    /groups/:id/create(.:format)                  groups#create

▼コンソールでGroupUser.allと打った結果

Answer 1

- if @group_id.nil? の書き方をみると　@group_id　は login ユーザーとshowされているユーザーとのgroupのidのことだと思われます。

しかし、UsersControllerの show ではそうなって居ません。
というよりそれ以前にmodelの設計がおかしいかも。
User、Group、GroupUserの関連定義を載せてください。
特に気になるのは
userAをshow、userBがcurrent_user の時のGroupと
userBをshow、userAがcurrent_user の時のGroupとは
同じGroupなのですよね？それが実現するようになっているかどうか。

追記

となると、そのgroupをどうやって得るかなぁ

1. @user.groups と current_user.groups で一致するもの
2. @user.group_users.pluck(:group_id) の中にcurrent_userと同じgroup_idのものがあるか

か
前者は
@group = (@user.groups & current_user.groups).first
(ないときはnil。 なので nil.idとならないように注意)
後者は
group_user = GroupUser.where(group_id: @user.group_users.pluck(:group_id).first @group = group_user.try(:group)
両方共DB２回アクセスするなぁ。。。

Comments

[kumamin]

コメントありがとうございます。
まさに、@group_idは、 ログイン中のユーザーとshowページのユーザーとのgroupのidです。
userとgroupのモデルとgroupsコントローラーのコード、GroupUserの中身がわかるターミナルの画像を追加しましたので、ご確認いただけると助かります。

[kumamin]

すみませんなぜかわからないのですが、解決しました。
これから何が原因だったか探ってみたいと思います。
winterboumさんには以前にもご回答いただいたことがあり、大変感謝しています。
ありがとうございました。

[kumamin]

追記ありがとうございます。
showアクション内でgroupのidを取り出すのに5行も書いていたので、
追記の1番のように書いてみたところコードをとてもシンプルにすることができ、グループ作成も思うように実装できたのでよかったです。
ありがとうございます！