中間テーブルに保存したい。

Question

閲覧して頂き、ありがとうございます。

かなり長文になってしまいましたが、ご確認頂き、アドバイス頂けますと幸いです。

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

実現したい機能とデータベース設計

mixiのコミュニティ（当アプリではgroup）のような機能のアプリを作成しております。

そのためにはデータベース設計として、

groupテーブル
userテーブル
中間テーブルとしてgroup_users

が必要であると考えました。

保存の手順としては、

手順1.ログインしたユーザーはgroupを作成でき、作成されるとgroupテーブルにgroupの情報が保存され、中間テーブルであるgroup_usersテーブルに作成されたgroupのidと作成したuserのidが保存される。（views/groups/new.html.erbのgroup作成フォーム）

手順2.作成されたグループに新たなユーザーが参加した際は、中間テーブルであるgroup_usersテーブルに参加したgroupのidと参加したuserのidが保存される。（views/groups/index.html.erbのgroup参加フォーム）

という二段階のデータベース設計が必要かと考えました。

mixiのコミュニティのような機能を作成する際はこのようなデータベース設計、フォームの実装で方向性は間違っていませんでしょうか？

#現状の実装とエラーについて
現状は、

1.groupモデル、userモデル、group_usersテーブルの実装
2.groupコントローラー、userコントローラーの実装
3.deviseの実装
4.ルーティングの設定
5.viewの実装とformの作成
6.groupコントローラーの編集を行いました。

上記の手順で実装を行いました。

ですが、データベース設計の手順1.の部分の作成されたgroupのデータはgroupテーブルに保存は出来ていましたが、中間テーブルにはgroup_idとuser_idが保存されない状態でした。

※下記classについては過去に作成したアプリから引用しておりますので、ズレているところがあります。

###groupテーブルのみ保存された時のコントローラーとview

#Groupsのコントローラー
class GroupsController < ApplicationController
  
  def new
    @group = Group.new
  end
  
  def index
    @groups = Group.all
    @groups = Group.order("created_at DESC")
  end

  def create
    @group = Group.new(group_params)
    if @group.save
      redirect_to root_path
    else
      render :new
    end
  end

  private

    def group_params
      params.require(:group).permit(:name, :text, :category_id, :genre_id, :image, {user_id: []} )
    end

end

#groups/new.html.erbのview

<div class='chat-room-form'>
  <h1>新規グループ作成</h1>
  <%=form_with model: @group, local: true do |f|%>

      <%# user_idを送るための記述 %>
        <div class='group-room-form__field--left'>
        <%= f.label :メンバー, class: 'group-room-form__label'%>
      </div>
      <div class='group-room-form__field--right'>
        <%= text_field_tag 'group[user_id]', current_user.id %>
      </div>
      <%# //user_idを送るための記述 %>

    <div class='group-form__field'>
      <div class='group-room-form__field--left'>
        <%= f.label :グループ名, class: 'group-room-form__label'%>
      </div>
      <div class='group-room-form__field--right'>
        <%= f.text_field :name, class: 'group__room_name group-room-form__input', placeholder: 'グループ名を入力してください'%>
      </div>
      <div class="group-room-form__field--left">
        <%= f.label :image, "グループアイコン" %><br />
        <%= f.file_field :image %>
      </div>
      <div class='group-room-form__field--left'>
        <%= f.label :カテゴリー, class: 'group-room-form__label'%>
      </div>
      <div class='group-room-form__field--right'>
        <%= f.collection_select(:category_id, Category.all, :id, :name, {}, {class:"group-select"}) %>
      </div>
      <div class='group-room-form__field--left'>
        <%= f.label :ジャンル, class: 'group-room-form__label'%>
      </div>
      <div class='group-room-form__field--right'>
        <%= f.collection_select(:genre_id, Genre.all, :id, :name, {}, {class:"group-select"}) %>
      </div>
      <div class='group-room-form__field--left'>
        <%= f.label :グループ説明, class: 'group-room-form__label'%>
      </div>
      <div class='group-room-form__field--right'>
        <%= f.text_field :text, class: 'group__room_name group-room-form__input', placeholder: 'どんなグループ？'%>
      </div>
    </div>
    <div class='group-form__field'>
      <div class='group-form__field--left'></div>
      <div class='group-form__field--right'>
        <%= f.submit class: 'group-form__action-btn'%>
      </div>
    </div>
  <% end %>
</div>

user_idを送る記述がおかしいと思い、user_idを送るための記述を下記のように記述したらgroupも保存されませんでした。
<div class='chat-room-form__field--right'> <input name="[user_ids][]" type="hidden" value=<%= current_user.id %>> </div>

また、これ以外にも様々試みましたが、いずれも保存は出来ず、use_idが取得出来ませんでした。

