2

ディレマ：

複数のビューとコントローラーにわたってユーザーのアクティビティを表示する必要がある状況にあります。コードの匂いが少しし始めているので、コードをクリーンアップするためのガイダンスをいただければ幸いです。

ユーザーコントローラーには、サイドバーに最近のアクティビティを表示する「show」（/ user_id /）ビューと、ユーザーのすべてのアクティビティを表示する「activity」（user_id / activity）ビューがあります。また、ユーザーのアクティビティをレンダリングする「ダッシュボード」ビューを持つ「ページ」コントローラーもあります。

コード：

users_controller.rb次のようになります：

@activities = []
@activities += @user.skills.map { |skill| Activity.new("skill", skill, skill.created_at) }
@activities += @user.unions.map { |union| Activity.new("union", union, union.created_at) }
@activities += @user.companies.map { |company| Activity.new("company", company, company.created_at) }
@activities += @user.clients.map { |client| Activity.new("client", client, client.created_at) }
@activities += @user.schools.map { |school| Activity.new("school", school, school.created_at) }
@activities += @following_list.map { |following| Activity.new("following", following, following.created_at) }
@activities += @recommended.map { |recommend| Activity.new("recommend", recommend, recommend.created_at) }
@activities += @user.statuses.all(:limit => 5, :order => "created_at DESC").map { |status| Activity.new("status", status, status.created_at) }

sorted_activities = @activities.sort_by(&:date).reverse
@activities = sorted_activities[0..(4)]

pages_controller.rb次のようになります：

@your_activity = []
@your_activity += current_user.skills.map { |skill| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","skill", skill, skill.created_at) }
@your_activity += current_user.unions.map { |union| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","union", union, union.created_at) }
@your_activity += current_user.companies.map { |company| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","company", company, company.created_at) }
@your_activity += current_user.clients.map { |client| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","client", client, client.created_at) }
@your_activity += current_user.schools.map { |school| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","school", school, school.created_at) }
@your_activity += current_user.statuses.map { |status| ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}","status", status, status.created_at) }

sorted_activities = @your_activity.sort_by(&:date).reverse
@your_activity = sorted_activities[0..(35)]
4

1 に答える 1

1

あなたが本当にそれをすることができるいくつかの方法があります。それをモデルにオフロードし、libディレクトリにモジュールを作成し、それを使用してアクティビティ配列を構築することができます。重要なことの1つは、必ずヘルパーを使用してページに表示し、必要に応じてビューまたはパーシャルにロードすることです。

メタプログラミングを使用して、重複を少し減らすこともできます。要件に応じて調整する必要がある場合があります。最初のコードセットにも設定する必要があります

["skill", "union", "company", "client", "school", "status"].each do |type|
  plural = ActiveSupport::Inflector.pluralize(type)
  @your_activity += current_user.send(plural.to_sym).map do |object|
    ActivityDashboard.new("#{current_user.first_name} #{current_user.last_name}","#{current_user.id}","#{current_user.avatar}",type, object, object.created_at)
  end
end

私の作業例の要点：https ：//gist.github.com/3669117

于 2012-09-07T19:45:55.047 に答える