Лучшая структура кода для ассоциаций Rails

Есть четвертый вариант. Полностью удалите эту логику из модели:

class PostProcessor
  def initialize(posts)
    @posts = posts
  end

  def process
    @posts.each do |post|
      # ...
    end
  end
end

PostProcessor.new(User.find(1).posts).process

Иногда это называют шаблоном Service Object. Очень приятный бонус этого подхода в том, что он делает написание тестов для этой логики действительно простым. Вот отличная запись в блоге об этом и других способах рефакторинга «толстых» моделей: http://blog.codeclimate.com/blog/2012/10/17/7-ways-to-decompose-fat-activerecord-models/

Лично я считаю, что метод 1 самый чистый. Будет очень чисто и понятно написать что-то вроде этого:

Class User < ActiveRecord::Base
  has_many :posts

  def process_posts
    posts.each do |post|
     post.process
    end
  end
end

И поместите всю логику метода process в модель Post (с переменной экземпляра):

Class Post < ActiveRecord::Base
  belongs_to :user

  def process
     # Logic of your Post process
  end
end

Таким образом, сама логика Post процесса принадлежит Post классу. Даже если ваша User модель будет иметь много "процессных" функций, они будут очень простыми и маленькими. Мне как разработчику это кажется очень чистым.

Метод 3 имеет много технических последствий, которые довольно сложны и неинтуитивны (вам пришлось уточнить свой вопрос).

ПРИМЕЧАНИЕ. Если вы хотите повысить производительность, возможно, вам следует использовать eager loading для уменьшения количества вызовов ActiveRecord, но это выходит за рамки этого вопроса.

Прежде всего, извините меня за самоуверенный ответ.

Модели ActiveRecord — спорный вопрос. Его суть противоречит принципу единой ответственности, поскольку они обрабатывают как взаимодействие с базой данных через методы класса, так и объекты предметной области (которые используют для реализации собственного поведения) через свои экземпляры. В то же время они также нарушают принцип подстановки Лисков, поскольку модели не являются подвидами ActiveRecord::Base и реализуют собственный набор методов. И, наконец, парадигма ActiveRecord часто приводит к коду, нарушающему Закон Деметры, как в вашем предложении для третьего метода:

User.find(1).posts.process

Таким образом, существует тенденция, что для уменьшения связанности рекомендуется использовать объекты ActiveRecord только для взаимодействия с базой данных и поэтому не следует добавлять к ним какое-либо поведение (в вашем случае метод process). С моей точки зрения, это меньшее зло, хотя и не идеальное решение.

Поэтому, если бы я реализовал то, что вы описываете, у меня был бы объект ProcessablePostsCollection (где имя Processable можно настроить, чтобы лучше описать, о чем идет речь, или даже полностью пренебречь, чтобы вы просто имели класс PostsCollection), который, вероятно, был бы обертка над списком сообщений с использованием SimpleDelegator и будет иметь метод process.

class ProcessablePostsCollection < SimpleDelegator
  def self.from_collection(collection)
    new collection
  end

  def initialize(source)
    super source
  end

  def process
    # code of whatever 'process' does to posts
  end
end

И использование будет примерно таким:

ProcessablePostsCollection.from_collection(User.find(1).posts).process

несмотря на то, что from_collection и вызов process должны происходить в разных классах.

Кроме того, если у вас есть большая таблица сообщений, вероятно, было бы разумно обрабатывать данные в пакетном режиме. Для этого ваш метод process может вызывать find_in_batches для ваших сообщений ActiveRecord::Relation.

Но, как всегда, это зависит от ваших потребностей. Если вы просто строите прототип, вполне нормально, чтобы ваши модели разрастались, а если вы строите огромное приложение, Rails сам по себе, вероятно, не будет лучшим выбором, поскольку не поощряет некоторые передовые практики ООП с такими вещами, как модели ActiveRecord.

Вы не должны помещать это в модель User - поместите его в Post (если, конечно, объем process не включает модель User напрямую):

#app/models/post.rb
class Post < ActiveRecord::Base
    def process
       return false if post.published?
       # do something
    end
end

Затем вы можете использовать расширение ассоциации ActiveRecord, чтобы добавить функциональность к модели User:

#app/models/user.rb
class User < ActiveRecord::Base
   has_many :posts do
      def process
          proxy_association.target.each do |post|
             post.process
          end
      end
   end
end

Это позволит вам звонить...

@user = User.find 1
@user.posts.process