Targeting every object in an array syntax

Question

Newb question of the day:

I'm trying to select all the users with this condition, and then perform an action with each one :

User.find(:all).select { |u| u.organizations.count > 0} do |user|

Except, this isn't the right way to do this. Not entirely sure what the proper syntax is.

Any fellow rubyist offer a newb a hand?

Answer 1

To perform an action with each element of a collection use the each method, like this:

User.find(:all).select { |u| u.organizations.count > 0}.each do |user|

Answer 2

You'd probably be better folding the select into the query with:

User.find(:all, :conditions => "organization_id IS NOT NULL").each do |user|

This will only fetch the relevant results from the database so there should be less unnecessary data retrieved and thrown away.

EDIT: As suggested in the comments, the following would be correct for a many-to-many relationship assuming a join model called memberships (where user has_many :organisations, :through => :membership)...

User.all(:joins => "inner join memberships on memberships.user_id = users.id")