バリデーションがかからない

Question

前提・実現したいこと

現在UserテーブルとMemberテーブルが多対多の関係で紐づけられていてその中間テーブルに保存できる数を3つにしようとバリデーションをかけたいのですがかかりません

該当のソースコード

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

  validates :favorites, length: {maximum: 3}
 end

 class Member < ApplicationRecord
  has_many :member_comments
  has_many :comments, through: :member_comments
  
  has_many :favorites
  has_many :favorited_users, through: :favorites, source: :user
  validates :favorites, length: {maximum: 3}
end

class Favorite < ApplicationRecord
  belongs_to :user
  belongs_to :member
  validates_uniqueness_of :member_id, scope: :user_id
end

class FavoritesController < ApplicationController

  def index
    @favorites = Favorite.all
    @members = Member.all
  end


  def create
    @user_id = current_user.id
    @member_id = Member.find(params[:id]).id
    @favorite = Favorite.new(member_id: @member_id, user_id: @user_id)
    if @favorite.save
      redirect_to user_path(current_user.id)
    end
  end

  def destroy
    @favorite = Favorite.find(params[:id])
    if @favorite.destroy
      redirect_to user_path(current_user.id)
    end
  end

end

.members
  .member-index
    メンバー 一覧
  .member-selects
    .select
      - @members.each_with_index do |member, i|
        - if current_user.already_favorited?(member)
          .add-member
            = member.name
            .registration
            = link_to "推しメンから外す" , favorite_path(@favorites.ids), method: :delete, class: "delete"
        - else
          .add-member
            = member.name
            = link_to "推しメンに登録する", add_favorite_path(id: member.id), method: :post

試したこと

モデルへのかけ方がわからないのでビュー側で条件分岐で表示を変えようと
@favorites.length = 3で分岐させようとすると
NoMethodError in Favorites#index undefined method `lengthになります

Answer 1

class FavoritesController < ApplicationController
 def index
  @count = Favorite.where(user_id: current_user.id)　を追加
 end

-if @count.count == 3
　current_userのfavoriteテーブルのuser_idが3つ時の処理
-else
 それ以外の処理

これで中間テーブルであるfavoriteテーブルに保存できる数を制限できました。

Answer 2

「中間テーブルに保存できる数を3つにしようとバリデーションをかけたいのですがかかりません」と「@favorites.length = 3で分岐させようとすると NoMethodError」と二つあるのですが、
「バリデーションをかけたいのですがかかりません」というのはどのように調べましたか？

「@favorites.length = 3で分岐させようとすると NoMethodError」これは明
@favorites.size == 3 にしてみてください

Comments

[nasuk47]

まずモデルにバリデーションをかけて保存できないようにしようと思い「バリデーション　レコード　制限　」で調べました。
それでもうまくいかなかったのでビュー側で条件分岐でfavoriteのidを3つ持っている時に表示を変えればいいと思いました。

[winterboum]

validates :favorites, length: {maximum: 3}
でうまく行かなかった　というのではないのですか？

[nasuk47]

そうです.modelの方にvalidates :favorites, length: {maximum: 3}を記述したのですがうまくいきませんでした。
一応記述として残していました。

[winterboum]

うまく行かない、というのは、具体的に以下のように説明していただけますか？
こうしたら
こうなってほしいのに
こうなった

[nasuk47]

上記の記述をしたら保存ができないようなって欲しいのですが保存がされてしまいます。