validate（バリデーション）が機能しません

独自メソッドを用いてvalidateを行いたいと考えていましたがうまく行きません。ネット上にはhas_manyのバリデーションの仕方など掲載されておりましたがthroughを含めたものは見た限りありませんでした。
自分が開発しているアプリ内でフォローできる人数を8人以下にしたいです。そこでバリデーションを設置する事で8人以上の人をフォローをした時にエラーが表示されるようにしたいです。

以下のコードではバリデーションに引っかからず8人以上の人をフォローできるようになってしまいます。
よろしくお願いします。

やった事として、rails consoleを開き

first_following=Relationship.create(follower_id: "1", followed_id: "2")
second_following=Relationship.create(follower_id: "1", followed_id: "3")
third_following=Relationship.create(follower_id: "1", followed_id: "4")
fourth_following=Relationship.create(follower_id: "1", followed_id: "5")
fifth_following=Relationship.create(follower_id: "1", followed_id: "6")
sixth_following=Relationship.create(follower_id: "1", followed_id: "7")
seventh_following=Relationship.create(follower_id: "1", followed_id: "8")
eighth_following=Relationship.create(follower_id: "1", followed_id: "9")
ninth_following=Relationship.create(follower_id: "1", followed_id: "10")

上のコードを入力してレコードを作成し、９つともレコードが作成されました。エラーやレコードを作成できない
といったメッセージは表示されませんでした。
確認として,以下のコードを入力して９が出力され、９人フォローできていることも確認できました。

me=Account.find_by(id:"1")
me.following.count

これでバリデーションに問題があるというのはわかったのですが実際にどこに問題があり、なぜバリデーションが機能せず、９人以上フォローすることができてしまうのかがよくわかりません。

account.rb

class Account < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  
  
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
         
         
  has_many :active_relationships,
          class_name:  "Relationship",
          foreign_key: "follower_id",
          dependent:   :destroy
  has_many :passive_relationships,
          class_name:  "Relationship",
          foreign_key: "followed_id",
          dependent:   :destroy
                                   
  has_many :following, through: :active_relationships, source: :followed
  has_many :followers, through: :passive_relationships, source: :follower
 
  # ユーザーをフォローする
  def follow(other_account)
    following << other_account
  end

  # ユーザーをフォロー解除する
  def unfollow(other_account)
    active_relationships.find_by(followed_id: other_account.id).destroy
  end

  # 現在のユーザーがフォローしてたらtrueを返す
  def following?(other_account)
    following.include?(other_account)
  end
  
 
  
  
    
    
  
  
  
end

relationship.rb

class Relationship < ApplicationRecord
    validates :follower_id, presence: true
    validates :followed_id, presence: true
    belongs_to :follower, class_name: "Account", optional: true
    belongs_to :followed, class_name: "Account", optional: true
    validate :check_following

    def check_following
        
        if  Account.find(followed_id).following.count > 8
            
            errors.add(:followed_id, "too much following")
            
        
        end
    end
    
end

行動規範の内容に同意します

回答4件

ベストアンサー

できました。。。。。。。。

投稿2020/05/08 07:02

退会済みユーザー

総合スコア0

自分が開発しているアプリ内でフォローできる人数を1人以下にしたいです。

なら、モデルにfollowing_user_id列を作ってそこに値を入れる、という構造にすれば、バリデーションをかけるまでもなく1人以下のデータしか保存できなくなります。

投稿2020/04/16 05:39

maisumakun

総合スコア145184

退会済みユーザー

2020/04/16 05:43

アプリ内の別の機能で、フォローを８人以下にしたいのですが、その場合はどうなりますか？

maisumakun

2020/04/16 05:52

それならそうと先に言ってほしかったです。この方法は1人にしかならないのが前提です。

退会済みユーザー

2020/04/16 07:04

申し訳ありません

行動規範の内容に同意します

バリデーションに引っかからず４人以上に人をフォローできるようになってしまいます。

フォローするときに作られるレコードはRelationshipモデルのものなのでAccountモデルに定義されたバリデーションは意味を成しません。

投稿2020/02/28 08:21

rhiroe

総合スコア2349

退会済みユーザー

2020/02/29 02:00

Relationshipモデルで　if の条件文にfollower_id.count > 3 とした時に　undefined method `count' のエラーが出ており、count以外に個数を調べる方法を思いつかないです。ヒントをいただけないでしょうか？よろしくお願いします。

rhiroe

2020/03/02 00:31 編集

Account.find(followed_id).following.count >= 3 であれば既に3人以上フォローしている場合にそれ以上フォローできなくなると思います。 User ではなく Account でしたね、修正しました。

退会済みユーザー

2020/02/29 09:17

Relationship.find(followed_id).following.count >= 3をするとActiveRecord::RecordNotFound　Couldn't find Relationship with 'id'=2とエラーが出て、Account.find(followed_id).following.count >= 3をするとエラーが出ずにそのままエラーが出てしまいます。どこか他のところが間違っているのでしょうか？何度も何度も申し訳ありません。

rhiroe

2020/03/02 00:32

あれ、うまく行きませんか？どういうエラーが出てますか？

退会済みユーザー

2020/04/14 07:54

返信遅くなって申し訳ありません。エラーは何も出ずにそのままフォローできてしまいます。ただ、Account.find_by(id: followed_id).following.count >= 0というコードを入力してフォローすると、Validation failed: Followed too much followingというエラーが出ます。よろしくお願いします。

rhiroe

2020/04/16 00:27

そのフォローしたアカウントは3人以上フォローした状態でしたか？時間も空きすぎて当時何を思っていたかもよく覚えてないですが...。

退会済みユーザー

2020/04/16 00:34

いいえ、誰もフォローしていないし誰もフォローされていない状態です。本当に申し訳ありません。

退会済みユーザー

2020/04/16 00:37

consoleでRelationshipを作成して５人フォローして見てもerrors.messagesとした時でも何もエラーが出ていませんでした。

rhiroe

2020/04/16 03:23

現状をあまり把握できていないので、可能な限り最新の状態で質問を更新していただけますか？