0

'デフショー

 @dept = Dept.find(params[:id])
    @members = @dept.members_list.collect{|a|a.name}
    dif = @dept.users.collect{|a|[a.name,a.id]}
    @admin_list = @dept.admin_users.collect{|a|a.user}
   @not_member_users = User.all_users - dif
   @not_admin_user = (@dept.list_of_deptmembers - @dept.admin_users).collect{|a|[a.user.name, a.id]}'

@not_admin_user をリファクタリングするにはどうすればよいですか??

4

3 に答える 3

1

に基づいてテーブルを検索する方が高速で安価な場合idもあります -- これはそのようなケースの 1 つかもしれません。

@dept = Dept.find(params[:id])

# assuming both `members_list` and `admin_users` actually point to `User` objects
member_ids = @dept.members_list.collect(&:id)
admin_ids = @dept.admin_users.collect(&:id)
non_admin_id = member_ids - admin_ids

non_admin_users = User.where("id in ?", non_admin_id)
于 2013-02-28T02:49:28.787 に答える
0

For starters, you can push all of that logic down into the Dept model.

def show
  @dept = Dept.find(params[:id])
  @admin_list = @dept.admin_list
  @members = @dept.members
  @not_member_users = @dept.not_member_users
  @not_admin_user = @dept.not_admin_user
end

class Dept < ActiveRecord::Base
  def members
    self.members_list.collect{ |a| a.name }
  end

  def admin_list
    self.admin_users.collect{ |a| a.user }
  end

  def not_member_users(user = User)
    dif = self.users.collect{|a|[a.name,a.id]}
    user.all_users - dif
  end

  def not_admin_user
    (self.list_of_deptmembers - self.admin_users).collect{|a|[a.user.name, a.id]}
  end
end
于 2013-02-28T02:17:32.690 に答える
0

オブジェクトの属性の配列ではなく、オブジェクトのコレクションを扱う方がよいでしょう。そのため、名前の文字列の配列ではなく、またはオブジェクト@membersの配列にする必要があります。MemberUser

これはあなたの質問に対するあまり良い答えではないかもしれませんが、私は外部からの開発の原則に従い、次のようにあなたが望むものを手に入れることを目指します:

def show
  @dept = Dept.find(params[:id])
  @members          = @dept.members
  @admin_list       = @dept.admin_users
  @not_member_users = @dept.non_member_users
  @not_admin_users  = @dept.non_admin_users
end

Dept モデルで、上記で使用したメソッドに適切な結果を返すスコープ/関連付けを設定します。Ruby で結果をフィルタリングするのは最後の手段です。

于 2013-02-28T04:57:04.427 に答える