Working With Rails

06:47 25-06-09

#01

Hello, I would like some advice on how to optimise my code. I have different model

building has_many roles

role (which brings the type of role for the user for the building) belongs_to building belongs_to user

user has_many roles

And I have a lot of method in the building model to find the different type of member for a building like

def find_admins_of

roles = self.roles

admins = []

for role in roles

  admins &lt;&lt; User.find(role.user_id) if role.role_type_id == Role::ADMIN

end

admins

end

This is not the best way to do it but I would like to know which better way. I have thought of a has_many, through but I can't find the right syntax for the conditions on the role.role_type_id

I hope I am clear.

Thanks for your help Nicolas

Sign In To Rate Post

manohar

23:09 26-06-09

#02

Use following association in your model building-

has_many :roles, :incldue=>:user, :conditions=>"user.role_type = 'Admin' "

manohar

Sign In To Rate Post

patnakajima

01:27 27-06-09

#03

Here's my refactoring: "gist.github.com/136835":http://gist.github.com/136835

I renamed your "role_type_id" attribute to "name", but you can get the idea.

patnakajima

Sign In To Rate Post

21:25 28-06-09

#04

Thnaks very much, I am gonna give it a try. I'll let you know.

Nicolas