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

是否更简单地制造了该物体某些领域的 has? 如果情况非常明显,则会发生争执。 I m 从PHP过渡到Rub。

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.

利用记录:

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

Answer 2

可查阅<代码>#reject或#select。他们这样做或没有被列入你的名单。 http://www.ruby-doc.org/core-1.9.2/Hash.html#method-i-reject” rel=“nofollow”>rejectmeth。虽然仅仅增加属性并非罕见,但你已经这样做了。

Best regards

偏见

友情链接