Question

   list = []
    Merchant.all.each do |merchant|
      if merchant.url.present? && merchant.url != merchant.api_data.url
        list.push({
          :id => merchant.id,
          :name => merchant.name,
          :another => merchant.another,
          :another => merchant.another,
          :another => merchant.another
        })
      end
    end

Is there is simpler way to create a hash of only some of the fields of that object? Sorry if this is too obvious. I’m transitioning over to Ruby from PHP.

Answer 1

merchants = Merchant.all.map do |merchant|
  if merchant.url.present? && merchant.url != merchant.api_data.url
    merchant.attributes.slice("id", "name", "another")
  end
end.compact

Notes:

• Why do you need to extract attributes from a merchant instance instead of using the record itself?
• Don’t use generic names as list. Give meaningful names to variables.
• Consider moving some logic to the SQL level. That’s it, create scopes that filter the records you want, it will much faster.

Using records:

merchants = Merchant.all.select do |merchant|
  merchant.url.present? && merchant.url != merchant.api_data.url
end