rubocop文字列補間とサイズ条件I前

Question

。 to_sの最初のものはやや狂っているようです。私は他のメソッドをリファクタリングすることを考えていましたが、それは1行または2行になります。

思考？

コード例の一つ：

def destroy(resource_name, id) 
    delete "#{resource_name.to_s.pluralize}/#{id}" 
    end 

コード例2：

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 

Answer 1

コード例の一つでは、多分これを使用することができます。

delete [resource_name.to_s.pluralize, id].join('/') 

コード例えば2つ、はいあなたは間違いなくそれをリファクタリングする必要があります。
おそらく、variantsの部分のすべてのグループ化とマージなどを行う別個のメソッドを作成する必要があります。
これは良い習慣であるかどうかはわかりませんが、プライベートメソッドを作成できます。