rubocop String Interpolation und Größe Bedingung

Question

Bevor ich außer diesen beiden Methoden wollte ich sehen, ob jemand in der Gemeinschaft eine bessere Idee hatte, diese zu strukturieren und machen die Bullen passieren. Der erste mit to_s scheint auch ein bisschen verrückt. Ich dachte daran, die andere Methode zu überarbeiten, aber das wäre eine oder zwei Zeilen.

Gedanken?

Code-Beispiele One:

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

Codebeispiele zwei:

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

Code Beispiel zwei, ja:

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 Beispiel eines, vielleicht kann dies verwendet werden, Sie müssen es definitiv umgestalten.
Vielleicht müssen Sie eine separate Methode erstellen, die die gesamte Gruppierung und Zusammenführung usw. für den variants Teil übernimmt.
Ich bin nicht sicher, ob dies eine gute Praxis ist, aber Sie können eine private Methode dafür erstellen.