レコード数に基づいて真または偽を返すメソッドを作成する方法

Question

trueまたはfalseを返すメソッドを作成しようとしています。ここで

は私が持っているものである

ビュー

<%= @poll.has_user_voted?(current_user) %> 

モデル私が間違ってやっている

def has_user_voted?(user) 
    voted = self.poll_votes.where(:user_id => user.id).length 

    return !voted 
    end 

任意のアイデア。これは、空白

Answer 1

すべてが0が実際真実であることを意味しているnilとfalse,以外Rubyで真では何も返さないです。

ような何か試してみてください。これは一例で

def has_user_voted?(user) 
    self.poll_votes.where(:user_id => user.id).length > 0 
    end 

を、私は、あなただけ使用することがありますされていない場合、あなたの現実のバージョンがpoll_votes,を利用しますので、あなたも投票をしたいと推定します#count。

Answer 2

一致するレコードがあるかどうかだけを知りたい場合は、不要な行を取得してすべてのオブジェクトをインスタンス化するのではなく、組み込みのcountメソッドを使用する必要があります。デフhas_user_voted

：注意すべきもう一つのわずかな専門的

def has_user_voted?(user) 
    self.poll_votes.exists?(['user_id = ?', "%#{user.id}%"]) 
end 

Answer 3

これはかなり効率的な方法でなければなりませんか？ユーザ poll_votes.where（：USER_ID => user.id）.size> 0 端

.sizeは、DB .countを発行するか、または関連付けが既にロードされている場合、それは、配列の長さを取得しますSQL呼び出しもしません。私はいつもサイズを使います。

Answer 4

：例えば：

def has_user_voted? user 
    poll_votes.where(:user_id => user.id).count > 0 
end