gistfile1.rb

  def subproduct?
    !self.parent.blank?
  end
  
  def self.not_sub
    returning [] do |value|
      Product.each do |x|
        if !x.subproduct?
          value << x
        end
      end
    end
  end

1er commentaire : pourquoi !..blank? alors que rails à la méthode present? qui justement faire !blank? automatiquement ?

2ème commentaire : returning est deprecated. Il faut utiliser la méthode tap désormais.
Donc a la place de returning([]) do..., utilise [].tap do...

3eme : Vu que tu construit un Array a partir d'un autre, inject peut etre utilisé. Ex :

def self.not_sub
  Product.each.inject([]) { |value, product| value << x if !product.subproduct?; value; }
end

4eme : sinon effectivement ces commentaires sont pur ruby, y a moyen de faire en SQL.

En SQL c'est mieux en effet :

scope :not_sub, lambda {
joins("left join product_joins on products.id = product_joins.sub_product_id").where('product_joins.product_id' => nil)
}