[英]rubocop string interpolation and size condition
除了這兩種方法之外,我想先看看社區中是否有人有更好的主意來構造這兩種方法並使警察通過。 第一個帶有to_s
似乎也有些瘋狂。 我當時正在考慮重構另一種方法,但這將是一兩行。
有什么想法嗎?
代碼示例一:
def destroy(resource_name, id)
delete "#{resource_name.to_s.pluralize}/#{id}"
end
代碼示例二:
def all_products
products_map = fetch(:products).map { |x| [x['id'], x] }.to_h
variants = fetch :variants
variants.group_by { |x| x['product']['resource']['id'] }.to_a.map do |product_id, product_variants|
product.merge 'variants' => product_variants if product == products_map[product_id]
end.compact
end
對於代碼示例一,也許可以使用:
delete [resource_name.to_s.pluralize, id].join('/')
對於代碼示例2,是的,您肯定需要對其進行重構。
也許您需要創建一個單獨的方法來對variants
部分進行所有分組和合並等操作。
我不確定這是否是一個好習慣,但是您可以為其創建私有方法。
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.