そもそものデーターベース設計、または、送るデータの記述が間違っているのでしょうか？

アドバイス頂けましたら幸いです。

その他の関連するコード

#Groupモデル

class Group < ApplicationRecord
  extend ActiveHash::Associations::ActiveRecordExtensions

  has_one :category
  has_one :genre

  has_many :group_users
  has_many :users, through: :group_users, dependent: :destroy
  has_many :messages, dependent: :destroy

  has_one_attached :image

  with_options presence: true do
    validates :name, uniqueness: true
    validates :text
    validates :category_id
    validates :genre_id
  end

  with_options numericality:{other_than: 1} do
    validates :category_id
    validates :genre_id
  end

end

#Userモデル

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable



  validates :nickname,         presence: true
  validates :password,         presence: true
  PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?[\d])[a-z\d]+\z/i.freeze
  validates_format_of :password, with: PASSWORD_REGEX 
  with_options presence: true, format: { with: /\A[ぁ-んァ-ン一-龥]+\z/} do
  validates :last_name
  validates :first_name
  end
  with_options presence: true, format: { with: /\A[ァ-ン]+\z/} do
  validates :last_name_kana
  validates :first_name_kana
  end
  validates :birthday,         presence: true

  has_many :group_users
  has_many :groups, through: :group_users
    # accepts_nested_attributes_for :group_users

  has_many :messages
end

#GroupUserモデル

class GroupUser < ApplicationRecord
  belongs_to :group
  belongs_to :user
end

#GroupUsersのマイグレーションファイル

class CreateGroupUsers < ActiveRecord::Migration[6.0]
  def change
    create_table :group_users do |t|
      t.references :group,  foreign_key: true
      t.references :user,   foreign_key: true
      t.timestamps
    end
  end
end

#Groupのマイグレーションファイル

class CreateGroups < ActiveRecord::Migration[6.0]
  def change
    create_table :groups do |t|
      t.string    :name,          null: false
      t.text      :text,          null: false
      t.integer   :category_id,   null: false
      t.integer   :genre_id,      null: false
      t.references :user,         foreign_key: true
      t.timestamps
    end
  end
end

#userの関係のある部分のマイグレーションファイル

create_table :users do |t|
      ## Database authenticatable
      t.string  :nickname,             null: false
      t.string  :email,                null: false, default: ""
      t.string  :encrypted_password,   null: false, default: ""
      t.string  :last_name,            null: false
      t.string  :first_name,           null: false
      t.string  :last_name_kana,       null: false
      t.string  :first_name_kana,      null: false
      t.date    :birthday,             null: false
      t.integer :category
      t.integer :genre

長文になってしまいましたが、お読み頂きありがとうございます。

Comments

[winterboum]

手順１の話ということですね？
モデルGroupの関連定義を載せてください

[zoomer]

回答ありがとうございます。

は、手順１の話です。

groupモデル、コントローラー、マイグレーションファイル
userモデル、マイグレーションファイル
group_usersのモデル、マイグレーションファイル

記載致しました。

ご確認頂けますと幸いです。

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

Answer 1

方法は大きく分けて２つあります。
ひとつは create で @group.save　成功した後
@group.users = current_user
これはgroup作成時はcurrent_userしか入れないからこれで良いです。

ふたつ目は、この後の手順２でも使える方法で

1. モデル Groupに accepts_nested_attributes_for :group_users
2. viewの<%=form_with の中に

f.fields_for :griup_users do |griup_user| user.hidden_field :user_id
3. controller の group_params に group_users_attributes: :user_id を追加

Comments

[zoomer]

ご回答ありがとうございます。

重ね重ねの質問で申し訳ありません。

上記のご指摘頂いた一つめのコントローラーで保存記述をする場合は、今記載しているviewのform内にある

<%# user_idを送るための記述 %>
<div class='group-room-form__field--left'>
<%= f.label :メンバー, class: 'group-room-form__label'%>
</div>
<div class='group-room-form__field--right'>
<%= text_field_tag 'group[user_id]', current_user.id %>
</div>
<%# //user_idを送るための記述 %>

のような記載は不要ということでしょうか？

[winterboum]

はい不要です

[zoomer]

ありがとうございます。

上記のuser_idを送るための記述を削除して、ご指摘頂きました

@group.users = current_user

を@group.saveの下に記述して

`if @group.save
@group.users = current_user
redirect_to root_path`

と記載した際に

undefined method `each' for #<User:0x00007fcd555b88d8>

という表示が出て、eachはviewに記載されているので、viewに記載が必要なのかと考えておりました。

おそらく他のところが間違えているかと思いますので、

@group.users = current_user

をベースにエラー箇所を潰していきます。

ありがとうございます。

[winterboum]

念の為
@group.users = [current_user]
も試してください
undefined method 　がデタ時のエラーメッセージを見たいです