rubocop строка интерполяции и состояние размер

Question

Прежде, чем я, кроме этих двух методов, которые я хотел, чтобы увидеть, если кто-то в обществе было лучшее представление структурировать эти и сделать полицейские проходят. Первый с 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 

Answer 1

Например Code One, возможно, это может быть использовано:

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

Например Code Two, да вам определенно нужно реорганизовать его.
Возможно, вам нужно создать отдельный метод, который выполняет всю группировку и слияние и т. Д. Для части variants.
Я не уверен, что это хорошая практика, но вы можете создать для него частный метод.