Рефакторинг методов модели в Ruby On Rails

Question

Обычная идиома, которую мой лагерь использует в рельсах, такова:

def right_things(all_things, value)
    things = []
    for thing in all_things
       things << thing if thing.attribute == value
    end
    return things
end

как я могу сделать это лучше / быстрее / сильнее?

ТНХ

-C

Answer 1

def right_things(all_things, value)
    all_things.select{|x| x.attribute == value}
end

Answer 2

Если ваши вещи являются моделями ActiveRecord, и вам нужны только предметы, выбранные для вашей текущей цели, вы можете, если вы используете Rails 2.0 (определенно 2.1) или выше, найти named_scope s полезным.

class Thing
  named_scope :rightness, lambda { |value| :conditions => ['attribute = ?', value] }
end

Так что вы можете сказать

Thing.rightness(123)

, что (в данном случае) похоже на

Thing.find_by_attribute(123)

в том смысле, что он сводится к запросу SQL, но его проще модифицировать в цепочке. Если это полезно для вас, чего не может быть, конечно ...